Problem/Motivation

As a prerequisite for porting Project* to D8+ (probably straight to D9 at this point!), we need to get all the modules at least compatible with PHP 7+.

Proposed resolution

Investigate and solve any compatibility issues with Project* on PHP 7+. Initial target: 7.2.
This is a meta to track child issues for each part of the suite and its dependencies:

Remaining tasks

TBD.

User interface changes

None.

API changes

Perhaps.

Data model changes

Hopefully none.

Release notes snippet

TBD.

Comments

dww created an issue. See original summary.

dww’s picture

Issue summary: View changes
dww’s picture

Issue summary: View changes

machine_name is working fine, so I didn't bother spinning out a child issue for that one.
project.module itself is also fine without changes.

dww’s picture

Issue summary: View changes

Added #3138034: Make Version Control API compatible with PHP 7+ for a small problem in Version Control API.

dww’s picture

Issue summary: View changes

Crossing out projects from the initial list that have no warnings from phpcs --standard=PHPCompatibility.

joseph.olstad’s picture

Keep up the good work

dww’s picture

Thanks, @joseph.olstad! Curious: are you using Project* somewhere off d.o?

Note that this effort is somewhat on hold until we can spin up d.o sandbox sites using different versions of PHP. I've basically gotten things as far as I can with limited local testing and with static analysis...

Cheers,
-Derek

joseph.olstad’s picture

I'm using this project on drupal.org as is everyone else. Getting drupal.org onto php 7.3.x+ should be good for everyone.

raffaelj’s picture

Just to be sure - is this the module that powers everything on https://www.drupal.org/project/*?

And if so, does this mean, that drupal.org still runs on PHP 5.6, which reached EOL on 2018-12-31?

Initially I was testing and digging how to automate a module release. Than I noticed, that the project module needs v7 and that drupal.org still runs with v7. Now I'm a little bit concerned about security issues...

joseph.olstad’s picture

@raffaeilj, yes, over 40,000 project pages.
With that said, they may be using PHP 8 for all we know they could be using patches.

raffaelj’s picture

@joseph.olstad Yeah, I also hope that this is just a forgotten issue and drupal.org runs on modern systems. This issue being open just leaves too much room for speculation...

joseph.olstad’s picture

@dww, any progress on this?

I've been running PHP 8.1 with one of my more complex D7 instances, it's working well and possibly time for an upgrade to PHP 8.2.

joseph.olstad’s picture

PHP 8.3 is a worthy upgrade, significant performance improvements available. Could easily upgrade this project to work with PHP 8.3

https://kinsta.com/blog/php-benchmarks/