レコードを1行ずつ削除したい。

Question

Cakephpを学習中です。
簡単な掲示板を作っています。
削除ボタンを実装し(各テーブルに)、押下時に指定のレコードが削除される。
という実装を行いたいのですが、
削除ボタンを押下すると、errorが出てしまいます。
そして、indexに戻るとレコードが全て消えてしまいます。
どのように実装すればよいでしょうか。

<h1>Sample BBS</h1>

<?=$this->Form->create($entity,['url'=>['action'=>'addRecord']])?>
<fieldset>
  name:<?=$this->Form->text("user_name");?>
  title:<?=$this->Form->text("title");?>
  content:<?=$this->Form->text("content");?>
</fieldset>
  <?=$this->Form->button('投稿する');?>
  <?=$this->Form->end();?>

<table>
  <thead>
    <tr>
      <th>Id</th>
      <th>name</th>
      <th>title</th>
      <th>content</th>
      <th>delete</th>
    </tr>
  </thead>

  <tbody>
    <?php foreach($data as $obj):?>
    <tr>
      <td><?=h($obj->id)?></td>
      <td><?=h($obj->user_name)?></td>
      <td><?=h($obj->title)?></td>
      <td><?=h($obj->content)?></td>
      <!--削除する-->
      <td><?=$this->Form->postLink('削除',array('action'=>'delRecord'));?></td>
    </tr>
  <?php endforeach;?>
  </tbody>
</table>

<?php
namespace App\Controller;
use \Exception;
use Cake\log\log;

class PostsController extends AppController{
  //index action
  public function index(){
    $data = $this->Posts->find('all');
    $this->set('data',$data);
    $this->set('entity',$this->Posts->newEntity());
  }

  //レコードの追加
  public function addRecord(){
    if($this->request->is('post')){
      $p = $this->Posts->newEntity($this->request->data);
      $this->Posts->save($p);
    }
    return $this->redirect(['action'=>'index']);
  }

  //レコードの削除
  public function delRecord(){
    if($this->request->is('post')){
      $this->Posts->deleteAll();
    }
    return $this->redirect(['action'=>'index']);
  }
}

Answer 1

deleteAllを利用しているので全て削除されます
下記のいずれかで行うと良いでしょう。

• それぞれの行をformで囲い、hiddenでidを埋め込んでおき、そのidをdeleteに渡す
• 削除ボタンにidをdata属性やjavascriptの関数の引数に埋め込んでおき、ajaxを利用して削除処理を行う

いずれもidをdelete()に渡すことになると思います。

追記：
「エラーが出ます」とのことですが、そのエラーメッセージが提示されていないとなんとも言えませんので、エラーメッセージもあわせて提示いただけるとより得たい回答が得られるかもしれません。
私の回答はCakePHP独自というよりプログラミング言語がどれか関係なしのやり方なので、CakePHPでの表現方法は適宜調べてください。

追記２：
deleteAllはパラメータに何も渡さないと実行されない？ようなのでダミーデータを渡すと良いようですね。

• 【CakePHP】delete()/deleteAll()

エラーはこれではないでしょうか？
いずれにしても指定したデータだけ消したい場合はdeleteAllは使えません。その名の通り「すべて削除する」ものです。

Answer 2

CakePHP 3 だと思いますが、以下のような実装でいけそうです。

      <!--削除する-->
      <td>
        <?php echo $this->Form->postLink('削除', array(
          'action' => 'delRecord',
          $obj->id,
        )); ?>
      </td>

  //レコードの削除
  public function delRecord($id) {
    if($this->request->is('post')){
      $post = $this->Posts->get($id);
      $this->Posts->delete($post);
    }
    return $this->redirect(['action'=>'index']);
  }

ご参考: CakePHP 3.5 Red Velvet Cookbook > ブログチュートリアル - パート2 > 投稿記事の削除

Comments

[退会済みユーザー]

うまくいきました！ありがとうございます。

[Lulucom]

評価ありがとうございます。がんばってください。