Make it clear that 3.x requires PHP 8

Created on 9 February 2023, about 2 years ago

The 3.x version of this module is not compatible with PHP 7. It required at least PHP 8.

Reported by various people @ 🐛 Drupal\color_field\Plugin\Field\FieldFormatter\ColorFieldFormatterSwatch::getShape(): Return value must be of type array|string, Drupal\Core\StringTranslation\TranslatableMarkup Needs work .

The minimum that should be done is

  • Mention this on the project page
  • Add php: 8.0 to the info.yml file
  • Add "php": ">=8.0" to the composer json file
Feature request
Status

Active

Version

3.0

Component

Code

Created by

🇧🇪Belgium rp7

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

Comments & Activities

  • Issue created by @rp7
  • @rp7 opened merge request.
  • 🇧🇪Belgium rp7

    For whomever is on PHP 7.x and has already upgraded to version 3.x of this module, but doesn't want to or can't downgrade for whatever reason: patch attached makes the 3.x version module compatible again with PHP 7.x.

  • 🇧🇪Belgium rp7

    Can't immediately see what's wrong with the MR. The composer.json change loosk valid to me - but I'm probably missing something very obvious. Insights are welcome!

  • 🇫🇷France sashainparis Provins

    There is no composer.json in the patch.
    Might be that.

  • 🇧🇪Belgium rp7

    @sashainparis
    As mentioned in my comment in #3, the patch is only for people running on PHP 7.x
    It's the MR that is failing.

  • Status changed to Closed: outdated 7 days ago
  • 🇨🇦Canada nickdickinsonwilde Victoria, BC (T'So-uke lands)

    At this point considering this issue dead. PHP 7.4 was EOL before this issue was opened and no version of Drupal 10 has supported (officially) under PHP 8.1.
    Might've been a sensible thing to do at the time due to some people running PHP 7.4 past end of life.

Production build 0.71.5 2024