Drupal 11 compatibility

Created on 15 July 2024, about 2 months ago
Updated 5 August 2024, about 1 month ago

Problem/Motivation

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

πŸ“Œ Task
Status

Fixed

Version

3.0

Component

CAS

Created by

πŸ‡·πŸ‡΄Romania claudiu.cristea Arad πŸ‡·πŸ‡΄

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

Merge Requests

Comments & Activities

  • Issue created by @claudiu.cristea
  • Merge request !38Resolve #3461478 "D11" β†’ (Merged) created by claudiu.cristea
  • πŸ‡·πŸ‡΄Romania claudiu.cristea Arad πŸ‡·πŸ‡΄

    Fix branch

  • Assigned to claudiu.cristea
  • πŸ‡·πŸ‡΄Romania claudiu.cristea Arad πŸ‡·πŸ‡΄

    Working on this

  • Issue was unassigned.
  • Status changed to Needs review about 2 months ago
  • πŸ‡·πŸ‡΄Romania claudiu.cristea Arad πŸ‡·πŸ‡΄

    This is ready for review.

    Contains:

    • Drupal 11 compatibility
    • Testing extended to cover:
      • Drupal stable version (currently 10.3.x) which runs for lowest supported PHP (currently PHP 8.1) and highest supported PHP (currently PHP 8.3)
      • Drupal next major release (currently 11) which runs with highest supported PHP (currently PHP 8.3)
    • Converted ServiceControllerTest unit test to kernel test
    • Converted CasProxyHelperTest::testProxyAuthenticate() unit test to kernel test
    • Removed the backwards compatibility deprecated layer
    • Started modernization of code (this process should continue in the alpha stage):
      • Removed constructor documentation because is useless and only adds noise to the code. There are some exceptions where documenting the parameters is important.
      • Switched to constructor property promotion. Some exceptions were kept because of compatibility with Drupal 10.3.x.
  • Pipeline finished with Failed
    about 2 months ago
    Total: 220s
    #229120
  • Pipeline finished with Failed
    about 2 months ago
    Total: 223s
    #229132
  • Pipeline finished with Success
    about 2 months ago
    Total: 225s
    #229596
  • Pipeline finished with Failed
    about 2 months ago
    Total: 205s
    #229687
  • Pipeline finished with Failed
    about 2 months ago
    #229729
  • Pipeline finished with Success
    about 2 months ago
    Total: 208s
    #229730
  • Pipeline finished with Failed
    about 2 months ago
    Total: 198s
    #229762
  • Pipeline finished with Failed
    about 2 months ago
    Total: 200s
    #229765
  • Pipeline finished with Success
    about 2 months ago
    Total: 206s
    #229774
  • Pipeline finished with Failed
    about 2 months ago
    Total: 210s
    #229787
  • Pipeline finished with Success
    about 2 months ago
    Total: 322s
    #230167
  • Pipeline finished with Failed
    about 2 months ago
    Total: 251s
    #230171
  • Pipeline finished with Success
    about 2 months ago
    #230190
  • Pipeline finished with Success
    about 2 months ago
    Total: 207s
    #230274
  • Pipeline finished with Success
    about 2 months ago
    Total: 210s
    #230281
  • Pipeline finished with Success
    about 2 months ago
    Total: 210s
    #230287
  • Status changed to Needs work about 2 months ago
  • πŸ‡ΊπŸ‡ΈUnited States bkosborne New Jersey, USA
  • Status changed to Needs review about 2 months ago
  • πŸ‡·πŸ‡΄Romania claudiu.cristea Arad πŸ‡·πŸ‡΄

    Ready for a 2nd round of review. Fixed some and answered the other

  • Status changed to RTBC about 2 months ago
  • πŸ‡ΊπŸ‡ΈUnited States bkosborne New Jersey, USA

    Looks good to me.

  • Pipeline finished with Skipped
    about 2 months ago
    #231475
  • Status changed to Fixed about 2 months ago
  • πŸ‡·πŸ‡΄Romania claudiu.cristea Arad πŸ‡·πŸ‡΄

    Thanks

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

Production build 0.71.5 2024