• Type: Bug
• Resolution: Fixed
• Priority: Blocker
• Fix Version/s: 3.0 M4
• Affects Version/s: 3.0 M3
• Component/s: Plugin : Web
• Labels:

  None

[CMS-1226] Unable to add a page on the English site

Collapse comment: Raphaël Franchet added a comment - 02/Dec/09 11:59

Raphaël Franchet added a comment - 02/Dec/09 11:59

node.setId

will be fine

Expand comment: Raphaël Franchet added a comment - 02/Dec/09 11:59

Raphaël Franchet added a comment - 02/Dec/09 11:59 node.setId will be fine

Collapse comment: Laurence Aumeunier added a comment - 02/Dec/09 11:45

Laurence Aumeunier added a comment - 02/Dec/09 11:45

The root node of a tree is invariable.

To fix it, need to override the setRootNode method to manually unregister the old root node :

org.ametys.web.tool.sitemap.SitemapTreePanel.prototype.setRootNode = function (node)
{
	var oldRoot = this.root;
	
	node = org.ametys.web.tool.sitemap.SitemapTreePanel.superclass.setRootNode.call(this, node);
	
	// Unregister old root node if exists
	if (oldRoot != null)
		this.unregisterNode(oldRoot);
	
    return node;
}

Expand comment: Laurence Aumeunier added a comment - 02/Dec/09 11:45

Laurence Aumeunier added a comment - 02/Dec/09 11:45 The root node of a tree is invariable. To fix it, need to override the setRootNode method to manually unregister the old root node : org.ametys.web.tool.sitemap.SitemapTreePanel.prototype.setRootNode = function (node) { var oldRoot = this.root; node = org.ametys.web.tool.sitemap.SitemapTreePanel.superclass.setRootNode.call(this, node); // Unregister old root node if exists if (oldRoot != null) this.unregisterNode(oldRoot); return node; }

Collapse comment: Raphaël Franchet added a comment - 01/Dec/09 21:52

Raphaël Franchet added a comment - 01/Dec/09 21:52

org.ametys.web.tool.sitemap.SitemapTreePanel.prototype._reloadSite = function ()
{
	var rootNode = this.getRootNode();
	var rootID = this._getRootNodeId ();
	
	if (rootID != null)
	{
		rootNode.id = rootID;
		rootNode.reload ();
	}
}

You forgot to send the new selection event

Expand comment: Raphaël Franchet added a comment - 01/Dec/09 21:52

Raphaël Franchet added a comment - 01/Dec/09 21:52 org.ametys.web.tool.sitemap.SitemapTreePanel. prototype ._reloadSite = function () { var rootNode = this .getRootNode(); var rootID = this ._getRootNodeId (); if (rootID != null ) { rootNode.id = rootID; rootNode.reload (); } } You forgot to send the new selection event