Saves 20 KB on most requests.

CommentFileSizeAuthor
drupal7-sun.system-move-trigger.patch40.58 KBsun

Comments

catch’s picture

catch
he/him
Primary language English
commented

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

sun’s picture

sun
Primary language German
Location Karlsruhe
commented

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

In short: Doesn't matter.

catch’s picture

catch
he/him
Primary language English
commented
Status: Needs review » Reviewed & tested by the community

Thought as much.

Crell’s picture

Crell commented

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

catch’s picture

catch
he/him
Primary language English
commented
Status: Reviewed & tested by the community » Needs review

probably.

Status: Needs review » Needs work

The last submitted patch failed testing.

sun’s picture

sun
Primary language German
Location Karlsruhe
commented
Status: Needs work » Closed (duplicate)

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