Re: session_name("CCLTrolley")
| From: | John Taylor-Johnston | Date: | Sun, 02 Oct 2005 04:18:12 +0000 |
| Subject: | Re: session_name("CCLTrolley") | ||
| References: | 1 2 | Groups: | php.general |
| Request: | Send a blank email to php-general+get-223509@lists.php.net to get a copy of this message | ||
Robert Cummings wrote:
On Sat, 2005-10-01 at 23:57, John Taylor-Johnston wrote:P.S. do folks prefer I reply on top or on bottom?$TrolleyContents is a string. Basically what I want to accomplish here is if $TrolleyContents already exists append $AddToTrolley to $TrolleyContents, if not register $TrolleyContents. Am I going about it right? John <?php #printcontents.php session_name("CCLTrolley"); session_start(); if (isset($HTTP_POST_VARS["AddToTrolley"])) { $TrolleyContents = $TrolleyContents.",".$HTTP_POST_VARS["AddToTrolley"]; }else{ session_register("TrolleyContents"); } echo $TrolleyContents; phpinfo(); ?>Looks a bit odd to me :) But could be because you're using outdated semantics. It should be sufficient to do the following: <?php session_name( 'CCLTrolley' ); session_start(); // // Initialize the trolley. // if( !isset( $_SESSION['TrolleyContents'] ) ) { $_SESSION['TrolleyContents'] = ''; } // // Add new entry. // if( isset( $_POST['AddToTrolley'] ) ) { if( $_SESSION['TrolleyContents'] ) == '' ) {$_SESSION['TrolleyContents'] = $_POST['AddToTrolley'];} else {$_SESSION['TrolleyContents'] .= ','.$_POST['AddToTrolley'];} } echo $_SESSION['TrolleyContents']; phpinfo(); ?> Is there a reason you're using a comma delimited string? I would recommend using an array instead: <?php session_name( 'CCLTrolley' ); session_start(); // // Initialize the trolley. // if( !isset( $_SESSION['TrolleyContents'] ) ) { $_SESSION['TrolleyContents'] = array(); } // // Add new entry. // if( isset( $_POST['AddToTrolley'] ) ) { $_SESSION['TrolleyContents'][$_POST['AddToTrolley']] = $_POST['AddToTrolley'] } echo implode( ',', $_SESSION['TrolleyContents'] ); phpinfo(); ?> Why is it outdated semantics?