Fix the condition for product type based shipping calculation.

[richardhj]

Hallo,

dies ist ein Fix, damit das Vorhaben in #1970 gelingt.

Es handelt sich um die Einstellung "Preis für diese Produkttypen berechnen" bei den Versandarten.

Ich habe mir den Code angeschaut und deshalb habe ich von dieser Einstellung erwartet, in der Preisberechnung nur die Produkte mit zutreffendem Produkttyp zu berücksichtigen.

Allerdings war ein Vorzeichen falsch.

Die betroffene Stelle wurde das letze mal in 25aa89b#diff-b5a411f80105f8e2809e6e9f24529307 bearbeitet.

---

Vorher: Diese Einstellung resultiert in Versandkosten in Höhe von 0, auch wenn ein Produkt mit "Test-Produkttyp 1" im Warenkorb liegt.

Nach diesem Fix: Diese Einstellung resultiert in Versandkosten, wie erwartet, wenn ein Produkt mit "Test-Produkttyp 1" im Warenkorb liegt.
Edit: Wie erwartet heißt, dass die Versandkosten nur Anwendung finden, für diesen Produkttyp, und ignoriert wird, für andere Produkktypen. Details siehe #1970.

[richardhj]

-           if (is_array($allowedTypes) || !in_array($productType->id, $allowedTypes, false)) {
+           if (!is_array($allowedTypes) || !in_array($productType->id, $allowedTypes, false)) {
                continue;
            }

Das ist meine Lösung in 269cc83.
Dies bedeutet, dass wenn keine Produkttypen ausgewählt wurden, die Versandart auch Versandkosten i.H.v. 0 berechnent.

-           if (is_array($allowedTypes) || !in_array($productType->id, $allowedTypes, false)) {
+           if (is_array($allowedTypes) && !in_array($productType->id, $allowedTypes, false)) {
                continue;
            }

Das wäre eine alternative Lösung.
Dies bedeutet, dass wenn keine Produkttypen ausgewählt wurden, die Versandart trotzdem Anwendung findet.

[aschempp]

Dies bedeutet, dass wenn keine Produkttypen ausgewählt wurden, die Versandart trotzdem Anwendung findet.

Das wäre die richtige Variante. Die Einschränkungen gelten immer für den Fall, dass eine Eingabe vorliegt. Keine Eingabe entspricht keiner Auswahl. So wie auch wenn du keinen Mindestpreis eingibst, gilt die Versandart für jeden Preis.

Magst du den PR entsprechend anpassen?

[richardhj]

Hab ich entsprechend geändert.

[aschempp]

Danke! Lässt sich damit das vorhaben aus #1970 noch durchführen?