Skip to main content
4 de 4
se añadieron 833 caracteres en el cuerpo
alfredjmg
  • 511
  • 6
  • 31

Error al subir imagenes en Laravel

Estoy haciendo un blog en Laravel, los posts tienen imágenes y al crear un post se sube una imagen. Si se crea el post la imagen se sube perfectamente pero cuando se edita el post y se sube otra imagen esta no se sube y ademas la direccion en la base de datos cambia a algo como esto:

C:\xampp\tmp\phpCC3D.tmp

adjunto el código del controlador de los posts:

<?php

namespace App\Http\Controllers\Admin;

use Illuminate\Http\Request;  
use App\Http\Requests\PostStoreRequest;
use App\Http\Requests\PostUpdateRequest;
use App\Http\Controllers\Controller;

use Illuminate\Support\Facades\Storage;


use App\Post;
use App\Category;
use App\Tag;

class PostController extends Controller
{

public function __construct()
{
    $this->middleware('auth');
}

/**
 * Display a listing of the resource.
 *
 * @return \Illuminate\Http\Response
 */
public function index()
{
    $posts = Post::orderBy('id', 'DESC')
    ->where('user_id', auth()->user()->id)
    ->paginate();


    return view('admin.posts.index', compact('posts'));
}

/**
 * Show the form for creating a new resource.
 *
 * @return \Illuminate\Http\Response
 */
public function create()
{
    $categories = Category::orderBy('name', 'ASC')->pluck('name', 'id');
    $tags       = Tag::orderBy('name', 'ASC')->get();

     return view('admin.posts.create', compact('categories', 'tags'));
}

/**
 * Store a newly created resource in storage.
 *
 * @param  \Illuminate\Http\Request  $request
 * @return \Illuminate\Http\Response
 */
public function store(PostStoreRequest $request)
{
    //validacion
    $post = Post::create($request->all());

    //IMAGE
    if($request->file('file')){
        $path = Storage::disk('public')->put('image', $request->file('file'));

        $post->fill(['file' => asset($path)])->save();
    }

    //TAGS
    $post->tags()->sync($request->get('tags'));

    return redirect()->route('posts.edit', $post->id)
        ->with('info', 'Entrada creada con exito');
}

/**
 * Display the specified resource.
 *
 * @param  int  $id
 * @return \Illuminate\Http\Response
 */
public function show($id)
{
    $post = Post::find($id);
    $this->authorize('pass', $post);

    return view('admin.posts.show', compact('post'));
}

/**
 * Show the form for editing the specified resource.
 *
 * @param  int  $id
 * @return \Illuminate\Http\Response
 */
public function edit($id)
{
    $post       = Post::find($id);
    $this->authorize('pass', $post);

    $categories = Category::orderBy('name', 'ASC')->pluck('name', 'id');
    $tags       = Tag::orderBy('name', 'ASC')->get();

    return view('admin.posts.edit', compact('post', 'categories', 'tags'));
}

/**
 * Update the specified resource in storage.
 *
 * @param  \Illuminate\Http\Request  $request
 * @param  int  $id
 * @return \Illuminate\Http\Response
 */
public function update(PostUpdateRequest $request, $id)
{
    $post = Post::find($id);
    $this->authorize('pass', $post);
    $post->fill($request->all())->save();

    //IMAGE 
    if($request->file('image')){
        $path = Storage::disk('public')->put('image',  $request->file('image'));
        $post->fill(['file' => asset($path)])->save();
    }

    //TAGS
    $post->tags()->sync($request->get('tags'));

    return redirect()->route('posts.edit', $post->id)->with('info', 'Entrada actualizada con éxito');
}

/**
 * Remove the specified resource from storage.
 *
 * @param  int  $id
 * @return \Illuminate\Http\Response
 */
public function destroy($id)
{
    $post = Post::find($id);
    $this->authorize('pass', $post);
    $post->delete();

    return back()->with('info', 'Eliminado correctamente');
}
}

Lo que considero debo cambiar en el código es la parte del update, solo que no veo como agregarle para que elimine la imagen anterior:

 public function update(PostUpdateRequest $request, $id)
 {
    $post = Post::find($id);
    $this->authorize('pass', $post);
    $post->fill($request->all())->save();

    //IMAGE 
    if($request->file('image')){
        $path = Storage::disk('public')->put('image',  $request->file('image'));
        $post->fill(['file' => asset($path)])->save();
    }

    //TAGS
    $post->tags()->sync($request->get('tags'));

    return redirect()->route('posts.edit', $post->id)->with('info', 'Entrada actualizada con éxito');
}

¿Que debo cambiar en el código para borrar la imagen ya existente y subir la nueva?

Edit1: Pude resolverlo, el problema era aqui:

public function update(PostUpdateRequest $request, $id)
{
$post = Post::find($id);
$this->authorize('pass', $post);
$post->fill($request->all())->save();

//IMAGE 
if($request->file('image')){
    $path = Storage::disk('public')->put('image',  $request->file('image'));
    $post->fill(['file' => asset($path)])->save();
}

Realmente debería ser asi:

public function update(PostUpdateRequest $request, $id)
{
$post = Post::find($id);
$this->authorize('pass', $post);
$post->fill($request->all())->save();

//IMAGE 
if($request->file('file')){
    $path = Storage::disk('public')->put('image',  $request->file('file'));
    $post->fill(['file' => asset($path)])->save();
}
alfredjmg
  • 511
  • 6
  • 31