CODEWITHSUNDEEP
phplaravellaravel-5
Jija
Jija

Reputation: 277

Try catch for laravel is not working for duplicate entry

I am using below code in laravel controller. And getting duplicate error for username but I need to handle it by try-catch. This code is not working.

<?php
namespace App\Http\Controllers;
use Illuminate\Http\Request;
use App\Http\Requests;
use App\Http\Controllers\Controller;
use Response;
use DB;

class StaffController extends Controller
{
    public function saveMember(Request $request){
        $errormsg = "";
        $result = false;
        try{
            $result = DB::table('members')->insert(
                    [
                        'username' => $request->username,
                        'phone' => $request->phone,
                        'status' => 1
                    ]
                );
        }catch(Exception $exception)
        {
            $errormsg = 'Database error! ' . $exception->getCode();
        }
        return Response::json(['success'=>$result,'errormsg'=>$errormsg]);
    }
}

I am getting this error, which I need to handle by try and catch 

SQLSTATE[23000]: Integrity constraint violation: 1062 Duplicate entry 'test1' for key 'username'

Thanks for your help.

Upvotes: 5

Views: 12502

Answers (4)

Mohammad Ali Abdullah
Mohammad Ali Abdullah

Reputation: 331

   if($request->isMethod('post')){
        $data       = $request->all();
        $user       = Auth::user()->id;
        try {
            DB::beginTransaction();
            Company::Save($data,$user);
            BankBranch::updateBranch($data,$user,$id);
            DB::commit();
            $status = true;
        } catch (\Exception $e) {
            var_dump('Exception Message: '. $e->getMessage());
            var_dump('Exception Code: '. $e->getCode());
            var_dump('Exception String: '. $e->__toString());
            DB::rollback();
            $status = false;
        }
    }

Upvotes: 0

saber tabatabaee yazdi
saber tabatabaee yazdi

Reputation: 4959

according to this sample:

public function store(ServiceStoreRequest $request, int $id)
{
    $service = new Serviceable();
    $service->package_id = $id;
    $service->fill($request->all());
    try {
        $service->save();
    } catch (\Exception $exception) {
        return response(array(
            "code"=> 409,
            "error"=>"duplicate " . $exception->getMessage()));
    }
    return response($service);
}

Upvotes: 0

Idan Ptichi
Idan Ptichi

Reputation: 316

you can easily avoid this try & catch block by first validating the uniqness of the username against the desired column in your db table. you can do it with your $request object or (better) by setting a custom Request class that will do this validation before excecution the controller method. https://laravel.com/docs/5.1/validation#rule-unique

Upvotes: 4

Niklesh Raut
Niklesh Raut

Reputation: 34924

You need to make Exception as global,

  1. Either by using.

    use Exception;

and then use

catch(Exception $exception)

  1. Or by using

    catch(\Exception $exception)

Instead of this

catch(Exception $exception)

Upvotes: 18

Related Questions