Replace-Hook wirft unter bestimmten Umständen eine Exception

[MrMDeluxe]

Hallo,

der Replace-Hook von sGetOrderNumber() verursacht leider Probleme mit anderen Plugins, die diese Funktion im Backend Kontext benutzen. Denn der Zugriff auf Shopware()->Session() funktioniert bekanntlich im Backend nicht. Da die public Core Funktion sGetOrderNumber aber keinen Kontext vorschreibt, sollte der Hook hier die Verantwortung einer Prüfung für den richtigen Kontext haben.

Durch folgende Anpassung des Hooks wird hier der Session Zugriff auf den Frontend Kontext eingeschränkt:

 public function onGetOrderNumber(\Enlight_Hook_HookArgs $args) {

        if(Shopware()->Container()->has('shop')){
            $session = Shopware()->Session();
            $moptRatepayOrdernum = $session->offsetGet('moptRatepayOrdernum');
        }
        if ($moptRatepayOrdernum) {
            return $moptRatepayOrdernum;
        } else {
            // standard behaviour
            $args->setReturn($args->getSubject()->executeParent(
                $args->getMethod(),
                $args->getArgs()
            ));
        }
    }

Wäre super, wenn Ihr den Fix übernehmen könntet.

[fjbender]

Klingt gut, wir prüfen das und übernehmen das dann wahrscheinlich.

[fjbender]

@MrMDeluxe Nur zur Vollständigkeit: Mit welchem Drittplugin können wir diesen Fehler reproduzieren?

[rmaassen]

Premium Gutschriften von Rhiem Intermedia GmbH.

https://store.shopware.com/rhiem20364534336/premium-gutschriften.html

[rmaassen]

@fjbender wie ist denn hier der Status?

[fjbender]

Wir arbeiten noch dran, im Zuge der Analyse ist noch ein andere Bug mit Hooks aufgefallen, den wir gleich mit reparieren wollen.

[mediagraphik]

Wie ist der momentane Status hierzu?

[fjbender]

Sollte eigentlich mit #243 behoben worden sein - wir prüfen allerdings zur Zeit, ob das Problem nach #254 wieder auftritt.