This is prob really simple for you guys but help needed really fast

rn14 · February 7, 2007

My problem is whenever I run the code below I get the following error

Warning: Invalid argument supplied for foreach() in /home/seachange/domains/sea-change.ie/public_html/cart1/show_cart.php on line 122

However when I just run the php part of this code on its own it runs perfectly and executes everything.

As you prob can see Im a beginner but if their is a fix to this please let me know

<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN">

<html>

<head>

<title>Untitled Document</title>

.newStyle1 {

border-right-style: dotted;

border-right-width: thin;

border-right-color: #0000FF;

}

.newStyle2 {

border: thin solid #0000FF;

}

.style4 {

border: 0px solid #0000FF;

}

.style6 {

border: 1px solid #0000FF;

}

.newStyle3 {

border-width: thin;

border-color: #0000FF;

}

.style10 {

color: #0000FF;

}

.style16 {

text-decoration: none;

}

.newStyle4 {

background-color: #3366FF;

}

.newStyle5 {

background-color: #FFFFFF;

}

</style>

</head>

<tr>

<tr>

<tr>

</td>

</tr>

</table>

<tr>

</tr>

<tr>

About Us</a></td>

</tr>

<tr>

Our Events</a></td>

</tr>

<tr>

Newsletter</a></td>

</tr>

<tr>

Donate Online</a></td>

</tr>

<tr>

Our Work</a></td>

</tr>

<tr>

Contact Us</a></td>

</tr>

</table></td>

<p align="left </p><?session_start();

?>

"><?

include ('book_sc_fns.php');

// The shopping cart needs sessions, so start one

if($new)

{

//new item selected

if(!session_is_registered("cart"))

{

$cart = array();

session_register("cart");

$items = 0;

session_register("items");

$total_price = "0.00";

session_register("total_price");

}

if($cart[$new])

$cart[$new]++;

else

$cart[$new] = 1;

$total_price = calculate_price($cart);

$items = calculate_items($cart);

}

if($save)

{

foreach ($cart as $isbn => $qty)

{

if($$isbn=="0")

unset($cart[$isbn]);

else

$cart[$isbn] = $$isbn;

}

$total_price = calculate_price($cart);

$items = calculate_items($cart);

}

do_html_header("Your shopping cart");

if($cart&&array_count_values($cart))

display_cart($cart);

else

{

echo "<p>There are no items in your cart";

echo "<hr>";

}

$target = "index.php";

// if we have just added an item to the cart, continue shopping in that category

if($new)

{

$details = get_book_details($new);

if($details["catid"])

$target = "show_cat.php?catid=".$details["catid"];

}

display_button($target, "continue-shopping", "Continue Shopping");

$path = $PHP_SELF;

$path = str_replace("show_cart.php", "", $path);

display_button("https://".$SERVER_NAME.$path."checkout.php", "go-to-checkout", "Go To Checkout");

do_html_footer();

?> </p>

<p><br>

</p>

</div>

</td>

</tr>

</table>

<hr>

</tr>

</table>

Jessica · February 7, 2007

Before the foreach on line 20, check that the array being sent has values.

if(count($array) > 0){

//foreach

}else{

//oops!

}

rn14 · February 7, 2007

Sorry Im really a beginner where exactly will I put this?

Please be patient!

I think it doesnt pass them when I have the html code but when I just hve the php code its all good

Jessica · February 7, 2007

Before the foreach on line 20

Are you saving it as a .php file?

rn14 · February 7, 2007

Yes

Jessica · February 7, 2007

You posted way too much code, and didn't use the code tags, so it's annoying to try to edit it.

As I said twice, before the foreach on the line which is causing problems, add another if statment to check the values of the array. It seems the array is empty.

rn14 · February 7, 2007

Added the php code on its own. no its not receiving values. When I use this php code on its own it works perfectly

<?session_start();

?>

"><?

include ('book_sc_fns.php');

// The shopping cart needs sessions, so start one

if($new)

{

//new item selected

if(!session_is_registered("cart"))

{

$cart = array();

session_register("cart");

$items = 0;

session_register("items");

$total_price = "0.00";

session_register("total_price");

}

if($cart[$new])

$cart[$new]++;

else

$cart[$new] = 1;

$total_price = calculate_price($cart);

$items = calculate_items($cart);

}

if($save)

{

foreach ($cart as $isbn => $qty)

{

if($$isbn=="0")

unset($cart[$isbn]);

else

$cart[$isbn] = $$isbn;

}

$total_price = calculate_price($cart);

$items = calculate_items($cart);

}

do_html_header("Your shopping cart");

if($cart&&array_count_values($cart))

display_cart($cart);

else

{

echo "<p>There are no items in your cart";

echo "

--------------------------------------------------------------------------------

";

}

$target = "index.php";

// if we have just added an item to the cart, continue shopping in that category

if($new)

{

$details = get_book_details($new);

if($details["catid"])

$target = "show_cat.php?catid=".$details["catid"];

}

display_button($target, "continue-shopping", "Continue Shopping");

$path = $PHP_SELF;

$path = str_replace("show_cart.php", "", $path);

display_button("https://".$SERVER_NAME.$path."checkout.php", "go-to-checkout", "Go To Checkout");

do_html_footer();

?>

Jessica · February 7, 2007

Your original post doesn't have session_start() on the top.

Please use the code tags! Highlight your code and hit the # button.

rn14 · February 7, 2007

This is the code that runs the script correctly.

Whenever i add any html to the script it produces that error

<?session_start();
?>
"><?
  include ('book_sc_fns.php');
  // The shopping cart needs sessions, so start one
  
  if($new)
  {
    //new item selected
    if(!session_is_registered("cart"))
    {
      $cart = array();
      session_register("cart");
      $items = 0;
      session_register("items");
      $total_price = "0.00";      
      session_register("total_price");
    }
    if($cart[$new])
      $cart[$new]++;
    else 
      $cart[$new] = 1;
    $total_price = calculate_price($cart);
    $items = calculate_items($cart);

  }
  if($save)
  {
   foreach ($cart as $isbn => $qty)
    {
    if($$isbn=="0")
     unset($cart[$isbn]);
    else 
     $cart[$isbn] = $$isbn;
    }
    $total_price = calculate_price($cart);
    $items = calculate_items($cart);
}

  do_html_header("Your shopping cart");

  if($cart&&array_count_values($cart))
    display_cart($cart);
  else
  {
    echo "<p>There are no items in your cart";
    echo "
--------------------------------------------------------------------------------
";
  }
  $target = "index.php";

  // if we have just added an item to the cart, continue shopping in that category
  if($new)
  {
    $details =  get_book_details($new);
    if($details["catid"])    
      $target = "show_cat.php?catid=".$details["catid"]; 
  }
  display_button($target, "continue-shopping", "Continue Shopping");  
  $path = $PHP_SELF;
  $path = str_replace("show_cart.php", "", $path);
  display_button("https://".$SERVER_NAME.$path."checkout.php", "go-to-checkout", "Go To Checkout");  
  do_html_footer();
?>

Jessica · February 7, 2007

If you're only going to read half of my post each time this is going to take forever.

Your original post doesn't have session_start(); When you add your HTML, add it AFTER your session_start(); Try it then.

Where does $cart get defined?

rn14 · February 7, 2007

That fixed it. Sometimes I wonder about myself!!! Just looked at your site you seem to be a bit of a legend with a very good attitude. Im from Ireland fairly big into rugby so look after myself too. If u have any tips on learning php or just in general please let me know!! Thanks again for your help!

Jessica · February 8, 2007

Wow, I think you may be the first person EVER to say I have a good attitude

I've found the best way to learn is to code as much as possible, so just keep working on it. Find fun projects to do.

Sign In

This is prob really simple for you guys but help needed really fast

Recommended Posts

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Join the conversation

Important Information