Merge branch 'master' of https://github.com/devcode-it/openstamanager

modules/fatture/row-list.php

@@ -33,8 +33,9 @@ foreach ($righe as $riga) {
     $riga['sconto'] = abs($riga['sconto']);
     $riga['iva'] = abs($riga['iva']);
 
-	if (empty($riga['is_descrizione']))
+    if (empty($riga['is_descrizione'])) {
-		$riga['descrizione_conto'] = $dbo->fetchOne("SELECT descrizione FROM co_pianodeiconti3 WHERE id = ".prepare($riga['idconto']))['descrizione'];
+        $riga['descrizione_conto'] = $dbo->fetchOne('SELECT descrizione FROM co_pianodeiconti3 WHERE id = '.prepare($riga['idconto']))['descrizione'];
+    }
 
     $extra = '';
 
@@ -52,7 +53,6 @@ foreach ($righe as $riga) {
 
         $extra = '';
         $mancanti = 0;
-		
     }
     // Intervento
     elseif (!empty($riga['idintervento'])) {
@@ -68,17 +68,15 @@ foreach ($righe as $riga) {
     }
     // Contratti
     elseif (!empty($riga['idcontratto'])) {
-		
         //$ref_modulo = Modules::get('Contratti')['id'];
         //$ref_id = $riga['idcontratto'];
 
-		$contratto = $dbo->fetchOne("SELECT codice_cig,codice_cup,id_documento_fe FROM co_contratti WHERE id = ".prepare($riga['idcontratto']));
+        $contratto = $dbo->fetchOne('SELECT codice_cig,codice_cup,id_documento_fe FROM co_contratti WHERE id = '.prepare($riga['idcontratto']));
         $riga['codice_cig'] = $contratto['codice_cig'];
         $riga['codice_cup'] = $contratto['codice_cup'];
         $riga['id_documento_fe'] = $contratto['id_documento_fe'];
 
         $delete = 'unlink_contratto';
-		
     }
     // Righe generiche
     else {

modules/interventi/edit.php

@@ -142,8 +142,8 @@ $_SESSION['superselect']['idanagrafica'] = $record['idanagrafica'];
 
 <?php
     // Visualizzo solo se l'anagrafica cliente è un ente pubblico
-	if (!empty($record['idcontratto'])){
+    if (!empty($record['idcontratto'])) {
-		$contratto = $dbo->fetchOne("SELECT codice_cig,codice_cup,id_documento_fe FROM co_contratti WHERE id = ".prepare($record['idcontratto']));
+        $contratto = $dbo->fetchOne('SELECT codice_cig,codice_cup,id_documento_fe FROM co_contratti WHERE id = '.prepare($record['idcontratto']));
         $record['id_documento_fe'] = $contratto['id_documento_fe'];
         $record['codice_cup'] = $contratto['codice_cup'];
         $record['codice_cig'] = $contratto['codice_cig'];
@@ -154,24 +154,26 @@ $_SESSION['superselect']['idanagrafica'] = $record['idanagrafica'];
     <div class="panel panel-primary <?php echo (($record['tipo_anagrafica']) == 'Ente pubblico') ? 'show' : 'hide'; ?>" >
         <div class="panel-heading">
             <h3 class="panel-title"><?php echo tr('Dati appalto'); ?>
-			<?php if (!empty($record['idcontratto'])){ ?>
+			<?php if (!empty($record['idcontratto'])) {
+        ?>
 			<span class="tip" title="<?php echo tr('E\' possibile specificare i dati dell\'appalto solo se il cliente è di tipo \'Ente pubblico\' e l\'attività non risulta già collegata ad un contratto.'); ?>" > <i class="fa fa-question-circle-o"></i></span>
 			</h3>
-			<?php } ?>
+			<?php
+    } ?>
         </div>
 
         <div class="panel-body">
             <div class="row">
                 <div class="col-md-4">
-					{[ "type": "<?php echo (!empty($record['idcontratto']) ? 'span': 'text' ) ?>", "label": "<?php echo tr('Identificatore Documento'); ?>", "name": "id_documento_fe", "required": 0, "value": "<?php echo $record['id_documento_fe']; ?>", "maxlength": 20, "readonly": "<?php echo $record['flag_completato']; ?>", "extra": "" ]}
+					{[ "type": "<?php echo !empty($record['idcontratto']) ? 'span' : 'text'; ?>", "label": "<?php echo tr('Identificatore Documento'); ?>", "name": "id_documento_fe", "required": 0, "value": "<?php echo $record['id_documento_fe']; ?>", "maxlength": 20, "readonly": "<?php echo $record['flag_completato']; ?>", "extra": "" ]}
 				</div>
 
                 <div class="col-md-4">
-					{[ "type": "<?php echo (!empty($record['idcontratto']) ? 'span': 'text' ) ?>", "label": "<?php echo tr('Codice CIG'); ?>", "name": "codice_cig", "required": 0, "value": "<?php echo $record['codice_cig']; ?>", "maxlength": 15, "readonly": "<?php echo $record['flag_completato']; ?>", "extra": "" ]}
+					{[ "type": "<?php echo !empty($record['idcontratto']) ? 'span' : 'text'; ?>", "label": "<?php echo tr('Codice CIG'); ?>", "name": "codice_cig", "required": 0, "value": "<?php echo $record['codice_cig']; ?>", "maxlength": 15, "readonly": "<?php echo $record['flag_completato']; ?>", "extra": "" ]}
 				</div>
 
 				<div class="col-md-4">
-					{[ "type": "<?php echo (!empty($record['idcontratto']) ? 'span': 'text' ) ?>", "label": "<?php echo tr('Codice CUP'); ?>", "name": "codice_cup", "required": 0, "value": "<?php echo $record['codice_cup']; ?>", "maxlength": 15, "readonly": "<?php echo $record['flag_completato']; ?>", "extra": "" ]}
+					{[ "type": "<?php echo !empty($record['idcontratto']) ? 'span' : 'text'; ?>", "label": "<?php echo tr('Codice CUP'); ?>", "name": "codice_cup", "required": 0, "value": "<?php echo $record['codice_cup']; ?>", "maxlength": 15, "readonly": "<?php echo $record['flag_completato']; ?>", "extra": "" ]}
 				</div>
             </div>
         </div>

modules/primanota/add.php

@@ -351,9 +351,11 @@ include_once __DIR__.'/../../core.php';
 					var causale = $(this).find('option:selected').text();
 					
 					//aggiornava erroneamente anche la causale ed eventuale numero di fattura e data
-					<?php if (empty($iddocumento)) { ?>
+					<?php if (empty($iddocumento)) {
+        ?>
 						$('#bs-popup #desc').val(causale);
-					<?php } ?>
+					<?php
+    } ?>
 					
 					$.get('<?php echo $rootdir; ?>/ajax_complete.php?op=get_conti&idmastrino='+idmastrino, function(data){
 						var conti = data.split(',');

plugins/exportFE/src/FatturaElettronica.php

@@ -1577,7 +1577,6 @@ class FatturaElettronica
             $result[]['DettaglioPagamento'] = $pagamento;
         }
 
-
         return $result;
     }
 

plugins/importFE/src/FatturaElettronica.php

@@ -137,7 +137,6 @@ class FatturaElettronica
         // Informazioni sull'anagrafica
         $REA = $xml['IscrizioneREA'];
         if (!empty($REA)) {
-			
             if (!empty($REA['Ufficio']) and !empty($REA['NumeroREA'])) {
                 $anagrafica->codicerea = $REA['Ufficio'].'-'.$REA['NumeroREA'];
             }

src/Util/XML.php

@@ -2,6 +2,8 @@
 
 namespace Util;
 
+use UnexpectedValueException;
+
 /**
  * Classe dedicata all'interpretazione dei file XML.
  *
@@ -19,9 +21,16 @@ class XML
     public static function read($string)
     {
         $content = static::stripP7MData($string);
-        $content = static::sanitizeXML($content);
+
+        libxml_use_internal_errors(true);
 
         $xml = simplexml_load_string($content, 'SimpleXMLElement', LIBXML_NOCDATA);
+        if ($xml === false) {
+            $message = libxml_get_last_error()->message;
+
+            throw new UnexpectedValueException($message);
+        }
+
         $result = json_decode(json_encode($xml), true);
 
         return $result;