Coupons! Adding the Form

Yo :)
It would be really handy if a user could remove a product from the cart :)
my cart looks like this: https://imgur.com/a/H6Y6l

now when someone clicks on the x the product should be removed from the cart and the cart should be updated.
my template looks like this:
https://imgur.com/a/H6Y6l

and i started the javascript like this:

$('.removeproduct').on('click', function(e) {
    e.preventDefault();
    //here is something missing :P

i dont know how to send a request to the server and then remove the product or subscription.
a hint for me?
Thanks for anything :)

Hey Blueblazer!
Nice idea, definitely an user will find it helpful

You need two things here, one in your template for sending an AJAX to the server, and a Route in your controller, for handling that request

If you are using Jquery you can use $.ajax() method for seding that request, like this:

$.ajax({
  url: "{{ path('order_remove_product', {id: product.id}) }}",
  method: "DELETE"
})
  .done(function( data ) {
    //data variable contains the response from the server
    //logic for showing a successful message to the user
  });

Yep, you can use twig inside javascript for simplifying some things, and you only have to adjust that path and product id

Controller:

    /**
     * @Route("/cart/product/{id}", name="order_remove_product")
     * @Method("DELETE")
     */
    public function removeProductAction(Product $product)
    {
        $this->get('shopping_cart')
            ->removeProduct($product);
        
        return new JsonResponse('Product removed!');
    }

You can read more about Jquery AJAX here:
http://api.jquery.com/jquery.ajax/

Cheers!

thanks :)
but how should the removeProduct function look like? :P

Hey Blueblazer,
Sorry! I forgot about that method :p
Your implementation looks fine to me, but I think this way may be faster

    public function removeProduct(Product $product){
        $products = $this->getProducts();
        
        if(in_array($product, $products)){
            unset($products[$product->getId()]);
            $this->updateProducts($products);
         }
    }

Products are only updated if the item was found, if it wasn't found you might want to throw an exception because it shouldn't happen

Cheers!

thanks you are awasome :)

but now i get this nasty error: http://imgur.com/a/hU9Kx why does it not recognice the product variable, because it is existing?

Hey Blueblazer172 ,

Please, ensure you're passing "product" variable in all places where you render "order/checkout.html.twig" template - I bet you just missed it somewhere.

Cheers!

yeah my problem is passing the variable to the template, but how can i if the user hasn't selected something.

i tried like this in ordercontroller.php:

/**
 * @Route("/checkout", name="order_checkout", schemes={"%secure_channel%"})
 * @Security("is_granted('ROLE_USER')")
 */
public function checkoutAction(Request $request, Product $product)
{
    if (!$products = $this->get('shopping_cart')->getProducts()) {
        $this->addFlash('warning', 'No Products in Cart');
        return $this->redirectToRoute('homepage');
    }

    $error = false;
    if ($request->isMethod('POST')) {
        $token = $request->request->get('stripeToken');

        try {
            $this->chargeCustomer($token);
        } catch (\Stripe\Error\Card $e) {
            $error = 'There was a problem charging your card: '.$e->getMessage();
        }

        if (!$error) {
            $this->em = $this->get('fos_user.user_manager');
            $role = $this->getUser()->addRole('ROLE_BUYER_USER');
            $this->em->updateUser($role);

            $this->get('shopping_cart')->emptyCart();
            $this->addFlash('success', 'Yay! Order fully completed!');

            return $this->redirectToRoute('homepage');
        }
    }

    return $this->render('order/checkout.html.twig', array(
        'products' => $products,
        'cart' => $this->get('shopping_cart'),
        'stripe_public_key' => $this->getParameter('stripe_public_key'),
        'error' => $error,
        'product' => $this->get('shopping_cart')->getProductId($product)
    ));

and the getProductId() funtion in the shoppingcart.php:

public function getProductId(Product $product)
{
    return $product->getId();
}

but then i get this error:

Unable to guess how to get a Doctrine instance from the request information for parameter "product".

Hmm, looks like you are mixing some things:
- You shouldn't need to pass the shopping cart to the template, only the products
- Instead of asking for product ID to the shopping cart, you can ask it directly to the product itself
- And if the user hasn't selected anything, then he can't remove anything :)

I hope this makes sense to you :)

Have a nice day!

Hey Blueblazer172
As Victor said, you might have forgotten to pass "product" variable to the template

Have a nice day!

currently my 2 functions in the shoppingcart look like this:

public function addProduct(Product $product)
{
    $products = $this->getProducts();

    if (!in_array($product, $products)) {
        $products[] = $product;
    }

    $this->updateProducts($products);
}

public function removeProduct(Product $product)
{
    $products = $this->getProducts();
    $id = $product->getId();

    if (($key = array_search($id, $products)) !== false) {
        unset($products[$key]);
    }

    $this->updateProducts($products);
}