The documentation of update_manager_archive_extract() in update.manager.inc implies that it should take project name $project as an argument, but it doesn't in the current implementation.

See #538660: Move update manager upgrade process into new authorize.php file (and make it actually work) and #1019834: No .info files reported on theme upload when zip created with certain programs for the original documentation history and project name auto-discovery, respectively.

We ran into this because of a malformed tarball where the project name wasn't easily deduced from the archive contents. The function is called by another module (the openpublish install profile).

In most circumstances when invoking this function in code (as is the case with openpublish) you generally have the $project already defined, so there isn't a need to automatically determine the name.

In cases like #1019834: No .info files reported on theme upload when zip created with certain programs where someone uploads a project to be installed, there may be a need for the automated stuff.

Comments

dooug’s picture

Issue tags: +Documentation
dooug’s picture

Issue summary: View changes

fix project node links

Version: 7.9 » 7.x-dev

Core issues are now filed against the dev versions where changes will be made. Document the specific release you are using in your issue comment. More information about choosing a version.

Status: Active » Closed (outdated)

Automatically closed because Drupal 7 security and bugfix support has ended as of 5 January 2025. If the issue verifiably applies to later versions, please reopen with details and update the version.

quietone’s picture