Merge pull request 'funcion/arreglar-planillas' (#43) from funcion/arreglar-planillas into master

Reviewed-on: #43

app/Console/Commands/AgregarEsBonoAPedidosAprobados.php

@@ -0,0 +1,72 @@
+<?php
+
+namespace App\Console\Commands;
+
+use Illuminate\Console\Command;
+use Illuminate\Support\Facades\DB;
+
+class AgregarEsBonoAPedidosAprobados extends Command
+{
+    /**
+     * The name and signature of the console command.
+     *
+     * @var string
+     */
+    protected $signature = 'command:AgregarEsBonoAPedidosAprobados';
+
+    /**
+     * The console command description.
+     *
+     * @var string
+     */
+    protected $description = 'Agrega "producto_bono" a la view PedidosAprobados';
+
+    /**
+     * Create a new command instance.
+     *
+     * @return void
+     */
+    public function __construct()
+    {
+        parent::__construct();
+    }
+
+    /**
+     * Execute the console command.
+     *
+     * @return int
+     */
+    public function handle(): int
+    {
+        DB::statement("
+            ALTER VIEW pedidos_aprobados(
+                grupo_de_compra_id,
+                grupo_de_compra_nombre,
+                grupo_de_compra_region,
+                producto_id,
+                producto_nombre,
+                producto_precio,
+                cantidad_pedida,
+                total_por_producto,
+                producto_es_bono
+            ) AS
+            SELECT
+              g.id as grupo_de_compra_id,
+              g.nombre as grupo_de_compra_nombre,
+              g.region as grupo_de_compra_region,
+              pr.id AS producto_id,
+              pr.nombre as producto_nombre,
+              pr.precio as producto_precio,
+              SUM(ps.cantidad) as cantidad_pedida,
+              pr.precio*SUM(ps.cantidad) as total_por_producto,
+              pr.bono
+            FROM grupos_de_compra g
+              JOIN subpedidos s ON (s.grupo_de_compra_id = g.id AND s.aprobado=1)
+              JOIN producto_subpedido ps ON (ps.subpedido_id = s.id)
+              JOIN productos pr ON (pr.id = ps.producto_id)
+            GROUP BY
+              g.id, g.nombre, pr.id, pr.nombre
+        ;");
+        return 0;
+    }
+}

app/Console/Commands/CrearPedidosAprobadosViewCommand.php

@@ -36,7 +36,7 @@ class CrearPedidosAprobadosViewCommand extends Command
      *
      * @return int
      */
-    public function handle()
+    public function handle(): int
     {
     	DB::statement("
             CREATE OR REPLACE VIEW pedidos_aprobados

app/Filtros/Filtro.php

@@ -5,12 +5,15 @@ namespace App\Filtros;
 use Illuminate\Database\Eloquent\Model;
 use Illuminate\Http\Request;
 use Illuminate\Database\Eloquent\Builder;
+use Symfony\Component\HttpKernel\Exception\HttpException;
+use Throwable;
+use TypeError;
 
 class Filtro extends Model
 {
-    protected $request;
+    protected Request $request;
     protected $builder;
-    protected $MENSAJES_ERROR = [
+    protected array $MENSAJES_ERROR = [
     	'ARGUMENTO' => 'Argumento inválido para el parámetro %s. Revise la documentación.'
     ];
 
@@ -22,10 +25,10 @@ class Filtro extends Model
     /**
      * Apply all existing filters, if available.
      *
-     * @param  \Illuminate\Database\Eloquent\Builder  $builder
+     * @param Builder $builder
-     * @return \Illuminate\Database\Eloquent\Builder
+     * @return Builder
      */
-    public function aplicar(Builder $builder)
+    public function aplicar(Builder $builder): Builder
     {
         $this->builder = $builder;
 
@@ -47,11 +50,11 @@ class Filtro extends Model
 
             //Llamar métodos sin argumentos
             if ($valor === null|| (is_a($valor,'String') && trim($valor)=='')){ $this->$metodo(); continue; }
-            
+
             //Llamar métodos con argumentos
             try {
                 $this->$metodo($valor);
-            } catch (\Throwable $th) {
+            } catch (Throwable $th) {
                 if (is_a($th,'TypeError') ) { throw new HttpException(400, sprintf($this->MENSAJES_ERROR['ARGUMENTO'],$filtro)); }
                 throw $th;
             }
@@ -63,7 +66,7 @@ class Filtro extends Model
     //Buscar un término en el nombre
     public function nombre(String $valor)
     {
-        $this->builder->where('nombre', "LIKE", "%" . $valor . "%")->orderByRaw("IF(nombre = '{$valor}',2,IF(nombre LIKE '{$valor}%',1,0)) DESC");
+        $this->builder->where('nombre', "LIKE", "%" . $valor . "%")->orderByRaw("IF(nombre = '$valor',2,IF(nombre LIKE '$valor%',1,0)) DESC");
     }
 
     public function alfabetico(String $order = 'asc')

app/Filtros/FiltroDeProducto.php

@@ -1,7 +1,6 @@
 <?php
 
 namespace App\Filtros;
-use Illuminate\Database\Eloquent\Builder;
 
 class FiltroDeProducto extends Filtro {
 
@@ -10,4 +9,4 @@ class FiltroDeProducto extends Filtro {
         $this->builder->where('categoria', $valor);
     }
 
-}
+}

app/Filtros/FiltroDeSubpedido.php

@@ -2,7 +2,7 @@
 
 namespace App\Filtros;
 
-use Illuminate\Database\Eloquent\Model;
+use TypeError;
 
 class FiltroDeSubpedido extends Filtro
 {

app/GrupoDeCompra.php

@@ -2,13 +2,15 @@
 
 namespace App;
 
+use App\Helpers\CsvHelper;
+use App\Helpers\PdfHelper;
 use App\Helpers\TransporteHelper;
+use Illuminate\Database\Eloquent\Builder;
 use Illuminate\Database\Eloquent\Model;
+use Illuminate\Database\Eloquent\Relations\HasMany;
+use Illuminate\Support\Collection;
 use Illuminate\Support\Facades\DB;
-use League\Csv\CannotInsertRecord;
+use Illuminate\Support\Facades\Log;
-use League\Csv\Reader;
-use League\Csv\Writer;
-use Mpdf\Mpdf;
 
 class GrupoDeCompra extends Model
 {
@@ -17,12 +19,12 @@ class GrupoDeCompra extends Model
     protected $table = 'grupos_de_compra';
     protected $hidden = ['password'];
 
-    public function subpedidos()
+    public function subpedidos(): HasMany
     {
-        return $this->hasMany('App\Subpedido');
+        return $this->hasMany(Subpedido::class);
     }
 
-    public function toggleDevoluciones()
+    public function toggleDevoluciones(): bool
     {
         $this->devoluciones_habilitadas = !$this->devoluciones_habilitadas;
         $this->save();
@@ -91,25 +93,52 @@ class GrupoDeCompra extends Model
         return TransporteHelper::totalTransporte($this->totalCentralesQuePaganTransporte());
     }
 
-    public function cantidadTransporte()
+    /**
+     * @return int
+     * Calcula la cantidad de bonos de transporte del barrio
+     */
+    public function cantidadTransporte(): int
     {
         return TransporteHelper::cantidadTransporte($this->totalCentralesQuePaganTransporte());
     }
 
-    public function exportarPlanillasAPdf()
+    public function exportarPedidosAPdf()
     {
         $subpedidos = $this->pedidosAprobados();
-        //generar pdf
+        PdfHelper::exportarPedidos($this->nombre . '.pdf', $subpedidos);
-        $mpdf = new Mpdf();
+    }
-        foreach ($subpedidos as $subpedido) {
+
-            $tabla = $subpedido->generarHTML();
+    function pedidoParaPdf(): array
-            // agregar la tabla al pdf en una nueva página
+    {
-            $mpdf->WriteHTML($tabla);
+        $productos = $this->productosPedidos(true, true);
-            $mpdf->AddPage();
+        $pedido = [];
+        $pedido['productos'] = [];
+
+        $pedido['nombre'] = $this->nombre;
+        foreach ($productos as $producto) {
+            $productoParaPdf = [];
+            $productoParaPdf['pivot'] = [];
+            $productoParaPdf['nombre'] = $producto->producto_nombre;
+            $productoParaPdf['pivot']['cantidad'] = $producto->cantidad_pedida;
+            $productoParaPdf['pivot']['notas'] = false;
+            $productoParaPdf['bono'] = $producto->producto_es_bono;
+
+            $pedido['productos'][] = $productoParaPdf;
         }
-        $filename = $this->nombre . '.pdf';
+
-        // imprimir el pdf
+        return $pedido;
-        $mpdf->Output($filename, "D");
+    }
+
+    public function generarHTML()
+    {
+        $view = view("pdfgen.pedido_tabla", ["pedido" => $this->pedidoParaPdf()]);
+        return $view->render();
+    }
+
+    public static function exportarPedidosBarrialesAPdf()
+    {
+        $barrios = GrupoDeCompra::barriosMenosPrueba()->get();
+        PdfHelper::exportarPedidos('pedidos_por_barrio.pdf', $barrios);
     }
 
     static function filaVacia(string $product, int $columns): array
@@ -122,7 +151,7 @@ class GrupoDeCompra extends Model
     }
 
     //Asume que los productos están gruadados en orden de fila
-    public static function obtenerTemplateDeFilasVacias(int $columns)
+    public static function obtenerTemplateDeFilasVacias(int $columns): array
     {
         $productosFilaID = Producto::productosFilaID();
         $productosIDNombre = Producto::productosIDNombre();
@@ -135,104 +164,53 @@ class GrupoDeCompra extends Model
             $template[$fila] = GrupoDeCompra::filaVacia($productosIDNombre[$id], $columns);
             $num_fila = $fila + 1;
         }
-        $template[GrupoDeCompra::obtenerFilaDeBonoTransporte()] = GrupoDeCompra::filaVacia("Bonos de transporte", $columns);
+        $template[TransporteHelper::filaTransporte()] = GrupoDeCompra::filaVacia("Bonos de transporte", $columns);
         return $template;
     }
 
-    private static function obtenerFilaDeBonoTransporte()
-    {
-        $csv = Reader::createFromPath(resource_path('csv/productos.csv'), 'r');
-        $csv->setDelimiter("|");
-        $csv->setEnclosure("'");
-        $registros = $csv->getRecords();
-
-        foreach ($registros as $key => $registro)
-            if ($registro[0] == 'T') return $key;
-
-        throw new Exception('No hay bono de transporte');
-    }
-
-    private function totalPedidosSinBonos()
-    {
-        $total = 0;
-        foreach ($this->pedidosAprobados() as $pedido) {
-            $total += ceil($pedido->totalSinBonos());
-        }
-        return $total;
-    }
-
-    public function calcularCantidadBDT()
-    {
-        $total = 0;
-        foreach ($this->pedidosAprobados() as $pedido) {
-            $total += $pedido->totalParaTransporte();
-        }
-        return ceil($total / 500);
-    }
-
-    public function totalBonosBarriales()
-    {
-        $total = 0;
-        $bonoBarrial = Producto::where('nombre', 'LIKE', '%barrial%')->first();
-        if ($bonoBarrial) {
-            $pedidos = $this->pedidosAprobados();
-            foreach ($pedidos as $pedido) {
-                $bonoPedido = $pedido->productos()->find($bonoBarrial["id"]);
-                if ($bonoPedido) {
-                    $total += $bonoPedido["pivot"]["total"];
-                }
-            }
-        }
-        return $total;
-    }
-
     public function exportarPedidoEnCSV()
     {
         $records = $this->generarColumnaCantidades();
-        try {
+
-            $writer = Writer::createFromPath(resource_path('csv/exports/' . $this->nombre . '.csv'), 'w');
+        CsvHelper::generarCsv('csv/exports/' . $this->nombre . '.csv', $records);
-            $writer->insertAll($records);
-        } catch (CannotInsertRecord $e) {
-            var_export($e->getRecords());
-        }
     }
 
-    public function generarColumnaCantidades()
+    public function generarColumnaCantidades(): array
     {
-        $productos_en_pedido = DB::table('pedidos_aprobados')->where('grupo_de_compra_id', $this->id)->get()->keyBy('producto_id');
+        $productos_en_pedido = $this->productosPedidos();
 
         //si no hay pedidos aprobados, salir
         if ($productos_en_pedido->count() == 0) {
-            \Log::debug("El grupo de compra " . $this->nombre . " no tiene pedidos aprobados.");
+            Log::debug("El grupo de compra " . $this->nombre . " no tiene pedidos aprobados.");
             return [];
         }
 
         $records = $this->obtenerTemplateDeFilasVacias(1);
-        $productos_id_fila = Producto::productosIdFila();
+        $productos_id_fila = Producto::productosIDFila();
         foreach ($productos_en_pedido as $id => $producto_pedido) {
             $fila = $productos_id_fila[$id];
             $records[$fila][1] = $producto_pedido->cantidad_pedida;
         }
 
-        $records[$this->obtenerFilaDeBonoTransporte()][1] = $this->calcularCantidadBDT();
+        $records[TransporteHelper::filaTransporte()][1] = $this->cantidadTransporte();
 
         return $records;
     }
 
     public function exportarPedidoConNucleosEnCSV()
     {
-        $productos_en_pedido = DB::table('pedidos_aprobados')->where('grupo_de_compra_id', $this->id)->get()->keyBy('producto_id');
+        $productos_en_pedido = $this->productosPedidos();
 
         // si no hay pedidos aprobados, salir
         if ($productos_en_pedido->count() == 0) {
-            \Log::debug("El grupo de compra " . $this->nombre . " no tiene pedidos aprobados.");
+            Log::debug("El grupo de compra " . $this->nombre . " no tiene pedidos aprobados.");
             return;
         }
 
         $pedidos = $this->pedidosAprobados();
         // Generar tabla vacía con una columna por núcleo
         $records = $this->obtenerTemplateDeFilasVacias($pedidos->count());
-        $productos_id_fila = Producto::productosIdFila();
+        $productos_id_fila = Producto::productosIDFila();
 
         foreach ($productos_en_pedido as $id => $producto_pedido) {
             $fila = $productos_id_fila[$id];
@@ -251,13 +229,7 @@ class GrupoDeCompra extends Model
         }
         array_splice($records, 0, 0, array($nucleos));
 
-        // Guardar en un archivo .csv
+        CsvHelper::generarCsv('csv/exports/' . $this->nombre . '-completo.csv', $records);
-        try {
-            $writer = Writer::createFromPath(resource_path('csv/exports/' . $this->nombre . '-completo.csv'), 'w');
-            $writer->insertAll($records);
-        } catch (CannotInsertRecord $e) {
-            var_export($e->getRecords());
-        }
     }
 
     public function agregarCantidad($pedido, $id, array $records, $fila, int $i): array
@@ -269,52 +241,40 @@ class GrupoDeCompra extends Model
         return array($records, $i, $cantidad);
     }
 
-    static public function totalesParaTransportePorBarrio() {
+    public static function barriosMenosPrueba(): Builder
-        return DB::table('grupos_de_compra')
+    {
-            ->leftJoin('subpedidos', 'grupos_de_compra.id', '=', 'subpedidos.grupo_de_compra_id')
+        return self::where('nombre', '<>', 'PRUEBA')
-            ->leftJoin('producto_subpedido', 'subpedidos.id', '=', 'producto_subpedido.subpedido_id')
+            ->orderBy('region')
-            ->leftJoin('productos', 'producto_subpedido.producto_id', '=', 'productos.id')
+            ->orderBy('nombre');
-            ->where(function ($query) {
-                $query->whereNull('productos.categoria')
-                ->orWhere('productos.categoria', 'not like', '%SUBSIDIADO%');
-            })
-            ->where(function ($query) {
-                $query->whereNull('productos.bono')
-                ->orWhere('productos.bono', 0);
-            })
-            ->where(function ($query) {
-                $query->whereNull('subpedidos.aprobado')
-                ->orWhere('subpedidos.aprobado', 1);
-            })
-            ->select(
-                'grupos_de_compra.id as id',
-                'grupos_de_compra.nombre as barrio',
-                DB::raw('COALESCE(SUM(producto_subpedido.cantidad * productos.precio), 0) as total')
-            )
-            ->groupBy('grupos_de_compra.id')
-            ->get();
     }
 
+    public static function transportePorBarrio(): array
+    {
+        $result = [];
+        $barrios = GrupoDeCompra::barriosMenosPrueba()->get();
 
-    static public function planillaTransporte() {
+        foreach ($barrios as $barrio) {
-        $totalesPorBarrio = self::totalesParaTransportePorBarrio();
+            $result[] = $barrio->cantidadTransporte();
-        $barrios = [];
-        $bonosDeTransporte = [];
-
-        foreach ($totalesPorBarrio as $totalBarrio) {
-            $barrios[] = $totalBarrio->barrio;
-            $bonosDeTransporte[] = ceil($totalBarrio->total / 500);
         }
 
-        $planilla = [];
+        return $result;
-        $planilla[] = array_merge(['Barrio'], $barrios);
+    }
-        $planilla[] = array_merge(['Cant. bonos de transporte'], $bonosDeTransporte);
 
-        try {
+    /**
-            $writer = Writer::createFromPath(resource_path('csv/exports/transporte-por-barrio.csv'), 'w');
+     * @return Collection
-            $writer->insertAll($planilla);
+     */
-        } catch (CannotInsertRecord $e) {
+    public function productosPedidos($excluirBarriales = false, $excluirBonos = false): Collection
-            var_export($e->getRecords());
+    {
-        }
+        $query = DB::table('pedidos_aprobados')
+            ->where('grupo_de_compra_id', $this->id);
+
+        if ($excluirBarriales)
+            $query = $query->where('producto_nombre','NOT LIKE','%barrial%');
+        if ($excluirBonos)
+            $query = $query->where('producto_es_bono',false);
+
+        return $query
+            ->get()
+            ->keyBy('producto_id');
     }
 }

app/Helpers/CanastaHelper.php

@@ -9,14 +9,15 @@ use DatabaseSeeder;
 use Illuminate\Support\Arr;
 use Illuminate\Support\Facades\DB;
 use Illuminate\Support\Str;
-use League\Csv\Reader;
+
 
 class CanastaHelper
 {
-    const FILA_HEADER = "Tipo";
+    const TIPO = "Tipo";
-    const ULTIMA_FILA = "TOTAL";
+    const TOTAL = "TOTAL";
     const ARCHIVO_SUBIDO = 'Archivo subido';
     const CANASTA_CARGADA = 'Canasta cargada';
+    const TIPOS_BONO = ["B", "F", "BE"];
 
     public static function guardarCanasta($data, $path): string {
         $nombre = $data->getClientOriginalName();
@@ -31,53 +32,43 @@ class CanastaHelper
     public static function cargarCanasta($archivo) {
         self::limpiarTablas();
 
-        $csv = Reader::createFromPath(resource_path($archivo), 'r');
+        $registros = CsvHelper::getRecords($archivo);
-        $csv->setDelimiter("|");
-        $iHeader = self::obtenerIndiceDeHeader($csv);
-        $csv->setHeaderOffset($iHeader);
-        $registros = $csv->getRecords();
-
         $toInsert = [];
         $categoria = '';
-        foreach($registros as $i => $registro){
-            //filas que están arriba del header
-            if ($i <= $iHeader){
-                continue;
-            }
 
-            //finalizar
+        foreach($registros as $i => $registro) {
-            if ($registro[self::FILA_HEADER] == self::ULTIMA_FILA) {
+            // finalizar
+            if ($registro[self::TIPO] == self::TOTAL)
                 break;
-            }
 
-            //filas que no tienen tipo
+            // saltear filas que no tienen tipo
-            if (!Arr::has($registro,self::FILA_HEADER)|| trim($registro[self::FILA_HEADER]) == ''){
+            if (self::noTieneTipo($registro)) {
                 var_dump("no hay tipo en la fila " . $i);
                 continue;
             }
 
-            //saltear bono de transporte
+            // saltear bono de transporte
-            if ($registro[self::FILA_HEADER] == "T"){
+            if ($registro[self::TIPO] == "T"){
                 continue;
             }
 
-            //obtener categoria
+            // obtener categoria si no hay producto
             if ($registro['Producto'] == '') {
-                //es la pregunta de la copa?
+                // no es la pregunta de la copa?
-                if (Str::contains($registro[self::FILA_HEADER],"¿")) { continue; }
+                if (!Str::contains($registro[self::TIPO],"¿"))
-                $categoria = $registro[self::FILA_HEADER];
+                    $categoria = $registro[self::TIPO];
                 continue;
             }
 
-            //completar producto
+            // completar producto
             $toInsert[] = [
                 'fila'          => $i,
                 'categoria'     => $categoria,
-                'nombre'        => trim(str_replace('*', ' ',$registro['Producto'])),
+                'nombre'        => trim(str_replace('*', '',$registro['Producto'])),
                 'precio'        => $registro['Precio'],
                 'proveedor_id'  => self::obtenerProveedor($registro['Producto']),
-                'bono'          => $registro[self::FILA_HEADER] == "B",
+                'bono'          => in_array($registro[self::TIPO], self::TIPOS_BONO),
-                'requiere_notas'=> $registro[self::FILA_HEADER] =="PTC",
+                'requiere_notas'=> $registro[self::TIPO] =="PTC",
             ];
         }
 
@@ -90,18 +81,6 @@ class CanastaHelper
         self::log($archivo, self::CANASTA_CARGADA);
     }
 
-    private static function obtenerIndiceDeHeader($csv){
-        $registros = $csv->getRecords();
-        $iheader = 0;
-        foreach ($registros as $i => $registro){
-            if (strtolower($registro[0]) == strtolower(self::FILA_HEADER)) {
-                $iheader = $i;
-                break;
-            }
-        }
-        return $iheader;
-    }
-
     private static function obtenerProveedor($nombre) {
         $result = null;
         if (Str::contains($nombre,"*")){
@@ -137,7 +116,14 @@ class CanastaHelper
 
     private static function agregarBonoBarrial()
     {
-        $categoria = Producto::all()->pluck('categoria')->unique()->flatten()->first(function ($c) { return Str::contains($c, 'BONO'); });
+        $categoria = Producto::all()
+            ->pluck('categoria')
+            ->unique()
+            ->flatten()
+            ->first(function ($c) {
+                return Str::contains($c, 'BONO');
+            });
+
         DB::table('productos')->insert([
             'fila' => 420,
             'nombre' => "Bono barrial",
@@ -148,4 +134,13 @@ class CanastaHelper
             'requiere_notas'=> false,
         ]);
     }
+
+    /**
+     * @param $registro
+     * @return bool
+     */
+    public static function noTieneTipo($registro): bool
+    {
+        return !Arr::has($registro, self::TIPO) || trim($registro[self::TIPO]) == '';
+    }
 }

app/Helpers/CsvHelper.php

@@ -0,0 +1,40 @@
+<?php
+
+namespace App\Helpers;
+
+use Illuminate\Support\Facades\Log;
+use Iterator;
+use League\Csv\CannotInsertRecord;
+use League\Csv\Exception;
+use League\Csv\InvalidArgument;
+use League\Csv\Reader;
+use League\Csv\Writer;
+
+class CsvHelper
+{
+    public static function getRecords($filePath): Iterator {
+        $csv = Reader::createFromPath(resource_path($filePath));
+        try {
+            $csv->setDelimiter("|");
+            $csv->setEnclosure("'");
+            $csv->setHeaderOffset(0);
+            return $csv->getRecords();
+        } catch (InvalidArgument|Exception $e) {
+            Log::error($e->getMessage());
+            return null;
+        }
+    }
+
+    public static function generarCsv($filePath, $contenido, $headers = null): void
+    {
+        try {
+            $writer = Writer::createFromPath(resource_path($filePath), 'w');
+            if ($headers) {
+                $writer->insertOne($headers);
+            }
+            $writer->insertAll($contenido);
+        } catch (CannotInsertRecord $e) {
+            Log::error($e->getMessage(), $e->getTrace());
+        }
+    }
+}

app/Helpers/PdfHelper.php

@@ -0,0 +1,29 @@
+<?php
+
+namespace App\Helpers;
+
+use Mpdf\Mpdf;
+use Mpdf\MpdfException;
+
+class PdfHelper
+{
+    /**
+     * Requiere que el segundo argumento tenga definida la función generarHTML()
+     * para crear la tabla con los datos del pedido que se inserta en el pdf.
+     *
+     * @return void
+     * @throws MpdfException
+     */
+    public static function exportarPedidos($filepath, $pedidos)
+    {
+        $mpdf = new Mpdf();
+
+        foreach ($pedidos as $pedido) {
+            $html = $pedido->generarHTML();
+            $mpdf->WriteHTML($html);
+            $mpdf->AddPage();
+        }
+
+        $mpdf->Output($filepath, 'D');
+    }
+}

app/Helpers/TransporteHelper.php

@@ -2,6 +2,9 @@
 
 namespace App\Helpers;
 
+use App\CanastaLog;
+use Illuminate\Support\Facades\Log;
+
 class TransporteHelper
 {
     const COSTO_TRANSPORTE = 15;
@@ -16,4 +19,20 @@ class TransporteHelper
     {
         return self::cantidadTransporte($monto) * self::COSTO_TRANSPORTE;
     }
+
+    public static function filaTransporte()
+    {
+        $ultimaCanasta = CanastaLog::where('descripcion', CanastaHelper::CANASTA_CARGADA)
+            ->orderBy('created_at', 'desc')
+            ->pluck('path')
+            ->first();
+
+        $registros = CsvHelper::getRecords($ultimaCanasta);
+
+        foreach ($registros as $key => $registro)
+            if ($registro[CanastaHelper::TIPO] == 'T') return $key;
+
+        Log::error('No hay fila de tipo T en la planilla: ' . $ultimaCanasta);
+        return null;
+    }
 }

app/Http/Controllers/AdminController.php

@@ -3,8 +3,7 @@
 namespace App\Http\Controllers;
 
 use App\GrupoDeCompra;
-use Illuminate\Http\Request;
+use Symfony\Component\HttpFoundation\BinaryFileResponse;
-use Response;
 
 class AdminController extends Controller
 {
@@ -17,17 +16,19 @@ class AdminController extends Controller
         return view('auth/admin_subpedidos');
     }
 
-    public function exportarPlanillasAPdf(GrupoDeCompra $gdc) {
+    public function exportarPedidosAPdf(GrupoDeCompra $gdc) {
-        return $gdc->exportarPlanillasAPdf();
+        $gdc->exportarPedidosAPdf();
     }
 
-    public function exportarPedidoACSV(GrupoDeCompra $gdc) {
+    public function exportarPedidoACSV(GrupoDeCompra $gdc): BinaryFileResponse
+    {
         $gdc->exportarPedidoEnCSV();
         $file = resource_path('csv/exports/'.$gdc->nombre.'.csv');
         return response()->download($file);
     }
 
-    public function exportarPedidoConNucleosACSV(GrupoDeCompra $gdc) {
+    public function exportarPedidoConNucleosACSV(GrupoDeCompra $gdc): BinaryFileResponse
+    {
         $gdc->exportarPedidoConNucleosEnCSV();
         $file = resource_path('csv/exports/'.$gdc->nombre.'-completo.csv');
         return response()->download($file);

app/Http/Controllers/Api/ProductoController.php

@@ -10,24 +10,11 @@ use App\Producto;
 
 class ProductoController extends Controller
 {
-    /**
-     * Mostrar una lista de productos.
-     *
-     * @param App\Filtros\FiltroDeProducto $filtros
-     * @param \Illuminate\Http\Request  $request
-     * @return \Illuminate\Http\Response
-     */
     public function index(FiltroDeProducto $filtros, Request $request)
     {
         return ProductoResource::collection(Producto::filtrar($filtros)->paginate(Producto::getPaginar($request)));
     }
 
-    /**
-     * Display the specified resource.
-     *
-     * @param  \App\Producto  $producto
-     * @return \Illuminate\Http\Response
-     */
     public function show(Producto $producto)
     {
         return new ProductoResource($producto);

app/Http/Controllers/Api/SubpedidoController.php

@@ -15,13 +15,6 @@ use Symfony\Component\HttpKernel\Exception\HttpException;
 
 class SubpedidoController extends Controller
 {
-    /**
-     * Mostrar una lista de productos.
-     *
-     * @param App\Filtros\FiltroDeSubpedido $filtros
-     * @param \Illuminate\Http\Request  $request
-     * @return \Illuminate\Http\Response
-     */
     public function index(FiltroDeSubpedido $filtros, Request $request)
     {
         return Subpedido::filtrar($filtros)->get();
@@ -32,12 +25,6 @@ class SubpedidoController extends Controller
         return SubpedidoResource::collection(Subpedido::filtrar($filtros)->get());
     }
 
-    /**
-     * Store a newly created resource in storage.
-     *
-     * @param  \Illuminate\Http\Request  $request
-     * @return \Illuminate\Http\Response
-     */
     public function store(Request  $request)
     {
         $validado = $this->validateSubpedido();
@@ -51,7 +38,8 @@ class SubpedidoController extends Controller
         return $s;
     }
 
-    protected function validateSubpedido(){
+    protected function validateSubpedido(): array
+    {
         return request()->validate([
             'nombre'                => 'required|max:255',
             'grupo_de_compra_id'    => [
@@ -61,12 +49,6 @@ class SubpedidoController extends Controller
         ]);
     }
 
-    /**
-     * Display the specified resource.
-     *
-     * @param  \App\Subpedido  $subpedido
-     * @return \Illuminate\Http\Response
-     */
     public function show(Subpedido $subpedido)
     {
         return new SubpedidoResource($subpedido);

app/Http/Controllers/ComprasController.php

@@ -5,7 +5,9 @@ namespace App\Http\Controllers;
 use App\GrupoDeCompra;
 use App\Helpers\CanastaHelper;
 use App\Producto;
+use Illuminate\Http\JsonResponse;
 use Illuminate\Http\Request;
+use Symfony\Component\HttpFoundation\BinaryFileResponse;
 
 class ComprasController
 {
@@ -15,22 +17,22 @@ class ComprasController
         return view('compras_pedidos');
     }
 
-    public function descargarPedidos() {
+    public function descargarPedidos(): BinaryFileResponse
+    {
         Producto::planillaTotales();
         $file = resource_path('csv/exports/pedidos-por-barrio.csv');
         return response()->download($file);
     }
 
-    public function descargarNotas() {
+    public function descargarNotas(): BinaryFileResponse
+    {
         Producto::planillaNotas();
         $file = resource_path('csv/exports/notas-por-barrio.csv');
         return response()->download($file);
     }
 
-    public function descargarTransporte() {
+    public function pdf() {
-        GrupoDeCompra::planillaTransporte();
+        GrupoDeCompra::exportarPedidosBarrialesAPdf();
-        $file = resource_path('csv/exports/transporte-por-barrio.csv');
-        return response()->download($file);
     }
 
     public function show()
@@ -38,7 +40,7 @@ class ComprasController
         return view('auth/compras_login');
     }
 
-    public function cargarCanasta(Request $request)
+    public function cargarCanasta(Request $request): JsonResponse
     {
         $request->validate([
             'data' => 'required|file|mimes:csv,txt|max:2048',
@@ -49,10 +51,11 @@ class ComprasController
 
         return response()->json([
             'message' => 'Canasta cargada exitosamente',
-        ], 200);
+        ]);
     }
 
-    public function descargarCanastaEjemplo() {
+    public function descargarCanastaEjemplo(): BinaryFileResponse
+    {
         $file = resource_path('csv/productos.csv');
         return response()->download($file);
     }

app/Http/Controllers/ProductoController.php

@@ -2,8 +2,6 @@
 
 namespace App\Http\Controllers;
 
-use Illuminate\Http\Request;
-
 class ProductoController extends Controller
 {
     /**
@@ -16,11 +14,6 @@ class ProductoController extends Controller
         $this->middleware(['auth','subpedido']);
     }
 
-    /**
-     * Show the application dashboard.
-     *
-     * @return \Illuminate\Contracts\Support\Renderable
-     */
     public function index()
     {
         return view('productos');

app/Http/Middleware/Admin.php

@@ -3,18 +3,12 @@
 namespace App\Http\Middleware;
 
 use Closure;
-use Auth;
+use Illuminate\Support\Facades\Auth;
+use Illuminate\Http\Request;
 
 class Admin
 {
-    /**
+    public function handle(Request $request, Closure $next)
-     * Handle an incoming request.
-     *
-     * @param  \Illuminate\Http\Request  $request
-     * @param  \Closure  $next
-     * @return mixed
-     */
-    public function handle($request, Closure $next)
     {
         $user = Auth::user();
         if ($user->is_admin) {

app/Http/Middleware/Subpedido.php

@@ -3,17 +3,18 @@
 namespace App\Http\Middleware;
 
 use Closure;
+use Illuminate\Http\Request;
 
 class Subpedido
 {
     /**
      * Handle an incoming request.
      *
-     * @param  \Illuminate\Http\Request  $request
+     * @param Request $request
-     * @param  \Closure  $next
+     * @param Closure $next
      * @return mixed
      */
-    public function handle($request, Closure $next)
+    public function handle(Request $request, Closure $next)
     {
         if (!session('subpedido_nombre') || !session('subpedido_id')) {
             return redirect()->route('subpedidos.create');

app/Http/Resources/GrupoDeCompraResource.php

@@ -2,6 +2,7 @@
 
 namespace App\Http\Resources;
 
+use Illuminate\Http\Request;
 use Illuminate\Http\Resources\Json\JsonResource;
 
 class GrupoDeCompraResource extends JsonResource
@@ -9,10 +10,10 @@ class GrupoDeCompraResource extends JsonResource
     /**
      * Transform the resource into an array.
      *
-     * @param  \Illuminate\Http\Request  $request
+     * @param  Request  $request
      * @return array
      */
-    public function toArray($request)
+    public function toArray($request): array
     {
         return [
             'id' => $this->id,
@@ -23,8 +24,8 @@ class GrupoDeCompraResource extends JsonResource
             'total_barrial' => number_format($this->totalBarrial(),2),
             'total_devoluciones' => number_format($this->totalDevoluciones(),2),
             'total_a_transferir' => number_format($this->totalATransferir(),2),
-            'total_transporte' => number_format($this->totalTransporte(),0),
+            'total_transporte' => number_format($this->totalTransporte()),
-            'cantidad_transporte' => number_format($this->cantidadTransporte(),0),
+            'cantidad_transporte' => number_format($this->cantidadTransporte()),
         ];
     }
 }

app/Http/Resources/ProductoResource.php

@@ -2,6 +2,7 @@
 
 namespace App\Http\Resources;
 
+use Illuminate\Http\Request;
 use Illuminate\Http\Resources\Json\JsonResource;
 
 class ProductoResource extends JsonResource
@@ -9,10 +10,10 @@ class ProductoResource extends JsonResource
     /**
      * Transform the resource into an array.
      *
-     * @param  \Illuminate\Http\Request  $request
+     * @param  Request  $request
      * @return array
      */
-    public function toArray($request)
+    public function toArray($request): array
     {
         return [
             'id' => $this->id,

app/Http/Resources/SubpedidoResource.php

@@ -2,6 +2,7 @@
 
 namespace App\Http\Resources;
 
+use Illuminate\Http\Request;
 use Illuminate\Http\Resources\Json\JsonResource;
 
 class SubpedidoResource extends JsonResource
@@ -9,10 +10,10 @@ class SubpedidoResource extends JsonResource
     /**
      * Transform the resource into an array.
      *
-     * @param  \Illuminate\Http\Request  $request
+     * @param  Request  $request
      * @return array
      */
-    public function toArray($request)
+    public function toArray($request): array
     {
         return [
             'id' => $this->id,
@@ -21,8 +22,8 @@ class SubpedidoResource extends JsonResource
             'productos' => $this->productos,
             'aprobado' => (bool) $this->aprobado,
             'total' => number_format($this->total(),2),
-            'total_transporte' => number_format($this->totalTransporte(),0),
+            'total_transporte' => number_format($this->totalTransporte()),
-            'cantidad_transporte' => number_format($this->cantidadTransporte(),0),
+            'cantidad_transporte' => number_format($this->cantidadTransporte()),
             'total_sin_devoluciones' => number_format($this->totalSinDevoluciones(),2),
             'devoluciones_total' => number_format($this->devoluciones_total,2),
             'devoluciones_notas' => $this->devoluciones_notas

app/Producto.php

@@ -3,65 +3,61 @@
 namespace App;
 
 use App\Filtros\FiltroDeProducto;
+use App\Helpers\CsvHelper;
+use App\Helpers\TransporteHelper;
+use Illuminate\Database\Eloquent\Builder;
 use Illuminate\Database\Eloquent\Model;
+use Illuminate\Database\Eloquent\Relations\BelongsTo;
+use Illuminate\Database\Eloquent\Relations\BelongsToMany;
 use Illuminate\Http\Request;
+use Illuminate\Support\Collection;
 use Illuminate\Support\Facades\DB;
-use Illuminate\Support\Str;
-use League\Csv\CannotInsertRecord;
-use League\Csv\Reader;
-use League\Csv\Writer;
 
 class Producto extends Model
 {
     public $timestamps = false;
     protected $fillable = ["nombre", "precio", "presentacion", "stock", "categoria"];
-    static $paginarPorDefecto = 10;
+    static int $paginarPorDefecto = 10;
 
-    public function subpedidos()
+    public function subpedidos(): BelongsToMany
     {
         return $this->belongsToMany('App\Subpedido', 'productos_subpedidos')->withPivot(["cantidad", "notas"]);
     }
 
-    public function proveedor()
+    public function proveedor(): BelongsTo
     {
         return $this->belongsTo('App\Proveedor');
     }
 
-    public function pagaTransporte()
-    {
-        return !($this->bono || Str::contains($this->categoria, 'SUBSIDIADO'));
-    }
-
     //Este método permite que se apliquen los filtros al hacer una request (por ejemplo, de búsqueda)
-    public function scopeFiltrar($query, FiltroDeProducto $filtros)
+    public function scopeFiltrar($query, FiltroDeProducto $filtros): Builder
     {
         return $filtros->aplicar($query);
     }
 
-    public static function getPaginar(Request $request)
+    public static function getPaginar(Request $request): int
     {
         return $request->has('paginar') && intval($request->input('paginar')) ? intval($request->input('paginar')) : self::$paginarPorDefecto;
     }
 
     public static function productosFilaID()
     {
-        return Producto::pluck('id', 'fila',)->all();
+        return Producto::pluck('id', 'fila')->all();
     }
 
     public static function productosIDFila()
     {
-        return Producto::pluck('fila', 'id',)->all();
+        return Producto::pluck('fila', 'id')->all();
     }
 
     public static function productosIDNombre()
     {
-        return Producto::pluck('nombre', 'id',)->all();
+        return Producto::pluck('nombre', 'id')->all();
     }
 
-    static public function cantidadesPorBarrio()
+    static public function cantidadesPorBarrio(): Collection
     {
-        $barrios = DB::table('grupos_de_compra')
+        $barrios = GrupoDeCompra::barriosMenosPrueba()
-            ->where('nombre', '<>', 'PRUEBA')
             ->pluck('id', 'nombre');
 
         $columnasBarrios = $barrios->map(function ($id, $nombre) {
@@ -82,22 +78,28 @@ class Producto extends Model
             ->get();
     }
 
-    static public function planillaTotales() {
+    static public function planillaTotales()
+    {
         $headers = ['Producto'];
-        $barrios = DB::table('grupos_de_compra')
+        $barrios = GrupoDeCompra::barriosMenosPrueba()
-            ->where('nombre', '<>', 'PRUEBA')
             ->pluck('nombre')->toArray();
         $headers = array_merge($headers, $barrios);
 
         $cantidadesPorBarrio = self::cantidadesPorBarrio();
+        $transportePorBarrio = GrupoDeCompra::transportePorBarrio();
         $planilla = [];
         $ultimaFila = 1;
+        $filaTransporte = TransporteHelper::filaTransporte();
 
         foreach ($cantidadesPorBarrio as $productoCantidades) {
             $fila = $productoCantidades->fila;
             while ($fila - $ultimaFila > 1) {
                 $ultimaFila++;
-                $planilla[$ultimaFila] = ['---'];
+                if ($ultimaFila == $filaTransporte) {
+                    $planilla[$ultimaFila] = ['Bono de transporte'];
+                } else {
+                    $planilla[$ultimaFila] = ['---'];
+                }
             }
             $planilla[$fila] = [$productoCantidades->producto];
             foreach ($barrios as $barrio) {
@@ -106,16 +108,14 @@ class Producto extends Model
             $ultimaFila = $fila;
         }
 
-        try {
+        foreach ($transportePorBarrio as $key => $cantidad) {
-            $writer = Writer::createFromPath(resource_path('csv/exports/pedidos-por-barrio.csv'), 'w');
+            $planilla[$filaTransporte][] = $cantidad;
-            $writer->insertOne($headers);
-            $writer->insertAll($planilla);
-        } catch (CannotInsertRecord $e) {
-            var_export($e->getRecords());
         }
+
+        CsvHelper::generarCsv('csv/exports/pedidos-por-barrio.csv', $planilla, $headers);
     }
 
-    public static function notasPorBarrio(): \Illuminate\Support\Collection
+    public static function notasPorBarrio(): Collection
     {
         return DB::table('productos')
             ->join('producto_subpedido', 'productos.id', '=', 'producto_subpedido.producto_id')
@@ -133,8 +133,7 @@ class Producto extends Model
 
     static public function planillaNotas() {
         $headers = ['Producto'];
-        $barrios = DB::table('grupos_de_compra')
+        $barrios = GrupoDeCompra::barriosMenosPrueba()
-            ->where('nombre', '<>', 'PRUEBA')
             ->pluck('nombre')->toArray();
         $headers = array_merge($headers, $barrios);
 
@@ -150,12 +149,6 @@ class Producto extends Model
             $planilla[] = $fila;
         }
 
-        try {
+        CsvHelper::generarCsv('csv/exports/notas-por-barrio.csv', $planilla, $headers);
-            $writer = Writer::createFromPath(resource_path('csv/exports/notas-por-barrio.csv'), 'w');
-            $writer->insertOne($headers);
-            $writer->insertAll($planilla);
-        } catch (CannotInsertRecord $e) {
-            var_export($e->getRecords());
-        }
     }
 }

app/Proveedor.php

@@ -3,14 +3,15 @@
 namespace App;
 
 use Illuminate\Database\Eloquent\Model;
+use Illuminate\Database\Eloquent\Relations\HasMany;
 
 class Proveedor extends Model
 {
   public $timestamps = false;
   protected $fillable = [ "nombre","direccion","telefono","correo","comentario" ];
   protected $table = 'proveedores';
-  
+
-  public function productos()
+  public function productos(): HasMany
   {
     return $this->hasMany('App\Producto');
   }

app/Subpedido.php

@@ -3,43 +3,33 @@
 namespace App;
 
 use App\Helpers\TransporteHelper;
+use Illuminate\Database\Eloquent\Builder;
 use Illuminate\Database\Eloquent\Model;
+use Illuminate\Database\Eloquent\Relations\BelongsTo;
+use Illuminate\Database\Eloquent\Relations\BelongsToMany;
 use Illuminate\Support\Facades\DB;
-use Log;
 use App\Filtros\FiltroDeSubpedido;
 
 class Subpedido extends Model
 {
-    const COSTO_TRANSPORTE = 15;
     public $timestamps = false;
-  protected $fillable = ['grupo_de_compra_id', 'aprobado', 'nombre', 'devoluciones_total', 'devoluciones_notas'];
+    protected $fillable = ['grupo_de_compra_id', 'aprobado', 'nombre', 'devoluciones_total', 'devoluciones_notas'];
 
-  public function productos()
+    public function productos(): BelongsToMany
-  {
+    {
-    return $this->belongsToMany('App\Producto')->withPivot(["cantidad","total", "notas"]);
+        return $this->belongsToMany('App\Producto')->withPivot(["cantidad", "total", "notas"]);
-  }
+    }
 
-  //Bonos del MPS, Sororo, etc. NO devuelve bonos de transporte
+    public function grupoDeCompra(): BelongsTo
-  private function bonos()
+    {
-  {
+        return $this->belongsTo('App\GrupoDeCompra');
-    return $this->productos()->where('bono',1);
+    }
-  }
 
-  public function productosSinBonos()
+    // Permite que se apliquen los filtros al hacer una request (por ejemplo, de búsqueda)
-  {
+    public function scopeFiltrar($query, FiltroDeSubpedido $filtros): Builder
-    return $this->productos()->where('bono',false);
+    {
-  }
+        return $filtros->aplicar($query);
-
+    }
-  public function grupoDeCompra()
-  {
-    return $this->belongsTo('App\GrupoDeCompra');
-  }
-
-  //Permite que se apliquen los filtros al hacer una request (por ejemplo, de búsqueda)
-  public function scopeFiltrar($query, FiltroDeSubpedido $filtros)
-  {
-      return $filtros->aplicar($query);
-  }
 
     public function total()
     {
@@ -102,36 +92,40 @@ class Subpedido extends Model
         return TransporteHelper::cantidadTransporte($this->totalCentralesQuePaganTransporte());
     }
 
-  //Actualiza el pedido, agregando o quitando del subpedido según sea necesario. Debe ser llamado desde el controlador de subpedidos, luego de validar que los parámetros $producto y $cantidad son correctos. También calcula el subtotal por producto.
+    //Actualiza el pedido, agregando o quitando del subpedido según sea necesario. Debe ser llamado desde el controlador de subpedidos, luego de validar que los parámetros $producto y $cantidad son correctos. También calcula el subtotal por producto.
-  public function syncProducto(Producto $producto, Int $cantidad, string $notas) {
+    public function syncProducto(Producto $producto, int $cantidad, string $notas)
-        if ($cantidad){
+    {
-          //si la cantidad es 1 o más se agrega el producto o actualiza la cantidad
+        if ($cantidad) {
-          $this->productos()->syncWithoutDetaching([
+            //si la cantidad es 1 o más se agrega el producto o actualiza la cantidad
-            $producto->id => [
+            $this->productos()->syncWithoutDetaching([
-              'cantidad' => $cantidad,
+                $producto->id => [
-              'total' => $cantidad * $producto->precio,
+                    'cantidad' => $cantidad,
-              'notas' => $notas,
+                    'total' => $cantidad * $producto->precio,
-            ]
+                    'notas' => $notas,
-          ]);
+                ]
+            ]);
         } else {
             //si la cantidad es 0, se elimina el producto del subpedido
             $this->productos()->detach($producto->id);
         }
-  }
+    }
 
-  public function toggleAprobacion(bool $aprobacion) {
+    public function toggleAprobacion(bool $aprobacion)
-      $this->aprobado = $aprobacion;
+    {
-      $this->save();
+        $this->aprobado = $aprobacion;
-  }
+        $this->save();
+    }
 
-  public function generarHTML() {
+    public function generarHTML()
-      $view = view("pdfgen.subpedido_tabla", ["subpedido" => $this]);
+    {
-      return $view->render();
+        $view = view("pdfgen.pedido_tabla", ["pedido" => $this]);
-  }
+        return $view->render();
+    }
 
-  public function syncDevoluciones(float $total, string $notas) {
+    public function syncDevoluciones(float $total, string $notas)
-    $this->devoluciones_total = $total;
+    {
-    $this->devoluciones_notas = $notas;
+        $this->devoluciones_total = $total;
-    $this->save();
+        $this->devoluciones_notas = $notas;
-  }
+        $this->save();
+    }
 }

app/User.php

@@ -2,7 +2,7 @@
 
 namespace App;
 
-use Illuminate\Contracts\Auth\MustVerifyEmail;
+use Illuminate\Database\Eloquent\Relations\BelongsTo;
 use Illuminate\Foundation\Auth\User as Authenticatable;
 use Illuminate\Notifications\Notifiable;
 
@@ -38,7 +38,7 @@ class User extends Authenticatable
     ];
 
 
-    public function grupoDeCompra()
+    public function grupoDeCompra(): BelongsTo
     {
         return $this->belongsTo('App\GrupoDeCompra');
     }

database/factories/UserFactory.php

@@ -1,9 +1,10 @@
 <?php
 
-/** @var \Illuminate\Database\Eloquent\Factory $factory */
+/** @var Factory $factory */
 
 use App\User;
 use Faker\Generator as Faker;
+use Illuminate\Database\Eloquent\Factory;
 use Illuminate\Support\Str;
 
 /*

database/migrations/2023_07_06_015926_call_crear_pedidos_aprobados.php

@@ -1,8 +1,6 @@
 <?php
 
 use Illuminate\Database\Migrations\Migration;
-use Illuminate\Database\Schema\Blueprint;
-use Illuminate\Support\Facades\Schema;
 
 class CallCrearPedidosAprobados extends Migration
 {

database/migrations/2025_05_12_214437_call_agregar_es_bono_a_pedidos_aprobados.php

@@ -0,0 +1,27 @@
+<?php
+
+use Illuminate\Database\Migrations\Migration;
+use Illuminate\Support\Facades\Artisan;
+
+class CallAgregarEsBonoAPedidosAprobados extends Migration
+{
+    /**
+     * Run the migrations.
+     *
+     * @return void
+     */
+    public function up()
+    {
+        Artisan::call("command:AgregarEsBonoAPedidosAprobados");
+    }
+
+    /**
+     * Reverse the migrations.
+     *
+     * @return void
+     */
+    public function down()
+    {
+        //
+    }
+}

database/seeds/GrupoDeCompraSeeder.php

@@ -1,9 +1,9 @@
 <?php
 
+use App\Helpers\CsvHelper as CsvHelperAlias;
 use Illuminate\Database\Seeder;
 use Illuminate\Support\Facades\DB;
 use Illuminate\Support\Facades\Hash;
-use League\Csv\Reader;
 
 class GrupoDeCompraSeeder extends Seeder
 {
@@ -14,11 +14,7 @@ class GrupoDeCompraSeeder extends Seeder
      */
     public function run()
     {
-        $csv = Reader::createFromPath(resource_path('csv/barrios.csv'), 'r');
+        $registros = CsvHelperAlias::getRecords('csv/barrios.csv');
-        $csv->setDelimiter("|");
-        $csv->setEnclosure("'");
-        $csv->setHeaderOffset(0);
-        $registros = $csv->getRecords();
         $gdcToInsert = [];
         $usersToInsert = [];
 

resources/js/components/compras/Body.vue

@@ -20,10 +20,9 @@
                             La planilla de la canasta tiene que tener el siguiente formato para que la aplicación la lea correctamente:
                             <ul>
                                 <li> Los precios deben usar punto y no coma decimal </li>
-                                <li> El nombre de la columna de precios debe ser "Precio" </li>
+                                <li> El nombre de las columnas deben ser "Tipo", "Producto", y "Precio" respectivamente </li>
                                 <li> Las celdas deben separarse con '|' </li>
                                 <li> No puede haber "enters" en ninguna celda </li>
-                                <li> Todos los bonos deben tener tipo 'B' para evitar que paguen transporte </li>
                                 <li> El bono de transporte debe tener tipo 'T' </li>
                             </ul>
                             <a class="has-text-info" href="/compras/canasta/ejemplo">Planilla de ejemplo.</a>

resources/js/components/compras/DropdownDescargar.vue

@@ -20,8 +20,8 @@
                     <a href="/compras/pedidos/notas" class="dropdown-item">
                         Notas por barrio
                     </a>
-                    <a href="/compras/pedidos/transporte" class="dropdown-item">
+                    <a href="/compras/pedidos/pdf" class="dropdown-item">
-                        Transporte por barrio
+                        Pedidos por barrio en pdf
                     </a>
                 </div>
             </div>

resources/views/pdfgen/pedido_tabla.blade.php

@@ -4,7 +4,7 @@
         background: #CCC
     }
 </style>
-<h3>{{$subpedido->nombre}}</h3>
+<h3>{{$pedido['nombre']}}</h3>
 
 <table style="width: 100%">
     <tr>
@@ -25,18 +25,18 @@
         </th>
     </tr>
 
-    @foreach($subpedido->productos as $producto)
+    @foreach($pedido['productos'] as $producto)
-        @if(!$producto->bono)
+        @if(!$producto['bono'])
-        
+
         <tr>
             <td>
-                {{ $producto->nombre }}
+                {{ $producto['nombre'] }}
-                @if($producto->pivot->notas)
+                @if($producto['pivot']['notas'])
-                <br /><b>Talle/Color:</b> {{ $producto->pivot->notas }}
+                <br /><b>Talle/Color:</b> {{ $producto['pivot']['notas'] }}
                 @endif
             </td>
             <td style="text-align: center">
-                {{ $producto->pivot->cantidad }}
+                {{ $producto['pivot']['cantidad'] }}
             </td>
         </tr>
         @endif

routes/api.php

@@ -1,8 +1,5 @@
 <?php
 
-use App\Filtros\FiltroDeProducto;
-use App\Filtros\FiltroDeSubpedido;
-use Illuminate\Http\Request;
 use Illuminate\Support\Facades\Route;
 use App\GrupoDeCompra;
 use App\Producto;

routes/web.php

@@ -1,6 +1,9 @@
 <?php
 
+use Illuminate\Support\Facades\App;
+use Illuminate\Support\Facades\Auth;
 use Illuminate\Support\Facades\Route;
+use Illuminate\Support\Facades\URL;
 use Symfony\Component\HttpKernel\Exception\HttpException;
 /*
 |--------------------------------------------------------------------------
@@ -26,16 +29,15 @@ Route::get('/productos', 'ProductoController@index')->name('productos.index');
 Route::get('/admin', 'AdminController@show')->name('admin_login.show');
 
 Route::get('/admin/obtener_sesion', function() {
-    $sesion = [
+    return [
         'gdc' => session("admin_gdc")
     ];
-    return $sesion;
 })->name('admin_obtener_sesion');
 
 Route::middleware(['auth', 'admin'])->group( function () {
 	Route::get('/admin/pedidos', 'AdminController@index')->name('admin_login.index');
 
-	Route::get('/admin/exportar-planillas-a-pdf/{gdc}', 'AdminController@exportarPlanillasAPdf');
+	Route::get('/admin/exportar-planillas-a-pdf/{gdc}', 'AdminController@exportarPedidosAPdf');
 
 	Route::get('/admin/exportar-pedido-a-csv/{gdc}', 'AdminController@exportarPedidoACSV');
 
@@ -64,14 +66,13 @@ Route::middleware('auth')->group( function() {
 		})->name('guardarSesion');
 
 		Route::get('obtener_sesion', function() {
-			$sesion = [
+            return [
-				'subpedido' => [
+                'subpedido' => [
-					'nombre' => session("subpedido_nombre"),
+                    'nombre' => session("subpedido_nombre"),
-					'id' => session("subpedido_id")
+                    'id' => session("subpedido_id")
-				],
+                ],
                 'gdc' => session("gdc")
-			];
+            ];
-	        return $sesion;
 		})->name('obtenerSesion');
 	});
 });
@@ -82,7 +83,7 @@ Route::middleware(['compras'])->group( function() {
     Route::get('/compras/pedidos', 'ComprasController@indexPedidos')->name('compras.pedidos');
     Route::get('/compras/pedidos/descargar', 'ComprasController@descargarPedidos')->name('compras.pedidos.descargar');
     Route::get('/compras/pedidos/notas', 'ComprasController@descargarNotas')->name('compras.pedidos.descargar');
-    Route::get('/compras/pedidos/transporte', 'ComprasController@descargarTransporte')->name('compras.pedidos.descargar');
+    Route::get('/compras/pedidos/pdf', 'ComprasController@pdf')->name('compras.pedidos.pdf');
     Route::post('/compras/canasta', 'ComprasController@cargarCanasta')->name('compras.canasta');
     Route::get('/compras/canasta/ejemplo', 'ComprasController@descargarCanastaEjemplo')->name('compras.canasta.ejemplo');
 });