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

Verhindert Fehlermeldung bei nicht existentem Key in getValue() #1520

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

christophboecker
Copy link
Contributor

Wenn mit $dataset->getValue($key) auf einen Key zugegriffen wird, der nicht existiert, wird ein Fehler ausgeworfen (Undefined array-key). Die unschöne Meldung lässt durch ein intern vorgeschaltetes if hasValue verhindern. Da hasValue bereits eine Abfrage auf dataLoaded durchführt, enthällt die Abfrage in der getValue-Methhode.

siehe FriendsOfREDAXO/neues#94 (comment)

Wenn mit `$dataset->getValue($key)` auf einen Key zugegriffen wird, der nicht existiert, wird ein Fehler ausgeworfen (Undefined array-key). Die unschöne Meldung lässt durch ein intern vorgeschaltetes `if hasValue` verhindern. Da `hasValue` bereits eine Abfrage auf `dataLoaded` durchführt, enthällt die Abfrage in der getValue-Methhode.

siehe FriendsOfREDAXO/neues#94 (comment)
@tbaddade
Copy link
Member

Damit erhält man aber keine Fehlermeldung mehr, wenn der Key nicht existiert und das halte ich für wichtig anstatt dann einfach null zu liefern.

@christophboecker
Copy link
Contributor Author

Wer Wert darauf legt, das Fehlen des Wertes von NULL zu unterscheiden, kann ja alternativ auch mit hasValue abfragen was Sache ist.

@tbaddade
Copy link
Member

Hier geht es doch um den Feldnamen und nicht um den Wert dessen.

Auch wenn man hasValue()vorschaltet kann immer noch das passieren.

if ($data->hasValue('field')) {
    echo $data->getValue('feild');
}

Und deshalb sollte das bisherige Verhalten beibehalten werden.

@christophboecker
Copy link
Contributor Author

Ist ja halt ein Vorschlag. Den muss man ja nicht übernehmen. :-)

@alxndr-w
Copy link
Contributor

Sollte dann nicht eine Exception geworfen werden? (Siehe Original-Thread)

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.

3 participants