[Regression] Tests fail on PHP 5.3

looks good thanks

Easy win here.

Existing test coverage is sufficient.

Bumping the priority up (this is very important), lots of sites are still on php 5.3.x , I recently worked at an engineering school in Montréal that had about 120 Drupal sites using php 5.3.x. They have quite an elaborate setup using aegir and multisite with impressive functionality.

Triage, this is a high priority issue so that we can re-queue any patches against php 5.3.x before committing to core.

Ha, I've found this during my Drupal 7 support PHP7.4 run, and the patch is pretty much the same (except for the wording).

For the reference, patch from #3081386-22: [META] Fully support PHP 7.4 in Drupal 7 and the wording:

• The invalid filename has been detected and RuntimeException has been thrown
• The invalid destination has been detected and RuntimeException has been thrown

+1 to RTBC

Purely putting to CNW do that a Novice sees this as actionable. (Else would have left RTBC)

We really like the strings from the PHP 7.4 issue much better.

Novice instructions

- Change descriptions per above comment
- Add new patch
- Set back to RTBC
- Remove Novice tag

Thank you!

I'll work on this

Here is a new patch with new error messages.

Back to RTBC

RTBM from me and over to mcdruid for commit.

Thank you everyone that contributed!

Thanks @mcdruid, and congrats on your first real core commit other than the MAINTAINERS.txt !

Many firsts: this was @jordanwood's first accepted patch to any project as well! Congratulations to both of you! 🙂