Unique key validation showing error while updating data in laravel

Question

Hi I am creating an application using laravel and i have a model where asset_number field is unique and the validation rules are as follows

MODEL

protected $rules = array
        (

            'asset_number'              => 'required|alpha_dash|min:2|max:12|unique:asset_details,asset_number,{id}',
            'asset_name'                => 'alpha_space',
            'asset_type_id'             => 'required',
            'shift'                     => 'required',

        );

Here is my controller code for create and edit

CONTROLLER

public function postCreate()
    {

        // get the POST data
        $new = Input::all();

        // create a new Assetdetail instance
        $assetdetail = new Assetdetail();

        // attempt validation
        if ($assetdetail->validate($new))
        {

            // Save the location data
            $assetdetail ->asset_number             = e(Input::get('asset_number'));
            $assetdetail ->asset_name               = e(Input::get('asset_name'));
            $assetdetail ->asset_type_id            = e(Input::get('asset_type_id'));
            $assetdetail ->shift                    = e(Input::get('shift'));

            // Was the asset created?
            if($assetdetail ->save())
            {
                // Redirect to the new location  page
                return Redirect::to("admin/settings/assetdetails")->with('success', Lang::get('admin/assetdetails/message.create.success'));
            }
        }
        else
        {
            // failure
            $errors = $assetdetail->errors();
            return Redirect::back()->withInput()->withErrors($errors);
        }

        // Redirect to the location create page
        return Redirect::to('admin/settings/assetdetails/create')->with('error', Lang::get('admin/assetdetails/message.create.error'));

    }


    public function getEdit($assetdetailId = null)
    {
        // Check if the location exists
        if (is_null($assetdetail = Assetdetail::find($assetdetailId)))
        {
            // Redirect to the blogs management page
            return Redirect::to('admin/settings/assetdetails')->with('error', Lang::get('admin/assetdetails/message.does_not_exist'));
        }

        //$assetlife = DB::table('asset_details')->where('id', '=', $assetdetailId)->lists('asset_life');
        $location_list = array('' => '') + Location::lists('name', 'id');
        $assettype_list = array('' => '') + Assettype::lists('asset_type', 'id');
        $assignTo_list = array('' => 'Select a User') + User::select(DB::raw('CONCAT(first_name, " ", last_name) AS full_name'), 'id') ->lists('full_name', 'id');

        $assetdetail_options = array('' => 'Top Level') + DB::table('asset_details')->where('id', '!=', $assetdetailId)->lists('asset_number', 'id');

        return View::make('backend/assetdetails/edit', compact('assetdetail'))->with('assetdetail_options',$assetdetail_options)->with('location_list',$location_list)->with('assettype_list',$assettype_list)->with('assignTo_list',$assignTo_list);

    }


    public function postEdit($assetdetailId = null)
    {
        // Check if the location exists
        if (is_null($assetdetail = Assetdetail::find($assetdetailId)))
        {
            // Redirect to the blogs management page
            return Redirect::to('admin/settings/assetdetails')->with('error', Lang::get('admin/assetdetails/message.does_not_exist'));
        }

        $new = Input::all();

        /*if ($assetdetail ->asset_number   == Input::old('asset_number') && 
        {
          $assetdetail ->asset_number = e(Input::get('asset_number'));
        }*/

        if ($assetdetail->validate($new))
        {

            // Update the asset data
            $assetdetail ->asset_number             = e(Input::get('asset_number'));
            $assetdetail ->asset_name               = e(Input::get('asset_name'));
            $assetdetail ->asset_type_id            = e(Input::get('asset_type_id'));
            $assetdetail ->shift                    = e(Input::get('shift'));


            // Was the asset created?
            if($assetdetail->save())
            {
                // Redirect to the saved location page
                return Redirect::to("admin/settings/assetdetails/$assetdetailId/edit")->with('success', Lang::get('admin/assetdetails/message.update.success'));
            }
        }

            else
            {
            // failure
            $errors = $assetdetail->errors();
            return Redirect::back()->withInput()->withErrors($errors);
            }


        // Redirect to the asset management page with error
        return Redirect::to("admin/settings/assetdetails/$assetdetailId/edit")->with('error', Lang::get('admin/assetdetails/message.update.error'));

    }

My Route.php

Route::post('create', array('as' => 'savenew/assetdetail','uses' => 'Controllers\Admin\AssetdetailsController@postCreate'));
            Route::get('{assetdetailId}/edit', array('as' => 'update/assetdetail', 'uses' => 'Controllers\Admin\AssetdetailsController@getEdit'));
            Route::post('{assetdetailId}/edit', 'Controllers\Admin\AssetdetailsController@postEdit');

Now the problem is whenever i try to update my form my asset_number field throws an Error: The asset_number has already been taken.How do i pass the id in my controller while updating my form so that it throws error only if an already existing asset_number is used and not while editing the form with current asset_number.

I tried the following with no luck

Model

'asset_number' => 'mobileNumber' => 'required|min:5|numeric|unique:asset_details,asset_number,' . $id

Controller

Assetdetail::$rules['mobileNumber'] = 'required|min:5|numeric|unique:asset_details,asset_number,' . $id

@Kyslik updated my question with errors in bold letters please take a look — megamage, Nov 10 '14 at 10:51
hello, please take a look at this answer http://stackoverflow.com/a/24205849/1564365 — Kyslik, Nov 10 '14 at 12:45
@Kyslik I tried that but still same error exists....any other solution please — megamage, Nov 11 '14 at 06:43

score 0 · Accepted Answer · answered Nov 13 '14 at 09:10

Hi i finally found a working solution myself here is what i did please take a look

Controller

class AssetdetailsController extends AdminController 
 {

     protected $validationRules = array
        (   
            'asset_number'              => 'required|alpha_dash|min:2|max:12|unique:asset_details,asset_number',
            'asset_name'                => 'alpha_space',
            'asset_type_id'             => 'required',
            'shift'                     => 'required',
        );

    public function postCreate()
    {

        $validator = Validator::make(Input::all(), $this->validationRules);

        // If validation fails, we'll exit the operation now.
        if ($validator->fails())
        {
            // Ooops.. something went wrong
            return Redirect::back()->withInput()->withErrors($validator);
        }

        // attempt validation
        else
        {

            $assetdetail ->asset_number             = e(Input::get('asset_number'));
            $assetdetail ->asset_name               = e(Input::get('asset_name'));
            $assetdetail ->asset_type_id            = e(Input::get('asset_type_id'));
            $assetdetail ->shift                    = e(Input::get('shift'));

            // Was the asset created?
            if($assetdetail ->save())
            {
                // Redirect to the new location  page
                return Redirect::to("admin/settings/assetdetails")->with('success', Lang::get('admin/assetdetails/message.create.success'));
            }
        }


        // Redirect to the location create page
        return Redirect::to('admin/settings/assetdetails/create')->with('error', Lang::get('admin/assetdetails/message.create.error'));

    }

    public function postEdit($assetdetailId = null)
    {
        // Check if the location exists
        if (is_null($assetdetail = Assetdetail::find($assetdetailId)))
        {
            // Redirect to the blogs management page
            return Redirect::to('admin/settings/assetdetails')->with('error', Lang::get('admin/assetdetails/message.assetdetail_not_exist'));
        }

        $this->validationRules['asset_number'] = "required|alpha_dash|min:2|max:12|unique:asset_details,asset_number,{$assetdetail->asset_number},asset_number";

        $validator = Validator::make(Input::all(), $this->validationRules);


        // If validation fails, we'll exit the operation now.
        if ($validator->fails())
        {
            // Ooops.. something went wrong
            return Redirect::back()->withInput()->withErrors($validator);
        }

        else
        {
            $assetdetail ->asset_number             = e(Input::get('asset_number'));
            $assetdetail ->asset_name               = e(Input::get('asset_name'));
            $assetdetail ->asset_type_id            = e(Input::get('asset_type_id'));
            $assetdetail ->shift                    = e(Input::get('shift'));

            if($assetdetail->save())
            {
                // Redirect to the saved location page
                return Redirect::to("admin/settings/assetdetails/$assetdetailId/edit")->with('success', Lang::get('admin/assetdetails/message.update.success'));
            }
        }

        // Redirect to the asset management page with error
        return Redirect::to("admin/settings/assetdetails/$assetdetailId/edit")->with('error', Lang::get('admin/assetdetails/message.update.error'));

    }
}

Unique key validation showing error while updating data in laravel

1 Answers1