Methode setStatus() zusätzlich zu setOnline()
alxndr-w opened this issue · 8 comments
Das Feld heißt status, demzufolge erwartet man auch eine Methode namens setStatus()
PR welcome
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
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.
ist es quatsch, setOnline(0) ausführen zu können.
Warum ist das quatsch? Aber ich bin jetzt hier raus …
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().
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.
Dann bitte möge man mein Issue dahingehend intepretieren, dass ich eine Methode setStatus() vorschlage, die genau das umsetzt.
Ich denke nicht, dass ich hier nochmal etwas machen werde und es scheint sonst keinen zu stören.