If value exist in database don't save Laravel

Question

I'm creating a wishlist where the user can save his own products and save them. What I want is that a user can't save a product more than one time. So, what I'm doing is, if the product_id exists in the DB, I don't want to save the new record.

This is my code, what am I doing wrong?

public function addWishlist(Wishlist $wishlist) {        
    $wishlist->user_id = request('user_id');
    $wishlist->product_id = request('product_id');

    if(DB::table('wishlists')->where('product_id' !== $wishlist->product_id)) {
        $wishlist->save();
        return redirect()->back();
    }
}

There's already an eloquent function for that `updateOrCreate`. https://laravel.com/docs/5.6/eloquent — hungrykoala, Jun 18 '18 at 05:49
use firstOrCreate and then check for wasRecentlyCreated https://stackoverflow.com/questions/18498518/how-to-check-if-a-record-is-new-in-laravel — Payam Khaninejad, Jun 18 '18 at 07:31
`where('product_id' !== $wishlist->product_id)` -- what do you expect from this conditon? It will select all items from `wishlists`with the condition `true` — Nico Haase, Jun 19 '18 at 07:26

rkj · Accepted Answer · 2018-06-18T06:28:30.227

2

what about this

public function addWishlist(Wishlist $wishlist) {  
    $userId = request('user_id');
    $productId = request('product_id');

    $found = Wishlist::where('user_id', $userId)->where('product_id', $productId)->count();

    if($found == 0) {
        $wishlist->user_id = $userId;
        $wishlist->product_id = $productId;
        $wishlist->save();
        return redirect()->back();
    }
}

edited Jun 18 '18 at 06:28

answered Jun 18 '18 at 05:53

rkj

8,067
2
27
33

Thanks, but it would be a mess change my code for calling this function. – Daniel Hernandez Jun 18 '18 at 06:02
@danhergir i updated my answer, you can make these changes in your code. it should work – rkj Jun 18 '18 at 06:29
You did it. Thanks a lot. – Daniel Hernandez Jun 18 '18 at 06:32

score 0 · Answer 2 · answered Jun 18 '18 at 05:52

public function addWishlist(Wishlist $wishlist) {   

    if(DB::table('wishlists')->where(['user_id'=>$user_id,'product_id'=>$wishlist->product_id])->exists()){
         // record already exist 
         return redirect()->back();
    }else{
         // record not exist , save the record
         $withlist = new Wishlist();     
         $wishlist->user_id = request('user_id');
         $wishlist->product_id = request('product_id');
         $wishlist->save();
         return redirect()->back();
    }
}

I don't understand why, but it keeps saving the record if the value of the product_id is saved in the DB. — Daniel Hernandez, Jun 18 '18 at 06:00

score 0 · Answer 3 · answered Jun 18 '18 at 05:53

0

You can use firstOrNew and you need to check with user_id and product_id combination

    $wishlist::firstOrNew(array('user_id' => request('user_id'),'product_id'=>request('product_id')));
    $wishlist->user_id = request('user_id');
    $wishlist->product_id = request('product_id');
    $wishlist->save();
    return redirect()->back();

answered Jun 18 '18 at 05:53

JYoThI

11,977
1
11
26

It isn't working correctly. It's saving the record if the product_id is repeated. So, I'm having the same problem – Daniel Hernandez Jun 18 '18 at 05:57
Did you noticed this line `you need to check with user_id and product_id combination` ? @danhergir – JYoThI Jun 18 '18 at 05:59
I only need to check the product_id. I changed your code, but it's not working anyway – Daniel Hernandez Jun 18 '18 at 06:07
if your not check with user id means .if A user choose one product means . B user can't choose that product . is it okay ? @danhergir – JYoThI Jun 18 '18 at 06:11
No, what I mean is if a user added that product to the wishlist, that user can't do it again. – Daniel Hernandez Jun 18 '18 at 06:17
I can understood your point . if `user A` added `product_id` 1 .Now `user B` trying to add that the same `product_id` 1 . now you can't insert because that `product_id` 1 already exists in table that's why i'm telling you need to check with combination column `user_id` and `product_id` – JYoThI Jun 18 '18 at 06:22
use firstOrCreate and then check for wasRecentlyCreated https://stackoverflow.com/questions/18498518/how-to-check-if-a-record-is-new-in-laravel – Payam Khaninejad Jun 18 '18 at 07:30

score 0 · Answer 4 · answered Jun 19 '18 at 07:25

0

Have you tried firstOrCreate()

 $user = User::firstOrCreate(['email' => $email])

answered Jun 19 '18 at 07:25

Faizan Fayaz

540
5
16

If value exist in database don't save Laravel

4 Answers4