includo nome degli utenti nei documenti esportati dal fornitore. closes #349

Author: madbob

code/app/Http/Controllers/PublicGateController.php

@@ -20,48 +20,43 @@ public function show(Request $request, $token, $action)
                     $order_id = $req->p->id;
                     $order = Order::find($order_id);
 
-                    switch($action) {
-                        case 'show':
-                            $links = [];
-                            $type = $order->supplier->notify_on_close_enabled;
+                    if ($action == 'show') {
+                        $links = [];
+                        $type = $order->supplier->notify_on_close_enabled;
 
-                            if ($type == 'shipping_summary') {
-                                $types = ['shipping', 'summary'];
-                            }
-                            else {
-                                $types = [$type];
-                            }
+                        if ($type == 'shipping_summary') {
+                            $types = ['shipping', 'summary'];
+                        }
+                        else {
+                            $types = [$type];
+                        }
 
-                            foreach($types as $t) {
-                                $base_filename = __('texts.orders.files.order.' . $t);
-                                $links[$base_filename . ' - CSV'] = publicGateGetLink('odoc', $t . '_csv', ['id' => $order->id]);
-                                $links[$base_filename . ' - PDF'] = publicGateGetLink('odoc', $t . '_pdf', ['id' => $order->id]);
-                            }
+                        foreach($types as $t) {
+                            $base_filename = __('texts.orders.files.order.' . $t);
+                            $links[$base_filename . ' - CSV'] = publicGateGetLink('odoc', $t . '_csv', ['id' => $order->id]);
+                            $links[$base_filename . ' - PDF'] = publicGateGetLink('odoc', $t . '_pdf', ['id' => $order->id]);
+                        }
 
-                            return view('public.order_documents', [
-                                'order' => $order,
-                                'links' => $links,
-                            ]);
-
-                            break;
-
-                        default:
-                            $printer = new OrderPrinter();
-                            list($type, $format) = explode('_', $action);
-
-                            return $printer->document($order, $type, [
-                                'format' => $format,
-                                'status' => 'pending',
-                                'extra_modifiers' => 0,
-                                'action' => 'download',
-                            ]);
+                        return view('public.order_documents', [
+                            'order' => $order,
+                            'links' => $links,
+                        ]);
+                    }
+                    else {
+                        $printer = new OrderPrinter();
+                        list($type, $format) = explode('_', $action);
 
-                            break;
+                        return $printer->document($order, $type, [
+                            'format' => $format,
+                            'status' => 'pending',
+                            'extra_modifiers' => 0,
+                            'action' => 'download',
+                        ]);
                     }
                     break;
 
                 default:
-                    throw new \Exception('Richiesta ' . $req->t . ' non riconosciuta');
+                    throw new \UnexpectedValueException('Richiesta ' . $req->t . ' non riconosciuta');
                     break;
             }
         }

code/app/Printers/PrintParams.php

@@ -1,5 +1,12 @@
 <?php
 
+/**
+ * Questa struttura dati è comune alle diverse tipologie di Printer, e ne
+ * include tutte le possibili varianti ed opzioni. Questo per accentrare in un
+ * posto solo tutti i possibili default, e rendere un po' più leggibile il
+ * codice dei Printer stessi
+ */
+
 namespace App\Printers;
 
 use Illuminate\Database\Eloquent\Model;
@@ -28,8 +35,8 @@ public function __construct($request, $obj)
         $this->subtype = $request['format'] ?? 'pdf';
         $this->status = $request['status'] ?? 'pending';
         $this->include_missing = $request['include_missing'] ?? 'no';
-        $this->isolate_friends = ($request['isolate_friends'] ?? false) == true;
-        $this->extra_modifiers = ($request['extra_modifiers'] ?? false) == true;
+        $this->isolate_friends = ($request['isolate_friends'] ?? false);
+        $this->extra_modifiers = ($request['extra_modifiers'] ?? false);
 
         $this->required_fields = $request['fields'] ?? [];
         if (empty($this->required_fields)) {
@@ -39,10 +46,10 @@ public function __construct($request, $obj)
         $this->fields = $this->splitFields($this->required_fields);
     }
 
-    /*
-        Questo serve a separare le colonne per utenti e prodotti quando si generano
-        i Dettagli Consegne che contengono tutto
-    */
+    /**
+     * Questo serve a separare le colonne per utenti e prodotti quando si
+     * generano i Dettagli Consegne che contengono tutto
+     */
     private function splitFields(array $fields): object
     {
         $formattable_user = User::formattableColumns('all');
@@ -92,10 +99,17 @@ private function splitFields(array $fields): object
         return $ret;
     }
 
+    /**
+     * Qui si enumerano le informazioni da includere nei files esportati quando
+     * non ne viene esplicitamente richiesta nessuna (e.g. quando genero il file
+     * scaricabile dal fornitore)
+     */
     private function autoGuessFields(Model $order): array
     {
         $guessed_fields = [];
 
+        $guessed_fields[] = 'fullname';
+
         if ($order->products->first(fn ($p) => !empty($p->code)) != null) {
             $guessed_fields[] = 'code';
         }