Hi,

The following seems to work (leaving the refresh action):

  node.Menu.RemoveRange(0, node.Menu.Count - 1);
  node.Menu.Add(ActionDelete.Instance);

Still curious as to the best way of checking the docTypeAlias of the current node.

TIA,

Hendy

Hi Richard,

Thanks for confirming that, I'll update the WIKI.

There's the node.TreeType property that also returns 'content', so would have thought node.NodeType would be a sensible place to return the docTypeAlias, although appreciate that could break existing code.

Thanks,

Hendy

I followed the advice in the wiki (it actually had an error in it that i fixed) and things work as expected when i right-click on my admin tree, ... but now am getting NullReferenceException error when spawning a Content Picker. Here's what is coming back from the TreeDataService.ashx file...

[NullReferenceException: Object reference not set to an instance of an object.]
   MyCustomAppBase.BaseTree_BeforeNodeRender(XmlTree& sender, XmlTreeNode& node, EventArgs e) +150
   umbraco.cms.presentation.Trees.BeforeNodeRenderEventHandler.Invoke(XmlTree& sender, XmlTreeNode& node, EventArgs e) +0
   umbraco.cms.presentation.Trees.BaseTree.OnBeforeNodeRender(XmlTree& sender, XmlTreeNode& node, EventArgs e) +30
   umbraco.cms.presentation.Trees.BaseContentTree.Render(XmlTree& Tree) +191
   umbraco.loadContent.Render(XmlTree& tree) +18
   umbraco.presentation.webservices.TreeDataService.LoadTree(TreeRequestParams treeParams) +77
   umbraco.presentation.webservices.TreeDataService.GetXmlTree() +140
   umbraco.presentation.webservices.TreeDataService.ProcessRequest(HttpContext context) +55
   System.Web.CallHandlerExecutionStep.System.Web.HttpApplication.IExecutionStep.Execute() +100
   System.Web.HttpApplication.ExecuteStep(IExecutionStep step, Boolean& completedSynchronously) +75

Has anyone had any recent experience working with this, or have any advice for something i might need to do differently? I am thinking I might need to adjust my code to first see if something called ActionDelete.Instance actually exists in node.Menu before just assuming to call Remove?

Check if node.Menu not is null. If it's null it's a treepicker and then you shouldn't remove the context menu's since there is no menu.

Hope this helps,

Richard

Thank you, Richard!  I tested this and you are exactly right. 

I went ahead and updated the wiki page again with this fix and a note.  On that same topic, as you'll see I also added to the bottom of the wiki page, is a note about ApplicationBase being deprecated in favor of umbraco.businesslogic.ApplicationStartupHandler, according the note in the source code? It seems they are mostly interchangeable from my own limited interaction, but for posterity and for anyone looking to do more with their menus, looking into these differences would probably be advised.

Thanks to everyone for this golden thread!

I know this is quite an old thread but I kept coming back here when looking for the answer to my question: how can I completely hide a menu item programmatically. The code above states;

node.Menu.Clear();// seems to stop node from being rendered

This didn't work for me using 4.11.1, but I found the following works just fine;

private void BaseTree_BeforeNodeRender(ref XmlTree sender, ref XmlTreeNode node, EventArgs e) {
    node = null;
}

Just to clarify for posterity, Robert, ... are you trying to hide the entire node, the entire context menu, or just an item in the context menu? The code snippet you posted seems like it will enitrely destroy all trees, correct?

Hi,

I know this task is been open a while but for anyone looking to remove menu items in Umbraco 7 then here it is.

private void ContentTreeController_MenuRendering(TreeControllerBase sender, MenuRenderingEventArgs e)
        {
            //remove unwanted menu items
            e.Menu.Items.Remove(e.Menu.Items.Where(x => x.Alias == "notify").FirstOrDefault());
            e.Menu.Items.Remove(e.Menu.Items.Where(x => x.Alias == "sendToTranslate").FirstOrDefault());
            e.Menu.Items.Remove(e.Menu.Items.Where(x => x.Alias == "rollback").FirstOrDefault());
            e.Menu.Items.Remove(e.Menu.Items.Where(x => x.Alias == "auditTrail").FirstOrDefault());
        }

is working on a reply...