Environment Indicator needs to declare a dependency on the Color module.

CommentFileSizeAuthor
#2 2776573-environment_indicator-color.patch383 bytesAlgeron
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Algeron created an issue. See original summary.

Algeron’s picture

Algeron CreditAttribution: Algeron at XIO commented
FileSize
2776573-environment_indicator-color.patch383 bytes
Algeron’s picture

Algeron CreditAttribution: Algeron at XIO commented
Status: Active » Needs review
kyberman’s picture

kyberman
Location Czech Rep. 🇨🇿
CreditAttribution: kyberman commented
Status: Needs review » Reviewed & tested by the community

Thank you for a patch, this worked for me to fix "missing theme" warning.

Library "drupal.environment_indicator" has dependency for "color/drupal.color", so module should also declare dependency for "color" module.

dman’s picture

dman CreditAttribution: dman as a volunteer and at Sparks Interactive commented

Yeah. +1

I would prefer if it could have a SOFT dependency on color.module in a way that would avoid throwing that error message at me every cache-clear...
Because I have no real need for color module normally, and that coupling is too close for comfort.

But if it is required, then it must be declared

attheshow’s picture

attheshow CreditAttribution: attheshow at FleetThought commented

Enabling Color fixed the warning for me as well. Thanks!

  • e0ipso committed c5884eb on 8.x-3.x authored by Algeron 
    Issue #2776573 by Algeron: The following theme is missing from the file...
e0ipso’s picture

e0ipso
Primary language Catalan
Location Can Picafort
CreditAttribution: e0ipso as a volunteer and at Lullabot commented
Status: Reviewed & tested by the community » Fixed

Thanks all. As you can see, it's taking me too long to fix issues that the community has already fixed.

I am currently looking for co-maintainers, so if you want to help me take good care of this module please drop me an email using the contact form.

I'll be merging this asap.

Status: Fixed » Closed (fixed)

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