Laravel SQLSTATE [23000]:完整性约束违规:1062重复条目

时间:2014-07-25 05:16:33

标签: php authentication laravel-4

我是Laravel的新手,这是我在Laravel的第一个项目。平时,首先我正在开发一个完整的用户身份验证系统。我可以注册一个用户,可以发送用户验证电子邮件,点击后链接我可以激活一个新的用户帐户,可以登录并可以注销。但是,当我尝试注册另一个新用户和点击验证链接后,我面临一个例外,即< / p>

Illuminate \ Database \ QueryException
SQLSTATE[23000]: Integrity constraint violation: 1062 Duplicate entry '' for key     'users_code_unique' (SQL: update `users` set `code` = , `active` = 1, `updated_at` = 2014-07-   25 04:26:06 where `id` = 41)

现在这是我的route.php,

<?php

Route::get('/',array(
     'as'   =>'home',
     'uses' =>'HomeController@index'
     ));

Route::get('/signin',array(
    'as'        =>'signin',
    'uses'      =>'AccountController@signinGet'
    ));

Route::get('/signup',array(
    'as' => 'signup',
    'uses' => 'AccountController@signupGet'
    ));

/* 

/* 
/Authenticated Group
*/
 Route::group(array('before' => 'auth'),function(){
 /* 
 /Sign Out(GET)
*/

 Route::get('/signout',array
    (
        'as' => 'signout',
        'uses' => 'AccountController@signoutGet'
    ));

 });

/* 
/UnAuthenticated Group
*/
Route::group(array('before' => 'guest'),function(){

/* CSRF Protect*/
Route::group(array('before' => 'csrf'),function(){
        /*
        / Create Account(POST)
        */
        Route::post('/signup',array(
                'as'=> 'signup',
                'uses'=>'AccountController@signupPost'
            ));

        /*
        / Sign In(POST)
        */

        Route::post('/signin',array(
                'as' => 'signin-post',
                'uses' => 'AccountController@signinPost'
            ));
 });

 /* 
 / Sign In (GET) 
 */

 Route::get('/signin',array(
        'as' => 'signin',
        'uses' => 'AccountController@signinGet'
    ));

 /* 
 /Create Account(GET) 
 */
 Route::get('/signup',array(
        'as' => 'signup',
        'uses'=> 'AccountController@signupGet'
    ));
 Route::get('signup/account/activate/{code}',array(
        'as'        =>'activate-account',
        'uses'      =>'AccountController@activatePost'
    ));
 });
 ?>

这是我的AccountController

<?php

class AccountController extends \BaseController {

public function signinGet()
{
    return View::make('account.signin');
}

public function signinPost(){

    $validator = Validator::make(Input::all(),array(
            'email' => 'required|email',
            'password' => 'required'
        ));

    if($validator->fails()){
        //redirect to the signin page
        return Redirect::route('signin')
                ->withErrors($validator)
                ->withInput();
    }else{
        //Attempt user singin

        $auth = Auth::attempt(array
            (
                'email' => Input::get('email'),
                'password' => Input::get('password'),
                'active' => 1
            ));

        if($auth){
            //Redirect To intented URL
            return Redirect::intended('/');
        }
        else
        {

            return Redirect::route('signin')
                                ->with('global','The username or password you provided is wrong or account not activated!');
        }

    }

            return Redirect::route('signin')    
                                ->with('global','There is a problem Signing You in.');
}


/**
 * Show the form for creating a new resource.
 *
 * @return Response
 */

public function signupGet()
{
    return View::make('account.signup');
}

public function signupPost()
{
    $validator = Validator::make(Input::all(), array(

        'email'             => 'required|max:255|email|unique:users',
        'username'          => 'required|min:3|unique:users',
        'password'          => 'required|min:6',
        'password_again'    =>  'required|same:password'

        )
    );

    if($validator->fails())
    {
        return Redirect::route('signup')
            ->withErrors($validator)
            ->withInput();
    }else
    {
        $email          = Input::get('email');
        $username       = Input::get('username');
        $password       = Input::get('password');

        //Activation Code
        $code = str_random(60);

        $user = User::create(array(
                    'email'     => $email,
                    'username'  => $username,
                    'password'  => Hash::make($password),
                    'code'      => $code,
                    'active'    => 0                
                    )
        );

        if($user){
            //User Activation Code Creation
            Mail::send('emails.auth.activate', array('link' => URL::route('activate-account',$code), 'username' => $username),function($message) use ($user)
                {
                    $message->to($user->email,$user->username)->subject('Activate Your Account');
                });

            return Redirect::route('signup')
                            ->with('global','Your Account has been created! We have sent you an email to activate your account.Please Check the both the Inbox and Spam Folder.');

        }

    }


    //return 'This is a Post Result';
}

public function activatePost($code){

    $user = User::where('code','=',$code)->where('active','=',0);
    if($user->count()){
        $user = $user->first();

        $user->active = 1;
        $user->code = '';
        if($user->save()){
            return Redirect::route('home')
                            ->with('global','Activated!.You can sign in now!'); 
        }
    }

    else{
        return Redirect::route('signup')
                        ->with('global','Sorry!We could not activate your acount,please try again later.');
    }
}


public function signoutGet(){

    Auth::logout();
    return Redirect::route('home');
}
}
?>

这是我的创建用户迁移文件

<?php

use Illuminate\Database\Schema\Blueprint;
use Illuminate\Database\Migrations\Migration;

class CreateUsersTable extends Migration {

public function up()
{
    Schema::create('users', function(Blueprint $table)
    {
        $table->increments('id');
        $table->string('username',255)->unique();
        $table->string('email',255)->unique();
        $table->string('password',60);
        $table->string('password_temp',60);
        $table->string('code',60)->unique();
        $table->integer('active');
        $table->timestamps();
    });
}

/**
 * Reverse the migrations.
 *
 * @return void
 */
public function down()
{
    Schema::drop('users');
}

}
?>

这是我的user.php

<?php

use Illuminate\Auth\UserTrait;
use Illuminate\Auth\UserInterface;
use Illuminate\Auth\Reminders\RemindableTrait;
use Illuminate\Auth\Reminders\RemindableInterface;

class User extends Eloquent implements UserInterface, RemindableInterface {


    public function getRememberToken()
    {
        return $this->remember_token;
    }

    public function setRememberToken($value)
    {
        $this->remember_token = $value;
    }

    public function getRememberTokenName()
    {
        return 'remember_token';
    }

protected $fillable = array('email','username','password','password_temp','code','active');

use UserTrait, RemindableTrait;

/**
 * The database table used by the model.
 *
 * @var string
 */
protected $table = 'users';



/**
 * The attributes excluded from the model's JSON form.
 *
 * @var array
 */
protected $hidden = array('password', 'remember_token');

}
?>

现在问题是什么?

4 个答案:

答案 0 :(得分:6)

确保您的code字段为nullable,然后将其设置为空字符串,而不是将其设为空:

$code = null;

然后,您将能够将其保存为NULL(MySQL),同时保持唯一。


同样改变这个:

 $user = User::where('code','=',$code)->where('active','=',0);
   if($user->count()){
     $user = $user->first();

要:

 $user = User::where('code','=',$code)->where('active','=',0)->first();
   if(count($user)){

您不需要两次调用db,只需检查返回的结果是否为空(count即可),这意味着它返回了一个User对象。

答案 1 :(得分:1)

我找到了。您已将代码列设置为唯一,但在用户单击激活链接后将其设置为空字符串。表格中已经有一行代码=&#39;&#39 ;;所以它抛出一个错误。问题出在这里(activatePost):

$user->code = '';

因此,要么将其清空,将其设置为其他内容,要么将db colums设置为不唯一。

我会保留代码而不会清空它,另外我会检查用户是否被激活 - 在activatePost中是一个简单的if。也许最好不仅根据代码验证用户,还要在链接中使用散列ID进行验证。

答案 2 :(得分:1)

您的问题可以使用验证器进行检查。只需使用:

use Validator;
use Request;
//...
//unique will pre check the key code weather if unique in tbl_name
public function yourfunc(Request $request) {
    // set the rules to check
    $rules = ['code'=>'required|unique:tbl_name'];
    $validator = Validator::make($request->all(), $rules);
    if ($validator->fails()) {
       // handler errors
       $erros = $validator->errors();
    }
    //... everything is ok here
}

您可以在laravel validation

了解详情

答案 3 :(得分:0)

您需要做一些改进代码的事情。但是,当您将列设置为唯一并尝试将相同数据重新插入另一行时,通常会发生重复条目。最让人困惑的是当你检查你的桌子并找到空列时。呐喊!当列设置为唯一且为空时,表示没有其他列可以包含空数据。

简单来说,列的形式不能复制,无论是null还是数据。