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

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
commented
Status: Active » Needs review
StatusFileSize
new 2705097-server-verify-platforms.patch977 bytes
ergonlogic’s picture

ergonlogic
he/him
Primary language English
Location Montréal, Québec 🇨🇦
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 🇨🇦
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
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
commented
StatusFileSize
new 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 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.