[development] Splitting hook_menu into hook_menu and hook_router

Darrel O'Pry dopry at thing.net
Tue Jan 16 23:32:36 UTC 2007


On Tue, 2007-01-16 at 23:04 +0100, Karoly Negyesi wrote:
> Hi,
> 
> We discussed renaming hook_menu and hook_router was a popular choice but  
> as Gerhard says "I don't like to rename funtions just for the sake of  
> renaming them" so it remained hook_menu .
> 
> However, looking at my brain dump at http://drupal4hu.com/node/28 tells me  
> that further down the road a split is needed.
> 
> hook_router has these things:
> 
> -- path as primary key. We use this as an array key, so an alter hook can  
> easily be provided.
> -- access callback and arguments
> -- page callback and arguments
> -- map callback and arguments
> 
> hook_menu has these:
> 
> -- path. This is not unique. It's often requested that one path could have  
> more than one visible link.
> -- parent. If you omit this, it can be autogenerated but for setting it to  
> some arbitrary value, let's say MENU_ROOT can be used to indicate a new  
> menu. With this (or via other means if someone has a better idea) we can  
> put links into any menu.
> -- title.
> -- weight.
> -- expanded.
> 
> With path being unique in one and not unique in the other, I fail to see  
> how the two could be unified. And I fail to see the point as well. We  
> could lose path being an array key and make it a property again in  
> hook_router as it was in old hook_menu but then what shall happen if  
> someone defines two router items to the same path? This is a mess. The  
> above is nice and clean.
> 
> Regards
> 
> NK


double plus good. We can remove silly locked menu items after this
split, no? What is the difference between page and map in hook_router?



More information about the development mailing list