[CON-444] FrontendUser::store() ohne Rückgabewert

Gesperrt
codefux
Beiträge: 25
Registriert: Fr 6. Apr 2007, 15:38
Wohnort: München
Kontaktdaten:

[CON-444] FrontendUser::store() ohne Rückgabewert

Beitrag von codefux » Do 22. Sep 2011, 21:13

Version: CONTENIDO 4.8.14
Die Methode FrontendUser::store() ruft in der letzten Zeil parent::store() auf, gibt jedoch den return dieses Aufrufes nicht als Rückgabewert zurück.

Statt...

Code: Alles auswählen

    public function store()
    {
        global $auth;

        $this->set("modified", date("Y-m-d H:i:s"), false);
        $this->set("modifiedby", $auth->auth["uid"]);
        parent::store();
    }
... würde ich ...

Code: Alles auswählen

    public function store()
    {
        global $auth;

        $this->set("modified", date("Y-m-d H:i:s"), false);
        $this->set("modifiedby", $auth->auth["uid"]);
        return parent::store();
    }
...erwarten.
What happens on the island, stays on the island ;-)
|| codefux.de ||
|| Advanced ModRewrite mit Windows IIS ||

xmurrix
Beiträge: 3143
Registriert: Do 21. Okt 2004, 11:08
Wohnort: Augsburg
Kontaktdaten:

Re: [BUG 4.8.14] FrontendUser::store() ohne Rückgabewert

Beitrag von xmurrix » Do 22. Sep 2011, 23:06

Hallo codefux,

da hast du recht, die funktion store() sollte überall das gleichr Verhalten aufweisen, also true oder false zurückliefern.

Der Beitrag wurde nach Bugs verschoben.

Gruß
xmurrix
CONTENIDO Downloads: CONTENIDO 4.10.1
CONTENIDO Links: Dokumentationsportal, FAQ, API-Dokumentation
CONTENIDO @ Github: CONTENIDO 4.10 - Mit einem Entwicklungszweig (develop-branch), das viele Verbesserungen/Optimierungen erhalten hat und auf Stabilität und Kompatibilität mit PHP 8.0 bis 8.2 getrimmt wurde.

xmurrix
Beiträge: 3143
Registriert: Do 21. Okt 2004, 11:08
Wohnort: Augsburg
Kontaktdaten:

Re: [CON-444] FrontendUser::store() ohne Rückgabewert

Beitrag von xmurrix » Do 22. Sep 2011, 23:30

Gefixt und commited.
CONTENIDO Downloads: CONTENIDO 4.10.1
CONTENIDO Links: Dokumentationsportal, FAQ, API-Dokumentation
CONTENIDO @ Github: CONTENIDO 4.10 - Mit einem Entwicklungszweig (develop-branch), das viele Verbesserungen/Optimierungen erhalten hat und auf Stabilität und Kompatibilität mit PHP 8.0 bis 8.2 getrimmt wurde.

Gesperrt