Problem/Motivation

#3182405: Do not use verb "Install" for things other than turning on modules/themes changed the link on update.php to direct to a Drupal 7 page. That was committed to 10.1.0. I discovered while working on #3414666: Improve text on database update page

Setting 'critical' because this is a significant regressions in user experience, developer experience, or documentation.

Steps to reproduce

Proposed resolution

Remaining tasks

Revert the link part of this change

-      '#markup' => '<p>' . $this->t('Use this utility to update your database whenever a new release of Drupal or a module is installed.') . '</p><p>' . $this->t('For more detailed information, see the <a href="https://www.drupal.org/docs/updating-drupal">Updating Drupal guide</a>. If you are unsure what these terms mean you should probably contact your hosting provider.') . '</p>',
+      '#markup' => '<p>' . $this->t('Use this utility to update your database whenever a module, theme, or the core software is updated.') . '</p><p>' . $this->t('For more detailed information, see the <a href="https://www.drupal.org/upgrade">upgrading handbook</a>. If you are unsure what these terms mean you should probably contact your hosting provider.') . '</p>',
     ];

User interface changes

Introduced terminology

API changes

Data model changes

Release notes snippet

Issue fork drupal-3534893

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

Comments

quietone created an issue. See original summary.

quietone’s picture

Issue summary: View changes
quietone’s picture

Status: Active » Needs review
smustgrave’s picture

Status: Needs review » Reviewed & tested by the community

Seems straight forward and verified the link

  • longwave committed d731332e on 10.5.x
    Issue #3534893 by quietone: [regression] Wrong doc link on database...

  • longwave committed d1db7dc0 on 10.6.x
    Issue #3534893 by quietone: [regression] Wrong doc link on database...

  • longwave committed fee3dc83 on 11.2.x
    Issue #3534893 by quietone: [regression] Wrong doc link on database...

  • longwave committed 054568a1 on 11.x
    Issue #3534893 by quietone: [regression] Wrong doc link on database...
longwave’s picture

Status: Reviewed & tested by the community » Fixed

Backported down to 10.5.x as a critical regression, this only affects the link and not the translated string.

Committed and pushed 054568a1f05 to 11.x and fee3dc830b3 to 11.2.x and d1db7dc0cb6 to 10.6.x and d731332ecfc to 10.5.x. Thanks!

longwave’s picture

Version: 11.x-dev » 10.5.x-dev

Status: Fixed » Closed (fixed)

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