How do I make a RemoveFromCart function? - php

Question

I've been struggling with a problem that doesn't allow me to delete my items from Cart. Whenever I try to do that, the error occurs with name "POST parameter id is required".
On the contary, it surprisingly works with adding items to the cart. This is the script I use for showing items in the cart, and deleting:
<div class="container">
<?php
foreach(Cart::GetRooms() as $room) {
?>
<div>
<div class="media">
<img class="align-self-start mr-3" src="<?=$room->GetData()["icon_url"]?>" alt="Room icon" width="200px">
<div class="media-body">
<h5 class="mb-0"><?=$room->GetData()["title"]?></h5>
<div class="clearfix">
<p>Cena: <span class="text-success font-weight-bold"><?=$room->GetData()["price"]?> <?=SHOP_CURRENCY?></span></p>
<form method="post" action="/api/cart/removeItem.php">
<input type="hidden" name="room_id" value="<?=$room->GetData()["id"]?>" />
<button class="btn btn-primary" type="submit">X</button>
</form>
</div>
</div>
</div>
</div>
<?php
}
?>
This is the script that is supposed to remove items from the cart
<?php
require_once "../../settings.php";
$id = POST("id", true);
Cart::RemoveFromCart((int) $id);
http_response_code(302);
header("Location: /cart.php");
public static function RemoveFromCart(int $id) : bool {
$db = MysqliDb::getInstance();
$db->where("id", $id);
$db->delete("cart");
return true;
}
and this is the POST function i wrote myself in case it was needed
function POST(string $key, bool $required=false, bool $secure=true) {
if(!isset($_POST[$key])) {
if($required) {
exit("POST parameter $key is required");
}
else {
}
}
$return_val = $_POST[$key];
if($secure) {
$return_val = htmlentities($return_val);
}
return $return_val;
}

Answer 1

in your form you have
<input type="hidden" name="room_id" value="<?=$room->GetData()["id"]?>" />
but in your script you are looking for a id property inside the request, so you should change it either to this
<input type="hidden" name="id" value="<?=$room->GetData()["id"]?>" />
or to this:
$id = POST("room_id", true);