RexStan: Rückgabe-Typen korrekt setzen

[christophboecker]

RexStan meckert z.B. hier:

base_quality_check/fragments/pagecontent.php

Line 58 in 5a92225

if ($task->getDescription() || $task->getSource() || $task->getLinks()) {

Der Auslöser ist die Rückgabedefinition (?string) der angesprochenen Methoden wie z.B.

base_quality_check/lib/BaseQualityCheck.php

Line 70 in 5a92225

public function getDescription(bool $asPlaintext = false): ?string

Warum wird null als mögliche Rückgabe angesehen? Das müsste doch immer string sein bedingt durch das Datenfeld. Und wenn nicht müsste BaseQualityCheck::getDescription sicherstellen, dass ein String zurückgegeben wird (oder ein Whoops).

Ich würde stumpf die Rückgabetypen auf den Soll-Typ setzen und fertig.