Up-to-date modules in the "Available updates" report are given the style "color-error" instead of "color-success" which matches the behavior of Drupal 7, apparently due to a missing statement in the update module.

CommentFileSizeAuthor
#9 head-2546078.png30 KBalbertski
#9 patch-2546078.png30.45 KBalbertski
#2 drupal-fix_update_report_color-2546078-1.patch629 bytesravenstar

Comments

ravenstar created an issue. See original summary.

ravenstar’s picture

ravenstar commented
StatusFileSize
new drupal-fix_update_report_color-2546078-1.patch629 bytes
ravenstar’s picture

ravenstar commented
Status: Active » Needs review
albertski’s picture

albertski commented
Status: Needs review » Reviewed & tested by the community

I was able to replicate the issue. Code looks perfect and applying patch solves the issue!

Status: Reviewed & tested by the community » Needs work

The last submitted patch, 2: drupal-fix_update_report_color-2546078-1.patch, failed testing.

Status: Needs work » Needs review

albertski queued 2: drupal-fix_update_report_color-2546078-1.patch for re-testing.

albertski’s picture

albertski commented
Status: Needs review » Reviewed & tested by the community

Reran the patch and it is still good.

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
commented
Status: Reviewed & tested by the community » Needs work
Issue tags: +Needs screenshots

Before and after screenshots would be nice.

albertski’s picture

albertski commented
Status: Needs work » Reviewed & tested by the community
Issue tags: -Needs screenshots
StatusFileSize
new patch-2546078.png30.45 KB
new head-2546078.png30 KB

Below are screen shots of the modules section of the Available Updates page (which can be found at /admin/reports/updates) before and after the patch:

HEAD

HEAD Screenshot (before patch)

PATCH

Patch Screenshot (after patch)

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
commented
Status: Reviewed & tested by the community » Fixed

Committed c89df1c and pushed to 8.0.x. Thanks!

Thanks for adding the screenshots.

  • alexpott committed c89df1c on 8.0.x 
    Issue #2546078 by ravenstar, albertski: Update report shows up-to-date...

Status: Fixed » Closed (fixed)

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