Modify

Opened 4 years ago

Closed 4 years ago

#2154 closed Erweiterung (fixed)

Bei Express-Checkout wird MAX_PRODUCTS_QTY nicht beachtet

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

Attachments (0)

Change History (7)

comment:1 by Torsten Riemer, 4 years ago

Description: modified (diff)

comment:2 by Gerhard Waldemair, 4 years ago

Resolution: invalid
Status: newclosed

das kann ich nur nicht nachstellen. Ich komme per Schnellkauf nicht in den Checkout mit mehr Artikel als MAX_PRODUCTS_QTY. Die Warenmenge wird immer reduziert.

comment:3 by Gerhard Waldemair, 4 years ago

Milestone: modified-shop-2.0.6.1

comment:4 by noRiddle, 4 years ago

Resolution: invalid
Status: closedreopened

Ich habe nicht behauptet, daß die Menge nicht reduziert würde sondern lediglich gesagt man könne in den Checkout gehen.
Findest du okay, daß der Kunde vor die vollendete Tatsache einer plötzlich reduzierten Stückzahl gesetzt wird, wenn er es überhaupt sieht ?
Ich finde es nicht nur nicht okay sondern absolut anti-user-freundlich.

Dazu kommt, daß man die Meldungen den WK betreffend ohnehin anzeigen sollte ohne in ihn zu leiten (wenn denn DISPLAY_CART nicht auf 'true' steht). Das passt wunderbar zu dem im Eingang verlinkten thema mit dem redirect nach login und man könnte es mit einem Schlag logisch und user-freundlich machen.

Gruß,
noRiddle

comment:5 by noRiddle, 4 years ago

*NACHTRAG*
Hatte mich allerdings mißverständlich ausgedrückt, sorry.

comment:6 by Torsten Riemer, 4 years ago

Milestone: modified-shop-2.0.6.1
Priority: hochnormal
Type: Bug/FehlerErweiterung

Ein Hinweis an der Stelle macht durchaus Sinn, da gebe ich dir Recht.

comment:7 by Gerhard Waldemair, 4 years ago

Resolution: fixed
Status: reopenedclosed

In 14082:

fix #2154 - show message if qty reduced

Modify Ticket

Action
as closed The owner will remain somebody.
The resolution will be deleted. Next status will be 'reopened'.

Add Comment


E-mail address and name can be saved in the Preferences .
 
Note: See TracTickets for help on using tickets.