这仍然与我在How to remove and unlink multiple images in json format
中的帖子有关我可以存储和删除所有多个图像。但是更新和删除旧图像时我仍然有问题。我的代码在更新新图像时仍然无法删除oldimage。
我的更新控制器,我使用handlerequest方法存储图像,其中oldImage是数据库中的变量,它将与新图像进行比较。如果不一致,则删除旧图像。
public function update(Requests\UpdatePostRequest $request, $id)
{
$post = Post::findOrFail($id);
$oldImage = $post->image;
$data = $this->handleRequest($request);
$post->update($data);
if($oldImage != $post->image)
{
foreach (json_decode($post->image, true) as $oldImage) {
$this->removeImage($oldImage);
}
}
if($request->submitbutton =='Publish')
{
Alert::success('Your post was updated successfully')->persistent('Close');
return redirect(route('admins-blogpost.index'));
}
if($request->submitbutton =='Save Draft')
{
Alert::success('Your draft was updated successfully')->persistent('Close');
return redirect(route('admins-blogpost.index'));
}
}
我的handlerequest方法用于存储多个图像,该方法正在使用store方法。所以这个问题不在这里。
private function handleRequest($request)
{
$data = $request->all();
if($request->hasFile('image'))
{
$images = $request->file('image');
foreach($images as $key=>$image){
$fileName = $image->getClientOriginalName();
$destination = $this->uploadPath;
$successUploaded = $image->move($destination, $fileName);
if($successUploaded)
{
$width = config('cms_blog.image.thumbnail.width');
$height = config('cms_blog.image.thumbnail.height');
$extension = $image->getClientOriginalExtension();
$thumbnail = str_replace(".{$extension}", "_thumb.{$extension}", $fileName);
Image::make($destination . '/' . $fileName)
->resize($width,$height)
->save($destination . '/' . $thumbnail);
$datax[] = $fileName;
}
$data['image'] = json_encode($datax);
}
}
return $data;
}
和删除方法
public function removeImage($image)
{
if( ! empty($image))
{
$imagePath = $this->uploadPath . '/' . $image;
$ext = substr(strrchr($image, '.'), 1);
$thumbnail = str_replace(".{$ext}", "_thumb.{$ext}", $image);
$thumbnailPath = $this->uploadPath . '/' . $thumbnail;
if(file_exists($imagePath) ) unlink($imagePath);
if(file_exists($thumbnailPath) ) unlink($thumbnailPath);
}
}
如何解决我的问题,以便在旧图像与新图像不相同时删除旧图像?
答案 0 :(得分:0)
在删除图像的for循环中,我相信您想遍历$oldImage
,而不是遍历$post->image
:
foreach (json_decode($oldImage, true) as $item) {
$this->removeImage($item);
}
答案 1 :(得分:0)
在单个列上存储多个图像名称,这不是一个好主意。我建议您创建另一个表post_images或其他内容。
然后从posts表和post_images表创建hasMany关系。然后,您在这里拥有很大的灵活性。另外,您可以创建多态关系以将同一图像模型与多个模型一起使用。