Shouldn't the hooks run by update. operate in the same order as other hooks, by module weight?

Patch adds ORDER BY clause where it is needed.

CommentFileSizeAuthor
#4 drupal_6_update_weight.diff879 bytesDave Cohen
#3 drupal_6_update_weight.diff850 bytesDave Cohen
drupal_6_update_weight.diff850 bytesDave Cohen
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Dave Cohen’s picture

Dave Cohen CreditAttribution: Dave Cohen commented

Sorry, in case it is not clear: file being patched is includes/install.inc.

Status: Needs review » Needs work

The last submitted patch, drupal_6_update_weight.diff, failed testing.

Dave Cohen’s picture

Dave Cohen CreditAttribution: Dave Cohen commented
Status: Needs work » Needs review
FileSize
drupal_6_update_weight.diff850 bytes

better patch.

Dave Cohen’s picture

Dave Cohen CreditAttribution: Dave Cohen commented
FileSize
drupal_6_update_weight.diff879 bytes

Sigh. Better better patch.

Status: Needs review » Needs work

The last submitted patch, drupal_6_update_weight.diff, failed testing.

Dave Cohen’s picture

Dave Cohen CreditAttribution: Dave Cohen commented
Status: Needs work » Needs review

#4: drupal_6_update_weight.diff queued for re-testing.

Status: Needs review » Needs work

The last submitted patch, drupal_6_update_weight.diff, failed testing.

Dave Cohen’s picture

Dave Cohen CreditAttribution: Dave Cohen commented

I give up. Can't get a one-line patch to apply.

But I still think it's a good idea!

Status: Needs work » Closed (outdated)

Automatically closed because Drupal 6 is no longer supported. If the issue verifiably applies to later versions, please reopen with details and update the version.