Q&A
データベースの materials テーブルの定義（特にuser_idの定義）を提示できますか？
前提・実現したいこと
資料の貸出管理のシステムを作っています。
Materialsテーブルのuser_idフィールドが空だったら貸し出していない状態
同user_idフィールドがセットされていたら、貸出状態という仕組みを考えています。
該当のソースコード
bin\cake bake all materials
を実行後、追加画面(add.ctp)をカスタマイズ中です。
add.ctp
PHP
1echo $this->Form->control('user_id', ['label' => '貸出先', 'options' => $users, 'empty' => '貸出なし']);
上記のようにemptyを指定して、Selectボックスの最初に「貸出なし」という項目をSelectボックスの最初に表示することに成功しました。
「貸出なし」以外のユーザーを選択しているときは何の問題もなくデータを追加できます。
しかし、「貸出なし」つまりEmptyの項目を選んでいると下記のエラーが発生します。
発生している問題・エラーメッセージ
Error: SQLSTATE[HY000]: General error: 1364 Field 'user_id' doesn't have a default value
気になるのはエラーメッセージに表示されているINSERT文にuser_idが見当たらないことです。
Emptyの値の場合、コントローラ側で何か処理をするべきなのか、
それともMaterialsTable.phpで定義を変えれば済む話なのかさっぱり分かりません。
CakePHPで開発を始めたばかりの手探り状態で、完全に行き詰りました。
皆さんのお知恵をお貸しください。
試したこと
Insert時に空の時のデフォルト値が指定されていないのが原因かと思い、テーブルのバリデーション定義を書き換えてみました。
MaterialsTable.php
PHP
1 $validator 2 ->integer('user_id') 3 ->allowEmptyString('user_id', null, 'create');
また、アソシエーションのjoinTypeにINNERを指定しているのが原因かと思い、LEFTに変えてみましたが変化ありませんでした。
PHP
1 $this->belongsTo('Users', [ 2 'foreignKey' => 'user_id', 3 'joinType' => 'LEFT' 4 ]);
質問の投稿以降に行った検証で気づいた点があります。
追加画面で貸出なし（empty）を選択しても、
MaterialsController.phpのadd()で
$material('user_id') =0;
と強制的に値を代入すれば、追加は成功するということです。
ソースコードは下記のとおりです。
PHP
1public function add() 2 { 3 $material = $this->Materials->newEntity(); 4 if ($this->request->is('post')) { 5 $material = $this->Materials->patchEntity($material, $this->request->getData()); 6 $material['status'] = ''; 7 $material['creator'] = $this->Auth->user('id'); 8 $material['user_id'] = 0; //この行が追加した部分です。 9 if ($this->Materials->save($material)) { 10 $this->Flash->success(__('資料情報は保存されました。')); 11 12 return $this->redirect(['action' => 'index']); 13 } 14 $this->Flash->error(__('資料情報は保存されませんでした。もう一度お試しください。')); 15 } 16 $users = $this->Materials->Users->find('list', ['limit' => 200]); 17 $this->set(compact('material', 'users')); 18 }
この結果から推測してみました。
貸出先でEmpty以外のユーザーが選ばれている場合、
$this->request->getData() にはuser_idが含まれた状態になります。
その値はpatchEntityによって、$material['user_id']に一括代入されるので、
無理に代入しなくても、正常にレコードの追加が実行されます。
しかし、貸出先でEmpty（貸出なし）を選択した場合、
$this->request->getData()にuser_idの項目が存在しなくなるのではないかと思います。
その場合、patchEntityも$material['user_id']に代入は行わず、
レコードの追加実行時に
「Field 'user_id' doesn't have a default value 」というエラーを出力するに至るのではないかと思うのです。
それで、前出のコードのようにpatchEntityの実施後、
$material['user_id'] = 0;
のように強制的に値を代入してやれば、レコードの追加実行時にもエラーにならずに済むということになります。
ということは$this->request->getData()に
user_id の値が含まれているかいないかを判定して、
含まれていない時のみ、貸出されていないことを示す 0 を強制的に代入すればいいはずです。
でも、その判定式の書き方が分かりません。
補足情報（FW/ツールのバージョンなど）
PHP 7.2.4
CakePHP 3.8.2
MySQL 8.0.17
bin\cake bake migration CreateMaterials name:string[100] user_id:integer status:string[10] created creator:integer
bin\cake bake migration CreateUsers username:string[100] password:string[100] sex:integer authority:string[10] status:string[10] created creator:integer
関連部分はこんな感じです。
migrationを使ってテーブル作りましたので、migrationに打ち込んだコマンドです。
MaterialsTable.phpではMaterials.user_idとUsers.idのアソシエーションが適切に設定されているように見えます。
回答2件
0
デフォルト値を与えたいのであれば以下のようarray_mergeでデフォルト値にPOST値をマージするのが簡単でしょう。
php
1$default = [ 2 'user_id' => 0; 3]; 4$data = array_merge($default, $this->request->getData()); 5 6$material = $this->Materials->patchEntity($material, $data);
投稿2019/09/11 03:51
総合スコア959
0
"Materialsテーブルのuser_idフィールドが空だったら貸し出していない状態" としたいのであれば、 materials.user_id のNOT NULL制約を外すのがよいと考えます。
sh
1bin/cake bake migration CreateMaterials name:string[100] user_id:integer status:string[10] created creator:integer
を実行した場合、マイグレーションファイルには以下のように書き出されているはずです。
php
1<?php 2use Migrations\AbstractMigration; 3 4class CreateMaterials extends AbstractMigration 5{ 6{ 7// （略） 8 $table->addColumn('user_id', 'integer', [ 9 'default' => null, 10 'limit' => 11, 11 'null' => false, // ここを true に置き換え 12 ]); 13// （略）
この 'null' => false, を 'null' => true, に置き換えて、user_idのNOT NULL制約を外しましょう。
投稿2019/09/11 03:45
総合スコア959
あなたの回答
tips
プレビュー
