Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

RexStan-Überprüfung: uninstall.php #108

Merged
merged 2 commits into from
Sep 8, 2024
Merged

RexStan-Überprüfung: uninstall.php #108

merged 2 commits into from
Sep 8, 2024

Conversation

christophboecker
Copy link
Member

Für beide "Fehler" findet hier keine Korrektur statt, auch kein ignore-next-line; Ich habe nur Kommentare eingefügt, damit man darum weiß, da sie nicht stören und vom Code her korrekt sind. Die Fehlerkorrektur muss woanders stattfinden.

'Unable to resolve the template type TFetchType in call to method rex_sql::getArray()'
'Parameter $fetchType of method rex_sql::getArray() expects 2|3|12, 7 given.'

Beide Meldungen kann man hier ignorieren. Das liegt an rex_sql; dort sind einige häufige PDO::FETCH_... hinterlegt,
aber eben nicht alle. In diesem Fall ist PDO::FETCH_COLUMN eben unbekannt. Es hat aber keinerlei funktionale
Auswirkungen.

'Strict comparison using !== between null and Url\Profile will always evaluate to true.'

Der Fehler hier ist ignorierbar, denn der Problembär sitzt eine Zeile darüber mit der Referenz auf das Url-Addon.
Die Methode $profile = Profile::get($profileId); liefert angeblich immer eine Profil-Instanz (@return self), aber niemals null.
So gesehen ist die Abfrage if (null !== $profile) tatsächlich überflüssig. In der Realität liefert Profile::get
aber doch u.U. null. Der Fehler liegt also dort im unvollständigen Return-Type.

Siehe PR beim URL-Addon

@alxndr-w alxndr-w merged commit 23d2c37 into main Sep 8, 2024
@alxndr-w alxndr-w deleted the cb-14 branch September 8, 2024 16:23
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