Problem/Motivation

As per documentation https://www.drupal.org/docs/8/upgrade/upgrade-using-web-browser
Failed upgrade count should be error message, but its showing as success message. refere Screen Shot 2018-08-25 at 11.30.08 AM.png

Steps:

  1. Upgrade Drupal8 as https://www.drupal.org/docs/8/upgrade/upgrade-using-web-browser
  2. Failures are shown in success message

Proposed resolution

Replace success with error. refer
Screen Shot 2018-08-25 at 11.31.00 AM.png

Remaining tasks

Submit patch for the same

User interface changes

NA

API changes

NA

CommentFileSizeAuthor
#2 2995033-02.patch833 bytesvj
Screen Shot 2018-08-25 at 11.31.00 AM.png50.78 KBvj
Screen Shot 2018-08-25 at 11.30.08 AM.png53.8 KBvj

Comments

Vj created an issue. See original summary.

vj’s picture

vj commented
Status: Active » Needs review
StatusFileSize
new 2995033-02.patch833 bytes

Add patch to resolve issue

prashantgajare’s picture

prashantgajare commented

Good catch! @vj +1 LGTM

xano’s picture

xano
he/they
Primary language English
Location Southampton
commented
Component: database update system » migration system
Priority: Normal » Minor
Status: Needs review » Reviewed & tested by the community

Hah, that's an embarrassing bug! Thanks for submitting a patch :)

maxocub’s picture

maxocub commented
Status: Reviewed & tested by the community » Closed (duplicate)

This is a duplicate of #2974654: Migrate upgrade shows errors in green. The other issue is tagged as 'Needs screenshots' to show the result after the patch is applied. Since you already have that screenshot here, please post it on the other issue so you will get credited for your work. Thanks!