FriendsOfShopware / FroshShareBasket

This plugin allows your customers to save the current basket and share it via link.
MIT License
8 stars 5 forks source link

Falsche Freitextfelder Tabelle #24

Closed rogrconvis closed 2 years ago

rogrconvis commented 2 years ago

PHP Version

7.4

Shopware Version

5.7.7

Actual behaviour

In Ihrer Config "Zusätzliche Felder speichern", beziehen sie sich auf die Tabelle "s_order_detail_attributes". Aber in Ihrer Funktion "getBasketAttributes" bauen sie einen SQL-Query, der sich auf die Tabelle "s_order_basket_attributes" bezieht. Was dazu führt, dass die folgende Abfrage bei Freitextfeldern von anderen Plugins nicht ausgeführt wird. Pfad: custom/plugins/FroshShareBasket/Components/ShareBasketService.php Zeile: 116 - 119 if ($value && in_array($attribute, $attributesToStore, false)) { $basketArticle['attributes'][$attribute] = $value; }

Expected behaviour

Wenn sie in ihrer Config die Tabelle "s_order_detail_attributes" mit der Tabelle "s_order_basket_attributes" austauschen, können die entsprechenden Freitextfelder ausgewählt und die Abfrage ausgeführt werden.

Steps to Reproduce?

Nachstellen können sie diesen Bug, mit jedem Freitextfeld welches nicht Standardmäßig von Shopware kommt.

shyim commented 2 years ago

Hey,

magst du ein PR machen?

stefanpoensgen commented 2 years ago

Ich kann hier keinen Fehler nachvollziehen. Das Plugin funktioniert in meinen Tests genau wie vorgesehen.

Felder die in s_order_detail_attributes angelegt werden, werden von Shopware automatisch in s_order_basket_attributes angelegt. Sollten dort Felder von zusätzlichen Plugins fehlen, ist davon auszugehen das diese Plugins die Felder nicht korrekt anlegen.

chrome_uBVuEaYwKi chrome_eEDopR17iv