hosting_server_post_hosting_verify_task() does a query on hosting_platform, which might not be enabled.

Going to add a "module_exists" check before calling it.

CommentFileSizeAuthor
#8 2705097-server-verify-platforms.patch1.54 KBJon Pugh
#3 2705097-server-verify-platforms.patch977 bytesJon Pugh
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Jon Pugh created an issue. See original summary.

  • Jon Pugh committed 9616319 on 2705097-server-verify-platforms 
    Issue #2705097: Check for hosting_platform before querying...
Jon Pugh’s picture

Jon Pugh
he/him
Primary language English
Location Newburgh, NY
CreditAttribution: Jon Pugh at OpenDevShop Inc (CLOSED) commented
Status: Active » Needs review
FileSize
2705097-server-verify-platforms.patch977 bytes
ergonlogic’s picture

ergonlogic
he/him
Primary language English
Location Montréal, Québec 🇨🇦
CreditAttribution: ergonlogic at Ergon Logic Enterprises for Aegir Cooperative commented

Shouldn't we just move that to hosting_platform_post_hosting_verify_task()?

ergonlogic’s picture

ergonlogic
he/him
Primary language English
Location Montréal, Québec 🇨🇦
CreditAttribution: ergonlogic at Ergon Logic Enterprises for Aegir Cooperative commented
Status: Needs review » Needs work

  • Jon Pugh committed 0dd6ce1 on 2705097-server-verify-platforms 
    Progress on Issue #2705097: Move code that triggers platform verifies to...
Jon Pugh’s picture

Jon Pugh
he/him
Primary language English
Location Newburgh, NY
CreditAttribution: Jon Pugh at OpenDevShop Inc (CLOSED) commented
Status: Needs work » Needs review

Good call I've made the fix

Jon Pugh’s picture

Jon Pugh
he/him
Primary language English
Location Newburgh, NY
CreditAttribution: Jon Pugh at OpenDevShop Inc (CLOSED) commented
FileSize
2705097-server-verify-platforms.patch1.54 KB

  • helmo committed 5d328b5 on 7.x-3.x authored by Jon Pugh 
    Issue #2705097 by Jon Pugh: Check for hosting_platform before querying...
helmo’s picture

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

There was a missing } in platform/hosting_platform.drush.inc ... but after adding that it looks good.... committed.

Status: Fixed » Closed (fixed)

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