Validación y Guardado de Datos en Laravel

Nunca es buena idea guardar archivos subidos por los usuarios dentro del servidor que ejecuta PHP ya que esto puede prestarse para fallas de seguridad que un atacante puede explotar, siempre es mejor almacenar dichos archivos en sistemas como S3, Azure Storage, entre otros

Sergio Ramirez

Que buen consejo y bien explicado, gracias

Gilberto Pérez Garrido

Hace 4 años

¿Cómo?, referencias por favor y gracias we

Andrés David Solarte Vidal

Tenia entendido que el all() no era conveniente usarlo.

Manuel Ojeda

Es correcto Andrés, al utilizar All podemos aceptar todo lo que se nos envía a través del Request, lo mejor es crear un Request personalizado o ya en el último caso aceptar la petición en el controlador con:

$data = $request->validate(
[
 // Validaciones aquí
]
)

Joseph Dakkak

Ciertamente usar el metodo all() del request no representa incluso una buena practica en el guardado de datos. Sin embargo hay una buena forma de hacerlo, cuando se utiliza una clase Request Validation como es el caso de Post, podriamos usar el metodo 'validated' y este solo retorna aquellas llaves que han sido estrictamente validadas.

$request->validated();

Quedando el ejemplo como:

$post = Post::create([
	 'user_id' => auth()->user()->id
 ] + $request->validated());

Tomas Goldenberg

RESUMEN GUARDADO DE LA INFO DEL FORM DE CREACION SISTEMA DE VALIDACION PARA EL FORM

A)
-Crear archivo de validacion: php artisan make:request PostRequest
 "app/Http/Requests/PostRequest.php

namespace App\Http\Requests; 	//ESTO LO IMPORTAREMOS DESDE PostController

 public function authorize()
    {
        return true;
    }
    public function rules()
    {
        return [
            "title" => "required",
            "body" => "required",

        ];
    }





"PostCotroller.php
Conectar el archivo de validacion al controller

CAMBIAMOS ESTO:		//use Illuminate\Http\Request;
POR ESTO :		//use App\Http\Requests\PostRequest;



public function store(PostRequest $request)
    {
        $post = Post::create([
            "user_id" => auth()->user()->id
 //obtener el campo user_id de post ||lo buscamos en autenticados/usuarios/id
        ]+ $request->all() );

        if($request->file("file")){

            $post->image= $request->file("file")->store("posts","public");
            $post->save();
        }
        
        return back()->with("status","Creado con exito");
                        
   }
 //logica FUNCION STORE:   
        //1//creamos un post con el user_id y los datos del form de creacion
        //2//agarramos toda la info del form
        //3// si recibimos img:
            // (la guardamos en carpeta storage/app/public
            //en public se generara la carpeta "posts"
            // ahi se generara una ruta, y esa ruta se guarda en base de datos )
        //4 guardamos
        //5 retornamos a la vista anterior con un alert que dice creado con exit






"Post.php
Debemos avisar que vamos a recibir muchos datos juntos en un form

class Post extends Model
{
    use Sluggable;
    
    
    protected $fillable = [
        'title', "body", 'iframe',"image" ,"user_id"
    ];



Listo ya se guarda el post con la url de la imagen

Jimmy Buriticá Londoño

Automáticamente Laravel cambia el nombre del archivo, lo almacena con un nombre que es una combinación de letras y números, solo conserva la extensión. Con esto garantiza que no se suba una archivo con el mismo nombre.

Esteban Ramírez García

Una alternativa a la linea en el PostController donde se extrae el id del usuario del que está de momento loggeado para el post puede ir de la siguiente manera

Usado en la clase

'user_id' => auth()->user()->id

Alternativa

'user_id' => Auth::id()

Jhon Fredy Plazas Hurtado

use Illuminate\Support\Facades\Auth; Auth::user()->id

Vsqz Gabriel

No se olvide que en la carpeta Request/PostRequest.php deben de poner true* en la siguiente function sino les lanzara un error 403

  public function authorize()
    {
        return true;
    }```

Juan Carlos Pinzón

Podrías decir por qué ocurre es error?

Saul Santiago Gongora Ku

Gracias

María Sierra

Implementación del guardado

❗ Los controladores que sean privados se encuentra dentro de la carpeta Controllers/Backend y los que sea público son en Controllers

Para crear un archivo request usamos este comando:

php artisan make:request PostRequest

🧇 Cuando creamos un archivos Request siempre ponemos como nombre el que tenemos como entidad en la database.

Cuando estamos modificando el método de crear, para poder obtener y salvar la imagen creamos esta configuración:

// img
if ($request->file('file')) {
    # Esta configuración está guardando la imagen en la carpeta public
    $post->image = $request->file('file')->store('posts', 'public');
}

Según la lógica que estamos haciendo es 'Al recibir un archivo debemos salvar ese archivo', y para esto la configuración sería 'Ese archivo debemos salvarlo para generar una ruta y esa ruta será lo que guardamos en la Database.

‼ Nunca guardamos en la Database un archivo directamente.

Recuerda que para recibir datos tienes que poner esto multipart/form-data en el formulario, así:

<form action="{{ route('posts.store') }}" method="post" enctype="multipart/form-data">

Victor Noguera

Si les aparece el error:

Function name must be a string

es por que tienen la variable $request de la siguiente forma:

$post = Post::create (['user_id' => Auth()->user()->id]
                 + $request()->all());

Y la forma correcta es

$post = Post::create (['user_id' => Auth()->user()->id]
                 + $request->all());

Sin los (). Ahora si le quieres agregar algo mas de seguridad usa:

$post = Post::create (['user_id' => Auth()->user()->id]+ $request->validated());

Franz Pool Nuñez Villazana

Estoy trabajando con laravel 8 ¿Como podria usar el auth()->user->id?

Carlos Eduardo Gomez García

¡Hola!, ¿No te funciona de esa manera? En teoría, en Laravel 8 el helper auth() sigue estando disponible, deberías poder usarlo de esa manera, aunque user es un método, prueba con esto:

auth()->user()->id

Franz Pool Nuñez Villazana

Me sale este error cuando lo uso

Trying to get property 'id' of non-object

Ferdinand M

alguien me puede ayudar cuando guardo la imagen me sale asi en mi base de datos

C:\xampp7.4\tmp\phpEE65.tmp

este es el código que tengo

if($request->file('file')){

        $post->image = $request->file('file')->store('posts','public');

        // Salvamos la información
        $post->save();
    }

y este es el dd($request)

array:5 [▼ "title" => "asdasd" "body" => "asdasd" "iframe" => "asdasd" "_token" => "vS25I5J3ZfxnPfDTgsHSfr5zk1YgGIImEQvewHlj" "image" => Illuminate\Http\UploadedFile {#1263 ▼ -test: false -originalName: "1581867541_678905_1581924335_album_grande.jpg" -mimeType: "image/jpeg" -error: 0 #hashName: null path: "C:\xampp7.4\tmp" filename: "phpC77F.tmp" basename: "phpC77F.tmp" pathname: "C:\xampp7.4\tmp\phpC77F.tmp" extension: "tmp" realPath: "C:\xampp7.4\tmp\phpC77F.tmp" aTime: 2020-12-21 21:53:23 mTime: 2020-12-21 21:53:23 cTime: 2020-12-21 21:53:23 inode: 633318697599137181 size: 174137 perms: 0100666 owner: 0 group: 0 type: "file" writable: true readable: true executable: false file: true dir: false link: false linkTarget: "C:\xampp7.4\tmp\phpC77F.tmp" } ]

Jhon Fabio Cardona Martinez

Comparta tu código para depurar el error.

Germán Alejandro Niño

Tuve el mismo problema y estuve cambiando el nombre de las variables en PostController, create.blade.php y Post. No se exactamente que fue lo que causo el problema pero puede que te sirva. Siempre puedes comparar tu codigo con el Github del curso.

Sergio VO

Si estas usando la version 7 de laravel y te indica 403 | This action is unauthorized., haz lo siguiente:

Cambiar el valor de false a true en el archivo de validacion: use App\Http\Requests\PostRequest;

 public function authorize()
    {
        return true; // cambiar de false a true. 
    }

Ya que como indica el error 403 | This action is unauthorized. aunque se pasa desapersibido al principio del video es un paso importante.

Carlos Eduardo Gomez García

Creo que algo muy importante a recalcar es que por sobre todo estamos creando código limpio y entendible, es genial todo lo que podemos lograr con pocas lineas de código y todo gracias a Laravel

Patricio Castro Andrade

para los que les da problema con la version 8 al guardar la imagen a la base de datos (si que me dio dolor de cabeza que no lograba que se guarde). la creación de la carpeta la cree manualmente pero luego la img le envio a esa dirección. hay que mencionar que le agregue una validación para la imagen en el archivo PostController.php espero les ayude con la nueva version de Laravel 8 public function store(PostRequest $request) {

//return $request->all();

    $post = Post::create([
        'user_id' => auth()->user()->id
    ] + $request->all());



    $request->validate([

        'title'=>'required|min:3|max:20',
        'file' =>'image|mimes:jpg,jpeg,gif,png,svg|max:2048',
        'body' =>'required'
        //'iframe'=>'required' 

    ]);

if ($request->file('file')) {

    $imagenes= $request->file('file')->store('public/posts');
    $url=storage::url($imagenes);
    $post->image = $imagenes;
    $post->save();

    }

return back()->with('status', 'Creado con éxito');

Juan Felipe Arias Trullo

Para los que tienen el error de que Requests no existe. Es debido a que el Requests se llama PostRequest. Deben cambiar en la función store el nombre del parametro Request a PostRequest.

victor saumelli

Buenas, cuando intento guardar no me arroja ningun error pero no me guarda en base de datos y ni me aparece el mensaje de status. Cuando inspecciono la pagina y me meto en network veo que el metodo de posts arrojo un mensaje que dice: "failed to load response data" verifique si todo lo tenia bien escrito y en efecto no tengo errores de tipeo en el codigo.. si alguien tiene alguna sugerencia o algo. por favor ayudarme, gracias.

Deyner Steven Rendon Valladales

Yo pienso que puede ser un problema al validar los datos en PostRequest, de pronto alguna validaciones no esta pasando.

Edwin Armando Perez Palma

Tengo exactamente el mismo problema, solo me recarga la pagina, no me guarda nada en la base de datos

Andrés David Solarte Vidal

Si poseen problemas con el almacenamiento de imágenes con la siguiente excepcion:

fopen(): Filename cannot be empty

Solo deben configurar el php.init para dar permito a la ruta TEMP

upload_tmp_dir =C:\Windows\Temp

Brian Sánchez Izquierdo

a que se refiere e profe cuando menciona la "variable de sesión de tipo flash que se va a eliminar cuando actualicemos el navegador?" en el minuto 5:59

Fabian Andres Villon Garcia

Es de un solo uso, es decir a la siguiente petición ya no existe status

Paul Cortes

Excelente por la subida de archivos, lo que guardamos en la BD es la ruta del archivo que subimos

Añaqui Apolinar Morales

¿Alguien lo esta trabajando con laravel 8? No me funciona el store, me manda un 302

Jose Luis Barboza Gonzales

lo solucionaste?

Añaqui Apolinar Morales

Si ya lo solcuione

Blas Rodriguez934

muy bueno.

Italo Morales Fantone

Profesor