【laravel】論理削除（ソフトデリート）したユーザをログイン不可にしたい

Question

いつもお世話になります。

標記の件ですが、下記サイトを参考にコーディングしてみました。

Laravelの認証機能でユーザ情報の論理削除（ソフトデリート）を実装する

・・・が、ソフトデリートしたユーザのメールアドレス、パスワードを
ログイン画面で入力するとログインできてしまいます。

そもそも論でお聞きしたいのですが、ソフトデリートしたユーザであっても
ログイン可能なのは仕様なのでしょうか？

ちなみに環境は以下の通りです。
Laravel 5.5
PHP 7.2.7
MySQL 5.7.22

以上、よろしくお願い致します。

※以下、コーディングした内容です。
マイグレーションファイルの作成、マイグレートの実行
$ php artisan make:migration add_column_softDeletes_users_table --table=users
$ view 2018_07_12_045301_add_column_soft_deletes_users_table.php

<?php use Illuminate\Support\Facades\Schema; use Illuminate\Database\Schema\Blueprint; use Illuminate\Database\Migrations\Migration; class AddColumnSoftDeletesUsersTable extends Migration { /** * Run the migrations. * * @return void */ public function up() { Schema::table('users', function (Blueprint $table) { $table->softDeletes(); }); } /** * Reverse the migrations. * * @return void */ public function down() { Schema::table('users', function (Blueprint $table) { $table->dropColumn('deleted_at'); }); } }

モデルの作成
$ php artisan make:model Models/Users
$ view Users.php

<?php namespace App\Models; use Illuminate\Database\Eloquent\Model; use Illuminate\Database\Eloquent\SoftDeletes; class Users extends Model { use SoftDeletes; protected $table = 'users'; protected $dates = ['deleted_at']; }

下記SQL文にて、ユーザをソフトデリートしました。

update users set deleted_at = '2018-07-12 01:03:26' where id = 1;

前項でソフトデリートしたユーザでログインを試みるとログインできてしまいます。

Comments

[退会済みユーザー]

〈参考にコーディング〉した結果が思い通りになってないのなら、どうコーディングしたかソースコードを示してもらわないと誰もわからないのではないかと。

[rx179mk5]

大変失礼しました。加筆致しましたので、ご確認をお願い致します。

[退会済みユーザー]

示されたコードには、ログインユーザーの検証部分が抜けています。追記を。マイグレーション部分は確実にやってあれば不要な部分。別手段（mysqlならphpmysqlなど）で狙ったとおりのテーブル構造になっているかを検証すること。

[rx179mk5]

ありがとうございます。追加で確認してみます。

[rx179mk5]

検証部分でありますが、そこはフレームワークが吸収してくれていると考えております。また、テーブル構造は、期待した通りになってました。

Answer 1

あなたが仕様を決めて、あなたがコーディングしているのであれば、
デバッグするのもあなたです。
第三者には何もわかりようがありません。

きちっとどういう仕様で処理を組み込むかを決めましょう。
その仕様を満たしているかどうかを、どうテストすればいいかもいっしょに考えて決めておきます。
あとはそれに向かって必要なコードやデータを揃えるだけです。

その参考サイトのページを見ると、
テーブルusersにdeleted_atというTIMESTAMP型のカラムが追加され、
初期値はNULLだがユーザー削除したら削除した日時が格納されるようになっているようです。

論理削除されたユーザーをデータベースから取得する際には、

public function getUsers()
{
  $users = DB::table($this->table)
    ->whereNull('deleted_at')
    ->get();
  return $users;
}

ってサンプルコードも示されているので、お好きなように組み込んでください。

Answer 2

ソフトデリート用のフラグをたてるカラムをログイン時にチェックすれば
よいでしょう。
それが難しそうなら削除したユーザーを別のテーブルに移動するなど
ソフトデリートの考え方を変えてください

Comments

[rx179mk5]

回答ありがとうございます。検討してみたいと思います。