Patch will follow.

CommentFileSizeAuthor
#1 2120761-nodejs-install-1.patch2.37 KBamitaibu
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

amitaibu’s picture

amitaibu
Primary language Hebrew
Location Israel
CreditAttribution: amitaibu commented
Status: Active » Needs review
FileSize
2120761-nodejs-install-1.patch2.37 KB

Patch makes sure to return an empty array if needed.

Anonymous’s picture

Anonymous (not verified) CreditAttribution: Anonymous commented
Status: Needs review » Fixed

thanks, committed as:

http://drupalcode.org/project/nodejs.git/commit/7288b2e

Amitaibu - let me know if you want a commit bit.

amitaibu’s picture

amitaibu
Primary language Hebrew
Location Israel
CreditAttribution: amitaibu commented

> Amitaibu - let me know if you want a commit bit.

Thanks, but I'll pass for now. Having new twins at home nearly killed my contribution time... ;)

Status: Fixed » Closed (fixed)
Issue tags: +

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

Issue tags: +
  • Commit 7288b2e on 7.x-1.x, 8.x-1.x, 8.x-1.x-head authored by Amitaibu, committed by beejeebus: 
    #2120761 - Fix nodejs_requirements() returns wrong result, causing...