[SOLUCIONADO] Dos dudas de Laravel

[Fylux]

Buenas, soy nuevo en Laravel y estoy intentando migrar un código y se me han planteado dos cuestiones.

La primera es que ya tengo las tablas de mi proyecto creadas así que, ¿debería volver a crearlas con migraciones?
Además en Laravel le añade create_at y update_at, ¿debo añadírselo a las tablas que ya tengo?

La segunda cuestión es, ¿debo creo un modelo por cada tabla exactamente?

Saludos.

[JaimeMSE]

Hola,

Te voy contestando por pasos:

La primera es que ya tengo las tablas de mi proyecto creadas así que, ¿debería volver a crearlas con migraciones?

No necesariamente. Las migraciones de laravel te ayudan a volver a un estado anterior de cuando creastes alguna tabla o conjunto de tablas etc (principalmente usando el comando php artisan migrate). Pero no es para nada necesario. Es sólo para que puedas crear nuevas tablas con el comando, eliminarlas, etc.

Además en Laravel le añade create_at y update_at, ¿debo añadírselo a las tablas que ya tengo?

Eso se arregla añadiendo la variable -- public $timestamps = false; -- a cada clase de cada modelo que no quieras que se le añada esos dos campos.

La segunda cuestión es, ¿debo creo un modelo por cada tabla exactamente?

Para nada, yo por ejemplo tengo un archivo en models que se llama Usuario.php y dentro tengo la clase Usuarios, otra clase llamada Mensajes_privados, otra llamada Mascotas (estoy haciendo una web relacionada con los animales).

Lo que yo entiendo es que intentes juntar todo lo necesario de un conjunto de datos de tu web. Me refiero a que si tienes un modelo que trata de usuarios y los mensajes privados y las mascotas van relacionadas directamente con el modelo (que si no existiese el modelo Usuario éstas no podrían existir), que las unas en mismo archivo.

Espero haberte ayudado.
Un saludo!

[Fylux]

Muchas gracias por responder y si no es mucha molestia quisiera exponer dos dudas que me salen entonces.
En los modelos he visto que en Laravel se relacionan en uno con uno, uno con varios o varios con varios.
Lo que ocurre es que yo tengo un sistema de mensajería donde cada mensaje tiene id_emisor e id_receptor. De forma que no se como hacer para que en unos casos busque el id del usuario en el emisor y en otros caso en el receptor.

Mi segunda duda es si hace falta que diga la key que quiero usar para relacionar una tabla con otra. Y si no hace falta, ¿qué llave busca?

Saludos y gracias de antemano.

[JaimeMSE]

Hola

Para la primera duda no podrías usar la función find() que usa laravel. Por ejemplo:

Código PHP:

$usuario = Usuario::find(1)
 

Eso te devolvería el primer usuario buscando por el campo ID. Pero como en tu caso no lo usas puedes hacer:

Código PHP:

$usuario = Usuario::where('id_receptor','=', 1)->first();
 

Y ya te daría el mensaje con el id_receptor 1.

Para la segunda duda consulta:
http://four.laravel.com/docs/eloquent#relationships

Un saludo!

[eternoaprendiz]

Iniciado por Fylux

Muchas gracias por responder y si no es mucha molestia quisiera exponer dos dudas que me salen entonces.
En los modelos he visto que en Laravel se relacionan en uno con uno, uno con varios o varios con varios.
Lo que ocurre es que yo tengo un sistema de mensajería donde cada mensaje tiene id_emisor e id_receptor. De forma que no se como hacer para que en unos casos busque el id del usuario en el emisor y en otros caso en el receptor.

Mi segunda duda es si hace falta que diga la key que quiero usar para relacionar una tabla con otra. Y si no hace falta, ¿qué llave busca?

Saludos y gracias de antemano.

En el modelo Mensaje puedes tener algo así:

Código PHP:

          // models/mensaje.php
 
    public function emisor()
    {
        return $this->BelongsTo('User', 'from_id');
    }
 
    public function receptor()
    {
        return $this->BelongsTo('User', 'to_id');
    } 
   

Y en el controlador algo así:

Código PHP:

    // controllers/mensajes.php
 
  $mensajes = Mensaje::with('emisor', 'receptor')->get(); 
   

Tanto from_id como to_id enganchan con el user.id, salvo que cambies el primary key del modelo user.

[eternoaprendiz]

Para buscar, puedes añadir algo así:

Código PHP:

      //models/mensaje.php
 
    public function scopeDondeEmisor($query, $from_id)
    {
        return $query->where('from_id', '=', $from_id);
    } 
   

Y en el controlador usarlo así:

Código PHP:

    // controllers/mensajes.php
 
  $mensajes = Mensaje::with('emisor', 'receptor')->DondeEmisor(3)->get(); 
   

Luego en la vista puedes:

Código PHP:

   
@foreach ($mensajes as $mensaje):
 {{ $mensaje->emisor->nombre }} -  {{ $mensaje->receptor->nombre }}
@endforeach 
   

Creo que algo así se puede hacer.

[Fylux]

Muchas gracias por las repuestas :D
Espero no abusar de ustedes si les pregunto una duda sobre como aplicar correctamente el patrón MVC en mi proyecto.
El proyecto tiene como un 'motor' que se ejecuta cada vez que se actualiza la página y hacer alrededor de unas 50 consultas a la base de datos.
¿Este debería en un método en el modelo, no?

Saludos.

[eternoaprendiz]

Iniciado por Fylux

Muchas gracias por las repuestas :D
Espero no abusar de ustedes si les pregunto una duda sobre como aplicar correctamente el patrón MVC en mi proyecto.
El proyecto tiene como un 'motor' que se ejecuta cada vez que se actualiza la página y hacer alrededor de unas 50 consultas a la base de datos.
¿Este debería en un método en el modelo, no?

Saludos.

Toda la lógica del negocio debe ir en la capa modelo. En tu caso deberías tener varios modelos para todas esas operaciones.

Trata de que las clases no pasen de las 200 líneas y los métodos ocupen menos de 20 líneas.

El controlador sólo debe proporcionar datos al modelo.

[Fylux]

¿Entonces este ejemplo no sería correcto?

Código PHP:

  class RegistrarController extends BaseController
{
    public function registrar()
    {
        $usuario = Input::get('usuario');
        $password = Input::get('password');
        $email = Input::get('email');
        $rules = array(
                'usuario' => 
                    array('required', 'min:3','max:20','alpha_dash','required'),
                'password'=>
                    array('required', 'min:6','max:50','alpha_dash','required'),
                'email'=>
                    array('required','email')
                    );
 
        $validator = Validator::make(Input::all(), $rules);
 
        if ($validator->fails())
        {
            return Redirect::to('login')->withErrors($validator);
        }
 
        $user = Usuario::where('nombre', '=', $usuario)->orWhere('correo','=',$email)->first();
        if (is_null($user))
        {
            $user = new Usuario;
            $user->nombre = $usuario;
            $user->password = $password;
            $user->correo = $email;
            $user->fecha_ingreso=time();
            $user->ip=Request::getClientIp();
 
            $user->save();
        }
        else
        {
            echo "Pong";
        }
    }
} 
   

Supongo que para que fuese más MVC debería hacer que el insert se hiciera con un método del modelo.
¿Estoy en lo cierto?

Saludos.

[eternoaprendiz]

Yo lo veo bien. Yo crearía algunos métodos protegidos en el controlador para validar y otro para crear el usuario. Por ejemplo:

Código PHP:

   $this->validarRegistro();
 $this->crearUsuario(); 
   

Y esto:

Código PHP:

   $user = Usuario::where('nombre', '=', $usuario)->orWhere('correo','=',$email)->first();
 if (is_null($user)) 
   

Lo sustituiría por:

Código PHP:

   if ( ! Usuario::existe($nombre, $email)) 
   

Obviamente tienes que crear el metodo existe en el modelo.

[Fylux]

Gracias.
Un problema estoy teniendo todo el día es el del Login.
Ya sabes, este es mi controlador para registrar

Código PHP:

  <?php
 
class RegistrarController extends BaseController
{
    public function registrar()
    {
        $usuario = Input::get('usuario');
        $password = Hash::make(Input::get('password'));
        $email = Input::get('email');
        $rules = array(
                'usuario' => 
                    array('required', 'min:3','max:20','alpha_dash','required'),
                'password'=>
                    array('required', 'min:6','max:50','alpha_dash','required'),
                'email'=>
                    array('required','email')
                    );
 
        $validator = Validator::make(Input::all(), $rules);
 
        if ($validator->fails())
        {
            return Redirect::to('login')->withInput()->withErrors($validator);
        }
 
        $user = Usuario::where('usuario', '=', $usuario)->orWhere('correo','=',$email)->orWhere('ip','=',Request::getClientIp())->first();
        if (is_null($user))
        {
            $user = new Usuario;
            $user->usuario = $usuario;
            $user->password = $password;
            $user->correo = $email;
            $user->fecha_ingreso=time();
            $user->ip=Request::getClientIp();
 
            $user->save();
 
            return Redirect::to('login')->with('msg','Te has registrado correctamente');
        }
        else
        {
            return Redirect::to('login')->with('msg','El usuario o el correo ya está en uso');
        }
    }
}
?>   

Y este mi controlador para loguear

Código PHP:

  <?php
 
class UsuarioController extends BaseController {
 
    public function login()
    {
        $user = array(
            'usuario' => Input::get('usuario'),
            'password' => Input::get('password')
        );
        
        if (Auth::attempt($user)) {
            return Redirect::to('login')
                ->with('msg', 'You are successfully logged in.');
        }
    }
 
}
?>   

Sin embargo no se porqué siempre me devuelve falso. He probado a no ponerle Hash y cosas así. Los campos de mi tabla usuarios son usuario y password.

¿Alguna idea de por qué no me loguea?

Saludos y gracias.

[eternoaprendiz]

Posiblemente porque estás usando el atributo usuario en vez de username en el modelo usuario.

Revisa esto:

Código PHP:

  
// models/usuario.php

public function getAuthIdentifier() {

   return $this->usuario;

} 
   

[Fylux]

¿Que debo hacer exactamente con esa función?
Saludos.

[eternoaprendiz]

Iniciado por Fylux

¿Que debo hacer exactamente con esa función?
Saludos.

Me parece que hay que cambiar el config/auth.php

y añadir 'username' => 'email' o el campo que sea a validar.

[Fylux]

Eso lo he cambiado para poner
'Model'=>'Usuario'
'table'=>'usuarios'

Pero lo que tu dices está en Laravel 3, pero no en el 4.

Código PHP:

  <?php
 
return array(
 
    /*
    |--------------------------------------------------------------------------
    | Default Authentication Driver
    |--------------------------------------------------------------------------
    |
    | This option controls the authentication driver that will be utilized.
    | This drivers manages the retrieval and authentication of the users
    | attempting to get access to protected areas of your application.
    |
    | Supported: "database", "eloquent"
    |
    */
 
    'driver' => 'eloquent',
 
    /*
    |--------------------------------------------------------------------------
    | Authentication Model
    |--------------------------------------------------------------------------
    |
    | When using the "Eloquent" authentication driver, we need to know which
    | Eloquent model should be used to retrieve your users. Of course, it
    | is often just the "User" model but you may use whatever you like.
    |
    */
 
    'model' => 'Usuario',
 
    /*
    |--------------------------------------------------------------------------
    | Authentication Table
    |--------------------------------------------------------------------------
    |
    | When using the "Database" authentication driver, we need to know which
    | table should be used to retrieve your users. We have chosen a basic
    | default value but you may easily change it to any table you like.
    |
    */
 
    'table' => 'usuarios',
 
    /*
    |--------------------------------------------------------------------------
    | Password Reminder Settings
    |--------------------------------------------------------------------------
    |
    | Here you may set the settings for password reminders, including a view
    | that should be used as your password reminder e-mail. You will also
    | be able to set the name of the table that holds the reset tokens.
    |
    */
 
    'reminder' => array(
 
        'email' => 'emails.auth.reminder', 'table' => 'password_reminders',
 
    ),
 
);   

Saludos.