After creating a Context with a "Page template suggestions" reaction, I decided to check my site to get some base template names and got a lot of errors popping up. This is due to changes to PHP 8.2 not tolerating unset values.
Create a context with a "Page template suggestions" reaction and start navigating your site before adding anything to that field. The site should be running on PHP 8.2
Add a default empty string to the execute method return value in the PageTemplateSuggestions class.
| Comment | File | Size | Author |
|---|---|---|---|
| PageTemplateSuggestions_fix.patch | 533 bytes | jaydee1818 |
Show commandsStart within a Git clone of the project using the version control instructions.
Or, if you do not have SSH keys set up on git.drupalcode.org:
changes
Comments
Comment #2
nfabing commentedComment #3
nfabing commentedHi
This issue happens when adding a new "Page template suggestions" or "Page title" and only if the form was not saved afterward.
I added the missing default configuration for the related plugins to fix the issue.
I've created a new MR. Rdy for review
Comment #7
aimadbachar commentedHi
I have tested the MR !60
It works fine for me, there are no more errors
Drupal Core: 10.3.5
PHP: 8.3.12
Comment #8
benstallings commentedafter rebasing on 5.x, Claude Code says,
Both changes are correct, follow the same pattern used by other reaction plugins (e.g. BodyClass), and prevent undefined index errors when the reactions are added to a context but not yet configured.
No issues. Good to merge.