Allow PHP 8.0 version in 2.x branch

Created on 4 April 2023, about 1 year ago
Updated 20 August 2023, 10 months ago

Problem/Motivation

In πŸ“Œ Drupal 10 compatibility Fixed the minimum PHP compatibility was set to PHP 8.1. That makes it a bit more complicated for folks wanting to prepare for an update to Drupal 10. PHP 8.0 is supported in Drupal 9.5. Is there a reason that the config_update 2.x branch cannot also support PHP 8.0?

✨ Feature request
Status

RTBC

Version

2.0

Component

Base module

Created by

πŸ‡ΊπŸ‡ΈUnited States jastraat

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

  • Issue created by @jastraat
  • πŸ‡ΊπŸ‡ΈUnited States FatherShawn New York

    Drupal core has supported PHP 8.1 since 9.3. You should be able to move to 8.1, running Drupal 9.5.x, upgrade your modules, then move to D10

  • πŸ‡©πŸ‡ͺGermany lukas_w

    I would support this request, since we have a customer running SLES which currently only provides PHP 8.0. I checked with php -l if there are any errors in the module's code with PHP 8.0 but none showed up.

  • Status changed to Needs review 10 months ago
  • Open in Jenkins β†’ Open on Drupal.org β†’
    Core: 9.5.x + Environment: PHP 7.4 & MySQL 5.7
    last update 10 months ago
    74 pass
  • πŸ‡³πŸ‡ΏNew Zealand RoSk0 Wellington

    Completely agree with request - if the module doesn't have strict requirements to use PHP 8.1 there is no reason to declare dependency on it.

    Also it would greatly simplify the upgrade path for the users.

    Can you please downgrade PHP requirement to 8.0? Patch attached.

    I can see that the test suite of the module currently passes on PHP 7.4, so declared 8.1 doesn't make any sense to me.

  • Open in Jenkins β†’ Open on Drupal.org β†’
    Core: 9.5.x + Environment: PHP 7.4 & MySQL 5.7
    last update 10 months ago
    74 pass
  • πŸ‡³πŸ‡ΏNew Zealand RoSk0 Wellington

    Updated UI submodule.

  • Status changed to RTBC 10 months ago
  • πŸ‡³πŸ‡ΏNew Zealand ericgsmith

    +1 for this.

    Patch applies cleanly - no issues found running on PHP8.0 and tests showing as green against 7.4

    While #2 is an ideal path, complete agree with the request that there is no reason to enforce this in this module.

Production build 0.69.0 2024