In the hosting_package_sync() function of the hosting_package.module file, it looks like the name of the $package variable is mis-typed in one of the conditionals:

function hosting_package_sync(&$data) {
...
      // we only call node save when the title, description changes
      // or when it's a new package.
      if (!$package->nid || 
        ($package->title != $name) || 
        ($package->description != $file['info']['description']) ||
	($pacakge->old_short_name != $file['info']['old_short_name'])) {  // TYPO
...
}

Attached is a patch which should fix this issue.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

GuyPaddock’s picture

Status: Active » Needs review

Setting to "needs review".

Steven Jones’s picture

Status: Needs review » Fixed

Thanks very much for the fix, pushed to all branches.

GuyPaddock’s picture

Thanks, again.

Status: Fixed » Closed (fixed)

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

  • Commit b00ab36 on 7.x-2.x, dev-ssl-ip-allocation-refactor, dev-1205458-move_sites_out_of_platforms, 7.x-3.x, dev-588728-views-integration, dev-1403208-new_roles, dev-helmo-3.x authored by GuyPaddock, committed by Steven Jones:
    Issue #1273468: Fixes Typo in variable name inside hosting_package_sync...

  • Commit b00ab36 on 7.x-2.x, dev-ssl-ip-allocation-refactor, dev-1205458-move_sites_out_of_platforms, 7.x-3.x, dev-588728-views-integration, dev-1403208-new_roles, dev-helmo-3.x authored by GuyPaddock, committed by Steven Jones:
    Issue #1273468: Fixes Typo in variable name inside hosting_package_sync...