Miglioramento generazione FE

devcode-it · Dec 7, 2018 · b4e7f7e · b4e7f7e
1 parent 7f7ae84
commit b4e7f7e
Show file tree

Hide file tree

Showing 3 changed files with 26 additions and 19 deletions.
diff --git a/plugins/exportFE/actions.php b/plugins/exportFE/actions.php
@@ -7,9 +7,6 @@
         if (!empty($fattura_pa)) {
             $file = $fattura_pa->save($upload_dir);
 
-            //Aggiorno la data di creazione della fattura elettronica
-            $dbo->query('UPDATE co_documenti SET xml_generated_at=NOW() WHERE co_documenti.id='.prepare($id_record));
-
             flash()->info(tr('Fattura elettronica generata correttamente!'));
 
             if (!$fattura_pa->isValid()) {

diff --git a/plugins/exportFE/edit.php b/plugins/exportFE/edit.php
@@ -6,13 +6,7 @@
 
 if (!empty($fattura_pa)) {
     $disabled = false;
-    $generated = file_exists($upload_dir.'/'.$fattura_pa->getFilename());
-
-    // Ulteriore controllo sulla data generazione file
-    $rs_generated = $dbo->fetchArray('SELECT xml_generated_at FROM co_documenti WHERE id='.prepare($id_record));
-    if (empty($rs_generated[0]['xml_generated_at'])) {
-        $generated = false;
-    }
+    $generated = $fattura_pa->isGenerated();
 } else {
     echo '
 <div class="alert alert-warning">

diff --git a/plugins/exportFE/src/FatturaElettronica.php b/plugins/exportFE/src/FatturaElettronica.php
@@ -59,6 +59,18 @@ public function __construct($id_documento)
         }
     }
 
+    /**
+     * Restituisce le informazioni sull'anagrafica azienda.
+     *
+     * @return array
+     */
+    public function isGenerated()
+    {
+        $documento = $this->getDocumento();
+
+        return !empty($documento['xml_generated_at']) && !empty($documento['progressivo_invio']) && file_exists(DOCROOT.'/'.static::getDirectory().'/'.$this->getFilename());
+    }
+
     /**
      * Restituisce le informazioni sull'anagrafica azienda.
      *
@@ -333,15 +345,14 @@ protected static function getCedentePrestatore($fattura)
 
         // IscrizioneREA
         if (!empty($azienda['codicerea'])) {
-
             $codice = explode('-', $azienda['codicerea']);
-			
-			if ( !empty($codice[0]) and !empty($codice[1]) ){
-				$result['IscrizioneREA'] = [
-					'Ufficio' => strtoupper($codice[0]),
-					'NumeroREA' => $codice[1],
-				];
-			}
+
+            if (!empty($codice[0]) && !empty($codice[1])) {
+                $result['IscrizioneREA'] = [
+                    'Ufficio' => strtoupper($codice[0]),
+                    'NumeroREA' => $codice[1],
+                ];
+            }
 
             if (!empty($azienda['capitale_sociale'])) {
                 $result['IscrizioneREA']['CapitaleSociale'] = $azienda['capitale_sociale'];
@@ -1075,6 +1086,12 @@ public function save($directory)
             'original' => $filename,
         ], $data));
 
+        // Aggiornamento effettivo
+        database()->update('co_documenti', [
+            'progressivo_invio' => $this->getDocumento()['progressivo_invio'],
+            'xml_generated_at' => date('Y-m-d H:i:s'),
+        ], ['id' => $this->getDocumento()['id']]);
+
         return ($result === false) ? null : $filename;
     }
 
@@ -1096,7 +1113,6 @@ public function getFilename($new = false)
             } while ($database->fetchNum('SELECT `id` FROM `co_documenti` WHERE `progressivo_invio` = '.prepare($code)) != 0);
 
             // Registrazione
-            $database->update('co_documenti', ['progressivo_invio' => $code], ['id' => $this->getDocumento()['id']]);
             $this->documento['progressivo_invio'] = $code;
         }