Improve experimental theme/module installation warning

Created on 9 October 2019, over 5 years ago
Updated 8 June 2025, about 12 hours ago

Problem/Motivation

\Drupal\system\Form\ModulesListConfirmForm::buildMessageList() contains:

$items[] = $this->formatPlural(count($dependencies), 'You must enable the @required module to install @module.', 'You must enable the @required modules to install @module.', [

and
\Drupal\system\Form\ThemeExperimentalConfirmForm::buildForm() contains:

$items[] = $this->formatPlural(count($dependencies), 'You must enable the @required theme to install @theme.', 'You must enable the @required themes to install @theme.', [

This results in:

You must enable the foo, bar, baz themes to install Claro.

This is awkward/bad English.

Proposed resolution

Improve to:

You must enable the foo, bar and baz themes to install Claro.

Remaining tasks

User interface changes

API changes

Data model changes

Release notes snippet

πŸ“Œ Task
Status

Active

Version

11.0 πŸ”₯

Component

extension system

Created by

πŸ‡§πŸ‡ͺBelgium wim leers Ghent πŸ‡§πŸ‡ͺπŸ‡ͺπŸ‡Ί

Live updates comments and jobs are added and updated live.
  • Novice

    It would make a good project for someone who is new to the Drupal contribution process. It's preferred over Newbie.

Sign in to follow issues

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

Production build 0.71.5 2024