Opened 4 years ago

Last modified 4 years ago

#2154 closed Erweiterung

Bei Express-Checkout wird MAX_PRODUCTS_QTY nicht beachtet — at Version 1

Reported by: noRiddle Owned by: somebody
Priority: normal Milestone: modified-shop-2.0.7.0
Component: Shop Version: 2.0.6.0
Keywords: Cc:
Blocked By: Blocking:

Description (last modified by Torsten Riemer)

In der /includes/cart_actions.php wird bei Express-Checkout MAX_PRODUCTS_QTY nicht beachtet. Das heißt man kann in den Checkout gehen auch wenn man die MAX_PRODUCTS_QTY überschritten hat.
Um einem Mißverständnis vorzubeugen:
Mit Express-Checkout ist nicht PayPal-Express oder *-Cart gemeint sondern "Schnellkauf" aus dem System-Modul.

Beheben ließe es sich z.B. so:
Statt diesem Code (unter case 'add_product)

        if ($cart_quantity > MAX_PRODUCTS_QTY) {            
          $cart_quantity = MAX_PRODUCTS_QTY;
          $_SESSION['err_max_prod'] = true;   // error message for exceeded product quantity, noRiddle
          $_GET['max_prod_id'] = (int)$_POST['products_id'];
          $goto = FILENAME_SHOPPING_CART;
        }

das

        if ($cart_quantity > MAX_PRODUCTS_QTY) {            
          $cart_quantity = MAX_PRODUCTS_QTY;
          $_SESSION['err_max_prod'] = true;   // error message for exceeded product quantity, noRiddle
          $_GET['max_prod_id'] = (int)$_POST['products_id'];
          $goto = FILENAME_SHOPPING_CART;
          
          if($co_express === true) {
            $co_express = false;
          }
        }

Allerdings wird man dann zum WK geleitet um die Message anzuzeigen.
Wenn der Kunde Express-Checkout machen wollte ist das ein wenig ärgerlich.

Vielleicht sollte man das so ändern, daß die Message auf der Seite angezeigt wird auf welcher man sich befindet (in unserem Fall die Produktseite), ohne Redirect zum WK.
(Wie auch in meinem Ticket #2152 für die Anzeige evtl. sich im WK befindlicher Artikel nach Login angeregt (im dazu gehörigen dort verlinkten Thread).)

Gruß,
noRiddle

Change History (1)

comment:1 by Torsten Riemer, 4 years ago

Description: modified (diff)
Note: See TracTickets for help on using tickets.