greg — Mon, 09 Jul 2007 21:33:59 +0000

Ok thanks, I have it sorted.

I broke out of PHP to make the form, that way I could understand the sytnax better
And also will keep server usage down!

For those interested, and others it may help, I'll explain what I did:

	<form action="session.php" method="post">
	<input type="hidden" name="check" value="addpack">
<input type="hidden" name="packadd" value="<?php echo $row['id']; ?>">
<input type="submit" name="submit" value="Add to Cart"/>
</form>

Because I have two other options for the users cart, remove one item and empty whole cart, I needed someway of tackling this altogether.

So I used one page, sessions.php

And to incoporate them all in one file I included in the form for the package add, delete and cart empty:

greg — Mon, 09 Jul 2007 20:03:09 +0000

I was wroking on seperate pages as you mentioned, but as I explained above, the $_GET['id'] from the url is assigned to a variable, that variable is used within the cart contents and session calculations

so the alternative I thought about is still have all the variables and cart session caclulations on the one page, as they were originaly, and change the link that users use to select the product they are choosing:
'Add to cart';

and maybe use a form?

now forms are fairly straight forward, and i presume i would use hidden data to submit the variable value to the next page
where the next page would still have the get, but would now be $_GET['form_input_name']

The products are listed into a table from the database in a standard loop
So the ID of the product is determined via the $row['id']
Each row has one product on it, the product is of course the ['id']

The bit i'm stuck with, how do i put into a form something like:

echo "<input type="hidden" name="var1" value="<?php $_POST['.$row['id']']; ?>"'

cheers

pr0gr4mm3r — Mon, 09 Jul 2007 18:18:41 +0000

Quote: So it doesnt know what that id was from the $_GET to display the required products

I thought that meant you wanted to get rid of the $action variable and keep the $id variable...so I'm confused.

Maybe it would be best to have the processing page separate from the cart page. In other words, the link to add the product will go to like maybe add_item.php. Then, redirect them to cart.php.

greg — Mon, 09 Jul 2007 18:09:57 +0000

pr0gr4mm3r;221410 wrote: If you need to save one of the variables, then something like this would work.

<?php header('Location: cart.php?id=' . $_GET['id']); ?>

erm
but that sends the id to the url
So I am back where I started, but now with another page inbetween
The point is to get to the cart to display the users cart contents without the ID=1 in the URL

pr0gr4mm3r — Mon, 09 Jul 2007 17:56:24 +0000

If you need to save one of the variables, then something like this would work.

<?php
header('Location: cart.php?id=' . $_GET['id']); 
?>

greg — Mon, 09 Jul 2007 17:51:37 +0000

hmm

The problem i am having is the cart.php has ALL the code on it, all the variables everything

So when the user clicks the link for the product they want, the whole workings from the $_GET to displaying products uses the same variables
So they all have to be on the same page

And of course using your redirect does solve the problem as the id in the url was retrieved from a page, then that redirect sends them to display their cart
but then it doesnt recognise the variables that is assigned to the $_GET when it goes to the page to display their cart contents
So it doesnt know what that id was from the $_GET to display the required products

The code i have doesnt simply $_GET then add to the session
otherwise i could simply get from the session on the next page
It uses a variable to perform a selection of possible actions

$product_id = $_GET['id'];     //the product id from the URL
     $action     = $_GET[action]; //the action from the URL

     //if there is an product_id and that product_id doesn't exist display an error message
     if($product_id && !productExists($product_id)) {
         die("Error. Product Doesn't Exist");
     }

     switch($action) { 
    
         case "add":
             $_SESSION['cart'][$product_id]++; //add one to the quantity of the product with id $product_id
         break;
        
         case "remove":
             $_SESSION['cart'][$product_id]--; //remove one item
             if($_SESSION['cart'][$product_id] == 0) unset($_SESSION['cart'][$product_id]); //if the quantity is zero, remove it completely to avoid quantity being -1, -2 etc upon further pressing of the link
         break;
        
         case "empty":
             unset($_SESSION['cart']); //empty the cart.
         break;
    
     }

So the code adds to the session but also gives the user the chance to remove items or empty all the cart
After that code it displays the cart, obviously using the session but also the variable $product_id and the options from the switch

I dont know much about registering global variables? would this help, as I would only need it to remember $product_id
And will it remember the switch actions on another page?

The other thing.
I am still learning, but isnt this a hellish way of doing things? :..

The user clicks a link for a product. That link sends an ID to the URL, another page GETS the ID from the URL, then the script assigns the data from the GET to a variable, then it adds it to the session

There surely must be a better way of doing all this
Cant i just assign the ID value to the required variable WHEN the user clicks the link?

Cheers

pr0gr4mm3r — Fri, 06 Jul 2007 17:54:36 +0000

In your case, add this after your code that processes the addition of the item:

<?php
header('Location: cart.php');
?>

That will clear the $_GET data and solve your problem.