Updated: Comment #N

Problem/Motivation

composer_manager_requirements() currently returns NULL if $phase is not runtime. Drupal expects an array as return value. This breaks the installer.

Proposed resolution

Change to return array().

Remaining tasks

User interface changes

API changes

CommentFileSizeAuthor
#1 composer-manager-requirements-2214607-1.patch366 bytesBerdir
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Berdir’s picture

Berdir
Primary language German
Location Switzerland
CreditAttribution: Berdir commented
Status: Active » Needs review
FileSize
composer-manager-requirements-2214607-1.patch366 bytes
cpliakas’s picture

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

Good catch. Thanks for the contribution!

cpliakas’s picture

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

Committed: http://drupalcode.org/project/composer_manager.git/commit/89ec7d7

Status: Fixed » Closed (fixed)

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