Problem/Motivation

Drupal::RECOMMENDED_PHP is still set at PHP 7.3, but it's going to be out of security support in December when Drupal 9.3 comes out.

Steps to reproduce

Proposed resolution

Set Drupal::RECOMMMENDED_PHP to PHP 7.4 in Drupal 9.2, so that sites get six months notice that they need to update.

This will also mean we only (or nearly only) have to worry about PHP 7.4 to PHP 8.0 updates when we're in 9.3/9.4/10.0 territory.

Remaining tasks

User interface changes

API changes

Data model changes

Release notes snippet

CommentFileSizeAuthor
#2 3214920.patch480 bytescatch
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

catch created an issue. See original summary.

catch’s picture

Status: Active » Needs review
FileSize
480 bytes
longwave’s picture

Status: Needs review » Reviewed & tested by the community

Totally makes sense, anyone still on 7.3 needs nudging to update.

Gábor Hojtsy’s picture

Agree this looks good :) https://www.drupal.org/docs/system-requirements/php-requirements will also need an update alongside this commit as it still says 7.3 is recommended.

  • 88a03ea committed on 9.3.x
    Issue #3214920 by catch, longwave, Gábor Hojtsy: Increase...

  • 9fdc3fc committed on 9.2.x
    Issue #3214920 by catch, longwave, Gábor Hojtsy: Increase...
Gábor Hojtsy’s picture

Status: Reviewed & tested by the community » Fixed
Issue tags: +Needs documentation updates

All right, committed to 9.3.x and cherry-picked to 9.2.x.

I went to update https://www.drupal.org/docs/system-requirements/php-requirements but noticed that it would need an update for Drupal 8.x as well. Went to check if this Drupal.php file is handling recommended PHP versions on Drupal 8 too but it does not seem to be doing that. Is there code in Drupal 8 to update for recommended PHP versions or should we just update the docs page?

catch’s picture

For Drupal 8 it's the DRUPAL_RECOMMENDED_PHP constant in bootstrap.inc, which is otherwise identical.

I think we can definitely update the docs page without updating Drupal 8.9. Mostly we are only backporting major bugs (or worse) to Drupal 8.9, but maybe as essentially a security hardening issue this might be eligible.

Gábor Hojtsy’s picture

Status: Fixed » Closed (fixed)

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