Ahora cuando hacemos una venta a un cliente cuya dirección de factura…

…ción está vacía: - Podemos escribir la dirección, código postal, etc desde el formulario de venta. - Al guardar el documento (insert) se actualiza la dirección de facturación del cliente con los datos introducidos en la venta.
NeoRazorX · Jul 25, 2022 · 7b9efc2 · 7b9efc2
1 parent 3792dd1
commit 7b9efc2
Show file tree

Hide file tree

Showing 2 changed files with 45 additions and 15 deletions.
diff --git a/Core/Base/AjaxForms/SalesHeaderHTML.php b/Core/Base/AjaxForms/SalesHeaderHTML.php
@@ -113,12 +113,23 @@ public static function apply(SalesDocument &$model, array $formData, User $user)
         } elseif ($dir->loadFromCode($formData['idcontactofact'])) {
             // update billing address
             $model->idcontactofact = $dir->idcontacto;
-            $model->direccion = $dir->direccion;
-            $model->apartado = $dir->apartado;
-            $model->codpostal = $dir->codpostal;
-            $model->ciudad = $dir->ciudad;
-            $model->provincia = $dir->provincia;
-            $model->codpais = $dir->codpais;
+
+            // Is Billing address empty?
+            if (empty($dir->direccion)) {
+                $model->direccion = $formData['direccion'] ?? $model->direccion;
+                $model->apartado = $formData['apartado'] ?? $model->apartado;
+                $model->codpostal = $formData['codpostal'] ?? $model->codpostal;
+                $model->ciudad = $formData['ciudad'] ?? $model->ciudad;
+                $model->provincia = $formData['provincia'] ?? $model->provincia;
+                $model->codpais = $formData['codpais'] ?? $model->codpais;
+            } else {
+                $model->direccion = $dir->direccion;
+                $model->apartado = $dir->apartado;
+                $model->codpostal = $dir->codpostal;
+                $model->ciudad = $dir->ciudad;
+                $model->provincia = $dir->provincia;
+                $model->codpais = $dir->codpais;
+            }
         }
 
         // set shipping address
@@ -165,7 +176,7 @@ public static function render(SalesDocument $model): string
 
     private static function addressField(Translator $i18n, SalesDocument $model, string $field, string $label, int $size, int $maxlength): string
     {
-        $attributes = $model->editable && empty($model->idcontactofact) ?
+        $attributes = $model->editable && (empty($model->idcontactofact) || empty($model->direccion)) ?
             'name="' . $field . '" maxlength="' . $maxlength . '" autocomplete="off"' :
             'disabled=""';
         return '<div class="col-sm-' . $size . '">'
@@ -256,7 +267,9 @@ private static function codpais(Translator $i18n, SalesDocument $model): string
         }
 
         $pais = new Pais();
-        $attributes = $model->editable && empty($model->idcontactofact) ? 'name="codpais"' : 'disabled=""';
+        $attributes = $model->editable && (empty($model->idcontactofact) || empty($model->direccion)) ?
+            'name="codpais"' :
+            'disabled=""';
         return '<div class="col-sm-6">'
             . '<div class="form-group">'
             . '<a href="' . $pais->url() . '">' . $i18n->trans('country') . '</a>'

diff --git a/Core/Model/Base/SalesDocument.php b/Core/Model/Base/SalesDocument.php
@@ -384,12 +384,16 @@ protected function onChange($field)
             return false;
         }
 
+        if (empty($this->idcontactofact)) {
+            return true;
+        }
+
         // after parent checks
+        $contact = new Contacto();
         switch ($field) {
             case 'direccion':
-                $contact = new Contacto();
                 // if address is changed and customer billing address is empty, then save new values
-                if ($contact->loadFromCode($this->idcontactofact) && empty($contact->direccion)) {
+                if ($this->direccion && $contact->loadFromCode($this->idcontactofact) && empty($contact->direccion)) {
                     $contact->apartado = $this->apartado;
                     $contact->ciudad = $this->ciudad;
                     $contact->codpais = $this->codpais;
@@ -401,26 +405,39 @@ protected function onChange($field)
                 break;
 
             case 'idcontactofact':
-                if (empty($this->idcontactofact)) {
-                    return true;
-                }
                 // if billing address is changed, then change all billing fields
-                $contact = new Contacto();
                 if ($contact->loadFromCode($this->idcontactofact)) {
                     $this->apartado = $contact->apartado;
                     $this->ciudad = $contact->ciudad;
                     $this->codpais = $contact->codpais;
                     $this->codpostal = $contact->codpostal;
                     $this->direccion = $contact->direccion;
                     $this->provincia = $contact->provincia;
-                    return true;
+                    break;
                 }
                 return false;
         }
 
         return true;
     }
 
+    protected function onInsert()
+    {
+        // if billing address is empty, then save new values
+        $contact = new Contacto();
+        if ($this->direccion && $contact->loadFromCode($this->idcontactofact) && empty($contact->direccion)) {
+            $contact->apartado = $this->apartado;
+            $contact->ciudad = $this->ciudad;
+            $contact->codpais = $this->codpais;
+            $contact->codpostal = $this->codpostal;
+            $contact->direccion = $this->direccion;
+            $contact->provincia = $this->provincia;
+            $contact->save();
+        }
+
+        parent::onInsert();
+    }
+
     /**
      * This method is called after a record is updated on the database (saveUpdate).
      */