From 86bac71fb16abddac4196f7114666e4a9cb939f2 Mon Sep 17 00:00:00 2001
From: Thomas Zilio <Dasc3er@users.noreply.github.com>
Date: Sun, 8 Oct 2017 18:28:26 +0200
Subject: [PATCH] Risoluzione di un bug nei tab

Risoluzione di una problematica con i tab dei plugin all'interno dei moduli.
---
 .gitignore                   |  1 +
 controller.php               |  2 +-
 editor.php                   | 17 ++++++++---------
 lib/functions.js             |  4 ++--
 modules/anagrafiche/edit.php | 15 +++++++--------
 5 files changed, 19 insertions(+), 20 deletions(-)

diff --git a/.gitignore b/.gitignore
index afbc586d9..3d92c3818 100644
--- a/.gitignore
+++ b/.gitignore
@@ -82,3 +82,4 @@ files/my_impianti/*
 !files/my_impianti/componente.ini
 config.inc.php
 REVISION
+.php_cs.cache
diff --git a/controller.php b/controller.php
index 7dccc86bf..1d9855f74 100644
--- a/controller.php
+++ b/controller.php
@@ -27,7 +27,7 @@ echo Widgets::addModuleWidgets($id_module, 'controller_top');
 // Lettura eventuali plugins modulo da inserire come tab
 echo '
 		<div class="nav-tabs-custom">
-			<ul class="nav nav-tabs pull-right" role="tablist">
+			<ul class="nav nav-tabs pull-right" id="tabs" role="tablist">
 				<li class="pull-left active header">';
 
 // Verifico se ho impostato un nome modulo personalizzato
diff --git a/editor.php b/editor.php
index 3a9dde64e..13aaa7854 100755
--- a/editor.php
+++ b/editor.php
@@ -54,7 +54,7 @@ if (empty($records)) {
         */
     echo '
 		<div class="nav-tabs-custom">
-			<ul class="nav nav-tabs pull-right" role="tablist">
+			<ul class="nav nav-tabs pull-right" id="tabs" role="tablist">
 				<li class="pull-left active header">';
 
     // Verifico se ho impostato un nome modulo personalizzato
@@ -63,17 +63,18 @@ if (empty($records)) {
     echo '
 					<a data-toggle="tab" href="#tab_0">
 						<i class="'.$module['icon'].'"></i> '.$name;
-// Pulsante "Aggiungi" solo se il modulo è di tipo "table" e se esiste il template per la popup
-if (file_exists($docroot.'/modules/'.$module_dir.'/add.php') && $module['permessi'] == 'rw') {
-    echo '
+    // Pulsante "Aggiungi" solo se il modulo è di tipo "table" e se esiste il template per la popup
+    if (file_exists($docroot.'/modules/'.$module_dir.'/add.php') && $module['permessi'] == 'rw') {
+        echo '
 						<button type="button" class="btn btn-primary" data-toggle="modal" data-title="'.tr('Aggiungi').'..." data-target="#bs-popup" data-href="add.php?id_module='.$id_module.'"><i class="fa fa-plus"></i></button>';
-}
+    }
     echo '
 					</a>
 					<a class="back-btn" href="controller.php?id_module='.$id_module.'"><i class="fa fa-chevron-left"></i> '.tr("Torna all'elenco").'</a>
 				</li>';
 
     $plugins = $dbo->fetchArray('SELECT id, title FROM zz_plugins WHERE idmodule_to='.prepare($id_module)." AND position='tab' AND enabled = 1");
+
     foreach ($plugins as $plugin) {
         echo '
 				<li>
@@ -148,7 +149,6 @@ if ($module['permessi'] == 'r') {
 				$('a.btn-info, button.btn-info, input[type=button].btn-info', 'section.content').show();
 			});
 <?php
-
 } ?>
 
 		var content_was_modified = false;
@@ -173,7 +173,7 @@ if ($module['permessi'] == 'r') {
 		};
 <?php
 if ($advanced_sessions) {
-    ?>
+        ?>
 
 		function getActiveUsers(){
 			$.getJSON('<?php echo ROOTDIR; ?>/call.php', {
@@ -196,8 +196,7 @@ if ($advanced_sessions) {
 
 		setInterval(getActiveUsers, <?php echo get_var('Timeout notifica di presenza (minuti)') * 1000; ?>);
 <?php
-
-}
+    }
 ?>
 	</script>
 <?php
diff --git a/lib/functions.js b/lib/functions.js
index a7c7af627..0b2784476 100644
--- a/lib/functions.js
+++ b/lib/functions.js
@@ -193,12 +193,12 @@ $(document).ready(function () {
     }
 
     // Tabs
-    $('#tabs').tabs();
+    $('.nav-tabs').tabs();
 
     // Entra nel tab indicato al caricamento della pagina
     var hash = window.location.hash ? window.location.hash : getUrlVars().hash;
     if (hash && hash != '#tab_0') {
-        $('ul.nav a[href="' + hash + '"]').tab('show');
+        $('ul.nav-tabs a[href="' + hash + '"]').tab('show');
     }
 
     // Nel caso la navigazione sia da mobile, disabilito il ritorno al punto precedente
diff --git a/modules/anagrafiche/edit.php b/modules/anagrafiche/edit.php
index 2fb2bb50f..8843be61f 100644
--- a/modules/anagrafiche/edit.php
+++ b/modules/anagrafiche/edit.php
@@ -13,15 +13,14 @@ if (!empty($google)) {
 }
 
 if (!$cliente) {
-    $plugins = $dbo->fetchArray("SELECT id FROM zz_plugins WHERE name='Impianti del cliente' OR name='Ddt del cliente'");
+    $ignore = $dbo->fetchArray("SELECT id FROM zz_plugins WHERE name='Impianti del cliente' OR name='Ddt del cliente'");
 
-    foreach ($plugins as $plugin) {
+    foreach ($ignore as $plugin) {
         echo '
 <script>
     $("#link-tab_'.$plugin['id'].'").addClass("disabled");
 </script>';
     }
-
 }
 
 ?><form action="" method="post" role="form">
@@ -411,15 +410,15 @@ if (!str_contains($records[0]['idtipianagrafica'], $id_azienda)) {
 
 <script>
 	$(document).ready( function(){
-		$('.colorpicker').colorpicker().on('changeColor', function(){
-			$('#colore').parent().find('.square').css('background', $('#colore').val());
+		$(".colorpicker").colorpicker().on("changeColor", function(){
+			$("#colore").parent().find(".square").css("background", $("#colore").val());
 		});
 
-		$('#colore').parent().find('.square').css('background', $('#colore').val());
+		$("#colore").parent().find(".square").css("background", $("#colore").val());
 
         $("#geocomplete input").geocomplete({
-            map: $('#map').length ? "#map" : false,
-            location: $('#gaddress').val() ? $('#gaddress').val() : [$('#lat').val(), $('#lng').val()],
+            map: $("#map").length ? "#map" : false,
+            location: $("#gaddress").val() ? $("#gaddress").val() : [$("#lat").val(), $("#lng").val()],
             details: ".details",
             detailsAttribute: "data-geo"
         }).bind("geocode:result", function (event, result) {