質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

新規登録して質問してみよう
ただいま回答率
85.49%
Laravel

LaravelとはTaylor Otwellによって開発された、オープンソースなPHPフレームワークです。Laravelはシンプルで表現的なシンタックスを持ち合わせており、ウェブアプリケーション開発の手助けをしてくれます。

PHP

PHPは、Webサイト構築に特化して開発されたプログラミング言語です。大きな特徴のひとつは、HTMLに直接プログラムを埋め込むことができるという点です。PHPを用いることで、HTMLを動的コンテンツとして出力できます。HTMLがそのままブラウザに表示されるのに対し、PHPプログラムはサーバ側で実行された結果がブラウザに表示されるため、PHPスクリプトは「サーバサイドスクリプト」と呼ばれています。

Q&A

0回答

560閲覧

編集画面でのパスワードバリデーション

pup-ganbaru

総合スコア29

Laravel

LaravelとはTaylor Otwellによって開発された、オープンソースなPHPフレームワークです。Laravelはシンプルで表現的なシンタックスを持ち合わせており、ウェブアプリケーション開発の手助けをしてくれます。

PHP

PHPは、Webサイト構築に特化して開発されたプログラミング言語です。大きな特徴のひとつは、HTMLに直接プログラムを埋め込むことができるという点です。PHPを用いることで、HTMLを動的コンテンツとして出力できます。HTMLがそのままブラウザに表示されるのに対し、PHPプログラムはサーバ側で実行された結果がブラウザに表示されるため、PHPスクリプトは「サーバサイドスクリプト」と呼ばれています。

0グッド

0クリップ

投稿2020/12/11 04:36

編集2020/12/11 04:46

フォームリクエストバリデーション機能を実装したのですが、編集時にパスワードを何も触らずに、保存したところ自身のパスワードで値が重複したと判定されてバリデーションがかかってしまうので、自身のパスワードは除外するといったような記述をしたのですが、うまくいきません 

試したコードと結果
'cast_password' => [Rule::unique('casts', 'cast_password')->where('user_id', $user_id)->whereNull('deleted_at')->ignore($this->input('cast_password'))]

編集画面 何も変更せずに保存するとパスワードがすでに使われていますとバリデーションがかかってしまいます。

試したコードと結果
'cast_password' => [Rule::unique('casts', 'cast_password')->whereNot('cast_password', $this->input('cast_password'))->where('user_id', $user_id)->whereNull('deleted_at')]

バリデーションがそもそも効かなくなる

ソースコード

<?php namespace App\Http\Requests; use Illuminate\Foundation\Http\FormRequest; use Illuminate\Validation\Rule; use Illuminate\Support\Facades\Auth; use App\Models\Cast; use Illuminate\Http\Request; use App\Http\Controllers\CastController; class CastRequest extends FormRequest { /** * Determine if the user is authorized to make this request. * * @return bool */ public function authorize() { return true; } /** * Get the validation rules that apply to the request. * * @return array */ public function rules(Request $request) { // $castpw = CastController::$castpw; $cast = Auth::user(); // $cast = Cast::find($request)->cast_password; // dd($cast); $user_id = Auth::id(); return [ 'cast_password' => [Rule::unique('casts', 'cast_password')->where('user_id', $user_id)->whereNull('deleted_at')->ignore($this->input('cast_password'))] ]; } }

気になる質問をクリップする

クリップした質問は、後からいつでもMYページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

phper.k

2020/12/11 05:04 編集

パスワードにUNIQ制約をつけること自体がまちがいだと思います。 そうしてしまうと、特定のパスワードが使われていることが他のユーザーにばれてしまうのでログイン情報が流出することになります。 また、password_hash で保存することから鑑みても、UNIQチェックに全く意味のない処理であるため、そもそもなんのために制約しなければいけないのかと思います。
pup-ganbaru

2020/12/11 06:10

本当におっしゃるとおりです・・・ 仕様から見直したいと思います。ありがとうございます
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

まだ回答がついていません

会員登録して回答してみよう

アカウントをお持ちの方は

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
85.49%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問