search

FriendsOfREDAXO / feeds

Importiert Daten aus API-Quellen wie YouTube, Vimeo, RSS und mehr
https://friendsofredaxo.github.io/
MIT License
45 stars 15 forks source link

Methode setStatus() zusätzlich zu setOnline() #106

Closed alxndr-w closed 4 years ago

alxndr-w commented 5 years ago

Das Feld heißt status, demzufolge erwartet man auch eine Methode namens setStatus()

skerbis commented 5 years ago

PR welcome

tbaddade commented 5 years ago

Das Feld heißt status, demzufolge erwartet man auch eine Methode namens setStatus()

Nicht unbedingt. ein

setOnline kann nur bool sein und könnte intern dann wieder setStatus aufrufen setStatus kann dagegen irgendetwas sein

alxndr-w commented 5 years ago

Wenn man bei der Argumentation bleibt, ist es quatsch, setOnline(0) ausführen zu können. Dann eben keine Alias-Methode. setStatus($value) wäre trotzdem korrekt und die gibt es nicht.

tbaddade commented 5 years ago

ist es quatsch, setOnline(0) ausführen zu können.

Warum ist das quatsch? Aber ich bin jetzt hier raus …

alxndr-w commented 5 years ago

Dann habe ich nicht verstanden, warum man setOnline() und setStatus() unterschiedlich behandeln sollte.

setOnline() benötigt doch gar keinen Parameter, weil setOnline(0) würde doch keiner schreiben, dazu bräuchte man dann setOffline().

skerbis commented 5 years ago

Ich schließe mich @tbaddade an:

setOnline() legt den Online-Status (bool) fest.

setStatus() würde ich verwenden wenn ich neben online auch gesperrt oder wartend oder sonstwas hätte.

Wir sollte setStatus() nicht für einen Alias verschwenden.

alxndr-w commented 5 years ago

Dann bitte möge man mein Issue dahingehend intepretieren, dass ich eine Methode setStatus() vorschlage, die genau das umsetzt.

alxndr-w commented 4 years ago

Ich denke nicht, dass ich hier nochmal etwas machen werde und es scheint sonst keinen zu stören.