Remove all button won't work properly

[Cyjm1120]

Hi guys, I have stuck with this issue for over 3 hours now. My remove all button just wouldn't work.

 

Can anyone tell me what's the issue? I have 2 pages, one parent and another pop-up page that takes the values (cartSubmit) from parent. And I wrote out the code for both pages but I just cannot get it to work. 

I apologize in advanced if my code looks messy or doesn't make any sense.

 

Please take a look at the code and let me know if you could spot anything wrong with it. Any help is greatly appreciated.

 

Here is the code related to the remove all button in the parent page:

if(isset($_POST['finalSelected']))
{	echo "<br/>inside";
	if(!empty($_POST['fSelected']))
	{	echo "<br/>if NOT empty";
		$chosen = $_POST['fSelected'];
		foreach ($chosen as $item)
			echo "</br>aID selected: $item </br>";
		//implode function to transform an array to a string
		//chosen(array) to $delimit(String)
		$delimitStr = implode(", ", $chosen);
		//Save the selected String to SESSION
		if(!isset($_SESSION["final"]))
		{	
			$_SESSION["final"] .= "$delimitStr";
		}
		else
		{
			$_SESSION["final"] .= ", $delimitStr";
		}
		//Explode function to transform the SESSION variable back to an array for further trimming.
		$arraySession = explode(', ', $_SESSION["final"]);
		//Array unique to filter out duplicates.
		$uniqueArray = array_unique($arraySession);
		$cartSubmit = implode(", ",$uniqueArray);
		$_SESSION['cartSubmit'] = $cartSubmit;
	}
	else
	{
		echo "<br/>else empty";
		$noSubmit = $_SESSION["final"];
		//Explode function to transform the SESSION variable back to an array for further trimming.
		$arraySession = explode(', ', $_SESSION["final"]);
		//Array unique to filter out duplicates.
		$uniqueArray = array_unique($arraySession);
		$cartSubmit = implode(", ",$uniqueArray);
		$_SESSION['cartSubmit'] = $cartSubmit;
		
	}
}


<form name= "finalForm" method="POST" action="test6-1.php" target = "_self">
<input type="Submit" name="finalSelected"/>
<a href="javascript:popup('test6-3.php?aID=
<?php if(empty($_SESSION['final']) && empty($_SESSION['cartSubmit'])) echo "0";
else if(empty($cartSubmit)) echo "0"; else echo $_SESSION['cartSubmit'];?>')">Cart</a>


<?php if(isset($FinalName)) 
{ foreach($FinalName as $key => $item) 
	{?>
	<tr><td>
	<input type="checkbox" name="fSelected[]" value="<?php echo htmlspecialchars($FinalID[$key])?>" />
	<?php echo "$FinalID[$key] & $item";?>
	</td></tr>
<?php }
}?>
	</form>

And here is the code in the pop-up page.

<script> window.onunload = refreshParent;
function refreshParent() 
{
window.opener.location.reload();
}
</script>
if(isset($_POST['removeAll'])) 
{ $aID = "0"; 
unset($_SESSION['cartSubmit']); 
unset($_SESSION['final']); 
header("Location : test6-3.php?aID=0"); 
}

Edited November 18, 2014 by Cyjm1120

[Ch0cu3r]

 

 

Here is the code related to the remove all button in the parent page:

What do you mean by remove all?  What is being removed? Where and when does it need to be removed?

 

Also the code you posted is just weird. You're converting the data submitted by your form from an array into a comma delimited string. Then you explode it back to an array and apply array_unique then you implode it back into a comma delimited string and saving it to the session each time. Why? It will be a lot better to keep it as an array and just apply array unique and then save it the session. Sessions can store any data type not just strings.