cakePHP3で複数フィールドの挿入時におけるバリテーション&flashメッセージの表示

Question

現在、複数フィールドの挿入のプログラムを書いているのですが、挿入はできるようになったものの、バリテーションによるエラーメッセージがうまく表示されません。
Entityの使い方についてよくわかってないと思うのですが、どのようにすべきなのでしょうか。

• リストやりたいこと

一つでもエラーがあれば、処理を中止させて、エラーのflashメッセージとバリテーションの個々のメッセージを出して、もし一つもエラーがなければ、投稿完了のflashメッセージを出したいです。

コントローラ
  public function addition5(){
    $this->ViewBuilder()->layout('auth');
    $articles = TableRegistry::get('articles');//必要
    $entities = $articles->newEntities($this->request->data());
    if($this->request->is('post')){
 		$articles->connection()->transactional(function () use ($articles, $entities) {
          foreach ($entities as $entity) {
        	$articles->save($entity, ['atomic' => false]);
          }
        });
      //var_dump($entities);これをするとエラーメッセージが格納されていることがわかる。
    }
    $this->set(compact('articles'));
 	$this->set('_serialize', ['articles']);

  }  

ビュー
echo $this->Form->create($articles,["class" => "article_form1"]);
for($i = 0; $i < 5;$i++){
	$j = $i + 1;//エラーの配列番号と合わせるために足し算する。
	echo '<h2 class="add5">登録記事('.$j.'/5)</h2>';
	echo $this->Form->hidden($i.'.user_id',['value' => h($this->request->session()->read('Auth.User.id'))]);
	echo $this->Form->input($i.'.title',['label' => 'タイトル','placeholder' => 'タイトル']);
	echo $this->Form->input($i.'.comment',['label' => 'コメント','placeholder' => 'コメント']);
}
echo $this->Form->button('登録',["class" => "ok"]);
echo $this->Form->end();

Answer 1

ビューに下記のコードを追加するとどうでしょうか。

// カラム名がuser_idの場合
echo $this->Form->error('user_id');

問題は解決したようですが、以下のコードの間違いを修正します。修正しないと気持ち悪いので…。
yajinさんの解決策通りフォームヘルパーには$entitiesを渡す方向です。

追記（修正）

public function addition5()
{
  $this->ViewBuilder()->layout('auth');
  $articles = TableRegistry::get('Articles');

  if ($this->request->is('post')) {
    // ここでバリデーションが行われる
    $entities = $articles->newEntities($this->request->data());

    // バリデーションでエラーが出た場合にここで処理を中断する
    foreach ($entities as $entity) {
      if ($entity->errors()) {
        $this->set(compact('articles'));
        return;
      }
    }

    // 保存処理
    $articles->connection()->transactional(function () use ($articles, $entities){
      foreach ($entities as $entity) {
        $articles->save($entity, ['atomic' => false]);
      }
    });
  } else {
    // POSTではない場合
    $entities = $articles->newEntity();
  }

  $this->set(compact('entities'));
}

Comments

[yajin]

ご返信ありがとうございます！

早速試してみたところ、
echo $this->Form->error($i.'.title');
echo $this->Form->error('.title');
両方だめですね・・・。

[coba-coba]

echo $this->Form->error('title');ではどうですか？

[yajin]

あ、申し訳ございません。それでもだめです。
上のは誤植でした。ドットを抜かしてもだめですね。

[yajin]

オブジェクト名がarticleなのかarticlesなのかentitiesなのかentityなのかが頭の中でごちゃごちゃしています・・・。

[coba-coba]

確かにごちゃごちゃしてきますよね。でも考え方は間違えていないと思います。
私も同じような書き方で試してみましたが、複数保存されて、ちゃんとバリデーションもされました。
謎ですね……。

ちなみにビューに
echo $this->Form->error('title');
は追記せずできました。

yajinさんの書き方だと細かいところでエラーが出たので、コントローラーのコードを書き直して上に追記しました。
これだとどうでしょうか。
（yajinさんと考え方は同じなのでいける気はしませんが…）

だめだったら、バリデーションのエラーメッセージが見れる場合の
debug($entities);
の出力を見せていただけますか？

[yajin]

大変申し訳ございません。ありがとうございます。

echo $this->Form->create($articles,["class" => "article_form1"]);
for($i = 0; $i < 5;$i++){
$j = $i + 1;//エラーの配列番号と合わせるために足し算する。
echo '<h2 class="add5">登録記事('.$j.'/5)</h2>';
echo $this->Form->hidden($i.'.user_id',['value' => h($this->request->session()->read('Auth.User.id'))]);
echo $this->Form->input($i.'.title',['label' => 'タイトル','placeholder' => 'タイトル']);
echo $this->Form->error('title');
echo $this->Form->input($i.'.comment',['label' => 'コメント','placeholder' => 'コメント']);
echo $this->Form->error('comment');
}
echo $this->Form->button('登録',["class" => "ok"]);
echo $this->Form->end();

こういうことでいいんでしょうか？

[yajin]

debug($articles);
[
(int) 0 => object(App\Model\Entity\Article) {

'user_id' => (int) 4,
'comment' => '',
'[new]' => true,
'[accessible]' => [
'user_id' => true,
'title' => true,
'comment' => true
],
'[dirty]' => [
'user_id' => true,
'comment' => true
],
'[original]' => [],
'[virtual]' => [],
'[errors]' => [
'title' => [
'length' => '2文字以上で入力してください。',
'custom' => '日本語のみは使用できません。'
]
],
'[invalid]' => [
'title' => 'a'
],
'[repository]' => 'Articles'

},
(int) 1 => object(App\Model\Entity\Article) {

'user_id' => (int) 4,
'comment' => '',
'[new]' => true,
'[accessible]' => [
'user_id' => true,
'title' => true,
'comment' => true
],
'[dirty]' => [
'user_id' => true,
'comment' => true
],
'[original]' => [],
'[virtual]' => [],
'[errors]' => [
'title' => [
'exist' => 'その記事名はすでに登録されています'
]
],
'[invalid]' => [
'title' => 'bb'
],
'[repository]' => 'Articles'

},
(int) 2 => object(App\Model\Entity\Article) {

'user_id' => (int) 4,
'comment' => '',
'[new]' => true,
'[accessible]' => [
'user_id' => true,
'title' => true,
'comment' => true
],
'[dirty]' => [
'user_id' => true,
'comment' => true
],
'[original]' => [],
'[virtual]' => [],
'[errors]' => [
'title' => [
'exist' => 'その記事名はすでに登録されています'
]
],
'[invalid]' => [
'title' => 'cc'
],
'[repository]' => 'Articles'

},
(int) 3 => object(App\Model\Entity\Article) {

'user_id' => (int) 4,
'comment' => '',
'[new]' => true,
'[accessible]' => [
'user_id' => true,
'title' => true,
'comment' => true
],
'[dirty]' => [
'user_id' => true,
'comment' => true
],
'[original]' => [],
'[virtual]' => [],
'[errors]' => [
'title' => [
'exist' => 'その記事名はすでに登録されています'
]
],
'[invalid]' => [
'title' => 'dd'
],
'[repository]' => 'Articles'

},
(int) 4 => object(App\Model\Entity\Article) {

'user_id' => (int) 4,
'comment' => '',
'[new]' => true,
'[accessible]' => [
'user_id' => true,
'title' => true,
'comment' => true
],
'[dirty]' => [
'user_id' => true,
'comment' => true
],
'[original]' => [],
'[virtual]' => [],
'[errors]' => [
'title' => [
'exist' => 'その記事名はすでに登録されています'
]
],
'[invalid]' => [
'title' => 'ee'
],
'[repository]' => 'Articles'

}
]

[coba-coba]

情報ありがとうございます。
ビューのecho $this->Form->error('……');は無くても大丈夫でした。
すみません、消してもらって構いません。

頂いた情報からは特に問題は無いように思えます。
ただ、更に2つ気になる点があります。
すでにチェック済みかもしれませんが、念のため教えていただけますか。

１．addition5()はPOSTするときだけに使うのでしょうか。
addition5()を呼び出すときURLが「/コントローラー/addition5」でアクセスすると思うのですが、POSTしない場合（最初にアクセスする時）はエラーになると思います。
（この部分 → $entities = $articles->newEntities($this->request->data());）

２．addition5()で処理が終わった後、$this->redirect([……]);で飛んでいませんか。
$articlesがフォームヘルパーに渡されなければエラーメッセージは出ません。

[yajin]

$entities = $articles->newEntities($this->request->data());
そうです。これがすごく悩んでました。postされたものを何度も書き直してバリテーションがでて嬉しくと思っていたら、再び新規に開いたらエラーと・・・
バリテーションがでると、新規で開けなくなり、また挿入できるようになるとバリテーションエラーが表示されないという繰り返しです。
何度もコメントアウトしてソースがいっぱいになり、ごちゃごちゃになってました。
ちょっと見返してみます。

[coba-coba]

Entityのあたりはまりますよね…。
行き詰まったら、bakeで骨組みを作ってそこから少しずつ修正していくのが吉ですね。

[yajin]

public function addition5(){
$this->ViewBuilder()->layout('auth');
$articles = TableRegistry::get('articles');//必要
if($this->request->is('post')){
$entities = $articles->newEntities($this->request->data());
foreach($entities as $entity){
if($entity->errors()){
$this->set(compact('articles'));
return;
}
}

$articles->connection()->transactional(function () use ($articles, $entities) {
foreach ($entities as $entity) {
$articles->save($entity, ['atomic' => false]);
}
});

}else{
$articles = $articles->newEntity();
}
// echo "<pre>";
// debug($entities);
// echo "</pre>";
$this->set(compact('articles'));
}
現在このような形です。

[yajin]

インデントが読みにくくてすみません・・・

[coba-coba]

1つのレコードのみを保存する場合はバリデーションされるのでしょうか。まずは1つのレコードで試してみた方がいいと思います。
もし動かなければテーブルやエンティティの設定を間違えているのかもしれません。

基本的にnewEntity()やnewEntities()でエンティティを作って、それをフォームヘルパーに渡すだけです。

[yajin]

大変恐れ入ります。

>1つのレコードのみを保存する場合はバリデーションされるのでしょうか。まずは1つのレコードで試してみた方がいいと思います。

これは可能です。しっかりとエラーメッセージが出ます。大文字小文字で意識するところはどんなところでしょうか？それぐらいしか考えられないです。error.logやdebug.logも何もでなくなりました。

[coba-coba]

可能でしたか。そうするとますます謎ですね。

大文字小文字だと、
TableRegistry::get('articles')
を'Articles'にするくらいでしょうか。
CakePHPのマニュアルには大文字で載っていますし。
全部小文字でも問題ないはずですが。

[yajin]

まとめておきます。
・複数レコードの挿入のプログラム
１、正しい情報が入った時には複数挿入はできる。
２、debug($entities)にはオブジェクト型で、errorsにバリテーションエラーメッセージの値が格納されていることが確認できる。
３、別アクションでの一つのレコード挿入時にはバリテーションエラーも出るし、挿入もできる。

・解決できたこと
１、Post送信されていない場合（初回のアクセス）におけるエラーはなくなった。

[yajin]

考えられたこと
・CSSのせい？
display:none的なことが考えられるが、postされたあと、近辺をGCで検証を行ってみるも、タグが生成されていない。
・inputをtextに代えてみた。
$this->Form->textに一度代えてみて、$this->Form->error('title')にしてみたが、表示されず。

[coba-coba]

まとめありがとうございます。

最初に思いつきで書いてしまったのが悪かったのですが、項目は複数あるので少なくとも
$this->Form->error('title');
ではなく、
$this->Form->error('0.title');
$this->Form->error('1.title');
となるはずです。

[yajin]

本当にありがとうございます。
試してみたところ、だめですね。
一度はでたことがある（けど、postされていない場合にはアクセスエラー状態になっていた）ので、やはりControllerのプログラムのせいだと思います。

[yajin]

coba-cobaさんいらっしゃいますでしょうか？

[coba-coba]

いますよー。ちょくちょく考えてはいますが、私の実力だと直接見て動かしてみないと分からない状態です…。

[yajin]

個人的に相談したいのですが、私のプロフィールからサイトに飛んでいただき、連絡取り合うこてゃ可能でしょうか？
すごく困っています。

[coba-coba]

すみません、そういうのはやらないようにしています。
内心では連絡を取り合えたら楽だとは思いますが。

でも、私も今回のご質問の件は解決したいので、どこかにプログラムとデータベースのダンプしたものをzipなどで置いておいてもらえれば、私も試してみることはします。
どうでしょうか？

[yajin]

そうですよね。大変申し訳ございませんでした。
コントローラとビューとモデル部分だけでいいのでしょうか？

[coba-coba]

遅くなってすみません。それで大丈夫です。

[yajin]

こちらこそ遅くなってすみません。
アップロードはやはり許可が下りずできませんでした。
ただ、進展したので、お話します。

view
ctpのForm->create($entities,[～～])とし、

controllerは以下にしたところvalitationエラーメッセージが出てきました！
ただ、foreachで回されているようで、同じ文が2度でてきました。

public function addition5(){
$this->ViewBuilder()->layout('auth');
$articles = TableRegistry::get('Articles');

if($this->request->is('post')){
//postされた場合
$entities = $articles->newEntities($this->request->data());

foreach ($entities as $entity) {
if($entity->errors()){
$this->set(compact('entities'));
// debug($entity->errors());
// $this->set('_serialize', ['articles']);

return;
}
}

$articles->connection()->transactional(function () use ($articles,$entities){
foreach($entities as $entity){
$articles->save($entity,['atomic' => false]);
}
});

}else{
//初回アクセス

$articles = $this->Articles->newEntity();
}
//var_dump($entities);
$this->set(compact('entities'));

}
あと少しだとは思うのですが、どのようにしたらよいでしょうか？

[coba-coba]

よかったです！
しかしすみませんでした。$articlesと$entitiesがごっちゃになっていました…。
newEntities()で作ったエンティティを渡してなかったんですね。
何で今まで見逃していたのか分からないくらいのミスでした。
時間をかけさせて本当にすみません。

初回アクセスの部分も$entitiesにすべきですね。

//初回アクセス
$entities = $this->Articles->newEntity();

バリデーションが2度出るというのは、ビューに$this->Form->error()を加えていませんか？
$this->Form->input()のみで結構です。
もし加えていなければ他の原因も考えてみます。

[yajin]

よし、完璧です。出来ました。
おっしゃるとおり、$this->Form->error()を加えていました。
フラッシュメッセージも出ました。
とりあえず動きました。あとは細かいところは自分で調整していきます。
長い間ありがとうございました！

[coba-coba]

あまりお役に立てなくてすみませんでした。
解決してよかったです！