Witam wszystkich.
Jeśli to zły dział proszę o przeniesienie do odpowiedniego.

Przechodząc do tematu problem polega na tym, że:
mam sobie taki kod - http://hastebin.com/ucopinadax.xml
Do 3 tysięcy rekordów wszystko było ok tzn można było zmienić ilość w koszyku bez żadnego problemu oraz można było usunąć produkt z koszyka.
Teraz rekordów jest 13 tysięcy i właśnie tu zaczynają się problemy:
1. przy próbie zmiany ilości jak jest więcej niż 1 produkt w koszyku dubluje mi zmieniany produkt (pierwszy zostaje z początkową ilością, a drugi jest z poprawioną)
2. nie można usunąć żadnego produktu powyżej ID 3000 jeśli jest w koszyku więcej niż 1 produkt, nawet tego zdublowanego.

Na próbę dodałem rekord o ID 1301 i odziwo ten działa poprawnie tzn można go zmieniać, usuwać

Siedzę już nad tym 4 dni i wymiękam więc proszę o pomoc.

Cześć.

Fragment do "update" wygląda tak:

[PHP] pobierz, plaintext 
case "update":
		$found = false;
			foreach($_SESSION['cart_item'] as $product)
			{
		if($_REQUEST['reference'] == $product['reference']) {
        $found = true;
        break;
[PHP] pobierz, plaintext 

i gdzie tutaj jest jakaś aktualizacja?

To akurat była moja kombinacja bo normalnie działało to na case"add"

http://www.php.net/manual/en/info.configur....max-input-vars

Lub
max_input_vars = 3000
suhosin.post.max_vars = 3000
suhosin.request.max_vars = 3000

Musisz zwiększyć te wartości

Niestety nic to nie daje

A ja Ci mówię, że to to.

Pokaż co masz w phpinfo?

@maly_swd zwroc uwage ze on wysyla oddzielnego forma dla kazdego produktu a nie wszystkie produkty w jednym formie

@slawek Twoj kod jest fatalny.
Skoro kluczem w $_SESSION["cart_item"] jest reference to:
1) Nie ma sensu bys latal po calej tablicy i sprawdzal kazdy elelement, skoro mozesz od razu zrobic:
$_SESSION["cart_item"]['szukanyrefence']

2) Skoro kluczem jest UNIKALNY reference to nie ma prawa sie nic dublowac bo jak zrobisz w tablicy
$tab[1] = 'jeden';
$tab[1] = 'dwa';
to drugie 1 nadpisze ci pierwsze i nie ma mowy o duplikatach
Tak wiec cos krecisz, albo nagle ci refence zmienia na inne.

Popraw ten kod, uporzadkuj bo teraz to jest, przepraszam, wielki burdel - pomimo ze kodu duzo nie ma.


Masz niepoprawna strukture html, np. <tr><form><td>
Zajrzyj do dokumentacji html i zobacz jak wyglada poprawna tabela, HINT: miedzy TR a TD nie ma prawa nic innego byc.

edit:
ten kod z akcji ADD a nie UPDATE wyglada ze moze robic to co mowisz

[PHP] pobierz, plaintext 
if(in_array($productByCode[0]["reference"],$_SESSION["cart_item"])) {
					foreach($_SESSION["cart_item"] as $k => $v) {
							if($productByCode[0]["reference"] == $k)
								$_SESSION["cart_item"][$k]["quantity"] = $_POST["quantity"];
					}
				} else {
					$_SESSION["cart_item"] = array_merge($_SESSION["cart_item"],$itemArray);
				}
[PHP] pobierz, plaintext 

Tylko ze to jest ADD a nie UPDATE jak pisales wczesniej. Do rzeczy:
to
if(in_array($productByCode[0]["reference"],$_SESSION["cart_item"])) {
nigdy nie jest prawdziwe bo szukasz reference na liscie tablic a nie liscie referencji wiec wchodzi co do ELSE czyli do
$_SESSION["cart_item"] = array_merge($_SESSION["cart_item"],$itemArray);
a array_merge gdy tablice maja indexy numeryczne to poprostu numeruje wszystko od nowa wiec tak, to ci moze dublowac i zmieniac totalnie wszystko wiec nic dziwnego ze potem nie jest w stanie kasowac jak indesky sie pozmienialy.

Dobrze by bylo jakbys pisal opis problemu troche bardziej porzadnie na przyszlosc bo opisales ze podczas UPDATE ci sie duplikuje a tu raczej podczas ADD jest duplikacja

Może zrobię tak, że podam link do tego co nie działa i już. Wtedy każdy sobie zobaczy w czym tkwi problem.
katalog online
PHP INFO

Pozdr.

sprawdzales za pomoca debuggera co sie dzieje w kodzie?

A moze poprostu popraw rzeczy ktore ci wskazalem?

Pozmieniałem i nadal nic :/

I oczywiscie ja wiem na co pozmieniales...
Tak trudno podac poprawiony kod?

Tzn zmieniłem form w html i wyrzuciłem case update. Tylko qrcze ciągle nie wiem jak zmienić to co pisałeś odnośnie tablic.

KOD

Czyli zrobiles cale nic ale oczywiscie musiales napisac:

Guzik pozmieniales, struktura html nadal nie poprawna. Prosilem, zajrzyj do dokumentacji jak ma wygladac poprawna struktura TABLE.

Co mi to da jak poprawię strukturę table poza czytelnością?

Czasami niepoprawna struktura html jest powodem blednie wysylanych formow. Nie mowie, ze tak jest w tym przypadku, nie mniej jednak strasznie kluje po oczach.

A masz, co mi tam...
Resztę sobie sam ogarniaj, koniec dobroci

[PHP] pobierz, plaintext 
<?php
require_once("dbcontroller.php");
$db_handle = new DBController();
if(!empty($_GET["action"])) {
 
    if(!isset($_SESSION["cart_item"])) {
        $_SESSION["cart_item"] = [];
    }
 
    switch($_GET["action"]) {
        case "add":
            if(!empty($_POST["quantity"])) {
                mysql_query("SET CHARSET utf8"); // polskie znaki
                mysql_query("SET NAMES `utf8` COLLATE `utf8_polish_ci`"); // polskie znaki
                $productByCode = $db_handle->runQuery("SELECT * FROM cenniki WHERE reference='" . $_GET["reference"] . "' ORDER BY reference LIMIT 1");
 
 
                $quantity = (int)$_POST["quantity"];
                $product = $productByCode[0];
                $product['quantity'] = $quantity;
 
                if(array_key_exists($product['reference'], $_SESSION["cart_item"])) {
                    $_SESSION["cart_item"][$product['reference']]['quantity'] = $quantity;
                    break;
                }
 
                $_SESSION["cart_item"][$product['reference']] = $product;
                break;
            }
            break;
        case "remove":
            if(array_key_exists($_GET["reference"], $_SESSION["cart_item"])) {
                unset($_SESSION["cart_item"][$_GET["reference"]]);
            }
            break;
        case "empty":
            $_SESSION["cart_item"] = [];
            break;
    }
}
?>
[PHP] pobierz, plaintext 

Oczywiście to może nie działać w 100% i na 100% nie jest wolne od wad...

Rozumiem, że mam teraz to połączyć ze swoim nieudolnym kodem?

Masz zastapic caly swoj kod php z poczatku twojego pliku tym co dostales od Pytona

Początkowo zastąpiłem, ale jest błąd 500

Nauczysz sie w koncu czy nie? Nie pisz, ze cos poprawiles, tylko pokaz kod po poprawkach. No skad mamy wiedziec co znowu napsules.

No i Temat: Najczestsze bledy

Kod po zmianach

I ten kod daje ci teraz error 500?

Dokładnie tak

To chyba przez [] ktore nie sa obslugiwane w twoim php. zmien
$_SESSION["cart_item"] = [];
na
$_SESSION["cart_item"] = array();
i wszedzie indziej gdzie zdefiniowana jest w ten sposob tablica

WOW! W końcu działa jak należy

Dobrze rozumiem, że żebym mógł wpisać rabat z ręki muszę zrobić ?

[PHP] pobierz, plaintext 
                $rabat = (int)$_POST["rabat"];
                $product = $productByCode[0];
                $product['rabat'] = $rabat;
 
         if(array_key_exists($product['reference'], $_SESSION["cart_item"])) {
                    $_SESSION["cart_item"][$product['reference']]['quantity'] = $quantity;
                    $_SESSION["cart_item"][$product['reference']]['rabat'] = $rabat;
                    break;
                }
[PHP] pobierz, plaintext 

Tak

No i teraz jak zwykle musiałem coś zepsuć. Jak dodałem opcję rabatu z ręki to po wkładaniu do koszyka niema ilości tylko muszę ją wpisywać ręcznie :/

Zauwazyles, iz poraz trzeci nie podales poprawionego kodu?

Poprawiony kod
Form pewnie znów jest błędnie, ale inaczej nie chce działać :/

Po grzyba drugi raz z rzedu dajesz to:
$product = $productByCode[0];
zaraz po pierwszym?
Przeciez w ten sposob nadpisujesz $product ktory ustaliles dwie linie wczesniej

Poprawiłem. Teraz nie pobiera mi rabatu :/

EDIT:

Tzn wkłada same zera zamiast np 48.00%

Ostatni raz mowie POKAZ KOD. Potem poprostu zaczne ignorowac twoj temat.

Już zrobiłem.

Dopisałem

[PHP] pobierz, plaintext 
if(!empty($_POST["quantity"]["rabat"])) {
[PHP] pobierz, plaintext 

Proszę bardzo cały kod

czyli juz dziala?
Zdajesz sobie sprawe ze teraz bez rabatu nie bedziesz w stanie dodac nic do koszyka?

edit:
a co to jest
if(!empty($_POST["quantity"]["rabat"])) {
?
Bierzesz jakies losowe kody i strzelasz czy ten kod moze jednak wyszedl po wiekszej zadumie?

Każda pozycja ma rabat chociażby 0.
Ten kod to był strzał być może sobie w kolano.

Co do rabatu byłoby super gdyby pobierał wartość np 54.13% a nie 54% :/

Kolumna rabat: rabat decimal(20,2) [0.00]