In both 8.x-1.1 and the 8.x dev version, function "safeNodeServerVersion" is misspelled as "safeNodeSeverVerion" in both src/nodejs.php and nodejs.install

In addition, the bellow snippet of code in safeNodeServerVersion:

    return $this->nodeServerVersion && $this->nodeServerVersion >= SAFE_NODE_SERVER_VERSION;

should read:

    return $this->nodeServerVersion && $this->nodeServerVersion >= self::SAFE_NODE_SERVER_VERSION;
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Erica_der_Whatkin created an issue. See original summary.

jalpesh’s picture

Status: Active » Needs review
FileSize
1.15 KB

Yup, you are right. I have check both file function name is misspelled.

  • beejeebus committed a20f714 on 8.x-1.x authored by jalpesh
    Issue #2761093 by jalpesh, Erica_der_Whatkin: safeNodeServerVersion...
Anonymous’s picture

Status: Needs review » Fixed

thanks! committed and pushed.

Status: Fixed » Closed (fixed)

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