Split admin functions into their own file

Freso - May 7, 2008 - 08:16
Project:Pathauto
Version:6.x-1.x-dev
Component:Code
Category:task
Priority:normal
Assigned:Unassigned
Status:closed
Description

For speed increase and load reduction. PATCH NOT TESTED. Be careful.

AttachmentSize
pathauto_admin_split.d6.patch37.09 KB

#1

Freso - May 8, 2008 - 15:41

Fixed some Doxygen but otherwise not much (if anything) has changed since last. Will test now.

AttachmentSize
pathauto_admin_split.d6.patch 37.21 KB

#2

Freso - May 8, 2008 - 15:48

Okay. Both admin pages work. I'll commit this later if noone cries foul. If there are missing functions that could be moved, they can still be moved after this patch has gone in (unless I'm told before committing the code ;)).

#3

greggles - May 8, 2008 - 16:38

To any other testers - be sure to clear the cache after installing this. Fortunately there is that table deletion which is in between 6.x-1.0 and 6.x-1.1 so that people will need to visit update.php which I think would clear the menu cache...

Otherwise, it worked for me and seems good to me.

#4

Freso - May 8, 2008 - 17:51
Status:needs review» fixed

Committed! Another victory for the code load reducteurs! :)

#5

Anonymous (not verified) - May 22, 2008 - 17:51
Status:fixed» closed

Automatically closed -- issue fixed for two weeks with no activity.

 
 

Drupal is a registered trademark of Dries Buytaert.