The button in update.php that says "apply pending updates" should be "Apply pending updates" instead.

CommentFileSizeAuthor
#4 371886.002.patch0 byteskarschsp
#3 371886.001.patch518 byteskarschsp
#1 371886-update-php-capitalization.patch1.7 KBdamien tournoud

Comments

damien tournoud’s picture

damien tournoud commented
Status: Active » Needs review
StatusFileSize
new 371886-update-php-capitalization.patch1.7 KB

Free patch courtesy of the drupal.org sprint in Paris.

keith.smith’s picture

keith.smith commented
Status: Needs review » Needs work

This looks good, except for some unrelated search module stuff in there. (At least, I believe it to be unrelated.)

karschsp’s picture

karschsp commented
Assigned: Unassigned » karschsp
Status: Needs work » Needs review
StatusFileSize
new 371886.001.patch518 bytes

Patch attached.

karschsp’s picture

karschsp commented
StatusFileSize
new 371886.002.patch0 bytes

Quick re-roll here, sorry about that.

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
commented
Status: Needs review » Fixed

Committed #3 to HEAD. Thanks! Not sure what was up with #4, but it came back as 0 bytes.

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
commented

Unfortunately, this missed another instance in update.php which caused a WSOD. Fixed.

http://cvs.drupal.org/viewvc.py/drupal/drupal/update.php?r1=1.276&r2=1.277

Status: Fixed » Closed (fixed)
Issue tags: -Novice, -String freeze

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

Issue tags: +Novice, +String freeze

Restoring issue tags, see #2125755: System messages removed all issue tags during D7 upgrade.