Cakephp3でアソシエーション使って３つのテーブルに一括入力したいです。

Question

こんにちは、現在受講管理システムを作っています。
Cakephp3 のアソシエーションというものを使うと一括でできると聞いたのでマニュアルにそって試してみましたが、だめでした。POST値は正しく受け取っていますが、SaveMany()がTrueになるのにDBには何もInsertされていません。

TableはSubjectとsubject_equipment,subject_participantの３つで、DB上には関連キーは設定していません。
やりたいのは、１つのページのPOSTデータを分けて３つのテーブルに入れたいです。

SubjectTableでは以下のようにアソシエーションを設定しました。JoinTableも作りました。

class SubjectTable extends Table
{  public function initialize(array $config)
    {
        parent::initialize($config);

        $this->setTable('subject');
        $this->setDisplayField('subject_idx');
        $this->setPrimaryKey('subject_idx');

        $this->addBehavior('Timestamp');

        $this->belongsToMany('SubjectEquipment', ['foreignKey' => 'subject_idx', 'joinTable' => 'subject_equipment']);
        $this->belongsToMany('SubjectParticipant', ['foreignKey' => 'subject_idx', 'joinTable' => 'subject_participant']);

    }

add.ctpでは要素ごとに名前をつけておきました。データはちゃんと取れているので問題ないかと思います。

<?= $this->Form->hidden('subject.subject_idx',['value' => 1]); ?>
<?= $this->Form->multiCheckbox('subject_equipment.equipment_idx',$equipments)?>
<?= $this->Form->control('subject_participant.participant_idx',['options' =>$participants, 'label'=>false, 'multiple' => true])?>

コントローラーに問題がありそうですがなかなか見つかりません。PatchEntityをPatchEntitiesにしたり、saveMany()をsave()に変えたりするとerrorのFlashメッセージが示されます。このコードでは、SuccessのFlashメッセージが表示されますがDBには何も起こりませんでした。

    public function add()
    {
        $subject = $this->Subject->newEntity();

　　　　　if ($this->request->is('post')) {
            $subject = $this->Subject->patchEntity($subject, $this->request->getData(),[
              'associated' => [
                'SubjectEquipment',
                'SubjectParticipant',
              ]
            ]);
            
            if ($this->Subject->saveMany($subject)) {
                $this->Flash->success(__('The subject has been saved.'));
                return $this->redirect(['action' => 'index']);
            }
            $this->Flash->error(__('The subject could not be saved. Please, try again.'));
        }
        $this->set(compact('subject'));
        $this->set('_serialize', ['subject']);

アソシエーションがまだ理解できてないのもありますが、マニュアルが分かりにくく同じくしてみても動きませんでした。
どこか間違っているところがありましたら、教えていただけますでしょうか。
よろしくお願いいたします。

Answer 1

回答依頼が来ていたので、分かる範囲で回答しますね...

まず、コントローラ側の$this->Subject->saveMany($subject)は$this->Subject->save($subject)が正しいと思います。保存しようとしてるエンティティは1つだからです。アソシエーションを一緒に保存する場合でもsave()です。

後は、テンプレート側も問題だと思います。$this->Form->control()の第一引数であるフィールド名の指定の仕方は若干ややこしいです。

• subject.subject_idxは、subject_idxでいいはずです。subject自身であれば、subject.を指定しません。
• subject_equipment.equipment_idxは、subject_equipments._idsかsubject_equipment._idsだと思います。

※_idsというのはIDのリストを意味する特別な単語です
※通常、複数形なんですがモデル自体が単数形になっているので、単数形の可能性もあるかもしれません
※subject_participant.participant_idxも同じような感じかと

CakePHPのマニュアルとしては、BelongsToMany アソシエーションの保存が参考になると思います。

追記
まずは、フォームの$this->request->getData()を使わず、以下の配列の値で保存できるかやってみたらどうでしょうか

$data = [
    'subject_idx' => 1,
    'subject_equipments' => [
        '_ids' => [1, 2, 3, 4]
    ]
];

Comments

[lovelydai]

誠にありがとうございます。大変参考になりました。また試してみます。

[popobot]

もしうまくいかなかったら、マニュアルのように、まずはFormを使わないで、コントローラで配列を用意して保存できるか試してみるといいかと思います。それがうまく行ったら、フォームからその配列が作れているか調べれてみてください。

[lovelydai]

テストしてみましたら、Subjectテーブルにはデータが正しく入りますが、残り２つのテーブルには入りませんでした。これをBelongsToにすると1列だけデータが入ります。複数の選択値があるのでなかなかうまくいかないです。涙

[lovelydai]

追記技術ありがとうございます。他の値は大丈夫でしたが、subject_equipment._idsで作られた連想配列が問題ではないかと思います。その配列に入る値は、subject_equipmentテーブルのequipment_idxフィールドに入れたいし、そのフィールドはnotEmptyで必須項目です。その配列の使い方とチェックボックスなどのMultipleFormがよく分からなくてどうすればいいのか悩んでいます。Foreachなどのループを使えばいいのか、別のオプションがあるのか、でまた色々さがしています。

[lovelydai]

データはこのように取れています。

subject_equipment (array)
_ids (array)
0 1
1 2
2 4
subject_idx 1
year 2017
customer_idx 1
creater keadmin
modifier keadmin
subject_participant (array)
_ids (array)
0 1
1 2
subject_idx 1
year 2017
customer_idx 1
creater keadmin
modifier keadmin

あの_idsの配列にあるデータをそれぞれのテーブルの名前‗idxフィールドに入れるとうまく行けそうな気がしますがどうすればいいのかまったく気がつきません。またアドバイスいただければ幸いです。よろしくお願いいたします。

[lovelydai]

何度もすみません。同じコードでbelongsToに変えて動かしてみたら、配列の中で1番目のデータはテーブルにInsertされました。この場合、equipment_idxフィールドはNULLでした。どうしてもあの_ids配列が問題で、その値をなんとかしてequipment_idxに入れるとよさそうですが…

[popobot]

まず、subject_equipmentをsubject_equipmentsに変えてやってみてください。最後にsがついていて複数形です。belongsToは1つなので単数形で、belongsToManyは複数なので複数形です。
あと、NULLになっている問題ですが、Table側のアソシエーションの設定が正しくないからかもしれません
以下のマニュアルにオプションがあります。
https://book.cakephp.org/3.0/ja/orm/associations.html
bindingKeyやtargetForeignKeyにequipment_idxを指定する必要がありそうですね

[popobot]

ちなみに、テーブル名やカラム名はなるべくCakePHPの規約に則ったほうが、設定が少なくてすみますし、ハマることも減ります。今からでも変更できるのであれば、そこから直した方が早いかもしれません
https://book.cakephp.org/3.0/ja/intro/conventions.html#model-and-database-conventions
※もちろん、規約にそってなくても正しく設定すれば動きますけど

[lovelydai]

icchiiさん、本当にありがとうございます。しかし、教えていただいたこと試してみましたが、まだうまくできません。DebugのSQLを見ると、subject_equipmentテーブルへのInsert文自体がでてきません。また、ひとつ気になったのが、今私がやりたいのは2つのbelongsToManyをひとつのpatchEntityとsaveで処理しようとするのがそもそもダメかも知らないと思いました。そこでequipmentだけ試してみましたがこれもダメでした。
時間も限られているし、自分の知識が足りないということでまずはSQLBuilderを使って一つ一つ入れることにしてみようと思っています。この件は今後やり直してみます。またアドバイスいただければと思います。ありがとうございます。

[popobot]

> ひとつ気になったのが、今私がやりたいのは2つのbelongsToManyをひとつのpatchEntityとsaveで処理しようとするのがそもそもダメかも知らないと思いました。

そんなことはないです、自分は10個以上のアソシエーションを含むエンティティをひとつのpatchEntityとsaveで処理しています。

> 時間も限られているし、自分の知識が足りないということでまずはSQLBuilderを使って一つ一つ入れることにしてみようと思っています。

そうですね、わりとこの辺の処理は難易度高いので、一旦別の方法で実装するのはいいと思いました。