条件に合うデータ削除

Question

お気に入り機能を実装しようと思っています。

jqueryでコントローラー側にデータを渡して処理をしようと思っています。
コントローラー側で削除処理できません。

//登録削除ボタンを押した時の処理
$('.deletefavorite').click(function(e){
	e.preventDefault();
	var post_id = $(this).attr('data-post-id');
	var user_id = $(this).attr('data-user-id');
	var $user = JSON.parse('<?php echo json_encode($userdata['id']);?>');//ユーザid
	var data = {postId:post_id,userId:user_id};
	$.ajax({
		url:'/cakephp/posts/deletefavoajax',
		type:'POST',
		data:data
	}).done(function(){
		//削除の成功処理
		alert('削除成功');
		$('.deletefavobtn').hide();
		$('.addfavobtn').show();
		console.log(data);
	}).fail(function(){
		//削除処理失敗の場合
		alert('再度実行してください。');
	});
});

{postId: "1", userId: "2"}

データはコントローラ側に渡ってきてると思うのですが、
データの削除ができません。追加はできました。
削除するデータは、Favoriteのテーブル内のpost_idとuser_idと同じ値のデータを削除したいと思っています。

削除できないのは何が原因なのでしょうか。

//お気に入り削除
		public function deletefavoajax(){
			$this->autoRender = FALSE; //viewの出力を停止
			if($this->request->is('ajax')) {
				$postid = $this->request->data['postId'];
				$userId = $this->request->data['userId'];
				$this->Post->Favorite->deleteAll(array('post_id' => $postid,'user_id' => $userId));
			}
		}

Answer 1

処理自体はあっているようにみえます。
ブラウザのデベロッパーツールで見た場合リクエストは200で成功しているのでしょうか？
また、以下を外してみたとき、エラー等がレスポンスにないですか？

$this->autoRender = FALSE; 

念のため、deleteAllの戻り値も見てみたほうがいいです。

あとは、deleteAllの直前で、以下のコードを入れるとlogs/debug.logにSQLが吐かれるの見てみるといいと思います

$this->Post->Favorite->connection()->logQueries(true);

Comments

[atsupoooon]

コメントありがとうございます。

確認しましたが、リクエストは200で成功しています。

$this->autoRender = FALSE;
を外してもエラーなどはありません。

Request URL:http://localhost/cakephp/posts/deletefavoajax
Request Method:POST
Status Code:200 OK

debug.logが吐かれません。
cakephp/app/tmp/logsで合っていますでしょうか。
CakePHP 2.7になります。

他に原因を調べる方法などありますでしょうか。

[popobot]

すみません、それCakephp3の場合でした。Cakephp2では以下のような感じみたいです
http://kwski.net/cakephp-2-x/1064/

あと、もしかしたらdeleteAllまで処理がいってないかもしれないので、
deleteAllをthrow new NotFoundException();に置き換えてみて、
想定通り404が返ってくるかやってみるのもいいかもです。

[atsupoooon]

原因がありました！！！
ありがとうございます。

認証の処理の部分でアクセス不可になっており、コントローラー側で処理できませんでした。
public function isAuthorized($userId) {
// 登録済ユーザーは投稿できる
if ($this->action === 'add') {
return true;
}

// 投稿のオーナーは編集や削除ができる
$userId = $this->Auth->user('id');
if (in_array($this->action, array('edit', 'delete'))) {
$postId = (int) $this->request->params['pass'][0];
if ($this->Post->isOwnedBy($postId, $userId)) {
return true;
}
}
echo 'アクセス無理';
}

deletefavoriteの処理を通すようにするには、どのように変更すれば良いかわかりません。

[popobot]

「登録済ユーザは記事のお気に入りを削除できる」でいいと思うのでaddと同じ条件でdeletefavoriteを加えればよいのではないかと。
ただし、deletefavoajaxメソッドないでuser_idを外部からもらっているのはまずいですね。他人のお気に入りが消せちゃう。
認証ユーザのID（$this->Auth->user('id');）を使って自分の記事しか消せないようにしたらよいかと

[atsupoooon]

ありがとうございます！！
一度実装してみます。

ご指摘ありがとうございました。