Download & Extend
Issues

Re-order functions in trigger.install

Posted by deekayen on August 4, 2009 at 7:14am
Project:Drupal core
Version:7.x-dev
Component:trigger.module
Category:task
Priority:minor
Assigned:Unassigned
Status:closed (fixed)
Issue tags:coding standards, Novice

Issue Summary

As part of breaking up #263956: Ye Grande Olde Drupal 7 .install File Cleanup, this moves the functions to be in order of schema, install, uninstall. No code was changed, just moved.

AttachmentSizeStatusTest resultOperations
trigger_install_cleanup.patch1.2 KBIdleFailed: Failed to apply patch.View details
Login or register to post comments

Comments

#1

Posted by boombatower on August 5, 2009 at 4:51am
Status:needs review» reviewed & tested by the community

Good to see this happening.

#2

Posted by System Message on September 10, 2009 at 9:30am
Status:reviewed & tested by the community» needs work

The last submitted patch failed testing.

#3

Posted by lambic on September 21, 2009 at 6:39pm
Status:needs work» needs review

rerolled patch

AttachmentSizeStatusTest resultOperations
trigger_install_cleanup.patch959 bytesIdlePassed: 14486 passes, 0 fails, 0 exceptionsView details

#4

Posted by andypost on September 25, 2009 at 8:00pm
Status:needs review» reviewed & tested by the community

more comfortable to read

#5

Posted by webchick on November 2, 2009 at 2:39am
Priority:normal» minor
Status:reviewed & tested by the community» fixed

Committed to HEAD. Thanks!

#6

Posted by System Message on November 16, 2009 at 2:40am
Status:fixed» closed (fixed)

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