SQLSTATE[22001]エラーの解消

Question

実現したいこと

新規登録時のエラーを解消したい

前提

Laravel8でSNSサイトを作っています。
新規登録時に下記のエラーが発生しました。
取得したデータ長がカラムの長さと一致しないためエラーが発生しているかと思いますが、
設計書通りに入力したはずなのにエラーが出ている理由がわかりません。
長さの認識が間違っているかもしれないので、説明いただけると助かります。

発生している問題・エラーメッセージ

SQLSTATE[22001]: String data, right truncated: 1406 Data too long for column 'password' at row 1 (SQL: insert into `users` (`name`, `email`, `password`, `updated_at`, `created_at`) values (test1, test1@test, $2y$10$37kkBJzrzcEAwN4TVpkZWe8A4Z7bAgFaHuQVo7.loTdpe6hyjwyge, 2023-09-26 14:00:15, 2023-09-26 14:00:15))

該当のソースコード

<?php

use Illuminate\Database\Migrations\Migration;
use Illuminate\Database\Schema\Blueprint;
use Illuminate\Support\Facades\Schema;
use Illuminate\Support\Facades\DB;

class CreateUsersTable extends Migration
{
    /**
     * Run the migrations.
     *
     * @return void
     */
    public function up()
    {
        Schema::create('users', function (Blueprint $table) {
            $table->increments('id');
            $table->string('name',12);
            $table->string('email',12)->unique();
            $table->string('password',12)->unique();
            $table->timestamps();
        });
    }

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

<?php

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

class ModifyUsersTable extends Migration
{
    /**
     * Run the migrations.
     *
     * @return void
     */
    public function up()
    {
        Schema::table('users', function (Blueprint $table) {
            $table->string('password')->nullable()->change();
        });
    }

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

試したこと

データ長については調べて理解しましたが、設計書通りにマイグレーションファイルを設定しているのにエラーが出ている理由がわかりません。

補足情報（FW/ツールのバージョンなど）

ER図
email
種類：テキストボックス
必須：○
長さ：12
データ型：文字列
password
種類：テキストボックス
必須：○
長さ：12
データ型：文字列
name
種類：テキストボックス
必須：○
長さ：12
データ型：文字列
パスワードはtest1234で試しています。

Answer 1

エラーメッセージから、パスワードの長さが12文字を超えているためにエラーが発生していることがわかります。

マイグレーションファイルでは、パスワードのカラムの長さを12文字に設定しています。しかし、実際に入力したパスワードは「test1234」で、これは12文字を超えています。そのため、データベースに保存する際に、パスワードの右側が切り捨てられてエラーが発生します。

解決策としては、パスワードの長さを12文字以下にするか、あるいはパスワードのバリデーションルールを変更して、12文字を超えるパスワードを入力できないようにする方法があります。

パスワードの長さを12文字以下にする場合、マイグレーションファイルを修正する必要があります。

// 2023_09_26_204941_modify_users_table.php

class ModifyUsersTable extends Migration
{
    public function up()
    {
        Schema::table('users', function (Blueprint $table) {
            $table->string('password', 12)->nullable()->change();
        });
    }

    // ...
}

パスワードのバリデーションルールを変更する場合、コントローラーやリクエストクラスでバリデーションを追加します。

// App\Http\Controllers\Auth\RegisterController.php

class RegisterController extends Controller
{
    public function register(Request $request)
    {
        $request->validate([
            'name' => 'required|string|max:12',
            'email' => 'required|email|unique:users',
            'password' => 'required|string|min:6|max:12',
        ]);

        // ...
    }
}

// App\Http\Requests\RegisterRequest.php

class RegisterRequest extends FormRequest
{
    public function rules()
    {
        return [
            'name' => 'required|string|max:12',
            'email' => 'required|email|unique:users',
            'password' => 'required|string|min:6|max:12',
        ];
    }
}

この場合、パスワードのバリデーションルールを「max:12」に変更します。これにより、12文字を超えるパスワードを入力できなくなります。
●●●●

Answer 2

$table->string('password',12)->unique();

上記マイグレーションのpassword定義を
$table->string('password')->nullable();に変更してください。

パスワード列はハッシュ化したものが入るので最大12文字としてしまうと足りなくなるので指定なしでよいです。

> Hash::make('test1234')
= "$2y$10$PM0Sec2I06vqgqE4xO8nqeO5wro6A8ubmsUBuEoEbN21h42QmKZ82"

指定なしだとMySQLではvarchar(255)という型になりハッシュ化したパスワードもサイズのエラーなく格納できます。

Comments

[moana]

ありがとうございます。
修正用ファイルを作ってphp artisan migrateをしてみたのですが、下記のエラーが出てしまいました。
RuntimeException

Changing columns for table "users" requires Doctrine DBAL. Please install the doctrine/dbal package.

at vendor/laravel/framework/src/Illuminate/Database/Schema/Grammars/ChangeColumn.php:30
26▕ */
27▕ public static function compile($grammar, Blueprint $blueprint, Fluent $command, Connection $connection)
28▕ {
29▕ if (! $connection->isDoctrineAvailable()) {
➜ 30▕ throw new RuntimeException(sprintf(
31▕ 'Changing columns for table "%s" requires Doctrine DBAL. Please install the doctrine/dbal package.',
32▕ $blueprint->getTable()
33▕ ));
34▕ }

+6 vendor frames
7 database/migrations/2023_09_26_204941_modify_users_table.php:16
Illuminate\Support\Facades\Facade::__callStatic("table")

+22 vendor frames
30 artisan:35

修正用ファイルの内容は質問内容のソースコードに追記しております。
これはコードが間違っているということでしょうか？

[niiyz]

カラム変更の処理にパッケージが必要です。
以下でインストールしてください。

composer require doctrine/dbal

エラーメッセージでググれば英語でも日本語でもかなりヒットする情報です。

Answer 3

クエリビルダーでプレースホルダー処理をしているということですよね？
基本的にはpasswordカラムに十分な長さのvarcharかtextをしてい指定することです

Comments

[moana]

後半の文に誤字がありどういう意味かわかりません。

Answer 4

SQLでは文字列はシングルクォートで囲む(シングルクォーティング)のがオキテ。
test1, test1@test, $2y$10$37kkBJzrzcEAwN4TVpkZWe8A4Z7bAgFaHuQVo7.loTdpe6hyjwyge, 2023-09-26 14:00:15, 2023-09-26 14:00:15
↓
'test1', 'test1@test', '$2y$10$37kkBJzrzcEAwN4TVpkZWe8A4Z7bAgFaHuQVo7.loTdpe6hyjwyge', '2023-09-26 14:00:15', '2023-09-26 14:00:15'

Comments

[moana]

エラー文に書いてあるのは実際にローカル環境で作成したサイトで新規登録しようとした際に入れた値です。(test1が名前、test1@testがメールアドレス、暗号化されているのがパスワード)
なのでシングルクオーテーションで囲むのは違うと思います。カラム名は記載の通りシングルクオーテーションで囲まれていますし…

[Orlofsky]

>SQLでは文字列はシングルクォートで囲む(シングルクォーティング)のがオキテ。