Semplificata apertura attività da calendario su nuova scheda

actions.php

@@ -489,7 +489,7 @@ if ($structure->permission == 'rw') {
         ($include_file = $structure->filepath('actions.php')) ? include $include_file : null;
 
         // Operazioni generiche per i campi personalizzati
-        if (post('op') != null) {
+        if (!empty(post('op'))) {
             $custom_where = !empty($id_plugin) ? '`id_plugin` = '.prepare($id_plugin) : '`id_module` = '.prepare($id_module);
 
             $query = 'SELECT `id`, `html_name` AS `title` FROM `zz_fields` WHERE '.$custom_where;

modules/categorie_articoli/actions.php

@@ -40,7 +40,7 @@ switch (filter('op')) {
         }
 
         // Redirect alla categoria se si sta modificando una sottocategoria
-        if ($id_original != null) {
+        if (!empty($id_original)) {
             $database->commitTransaction();
             redirect(base_path().'/editor.php?id_module='.$id_module.'&id_record='.($id_original ?: $id_record));
             exit;

modules/categorie_impianti/actions.php

@@ -45,7 +45,7 @@ switch (filter('op')) {
         }
 
         // Redirect alla categoria se si sta modificando una sottocategoria
-        if ($id_original != null) {
+        if (!empty($id_original)) {
             $database->commitTransaction();
             redirect(base_path().'/editor.php?id_module='.$id_module.'&id_record='.($id_original ?: $id_record));
             exit;

modules/dashboard/edit.php

@@ -393,7 +393,8 @@ foreach ($days as $key => $day) {
 $modulo_interventi = Module::find((new Module())->getByField('title', 'Attività', Models\Locale::getPredefined()->id));
 
 echo '
-<script type="text/javascript">    
+<script type="text/javascript">
+
 var Draggable = FullCalendar.Draggable;
 globals.dashboard = {
         load_url: "'.$structure->fileurl('ajax.php').'",
@@ -727,7 +728,12 @@ if (isMobile() && setting('Utilizzare i tooltip sul calendario')) {
             eventClick: function(info) {
                 if (info.event.extendedProps.link !== undefined) {
                     info.jsEvent.preventDefault();
-                    location.href = info.event.extendedProps.link;
+
+                    if (info.jsEvent.ctrlKey) {
+                        window.open(info.event.extendedProps.link , "_blank");
+                    } else {
+                        location.href = info.event.extendedProps.link;
+                    }
                 }
             },';
 }

modules/interventi/actions.php

@@ -89,7 +89,7 @@ switch (post('op')) {
             $tags_presenti[] = $tag['id_tag'];
         }
 
-        $tags = post('tags');
+        $tags = post('tags') ?: [];
         $tags_presenti = [];
 
         foreach ($tags as $tag) {
@@ -126,7 +126,7 @@ switch (post('op')) {
             }
         }
 
-        $tecnici_assegnati_array [] = post('tecnici_assegnati');
+        $tecnici_assegnati_array = post('tecnici_assegnati') ?: [];
         $tecnici_assegnati = [];
 
         foreach ($tecnici_assegnati_array as $tecnico_assegnato) {
@@ -504,7 +504,7 @@ switch (post('op')) {
         break;
 
     case 'manage_articolo':
-        if (post('idriga') != null) {
+        if (!empty(post('idriga'))) {
             $articolo = Articolo::find(post('idriga'));
         } else {
             $originale = ArticoloOriginale::find(post('idarticolo'));
@@ -533,7 +533,7 @@ switch (post('op')) {
 
         $articolo->save();
 
-        if (post('idriga') != null) {
+        if (!empty(post('idriga'))) {
             flash()->info(tr('Articolo modificato!'));
         } else {
             flash()->info(tr('Articolo aggiunto!'));
@@ -555,7 +555,7 @@ switch (post('op')) {
         break;
 
     case 'manage_sconto':
-        if (post('idriga') != null) {
+        if (!empty(post('idriga'))) {
             $sconto = Sconto::find(post('idriga'));
         } else {
             $sconto = Sconto::build($intervento);
@@ -566,7 +566,7 @@ switch (post('op')) {
         $sconto->note = post('note');
         $sconto->save();
 
-        if (post('idriga') != null) {
+        if (!empty(post('idriga'))) {
             flash()->info(tr('Sconto/maggiorazione modificato!'));
         } else {
             flash()->info(tr('Sconto/maggiorazione aggiunto!'));
@@ -575,7 +575,7 @@ switch (post('op')) {
         break;
 
     case 'manage_riga':
-        if (post('idriga') != null) {
+        if (!empty(post('idriga'))) {
             $riga = Riga::find(post('idriga'));
         } else {
             $riga = Riga::build($intervento);
@@ -596,7 +596,7 @@ switch (post('op')) {
 
         $riga->save();
 
-        if (post('idriga') != null) {
+        if (!empty(post('idriga'))){
             flash()->info(tr('Riga modificata!'));
         } else {
             flash()->info(tr('Riga aggiunta!'));
@@ -1233,7 +1233,7 @@ switch (post('op')) {
         $numero_totale = 0;
 
         foreach ($righe as $riga) {
-            if ($riga['id'] != null) {
+            if (!empty($riga['id'])) {
                 $articolo = Articolo::find($riga['id']);
             }
 

modules/interventi/header.php

@@ -92,13 +92,15 @@ $insoluti = Scadenza::where('idanagrafica', $intervento->idanagrafica)
     ->count();
 
 // Logo
-$logo = Upload::where('id_module', (new Module())->getByField('title', 'Anagrafiche'))->where('id_record', $intervento->idanagrafica)->where('name', 'Logo azienda')->first()->filename ?: App::getPaths()['img'].'/logo_header.png';
+$logo = Upload::where('id_module', (new Module())->getByField('title', 'Anagrafiche'))->where('id_record', $intervento->idanagrafica)->where('name', 'Logo azienda')->first()->filename;
+
+$logo = $logo ? base_path().'/files/anagrafiche/'.$logo : App::getPaths()['img'].'/logo_header.png';
 
 echo '
 <hr>
 <div class="row">
     <div class="col-md-1">
-        <img src="'.base_path().'/files/anagrafiche/'.$logo.'" class="img-responsive">
+        <img src="'.$logo.'" class="img-responsive">
     </div>';
 
 // Cliente

modules/interventi/init.php

@@ -69,12 +69,12 @@ if (!empty($id_record)) {
     $module_query = Util\Query::getQuery($structure, $where, $order);
 
     // Precedente
-    $prev_query = str_replace('2=2', '2=2 AND `posizione` ='.$posizione_attuale - 1, $module_query);
-    $database->FetchArray('SET @posizione = 0;');
-    $prev = $database->FetchOne($prev_query)['id'];
+    $prev_query = str_replace('2=2', '2=2 AND `posizione` ='.prepare($posizione_attuale - 1), $module_query);
+    $database->query('SET @posizione = 0');
+    $prev = $database->fetchOne($prev_query)['id'];
 
     // Successivo
-    $next_query = str_replace('2=2', '2=2 AND `posizione` ='.$posizione_attuale + 1, $module_query);
-    $database->FetchArray('SET @posizione = 0;');
-    $next = $database->FetchOne($next_query)['id'];
+    $next_query = str_replace('2=2', '2=2 AND `posizione` ='.prepare($posizione_attuale + 1), $module_query);
+    $database->query('SET @posizione = 0');
+    $next = $database->fetchOne($next_query)['id'];
 }