In update_manager_archive_verify(), the failure exception refers to $file, but there is no variable $file defined. The variable is actually $archive_file. Patch attached.

CommentFileSizeAuthor
update_manager_archive_verify.patch820 bytesgdd
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

gdd’s picture

gdd
he/him
Primary language English
Location Portland, OR
CreditAttribution: gdd commented
Status: Active » Needs review
aspilicious’s picture

aspilicious CreditAttribution: aspilicious commented
Status: Needs review » Reviewed & tested by the community

Nice catch!
....

Easy fix :)
Thnx!

This is rtbc.... no doubts about that...

Dries’s picture

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

Good catch. Committed to CVS HEAD. Thanks.

Status: Fixed » Closed (fixed)
Issue tags: -Update manager

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

Issue tags: +Update manager

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