Skip to content

Conversation

@iMattPro
Copy link
Member

No description provided.

@LukeWCS
Copy link
Contributor

LukeWCS commented Nov 20, 2025

Background:

Running Extension Pre Validator on directory ext.
Running tests.
PHP Warning:  Attempt to read property "key" on null in /www/htdocs/xxxxxxxx/www/lib/composer/epv/vendor/phpbb/epv/src/Tests/ArrayKeyVisitor.php on line 45

Warning: Attempt to read property "key" on null in /www/htdocs/xxxxxxxx/www/lib/composer/epv/vendor/phpbb/epv/src/Tests/ArrayKeyVisitor.php on line 45

PASSED:  Fatal: 0, Error: 0, Warning: 0, Notice: 0 
Test results for extension:
No issues found 

After receiving the ZIP file from the extension developer, I was able to reliably reproduce the error using phpBB Ext Check. It took me quite a while to find the source of the problem, though. ^^ The cause was a language file. Here's a small extract with 3 array entries (out of 47 total); the cause is in the second entry: double comma.

...
	'ACP_SG_SHOW_PROFILE_PREV_TOTAL_EXPLAIN'	=> 'Hier können Sie die Anzahl einstellen, wie viele Bilder als Vorschau auf der Benutzerprofilseite angezeigt werden sollen.<br><br><b>Wichtige Information: </b> Wird der Wert <b>0</b> eingestellt wird keine Vorschau im Benutzerprofil angezeigt. Dann ist der Schalter <b>Anzahl der Vorschaubilder im Benutzerprofil</b> wirkungslos.',
	'ACP_SG_MISSING_ERROR_PIC'					=> 'Diese Meldung wird angezeigt, da das Bild, welches angezeigt wird, wenn ein Album leer ist oder das Bild nicht da ist, als Ersatz angezeigt wird. Mit dem Absenden des Buttons kann das Bild wiederhergestellt werden.',,
	'ACP_COPY_MISSING_FILE_PIC'					=> 'Bild wiederherstellen',
...

#131 fixes the problem; I just tested it. 👍

@iMattPro iMattPro merged commit 7681929 into phpbb:master Nov 23, 2025
8 checks passed
@iMattPro iMattPro deleted the iMattPro-patch-1 branch November 23, 2025 15:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants