Problem/Motivation

After you fix a .info.yml file, unless you rebuild your cache (and clear the extension cache), upgrade_status still reports a warning.

Proposed resolution

Read the .info.yml files directly and bypass the extension cache.

Remaining tasks

  1. Do it.
  2. Review.
  3. Commit.

User interface changes

Less false positives.

API changes

N/A

Data model changes

N/A

Release notes snippet

TBD

CommentFileSizeAuthor
#2 3129810.patch2.94 KBGábor Hojtsy
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

dww created an issue. See original summary.

Gábor Hojtsy’s picture

Gábor Hojtsy
he/him
Primary language Hungarian
Location Hungary
CreditAttribution: Gábor Hojtsy at Acquia commented
FileSize
3129810.patch2.94 KB
Gábor Hojtsy’s picture

Gábor Hojtsy
he/him
Primary language Hungarian
Location Hungary
CreditAttribution: Gábor Hojtsy at Acquia commented
Status: Active » Needs review

Status: Needs review » Needs work

The last submitted patch, 2: 3129810.patch, failed testing. View results
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.

Gábor Hojtsy’s picture

Gábor Hojtsy
he/him
Primary language Hungarian
Location Hungary
CreditAttribution: Gábor Hojtsy at Acquia commented
Status: Needs work » Needs review

I could not reproduce locally, neither did the simpletest output on the testbot show what the fail said. When rerunning two more times, it worked. So I'll write this down as a test fluke.

Gábor Hojtsy credited DamienMcKenna.

Gábor Hojtsy’s picture

Gábor Hojtsy
he/him
Primary language Hungarian
Location Hungary
CreditAttribution: Gábor Hojtsy at Acquia commented

Crediting Damien for reporting the same issue on slack yesterday.

lauriii’s picture

lauriii
he/him
Primary language Finnish
Location Finland
CreditAttribution: lauriii at Acquia commented
Status: Needs review » Reviewed & tested by the community

+1 for this change. I run into this same issue when I was assessing a module for Drupal 9 compatibility. Reviewed the patch and all looks good! 👏

Gábor Hojtsy’s picture

Gábor Hojtsy
he/him
Primary language Hungarian
Location Hungary
CreditAttribution: Gábor Hojtsy at Acquia commented
Title: Bypass extension cache and read info.yml files directly » Bypass extension cache and read info.yml files directly to facilitate a smoother development workflow
Gábor Hojtsy’s picture

Gábor Hojtsy
he/him
Primary language Hungarian
Location Hungary
CreditAttribution: Gábor Hojtsy at Acquia commented
Status: Reviewed & tested by the community » Fixed

  • Gábor Hojtsy committed 52d4d86 on 8.x-2.x 
    Issue #3129810 by Gábor Hojtsy, dww, DamienMcKenna, lauriii: Bypass...
heddn’s picture

heddn
Primary language English
Location Nicaragua
CreditAttribution: heddn at MTech, LLC commented

+1 to this fix. I had seen it earlier and didn't report it myself.

Status: Fixed » Closed (fixed)

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