koszyk na zakupy na sesjach, nie działa przy register_globals = off Opcje

[Olimpia_ona]

Witam,
pisałam już kiedyć o koszyku. Wszystkie problemy z nim związane rozwiązałam i wydawało mi się, że już wszystko jest ok, ale zapomniałam zmienić ustawień serwera na register_globals = off, po tej zmianie nie trzyma mi tego koszyka. Nie pamięta zmiennej $stan, co wydaje się być logiczne, ale teraz proszę o pomoc, co muszę zrobić, by jednak ten koszyk działał z register_globals = off.

Fragment jak mam to zrobione:

[PHP] pobierz, plaintext 
<?php
session_start();
...
$koszyk=$_SESSION['koszyk'];
  $dodaj=$_GET['dodaj'];
  $usun=$_GET['usun'];
  $zmien=$_GET['zmien'];
  $first=$_GET['first'];
...
class koszyk 
  { 
	  /* tablica asocjacyjna zawierajaca stan koszyka */ 
	  /* klucz, to wartosc id produktu, wartosc to ilosc elementow */ 
	  var $zawartosc; 
 
	  function dodaj($element, $ilosc) 
	  { 
		  $this->zawartosc[$element] += $ilosc; 
	  } 
 
	  function usun($element, $ilosc) 
	  { 
		  if ($this->zawartosc[$element] > $ilosc) 
			  $this->zawartosc[$element] -= $ilosc; 
		  else 
			  unset($this->zawartosc[$element]); 
	  } 
 
	  function zmien($element, $ilosc) 
	  { 
		  $this->zawartosc[$element] = $ilosc; 
	  } 
 
	  function stan() 
	  { 
		  return $this->zawartosc; 
	  } 
  } 
 
  if (!$koszyk) 
  $koszyk = new koszyk; 
...
if ($dodaj) 
	  $koszyk->dodaj($dodaj1, 1); //dodaj($element, $ilosc) - dodaje $ilosc $elementow do koszyka
...
session_register('koszyk');
 
  $stan = $koszyk->stan(); //zwraca zawartosc w postaci hasza
...
//i wyświetlanie koszyka
if ($stan) 
{
 $ilewkosz=0;
 while (list($key, $value) = each($stan))
 { 
   echo "produkt: $key ilość:$value<br />";
 }
}
?>
[PHP] pobierz, plaintext 

próbowałam robić coś w stylu:
$stan=$_SESSION['stan'];
session_register("stan");

ale to nie pomogło.

[likemandrake]

Witam

To co tu napisze, bedzie dotyczyc kodu z Twojej pierwszej wypowiedzi w tym poscie, a wiec...

Usuwasz linie 46
Miedzy linie 58, a 59 wstaw: $_SESSION['koszyk']=$koszyk;

Mysle ze powinno dzialac, ale nie jestem kompilatorem, jak co to bedziemy dalej kombinowac (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)
Przeczytaj to
Wyraznie jest opisany Twoj problem w bloku Uwaga

Jak zapewne widzisz, do zmiennej sesyjnej zostaje zapisany caly obiekt klasy, mysle ze lepszym rozwiazaniem byloby zapisanie tablicy z samym stanem koszyka, przy kazdym wywolaniu skryptu, zawsze bylby w takim razie tworzony obiekt klasy koszyk i dolaczany do niego aktualny stan koszyka pobrany ze zmiennej sesyjnej.

Jeszczed lepszym rozwiazaniem bedzie utworzenie tabeli w bazie danych, ktora to moglaby trzymac niezatwierdzony stan koszyka, a dopiero potem jak koszyk zostaje zatwierdzony, to po zweryfikowaniu danych, dane sa te przeniesione do odpowiedniej tabeli - tabeli zamowien, gdzie beda juz trzymane ostateczne dane.

Trzymanie calej zawartosci obiektu w sesji nie jest bezpiecznym rozwiazaniem, wystarczy ze ktos przelapie SID uzytkownika X i w tym momencie ten uzytkownik ma wszystko to co zrobil użytkownik X, czyli aktualny stan koszyka, itp. Mam nadzieje, ze jakos zabezpieczasz sie przed tego typu defektami (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

Pozdrawiam serdecznie (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)