【PHP】投稿を編集する際に画像イメージをキープしておきたい。

Question

##実現したいこと
投稿編集をする際に画像の変更がない場合は再度アップロードせずにそのまま画像は保持した状態でアップデートしたい。

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

投稿した記事を編集する際に新しい画像を選択した場合は新しい画像がアップロードされますが、文字のみの編集で画像の変更がない場合でも同じ画像をアップロードしないといけないため、画像をアップロードしない状態でEDITボタンを押すと画像なしで投稿が編集されてしまいます。

###試したこと
imageが空だった場合は元々登録されている画像をデーターベースから取り出し、画像を保持した状態でアップデートしようと思い以下のコードのようにempty functionを使用してみましたが、画像は空のままアップデートされました。

if(empty($image)){
         $selected_image = $db->prepare("SELECT * FROM posts WHERE post_id={$the_post_id}");
         $selected_image->execute(array($the_post_id));
         $selected_images = $selected_image->fetch();
    }

該当のソースコード

<!-- Edit -->
<?php 
if(isset($_REQUEST['edit_post_id'])){
    $the_post_id = $_REQUEST['edit_post_id'];
    
    $posted = $db->prepare("SELECT * FROM posts WHERE post_id = $the_post_id ");
    $posted->execute(array($the_post_id));
    $posted_p = $posted->fetch();
}
?>
<?php
     if(isset($_POST['edit_post'])){
    
         $edit_posts = $db->prepare("UPDATE posts SET post_image=?, post_contents=? WHERE post_id ='{$the_post_id}' ");
         $edit_posts->execute(array(
             $image = date('YmdHis') . $_FILES['image']['name'],
             $_POST['post_contents']
             ));
        move_uploaded_file($_FILES['image']['tmp_name'], './images/' . $image);
        
     if(empty($image)){
         $selected_image = $db->prepare("SELECT * FROM posts WHERE post_id={$the_post_id}");
         $selected_image->execute(array($the_post_id));
         $selected_images = $selected_image->fetch();
    }
        header('Location: index.php');
        exit();
    }
?>

<!-- Edit form -->
<div class="container px-4 px-lg-5">
    <div class="row gx-4 gx-lg-5 justify-content-center">
        <div class="col-md-10 col-lg-8 col-xl-7">
            <div class="well">
                <form action="" method="post" enctype="multipart/form-data">
                    <div>
                        <label for="summernote">Edit</label>
                        <textarea class="form-control" name="post_contents" id="summernote" col="30" rows="10"><?php echo htmlspecialchars($posted_p['post_contents'], ENT_QUOTES); ?></textarea>
                    </div>
                    <div class="form-group">
                        <label for="post_image">Image</label>
                        <input type="file" name="image"  >
                        <img width="100" src="./images/<?php echo $posted_p['post_image']; ?>" >
                    </div>
                    <span class="form-group">
                        <p><input class="btn btn-primary" type="submit" name="edit_post" value="Edit"></p>
                    </span>
                </form>
            </div>
        </div>
    </div>
</div>

ご教授頂けると幸いです。
どうぞ宜しくお願い致します。

Answer 1

imageの有無に関わらず

$edit_posts = $db->prepare("UPDATE posts SET post_image=?, post_contents=? WHERE post_id ='{$the_post_id}' ");

が実行されるからではないかと思います

imageがないなら　update文は
UPDATE posts SET post_contents=? WHERE post_id ='{$the_post_id}'

するように分岐させてそもそも更新しないようにするとかどうでしょう

Comments

[Jonh]

アドバイスいただきありがとうございます。
ご教授いただきました方法で実行してみましたところ、イメージの更新はされずにイメージの保持はできるようになりました。
ただイメージを更新しようとするとイメージは更新されずに元のイメージのままになります。
分岐をさせてとのことですが、以下のコードのように分岐をするということでしょうか？
こちらのコードではイメージの更新はできませんでした…

<?php
if(isset($_POST['edit_post'])){
$edit_posts = $db->prepare("UPDATE posts SET post_contents=? WHERE post_id ='{$the_post_id}' ");
$edit_posts->execute(array(
$_POST['post_contents']
));
header('Location: index.php');
exit();
}
?>

<?php
if(isset($_POST['edit_post'])){
$edit_image = $db->prepare("UPDATE posts SET post_image=? WHERE post_id ='{$the_post_id}' ");
$edit_image->execute(array(
$image = date('YmdHis') . $_FILES['image']['name'],
));
move_uploaded_file($_FILES['image']['tmp_name'], './images/' . $image);

header('Location: index.php');
exit();
}
?>
```

[退会済みユーザー]

if(isset($_POST['edit_post'])){

if(empty($image)){
$edit_posts = $db->prepare("UPDATE posts SET post_contents=? WHERE post_id ='{$the_post_id}' ");
$edit_posts->execute(array(
$_POST['post_contents']
));
} else {
$edit_posts = $db->prepare("UPDATE posts SET post_image=?, post_contents=? WHERE post_id ='{$the_post_id}' ");
$edit_posts->execute(array(
$image = date('YmdHis') . $_FILES['image']['name'],
$_POST['post_contents']
));
move_uploaded_file($_FILES['image']['tmp_name'], './images/' . $image);
}
header('Location: index.php');
exit();
}

例えばempty($image) がimageの有無を確認できるか確認したら、こんな感じでも良いと思います

[Jonh]

ご返信いただきありがとうございます！
まずempty($image)にてimageの有無を確認できるようにしないといけない感じですね。
お忙しい中ご教授いただきありがとうございました！
こちらのアドバイスを基に頑張ってみようと思います！

[退会済みユーザー]

empty($image)でimageの有無を確認できるようにというよりは、
imageがPostされているかどうの判断方法を調べるという感じでしょうか

https://techplay.jp/column/619

[Jonh]

こんばんは。
何度もアドバイスいただきありがとうございます！
いただいたリンクを参考に以下のような形でimageがPostされているかを判断し、イメージを洗濯していない状態でもイメージを保持することができました！
本当にありがとうございました！

<?php
if(isset($_POST['edit_post'])){

$post_image = date('YmdHis') . $_FILES['image']['name'];
$post_image_temp = $_FILES['image']['tmp_name'];
$post_contents = $_POST['post_contents'];

move_uploaded_file($post_image_temp, './images/' . $post_image);

if(empty($post_image_temp)){
$edit_posts = $db->prepare("UPDATE posts SET post_contents=? WHERE post_id ='{$the_post_id}' ");
$edit_posts->execute(array(
$_POST['post_contents']
));

} else {
$edit_posts = $db->prepare("UPDATE posts SET post_image=?, post_contents=? WHERE post_id ='{$the_post_id}' ");
$edit_posts->execute(array(
$post_image,
$_POST['post_contents']
));

move_uploaded_file($post_image_temp, './images/' . $post_image);
}

header('Location: index.php');
exit();