Implement D6 menu/theme include file handlers
atuyo60 - August 4, 2008 - 11:27
| Project: | Guestbook |
| Version: | 6.x-1.x-dev |
| Component: | Code |
| Category: | feature request |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | active |
Jump to:
Description
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!
#1
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
#2
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.