Download & Extend
Issues

Move system's trigger functionality away

Posted by sun on May 20, 2009 at 1:19am
Project:Drupal core
Version:7.x-dev
Component:system.module
Category:task
Priority:normal
Assigned:sun
Status:closed (duplicate)
Issue tags:Registry

Issue Summary

Saves 20 KB on most requests.

AttachmentSizeStatusTest resultOperations
drupal7-sun.system-move-trigger.patch40.58 KBIdleFailed: 11286 passes, 20 fails, 6 exceptionsView details | Re-test
Login or register to post comments

Comments

#1

Posted by catch on May 20, 2009 at 1:37am

CVS ID tags in the new file can't be right (not sure if this actually matters though).

#2

Posted by sun on May 20, 2009 at 2:05am

CVS matches and updates ID tags using this pattern: \$Id.*\$

In short: Doesn't matter.

#3

Posted by catch on May 20, 2009 at 2:06am
Status:needs review» reviewed & tested by the community

Thought as much.

#4

Posted by Crell on May 20, 2009 at 3:14am

Shouldn't it be system.actions.inc, since it's using the actions system which is not strictly dependent on the trigger module?

#5

Posted by catch on May 20, 2009 at 3:22am
Status:reviewed & tested by the community» needs review

probably.

#6

Posted by System Message on May 20, 2009 at 4:40am
Status:needs review» needs work

The last submitted patch failed testing.

#7

Posted by sun on May 22, 2009 at 6:09pm
Status:needs work» closed (duplicate)

Now part of #345118: Performance: Split .module files by moving hooks