Hello,

As discussed on IRC.

I found in http://cgit.drupalcode.org/hosting/tree/package/hosting_package.module#n41 a wrong implementation of the hook_node_load().

Maybe due to D6 legacy code.

I will provide a patch to remove it.

CommentFileSizeAuthor
#3 remove-2832161-3.patch1.42 KBhelmo
#2 hosting-remove_hosting_package_node_load-2832161-2.patch983 bytesGrimreaper
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Grimreaper created an issue. See original summary.

Grimreaper’s picture

Grimreaper
Primary language French
Location France 🇫🇷
CreditAttribution: Grimreaper at Smile commented
Assigned: Grimreaper » Unassigned
Status: Active » Needs review
FileSize
hosting-remove_hosting_package_node_load-2832161-2.patch983 bytes

And here is the patch.

Thanks for the review.

helmo’s picture

helmo CreditAttribution: helmo at Initfour websolutions for Aegir Cooperative commented
FileSize
remove-2832161-3.patch1.42 KB

I also found hosting_client_node_load() which looks equally obsolete.

Testing on https://travis-ci.org/aegir-project/hosting/builds/182229490

  • helmo committed a11c386 on 7.x-3.x authored by Grimreaper 
    Issue #2832161 by Grimreaper, helmo: Remove hosting_package_node_load()
helmo’s picture

helmo CreditAttribution: helmo at Initfour websolutions for Aegir Cooperative commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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