Home » Download » Drupal project » Drupal » Issues

Re-order functions in trigger.install

deekayen - August 4, 2009 - 07:14
Project:Drupal
Version:7.x-dev
Component:trigger.module
Category:task
Priority:minor
Assigned:Unassigned
Status:closed
Issue tags:coding standards, Novice
Description

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 | Re-test
» Login or register to post comments

#1

boombatower - August 5, 2009 - 04:51
Status:needs review» reviewed & tested by the community

Good to see this happening.

Login or register to post comments

#2

System Message - September 10, 2009 - 09:30
Status:reviewed & tested by the community» needs work

The last submitted patch failed testing.

Login or register to post comments

#3

lambic - September 21, 2009 - 18:39
Status:needs work» needs review

rerolled patch

AttachmentSizeStatusTest resultOperations
trigger_install_cleanup.patch959 bytesIdlePassed: 14486 passes, 0 fails, 0 exceptionsView details | Re-test
Login or register to post comments

#4

andypost - September 25, 2009 - 20:00
Status:needs review» reviewed & tested by the community

more comfortable to read

Login or register to post comments

#5

webchick - November 2, 2009 - 02:39
Priority:normal» minor
Status:reviewed & tested by the community» fixed

Committed to HEAD. Thanks!

Login or register to post comments

#6

System Message - November 16, 2009 - 02:40
Status:fixed» closed

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

Login or register to post comments
 
 

Drupal is a registered trademark of Dries Buytaert.