Laravel error if en relationship

[ArturoGallegos]

El dia de hoy me he topado con un error algo curioso que no he savido solventar directamente con eloquent.,



Verán tengo la siguiente función para obtener los detalles de un inventario y sus ventas filtrado o no por vendedor
Relacion en inventario:

Código PHP:

public function Ventas(){
        return $this->hasMany('App\Ventas', 'inventario_id');
    }
 

Funcion en controlador

Código PHP:

public function detalle_evento($inventario_id, $vendedor = false){
    $inventario = Inventario::where('id', $inventario_id);
    if($vendedor){
        $inventario = $inventario->with(['Ventas' => function($query) use($vendedor){
            $query->where('vendedor', $vendedor);
        }]);
    }else{
        $inventario = $inventario->with('Ventas');
    }
    return $inventario->first();
}
 

Hasta aqui todo Ok, mando llamar la funcion dentro de otra funcion que genera la vista y hago un return en ese momento para verificar los datos, me las ventas filtradas por ID de vendedor en json dado que no he imprimido la vista.... Magnifico

Código PHP:

return $evento = $this->detalle_evento($inventario_id, $request->vendedor);
 

Pero requiero hacer un par de operaciones extra

Código PHP:

foreach ($evento->ventas as $venta) {
//.... aqui mis operaciones
}
return $evento;
 

Aquí es donde me quedo de 6.... me retorna todas las ventas sin filtrar, a lo que me pregunto como diablos es que tengo todas las ventas alli si anteriormente ya habian sido filtradas... ¿como lo soluciono?


he intentado crear un array independiente utilizando el mismo foreach conque hago las operaciones extra, pero obvio no funciona, al hacer el foreach ya tiene todo sin filtrar.


Inclusive a modo de prueba hice esto, todo bien, filtrado como debe.

Código PHP:

public function test(){
return $evento = $this->detalle_evento(1, 4);
}
 

Luego esto, ERROR, datos sin filtrar.

Código PHP:

public function test(){
$evento = $this->detalle_evento(1, 4);
return $evento->ventas;
}
 

En el momento que doy entre a la siguiente linea ya me muestra todo sin filtrar

[ArturoGallegos]

Solucion temporal al problema aunque no me guste mucho el metodo/forma en que lo he hecho.

Código PHP:

public function test(Request $request){
  $evento = $this->detalle_evento($request->inventario_id, $request->vendedor);
  foreach($evento->ventas as $key=>$venta){
    if(isset($request->vendedor) && $venta->vendedor !== $request->vendedor){
      unset($evento->ventas[$key]);
    }
  }
  return $evento->ventas;
}
 

[hhs]

El error es porque estas usando asignación dentro del if y no es necesario

Código PHP:

public function detalle_evento($inventario_id, $vendedor = false){
    $inventario = Inventario::where('id', $inventario_id);
    if($vendedor){
        $inventario->with(['Ventas' => function($query) use($vendedor){
            $query->where('vendedor', $vendedor);
        }]);
    }else{
        $inventario->with('Ventas');
    }
    return $inventario->first();
}
 

La razón de no usar la asignación es porque el resultado de la primera senctencia es un objeto Builder y hasta que no hagas un first, get o find puedes seguir armando el query

Tambien puedes usar when para sustituir el if. Por otro lado supongo que la segunda función esta como método del controller si es así separa todo en una nueva clase.
También hecha un vistazo al tema de pruebas en Laravel; te hacen la vida mucho mas sencilla

[ArturoGallegos]

Negativo, he probado como tu dices y obtengo el mismo resultado, hasta donde tengo entendido hacer la asignación no debería alterar el resultado pero igual lo probé sin asignación y sigue dando el mismo problema tal cual.

Utilizando el método When, he ahorrado código aunque el problema tampoco se solvento con este

Código PHP:

public function detalle_evento($inventario_id, $vendedor = false){
        return Inventario::where('id', $inventario_id)->when($vendedor, function ($query, $vendedor){
            $query->with(['Ventas' => function($query) use($vendedor){
                $query->where('vendedor', $vendedor);
            }]);
        }, function ($query){
            $query->with('Ventas');
        })->first();
    }
 

[hhs]

Eso es muy raro, para que te regrese sin filtro tendría que estar pasando por alto la condición o estas pasando por alto algo mas que no sabemos. Puedes proporcionar mas info?; muestra tu controller la relación entre modelos, porque asi como esta la consulta debe de funcionar sin problema.

[ArturoGallegos]

Pues acabo de limpiar todo para hacer la prueba y sigo con los mismos resultados, asi es como quedaron los modelos y el controlador, practicamente nada.

Modelo Inventario:

Código PHP:

<?php
 
namespace App;
 
use Illuminate\Database\Eloquent\Model;
use Illuminate\Database\Eloquent\ModelNotFoundException;
 
class Inventario extends Model
{
    protected $table = 'inventario_2';
 
    protected $casts = array(
        'ids' => 'array',
    );
 
    public function getProductos(){
        return $this->hasManyThrough('App\Productos', 'App\InventarioProductosPivote',
            'inventario_id', 'id', 'id', 'producto_id');
    }
 
    public function Ventas(){
        return $this->hasMany('App\Ventas', 'inventario_id');
    }
}
 

Modelo Ventas:

Código PHP:

<?php
 
namespace App;
 
use Illuminate\Database\Eloquent\Model;
 
class Ventas extends Model
{
    public function getInventario(){
        return $this->hasOne('App\Inventario', 'id', 'inventario_id');
    }
 
    public function Pagos(){
        return $this->hasMany('App\Pagos', 'venta_id');
    }
 
    public function Articulos(){
        return $this->hasMany('App\VentaArticulos', 'venta_id');
    }
}
 

Controlador:

Código PHP:

<?php
 
namespace App\Http\Controllers;
 
use App\Inventario;
use Illuminate\Http\Request;
 
class InventarioController extends Controller
{
    public function detalle_evento($evento_id, $vendedor = false){
        return Inventario::where('id', $evento_id)->when($vendedor, function ($query, $vendedor){
            $query->with(['Ventas' => function($query) use($vendedor){
                $query->where('vendedor', $vendedor);
            }, 'Ventas.Articulos']);
        }, function ($query){
            $query->with('Ventas', 'Ventas.Articulos');
        })->first();
    }
 
    public function detalle_evento_view_imprimir($evento_id, Request $request){
      return $evento = $this->detalle_evento($evento_id, $request->vendedor);// si hago el return aqui el filtro se respeta
      return $evento->ventas;// si hago el return aqui el filtro se ignora ,  porque???'
    }
}
 

[hhs]

Ya no alcanzo a hacer pruebas el día de hoy, pero de entrada algunas relaciones no son correctas
por ejemplo esta:

Código PHP:

public function getProductos(){
        return $this->hasManyThrough('App\Productos', 'App\InventarioProductosPivote',
            'inventario_id', 'id', 'id', 'producto_id');
    }
 

hasManyTrough solo es para relaciones 1 a N si tu relación en los modelos es N a M entonces tienes que usar belongsToMany.

Tambien la relación de invetario a ventas la colocas como hasMany, pero desde ventas la poner como hasOne y debe de ser belongsTo

Código PHP:

public function getInventario(){
        return $this->hasOne('App\Inventario', 'id', 'inventario_id');
    }
 

Esto debe de ser belongsTo ya que es el lado inverso de la relación 1 a N.
Verifica esas relaciones ya que puedes tener resultados inesperados si los dejas como están.

[hhs]

Realice algunas pruebas simulando un poco tus modelos y no obtengo el comportamiento que tienes a la hora de hacer el eager loading y obtener solo las ventas que tienen un vendedor. No se si obtuviste otro resultado con lo que te comente de las relaciones ?

[ArturoGallegos]

Buen día @HHS

Perdón por la demora, sobre las migraciones cumplen su cometido así como están, eso si, como bien dices con ese modelo y controlador funciona.

He hecho una instalación agregando unicamente dichos modelos y controladores y funciona todo correcto, así que algo mas esta interfiriendo aunque no lo vea., ya que se me han pedido mas cosas para el proyecto estoy mudando todo poco a poco a la nueva instalación, cuando encuentre la causa la comparto.

Muchas gracias por tu paciencia y tiempo.

[hhs]

Vaya que curioso, si no es evidente el problema puede ser algo en la lógica en algún punto de la aplicación, espero que encuentres donde esta, sería interesante saber como es que esta afectando el resultado.