Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Problem/Motivation
Removes PHP 7 code paths from Drupal 10 now that PHP 8.0 is the minimum version.
Proposed resolution
Leave legacy tests alone as they can be removed in the issue that removes the legacy code.
Remaining tasks
User interface changes
API changes
Data model changes
Release notes snippet
Comment | File | Size | Author |
---|---|---|---|
#8 | 3255350-8.patch | 20.1 KB | alexpott |
| |||
#8 | 6-8-interdiff.txt | 4.7 KB | alexpott |
#6 | 3255350-6.patch | 15.4 KB | alexpott |
| |||
#6 | 3-6-interdiff.txt | 884 bytes | alexpott |
#3 | 3255350-3.patch | 15.29 KB | alexpott |
|
Comments
Comment #2
alexpottComment #3
alexpottNow that #3186524: Fix htaccess files for PHP 8 has landed we can remove the PHP 7 stuff from htaccess too.
Comment #4
longwaveWonder if we want to update or even remove this comment now to avoid having to update it again - the module is now called
mod_php
so I suspect it won't change name in PHP 9?Comment #5
longwaveI think all these have their own longstanding issues somewhere?
Otherwise this catches everything that I can see apart from legacy tests which I agree we will deal with elsewhere.
Comment #6
alexpottRe #5 yep they do.
RE #4 - yep I think removing the comment is a good idea.
Comment #7
longwaveSome more references, do we want to fix/remove any of these here?
Probably time to revive #3001920: Investigate PHP 7.3 workarounds as well.
Comment #8
alexpottThanks @longwave - yep let's fix all these here.
Re #7.2 my initial thought was to leave it because you might be coming from a site running on 7.3 and upgrading to PHP 8 at the same time - but that'd requiring going via update.php which is going to rebuild the router - so yep we can remove this.
Re #7.4 \Drupal\Tests\migrate\Functional\process\DownloadFunctionalTest still fails on PHP 8 so the PHP version is irrelevant.
Comment #9
longwaveThanks - this all looks sensible to me and I can't find any remaining references to PHP 7* so this is RTBC.
* there are references to PHP 5 and even one reference to PHP 4 but those seem out of scope for fixing here
Comment #11
catchAll looks good, think it's fine to leave PHP 5 removal to the existing issues.
Committed 94ee839 and pushed to 10.0.x. Thanks!