Hello,

Here is a follow up of the issue #2832155: Modules not detected as enabled on D8 site.

Now that the previous issue is closed, enabled modules/themes/profile are well detected on Drupal 8 site.

Now we need to detect disabled modules/themes/profiles.

I will try to provide a patch for that. It will change the _provision_system_query() in provision/platform/drupal/packages_8.inc

I found for modules and themes. I am searching for profiles how D8 detects its.

CommentFileSizeAuthor
#3 provision-d8_package_disabled-2850164-3.patch1.68 KBGrimreaper
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
Parent issue: » #2832155: Modules not detected as enabled on D8 site
Grimreaper’s picture

Grimreaper
Primary language French
Location France 🇫🇷
CreditAttribution: Grimreaper at Smile commented
Assigned: Grimreaper » Unassigned
Status: Active » Needs review
FileSize
provision-d8_package_disabled-2850164-3.patch1.68 KB

Hello,

As a d8 site has only one profile activated at a time, and as _provision_system_query() function is only for modules and themes, here is a patch that fix the problem.

Thanks for the review.

Grimreaper’s picture

Grimreaper
Primary language French
Location France 🇫🇷
CreditAttribution: Grimreaper at Smile commented

The only "problem" I see now is that test modules/themes (more globally the hidden modules/themes) are not detected on packages for a site but are detected on packages for a platform.

helmo’s picture

helmo CreditAttribution: helmo at Initfour websolutions commented
Status: Needs review » Reviewed & tested by the community

looks ok

  • helmo committed 3af8f88 on 7.x-3.x authored by Grimreaper 
    Issue #2850164 by Grimreaper: Detect disabled module on Drupal 8 sites
helmo’s picture

helmo CreditAttribution: helmo at Initfour websolutions commented
Status: Reviewed & tested by the community » Closed (duplicate)

Committed.

Grimreaper’s picture

Grimreaper
Primary language French
Location France 🇫🇷
CreditAttribution: Grimreaper at Smile commented

Thanks @helmo for the commit.

colan’s picture

colan
he/him
Primary language English
Location Toronto 🇨🇦
CreditAttribution: colan at Colan Schwartz Consulting commented
Status: Closed (duplicate) » Fixed

Status: Fixed » Closed (fixed)

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