POST送信後、DB更新でクエリが失敗してしまう

前提・実現したいこと

PHPで数行程度の文章・画像投稿可・リプライ機能は無しの簡易
ブログ機能を作っています。

管理画面からDBへ記事データを保存、取得したものをforeachで回しリスト表示し、
各記事をクリックすると編集・削除画面へ遷移。
編集もしくは削除を行うと元の投稿ページへ戻るようにしたいです。

発生している問題・エラーメッセージ

編集ページにてPOST送信を行い、DB更新をしようとすると
クエリが失敗してしまいます。
削除機能はまだつけていません。
php.logを確認すると以下のように表示されています。

[03-Nov-2019 06:11:57 UTC] デバッグ：バリデーションOK。
[03-Nov-2019 06:11:57 UTC] デバッグ：DB更新。
[03-Nov-2019 06:11:57 UTC] デバッグ：
SQL：UPDATE infoblog SET name = :name, comment = :comment, pic1 = :pic1, WHERE user_id = :u_id 
AND id = :p_id
[03-Nov-2019 06:11:57 UTC] デバッグ：格納データ：Array
(
    [:name] => 投稿テスト
    [:comment] => テストしています。
    [:pic1] => 
    [:u_id] => 5
    [:p_id] => 88
)

[03-Nov-2019 06:11:57 UTC] デバッグ：クエリ失敗。
[03-Nov-2019 06:11:57 UTC] デバッグ：失敗したSQL:PDOStatement Object


(  [queryString] => UPDATE infoblog SET name = :name, comment = :comment, pic1 = :pic1, WHERE user_id = :u_id AND id = :p_id  )

編集画面の該当ソースコード

PHP
1<?php
2
3try {
4  $dsn = 'mysql:dbname=blog;host=localhost;charset=utf8';
5  $user = 'root';
6  $password = 'root';
7  $options = array(
8    PDO::ATTR_ERRMODE => PDO::ERRMODE_SILENT,
9    PDO::ATTR_DEFAULT_FETCH_MODE => PDO::FETCH_ASSOC,
10    PDO::MYSQL_ATTR_USE_BUFFERED_QUERY => true,
11  );
12  $dbh = new PDO($dsn, $user, $password, $options);
13  return $dbh;
14   //DB更新
15   if(!empty ($_GET['p_id'])){
16       debug('DB更新');
17     $sql = 'UPDATE infoblog SET name = :name, comment = :comment, pic1 = :pic1, WHERE user_id = :u_id AND id = :p_id';
18     $data = array(':name' => $name , ':comment' => $comment, ':pic1' => $pic1, ':u_id' => $_SESSION['user_id'], ':p_id' => $p_id);
19      }
20        
21     debug('SQL：'.$sql);
22     debug('格納データ：'.print_r($data,true));
23   
24     $stmt =  $dbh->prepare($sql);
25     $stmt->execute($data);
26     return $stmt;
27
28     if($stmt){
29     $_SESSION['msg_success'] = SUC04;
30     debug('投稿ページへ戻ります。');
31     unset($p_id);
32     header("Location:admin_infoblog.php"); 
33    }
34          
35    }catch (Exception $e) {
36      error_log('エラー発生:' . $e->getMessage());
37      $err_msg['common'] = MSG07;
38    }
39  }
40}

試したこと

WHERE以降を消すとDBの更新がうまくいくので、
p_id(記事ごとのID）、user_idの送信が問題のように見えるのですが、ログを見る限りでは
正常に配列内に格納されており、また、編集ページにてvar_dumpで表示させてみても、記事ID、ユーザIDの取得はうまくできています。何故クエリが失敗してしまうのかがわかりません。pic1は画像データがあるかどうかですがその有無は関係ないようです。

補足情報（FW/ツールのバージョンなど）

OS: Mac OSX　Mojave10.14.3
サーバソフト：MAMP5.3
データベースソフト：PHPMyAdmin

Orlofsky

2019/11/03 07:56

エラーメッセージやコードは https://teratail.com/help/question-tips#questionTips3-7 の [コード] に修正してください。今の使い方は間違っています。

takasima20

2019/11/03 08:22

queryPost() ってなに?

m.ts10806

2019/11/03 08:26

独自関数多くて読解が大変。

tanishi_a

2019/11/03 12:53

こんな感じの SQL が実行されるかと思うのですが、手で実行した場合はエラーなく実行できるでしょうか？ UPDATE infoblog SET name = '投稿テスト', comment = 'テストしています', pic1 = null, WHERE user_id = 5 AND id = 88

yk_dev

2019/11/03 14:56 編集

皆様、コメントありがとうございます。非常に見づらいコードを投稿してしまい申し訳ありませんでした。使い方を勘違いしておりました。独自関数部分の記載は削除し処理の内容を記載しました。 >tanishi_aさん PHPMyAdminにて手打ちで処理を実行してみたところ、pic1 = nullではエラーが出ましたが、 pic1='' とすることで該当テーブルの内容が書き換わりました。実際の編集画面でも画像を投稿しない場合にpic1にnullが入ってしまっているのかと思い、画像データを挿入しPOST送信も行ってみましたが、やはりクエリが失敗してしまいます。

tanishi_a

2019/11/03 16:13

> p_id(記事ごとのID）、user_idの送信が問題のように見えるのであれば片方ずつwhere文から消してどちらかが問題なのか確認してみては。（危険ですが、既にwhere文自体を消して確認されてるようなのを見るに、全件updateされて大丈夫なデータですよね？）また、それぞれカラムの型と、PHP側で渡してる値の型もわかるとよいと思います。

takasima20

2019/11/03 23:16

まさかとは思いますが、もし queryString が生のSQLだとしたらデータが置換されてないんじゃ? というか、$dataに入れてる変数って定義されてなくね?

yk_dev

2019/11/04 11:52 編集

すみません、実際のファイルでは変数の定義の記載があるのですが抜けておりました・・別回答の方がご指摘してくださったpic1,のカンマが原因だったということで解決しましたが、次回投稿時にはもっと内容を分かりやすく書けるように心がけます！皆様ありがとうございました！