RexStan-Überprüfung: fragments/neues/list.php #107
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
'PHPDoc tag @var for variable $posts contains generic class rex_yform_manager_collection but does not specify its types: T'
Da fehlt die Angabe der Collection-Inhalte, also der ModelClass
Entry
:/** @var rex_yform_manager_collection<Entry> $posts */
''Only booleans are allowed in an if condition, FriendsOfRedaxo\Neues\Entry|null given.''
Die Fehlermeldung führt etwas in die falsche Richtung; das eigentliche Problem ist ein anderes. Hier stimmt nämlich die
Struktur nicht so ganz.
Da die Schleife nur durchlaufen wird, wenn es auch $posts gibt, ist die Abfrage auf
if($post) ...
nicht erforderlich. Es gibt den Post, sonst wäre er nicht in der Collection. Somit wird der else-Zweig (
no entries
)nie erreicht, auch nicht bei einer leeren Liste. Daher den Code etwas umgebaut.
'Only booleans are allowed in an if condition, mixed given.'
Da
rex_config::get(...)
als Defaultnull
liefert, möchte RexStan dass expliziet darauf abgefragt wird.