The name of post_process is confusing to the extreme. Before more harm is done, let's rename it. (yay, my first critical task!)

CommentFileSizeAuthor
#5 after_build_3.patch3.72 KBchx
#4 after_build_2.patch3.71 KBRichard Archer
#3 after_build_1.patch3.68 KBRichard Archer
#1 after_build_0.patch2.95 KBchx
after_build.patch2.95 KBchx
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

chx’s picture

chx CreditAttribution: chx commented
FileSize
after_build_0.patch2.95 KB
adrian’s picture

adrian CreditAttribution: adrian commented

+1

a much better name

Richard Archer’s picture

Richard Archer CreditAttribution: Richard Archer commented
FileSize
after_build_1.patch3.68 KB

As discussed with chx, I have merged this patch with: http://drupal.org/node/37380

This patch now also removes two extraneous arguments from the post_process user function.

Richard Archer’s picture

Richard Archer CreditAttribution: Richard Archer commented
FileSize
after_build_2.patch3.71 KB

Keeping up with HEAD.

chx’s picture

chx CreditAttribution: chx commented
FileSize
after_build_3.patch3.72 KB

rerolled.

Dries’s picture

Dries CreditAttribution: Dries commented
Status: Reviewed & tested by the community » Fixed

Committed to HEAD. Thanks.

Anonymous’s picture

(not verified) CreditAttribution: commented
Status: Fixed » Closed (fixed)