2016-12-13 6 views
-1

私のラーベルアプリケーションには奇妙な問題があり、バリデーションルールが適用されるとこのコードが2回呼び出されます。私は検証ロジックを別のクラスに抽象化しましたが、 APIは(とjQueryを使って、ポストマンを使用してみました)まだ出力はこのように見えるで二回実行するように表示されます。ラーバルリクエストのカスタムバリエーションクラスでの2回の呼び出し

called{"email":["The email has already been taken."],"country":["The country must be a number."]}called{"email":["The email has already been taken."],"country":["The country must be a number."]} 

私は一つだけJSONレスポンスを期待しています。私は髪を引き裂いて、2つの異なる接続で試してみて、なぜカスタムリクエストが2回呼び出されたのか分からないようです。これは新しいLaravelアプリなので、それと競合するコードはあまりありません。任意のアイデアを高く評価

//Create User Request extends standard request. Handles Validation 
    public function __construct(CreateUserRequest $request){ 
     $this->request = $request; 
    } 

    public function register() 
    { 
    try{ 

     $array = DB::transaction(function(){ 

      $email = $this->request->input('email'); 
      $password = $this->request->input('password'); 
      $companyName = $this->request->input('companyName'); 
      $userName = $this->request->input('name'); 
      $country = $this->request->input('country'); 

      $company = Company::create([ 
       'name' => $companyName, 
       'active'=>true, 
       'country_id'=>$country 
      ]); 

      $user = User::create([ 
       'company_id' => $company->id, 
       'name'=>'admin', 
       'email' => $email, 
       'password' => $password, 
       'active' =>true 
      ]); 

      if(!$company || !$user) 
      { 
       throw new \Exception('User not created for account'); 
      } 

      return compact('company', 'user'); 
      }); 

     $token = JWTAuth::fromUser($array['user']); 
     return Response::json(compact('token')); 

    } 
    catch(Exception $e) 
    { 
     return Response::json(['error' => $e->getMessage() ], HttpResponse::HTTP_CONFLICT); 
    } 

    } 

その後の検証カスタム要求..

namespace App\Http\Requests; 

use Illuminate\Foundation\Http\FormRequest; 
use Illuminate\Contracts\Validation\Validator; 
use Response; 
class CreateUserRequest extends FormRequest 
{ 
    /** 
    * Determine if the user is authorized to make this request. 
    * 
    * @return bool 
    */ 
    public function authorize() 
    { 
     return true; 
    } 

    public function response(array $errors) 
    { 
     // return Response::json(['errorg' => $errors ], 200); 
     echo('called'); 
    } 
    /** 
    * Get the validation rules that apply to the request. 
    * 
    * @return array 
    */ 
    public function rules() 
    { 
    return [ 
      'email' => 'required|unique:users', 
      'password' => 'required', 
      'companyName' => 'required', 
      'name' => 'required', 
      'country' => 'required|numeric' 
     ]; 
    } 
} 

答えて

2

興味深いです。

__construct()からCreateUserRequest $requestパラメータを削除して、register(CreateUserRequest $request)のようにregister()メソッドに追加してみてください。 $this->requestの代わりに$requestを呼び出してリクエストを使用してください。

+0

伝説!これは何をしていたか? – Paul

+0

リクエストクラスは、コントローラごとではなく、メソッドごとのリクエストを処理するように作られています。 リクエストクラスが__construct()で初めて呼び出され、別のリクエストクラスがメソッドへの実際のリクエストで起動したと思います。実際にはわからない。 –

+0

@Paulお願いします。頼むような怠惰な方法を使う前に、まずドキュメントをお読みください。 – Kyslik

関連する問題