• Type: Bug
• Resolution: Fixed
• Priority: Major
• Fix Version/s: 2.2.3, 2.4.0
• Affects Version/s: 2.4.0
• Component/s: Edition
• Labels:

  None

1. ODF_fix_dnd.patch

   3 kB

   19/May/15 14:00

[ODF-1050] Impossible de déplacer une liste d'ELP dans un conteneur vide

Laurence Perier made changes - 02/Feb/16 17:15

Status

Original: Resolved [ 5 ]

New: Closed [ 6 ]

Laurence Aumeunier made changes - 20/May/15 12:26

Resolution		New: Fixed [ 1 ]
Status	Original: Open [ 1 ]	New: Resolved [ 5 ]

Quentin Glinel-Mortreuil (Inactive) made changes - 19/May/15 14:00

Assignee

Original: Quentin Glinel-Mortreuil [ quentin ]

New: Laurence Aumeunier [ laurence ]

Quentin Glinel-Mortreuil (Inactive) made changes - 19/May/15 14:00

Attachment

New: ODF_fix_dnd.patch [ 16123 ]

Collapse comment: Laurence Aumeunier added a comment - 18/May/15 15:53

Laurence Aumeunier added a comment - 18/May/15 15:53

Certainement à cause du leaf = true pour les container/sous-programme n'ayant pas encore de fils.
Forcer node.leaf = false; avant de valider le D&D comme cela est fait pour l'explorateur de ressources.

org.ametys.explorer.TreeDropZone = Ext.extend(Ext.tree.TreeDropZone, {
	
	/** Override isValidDropPoint */
	isValidDropPoint: function(n, pt, dd, e, data) 
	{
		var isLeaf = n.node.leaf;
	 	var targetNode = n.node;
	 	if (isLeaf)
	 	{
	 		targetNode.leaf = false;	// Change leaf property to allow DD for leaf nodes
	 	}
	 	
	 	if (targetNode.attributes.isModifiable != 'true' || targetNode.attributes.canCreateChild  != 'true')
		{
			return false;
		}
	 	
	 	var result = org.ametys.explorer.TreeDropZone.superclass.isValidDropPoint.call(this, n, pt, dd, e, data);
	 	
 	 	return result;
	}
});

Expand comment: Laurence Aumeunier added a comment - 18/May/15 15:53

Laurence Aumeunier added a comment - 18/May/15 15:53 Certainement à cause du leaf = true pour les container/sous-programme n'ayant pas encore de fils. Forcer node.leaf = false; avant de valider le D&D comme cela est fait pour l'explorateur de ressources. org.ametys.explorer.TreeDropZone = Ext.extend(Ext.tree.TreeDropZone, { /** Override isValidDropPoint */ isValidDropPoint: function(n, pt, dd, e, data) { var isLeaf = n.node.leaf; var targetNode = n.node; if (isLeaf) { targetNode.leaf = false ; // Change leaf property to allow DD for leaf nodes } if (targetNode.attributes.isModifiable != ' true ' || targetNode.attributes.canCreateChild != ' true ' ) { return false ; } var result = org.ametys.explorer.TreeDropZone.superclass.isValidDropPoint.call( this , n, pt, dd, e, data); return result; } });

Laurence Aumeunier made changes - 18/May/15 15:48

Assignee

Original: Laurence Aumeunier [ laurence ]

New: Quentin Glinel-Mortreuil [ quentin ]

Laurence Aumeunier made changes - 18/May/15 10:44

Fix Version/s

New: 2.2.3 [ 12596 ]

Laurence Aumeunier made changes - 18/May/15 10:44

Fix Version/s

New: 2.4.0 [ 12390 ]

Laurence Aumeunier made changes - 18/May/15 10:44

Assignee

New: Laurence Aumeunier [ laurence ]

Laurence Perier created issue - 12/May/15 11:48