Skip to content

[Meta] PHP 8.4 support #832

Open
Open

Description

@millnut

This issue will help us track LocalGov Drupal compatibility with PHP 8.4.

Currently the core project https://github.com/localgovdrupal/localgov_project and its included localgov_* modules support PHP 8.4 (with the upcoming D11 release), however a small number of contrib modules do not support PHP 8.4 so we cannot yet state full compatibility.

These affected modules are;

  • drupal/address

  • drupal/entity_hierarchy

    • We use 3.3, there are no tickets/fixes for this release. Only the 5.0.0-alpha6 has support for PHP 8.4, however this release has some big MySQL/MariaDB requirements

The 5.x branch makes use of Common Table Expressions (CTEs) and requires Drupal 10 or higher. IF you use MySQL you will need 8.0 or greater. The minimum versions of PostgreSQL and SQLite already support CTEs. It is currently in alpha version.

I've created an issue ticket with MR, so hopefully this can be merged into 3.x https://www.drupal.org/project/entity_hierarchy/issues/3511660

Activity

  1. millnut commented on Mar 7, 2025

    @millnut
    MemberAuthor

    @finnlewis @stephen-cox @ekes latest on the PHP 8.4 contrib based on the project failing tests

  2. pinned this issue on Mar 7, 2025
  3. stephen-cox commented on Jun 24, 2025

    @stephen-cox
    Member

    @tonypaulbarker this is an issue tracking PHP 8.4 failures

  4. stephen-cox commented on Sep 16, 2025

    @stephen-cox
    Member

    Just reviewing this. Last modules without full PHP 8.4:

    • drupal/layout_paragraphs is fixed in the dev branch, but there's not been a release yet.
    • drupal/pathauto is also fixed in dev, but no release yet
  5. millnut commented on Sep 16, 2025

    @millnut
    MemberAuthor

    @stephen-cox I thought I had updated to also add scheduled_transitions to the list as well, only fixed in 2.8 releases and for 2.7.x it's only in the dev version, not in a tagged release yet :(

    25) Drupal\Tests\localgov_workflows\Functional\ApprovalsDashboardTest::testApprovalsDashboardView
    Exception: Deprecated function: Drupal\scheduled_transitions\Plugin\Menu\LocalTask\ScheduledTransitionsLocalTask::getTitle(): Implicitly marking parameter $request as nullable is deprecated, the explicit nullable type must be used instead
    include()() (Line: 576)
    
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    Drupal 11Drupal 11 specific issues

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions