Drupal 6 has a new file inclusion feature for menu items, I hope a future release would make use of this to improve memory usage. Thanks for this module!

Comments

atuyo60’s picture

atuyo60 commented
Title: Implement D6 menu/theme include file handlers » Page handlers and uninstall?

I am also wondering if an uninstall should be included to remove this module cleanly? Editing of comments would be nice too...

Just a note, I modified the code to utilise page handlers for both the menu and theme hooks and this gave me at least a 50% memory reduction. Given that this module is quite large (~200,000 bytes) as it is, using page handlers would help those of us with PHP memory limitations. It also makes reading the code easier. My modification only separates the functions called by menu and theme hooks, there may be other functions used by these functions that could also be moved to the new file but I am not very familiar with the code to decide what can be moved... so I hope an official release does that.

http://drupal.org/node/146172

sun’s picture

sun
Primary language German
Location Karlsruhe
commented
Title: Page handlers » Implement D6 menu/theme include file handlers

Yes, the current port for D6 is still very basic. However, I'll implement this probably for 6.x-2.x only.
Providing a hook_uninstall() should be filed as a separate issue.

sun’s picture

sun
Primary language German
Location Karlsruhe
commented
Title: Page handlers and uninstall? » Implement D6 menu/theme include file handlers
Status: Active » Closed (works as designed)

Something like this will be part of the port to D7.