Merge pull request NeoRazorX#1764 from daniel89fg/bug-sendmail-sales-purchases

Bug sendmail sales purchases

Author: NeoRazorX

Core/Controller/SendMail.php

@@ -194,7 +194,16 @@ protected function loadDataDefault($model): void
         ];
         if ($notificationModel->loadWhere($where)) {
             $shortCodes = ['{code}', '{name}', '{date}', '{total}', '{number2}'];
-            $shortValues = [$model->codigo, $model->nombrecliente, $model->fecha, $model->total, $model->numero2];
+            $shortValues = [$model->codigo, '', $model->fecha, $model->total, ''];
+
+            $shortValues[1] = $model->hasColumn('nombrecliente')
+                ? $model->nombrecliente
+                : $model->nombre;
+
+            $shortValues[4] = $model->hasColumn('numero2')
+                ? $model->numero2
+                : $model->numproveedor;
+
             $this->newMail->title = str_replace($shortCodes, $shortValues, $notificationModel->subject);
             $this->newMail->text = str_replace($shortCodes, $shortValues, $notificationModel->body);
             return;
@@ -203,21 +212,25 @@ protected function loadDataDefault($model): void
         // si no hay notificación, usamos los datos de las traducciones
         switch ($model->modelClassName()) {
             case 'AlbaranCliente':
+            case 'AlbaranProveedor':
                 $this->newMail->title = Tools::trans('delivery-note-email-subject', ['%code%' => $model->codigo]);
                 $this->newMail->text = Tools::trans('delivery-note-email-text', ['%code%' => $model->codigo]);
                 break;
 
             case 'FacturaCliente':
+            case 'FacturaProveedor':
                 $this->newMail->title = Tools::trans('invoice-email-subject', ['%code%' => $model->codigo]);
                 $this->newMail->text = Tools::trans('invoice-email-text', ['%code%' => $model->codigo]);
                 break;
 
             case 'PedidoCliente':
+            case 'PedidoProveedor':
                 $this->newMail->title = Tools::trans('order-email-subject', ['%code%' => $model->codigo]);
                 $this->newMail->text = Tools::trans('order-email-text', ['%code%' => $model->codigo]);
                 break;
 
             case 'PresupuestoCliente':
+            case 'PresupuestoProveedor':
                 $this->newMail->title = Tools::trans('estimation-email-subject', ['%code%' => $model->codigo]);
                 $this->newMail->text = Tools::trans('estimation-email-text', ['%code%' => $model->codigo]);
                 break;

Core/Data/Lang/ES/emails_notifications.csv

@@ -2,4 +2,8 @@
 "sendmail-AlbaranCliente";true;"Su albarán {code}";"Buenos días {name}, le adjunto su albarán {code} del día {date}. Saludos."
 "sendmail-FacturaCliente";true;"Su factura {code}";"Buenos días {name}, le adjunto su factura {code} del día {date}. Saludos."
 "sendmail-PedidoCliente";true;"Su pedido {code}";"Buenos días {name}, le adjunto su pedido {code} del día {date}. Saludos."
-"sendmail-PresupuestoCliente";true;"Su presupuesto {code}";"Buenos días {name}, le adjunto su presupuesto {code} del día {date}. Saludos."
+"sendmail-PresupuestoCliente";true;"Su presupuesto {code}";"Buenos días {name}, le adjunto su presupuesto {code} del día {date}. Saludos."
+"sendmail-AlbaranProveedor";true;"Su albarán {code}";"Buenos días {name}, le adjunto su albarán {code} del día {date}. Saludos."
+"sendmail-FacturaProveedor";true;"Su factura {code}";"Buenos días {name}, le adjunto su factura {code} del día {date}. Saludos."
+"sendmail-PedidoProveedor";true;"Su pedido {code}";"Buenos días {name}, le adjunto su pedido {code} del día {date}. Saludos."
+"sendmail-PresupuestoProveedor";true;"Su presupuesto {code}";"Buenos días {name}, le adjunto su presupuesto {code} del día {date}. Saludos."