Closed (fixed)
Project:
Drupal core
Version:
10.0.x-dev
Component:
base system
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
20 Dec 2021 at 23:08 UTC
Updated:
4 Jan 2022 at 13:09 UTC
Jump to comment: Most recent, Most recent file
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_phpso 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!