- Issue created by @joachim
- Merge request !12476Issue #3531636: fix the text in system_requirements → (Open) created by Unnamed author
- 🇺🇸United States smustgrave
I thought it was clear enough to begin with but this doesn't hurt anything.
- 🇬🇧United Kingdom joachim
> https://www.drupal.org/node/3443018 → "),
'severity' => REQUIREMENT_WARNING,I don't think we normally put bare URLs like that.
The URL is beyond the scope of this issue anyway.
- 🇬🇧United Kingdom joachim
@smustgrave it's currently completely wrong.
> Drupal 11 will default to having state cache enabled.
means that on D11 it will be enabled, unless you change a setting to make it disabled.
In fact, on D11 it's always enabled, and there is no setting to disable it.
- 🇺🇸United States xjm
Thanks everyone for working on this issue. I agree that the current string is somewhat misleading, and the suggested change would improve it.
That said, the release managers (@catc h, @quietone, and I) discussed and agreed this change does not meet our criteria for an allowed change in a maintenance minor → , which would be limited to:
- Certain critical or major bug fixes
- Critical API compatibility backports
- PHP and JavaScript dependency updates
- PHP version compatibility fixes
In this case, changing this string would require translators to also update all their translations of this string. We would only make such a break to a string in a maintenance minor if the string were actively harmful, such as telling the site owner to do something insecure.
Therefore, I am closing the issue as "won't fix" since Drupal 11 is not affected.
Thanks!