From 9ff3022deba7d5b53f6b8e38a9f2886a802271d0 Mon Sep 17 00:00:00 2001 From: Pek5892 Date: Thu, 28 Mar 2024 09:26:26 +0100 Subject: [PATCH] Ottmizzazione con funzione getPredefined() --- modules/anagrafiche/add.php | 4 ++-- modules/anagrafiche/ajax/select.php | 16 ++++++++-------- 2 files changed, 10 insertions(+), 10 deletions(-) diff --git a/modules/anagrafiche/add.php b/modules/anagrafiche/add.php index 226bc5c15..63ff07f9b 100755 --- a/modules/anagrafiche/add.php +++ b/modules/anagrafiche/add.php @@ -24,8 +24,8 @@ include_once __DIR__.'/../../core.php'; $id_nazione_italia = (new Nazione())->getByField('name', 'Italia', Models\Locale::getPredefined()->id); $tipo = get('tipoanagrafica'); -$id_tipo = Tipo::find((new Tipo())->getByField('name', $tipo, Models\Locale::where('predefined', true)->first()->id))->id; -$id_tipo_azienda = Tipo::find((new Tipo())->getByField('name', 'Azienda', Models\Locale::where('predefined', true)->first()->id))->id; +$id_tipo = Tipo::find((new Tipo())->getByField('name', $tipo, Models\Locale::getPredefined()->id))->id; +$id_tipo_azienda = Tipo::find((new Tipo())->getByField('name', 'Azienda', Models\Locale::getPredefined()->id))->id; if (!empty($tipo)) { $rs = $dbo->fetchArray('SELECT `an_tipianagrafiche`.`id`, `an_tipianagrafiche_lang`.`name` as descrizione FROM `an_tipianagrafiche` LEFT JOIN `an_tipianagrafiche_lang` ON (`an_tipianagrafiche`.`id` = `an_tipianagrafiche_lang`.`id_record` AND `an_tipianagrafiche_lang`.`id_lang` = '.prepare(Models\Locale::getDefault()->id).') WHERE `an_tipianagrafiche`.`id`='.prepare($id_tipo)); diff --git a/modules/anagrafiche/ajax/select.php b/modules/anagrafiche/ajax/select.php index 1d461e94d..9fc007ff0 100755 --- a/modules/anagrafiche/ajax/select.php +++ b/modules/anagrafiche/ajax/select.php @@ -25,7 +25,7 @@ $filter_agente = Auth::user()['gruppo'] == 'Agenti'; switch ($resource) { case 'clienti': $id_azienda = setting('Azienda predefinita'); - $tipologia = Tipo::find((new Tipo())->getByField('name', 'Cliente', Models\Locale::where('predefined', true)->first()->id))->id; + $tipologia = Tipo::find((new Tipo())->getByField('name', 'Cliente', Models\Locale::getPredefined()->id))->id; $query = "SELECT `an_anagrafiche`.`idanagrafica` AS id, @@ -103,7 +103,7 @@ switch ($resource) { case 'fornitori': $id_azienda = setting('Azienda predefinita'); - $tipologia = Tipo::find((new Tipo())->getByField('name', 'Fornitore', Models\Locale::where('predefined', true)->first()->id))->id; + $tipologia = Tipo::find((new Tipo())->getByField('name', 'Fornitore', Models\Locale::getPredefined()->id))->id; $query = "SELECT `an_anagrafiche`.`idanagrafica` AS id, @@ -147,7 +147,7 @@ switch ($resource) { break; case 'vettori': - $tipologia = Tipo::find((new Tipo())->getByField('name', 'Vettore', Models\Locale::where('predefined', true)->first()->id))->id; + $tipologia = Tipo::find((new Tipo())->getByField('name', 'Vettore', Models\Locale::getPredefined()->id))->id; $query = "SELECT `an_anagrafiche`.`idanagrafica` AS id, CONCAT(`ragione_sociale`, IF(`citta` IS NULL OR `citta` = '', '', CONCAT(' (', `citta`, ')')), IF(`an_anagrafiche`.`deleted_at` IS NULL, '', ' (".tr('eliminata').")'),' - ', `an_anagrafiche`.`codice`) AS descrizione, `idtipointervento_default` AS idtipointervento FROM `an_anagrafiche` INNER JOIN (`an_tipianagrafiche_anagrafiche` INNER JOIN `an_tipianagrafiche` ON `an_tipianagrafiche_anagrafiche`.`idtipoanagrafica`=`an_tipianagrafiche`.`id` LEFT JOIN `an_tipianagrafiche_lang` ON (`an_tipianagrafiche`.`id` = `an_tipianagrafiche_lang`.`id_record` AND `an_tipianagrafiche_lang`.`id_lang` = ".prepare(Models\Locale::getDefault()->id).')) ON `an_anagrafiche`.`idanagrafica`=`an_tipianagrafiche_anagrafiche`.`idanagrafica` |where| ORDER BY `ragione_sociale`'; @@ -176,7 +176,7 @@ switch ($resource) { * - idanagrafica */ case 'agenti': - $tipologia = Tipo::find((new Tipo())->getByField('name', 'Agente', Models\Locale::where('predefined', true)->first()->id))->id; + $tipologia = Tipo::find((new Tipo())->getByField('name', 'Agente', Models\Locale::getPredefined()->id))->id; $query = "SELECT `an_anagrafiche`.`idanagrafica` AS id, CONCAT(`ragione_sociale`, IF(`citta` IS NULL OR `citta` = '', '', CONCAT(' (', `citta`, ')')), IF(`an_anagrafiche`.`deleted_at` IS NULL, '', ' (".tr('eliminata').")'),' - ', `an_anagrafiche`.`codice`) AS descrizione, `idtipointervento_default` FROM `an_anagrafiche` INNER JOIN (`an_tipianagrafiche_anagrafiche` INNER JOIN `an_tipianagrafiche` ON `an_tipianagrafiche_anagrafiche`.`idtipoanagrafica`=`an_tipianagrafiche`.`id` LEFT JOIN `an_tipianagrafiche_lang` ON (`an_tipianagrafiche`.`id` = `an_tipianagrafiche_lang`.`id_record` AND `an_tipianagrafiche_lang`.`id_lang` = ".prepare(Models\Locale::getDefault()->id).')) ON `an_anagrafiche`.`idanagrafica`=`an_tipianagrafiche_anagrafiche`.`idanagrafica` |where| ORDER BY `ragione_sociale`'; @@ -216,7 +216,7 @@ switch ($resource) { break; case 'tecnici': - $tipologia = Tipo::find((new Tipo())->getByField('name', 'Tecnico', Models\Locale::where('predefined', true)->first()->id))->id; + $tipologia = Tipo::find((new Tipo())->getByField('name', 'Tecnico', Models\Locale::getPredefined()->id))->id; $query = "SELECT `an_anagrafiche`.`idanagrafica` AS id, CONCAT(`ragione_sociale`, IF(`citta` IS NULL OR `citta` = '', '', CONCAT(' (', `citta`, ')')), IF(`an_anagrafiche`.`deleted_at` IS NULL, '', ' (".tr('eliminata').")'),' - ', `an_anagrafiche`.`codice`) AS descrizione, `idtipointervento_default` FROM `an_anagrafiche` INNER JOIN (`an_tipianagrafiche_anagrafiche` INNER JOIN `an_tipianagrafiche` ON `an_tipianagrafiche_anagrafiche`.`idtipoanagrafica`=`an_tipianagrafiche`.`id` LEFT JOIN `an_tipianagrafiche_lang` ON (`an_tipianagrafiche`.`id` = `an_tipianagrafiche_lang`.`id_record` AND `an_tipianagrafiche_lang`.`id_lang` = ".prepare(Models\Locale::getDefault()->id).')) ON `an_anagrafiche`.`idanagrafica`=`an_tipianagrafiche_anagrafiche`.`idanagrafica` |where| ORDER BY `ragione_sociale`'; @@ -250,9 +250,9 @@ switch ($resource) { break; case 'clienti_fornitori': - $id_cliente = Tipo::find((new Tipo())->getByField('name', 'Cliente', Models\Locale::where('predefined', true)->first()->id))->id; - $id_fornitore = Tipo::find((new Tipo())->getByField('name', 'Fornitore', Models\Locale::where('predefined', true)->first()->id))->id; - $id_azienda = Tipo::find((new Tipo())->getByField('name', 'Fornitore', Models\Locale::where('predefined', true)->first()->id))->id; + $id_cliente = Tipo::find((new Tipo())->getByField('name', 'Cliente', Models\Locale::getPredefined()->id))->id; + $id_fornitore = Tipo::find((new Tipo())->getByField('name', 'Fornitore', Models\Locale::getPredefined()->id))->id; + $id_azienda = Tipo::find((new Tipo())->getByField('name', 'Fornitore', Models\Locale::getPredefined()->id))->id; $query = "SELECT `an_anagrafiche`.`idanagrafica` AS id, CONCAT_WS('', `ragione_sociale`, IF(`citta` !='' OR `provincia` != '', CONCAT(' (', `citta`, IF(`provincia`!='', CONCAT(' ', `provincia`), ''), ')'), ''), IF(`an_anagrafiche`.`deleted_at` IS NULL, '', ' (".tr('eliminata').")'),' - ', `an_anagrafiche`.`codice`) AS descrizione, `an_tipianagrafiche_lang`.`name` AS optgroup, `idtipointervento_default`, `an_tipianagrafiche`.`id` as id_tipo FROM `an_tipianagrafiche` LEFT JOIN `an_tipianagrafiche_lang` ON (`an_tipianagrafiche`.`id` = `an_tipianagrafiche_lang`.`id_record` AND `an_tipianagrafiche_lang`.`id_lang` = ".prepare(Models\Locale::getDefault()->id).') INNER JOIN `an_tipianagrafiche_anagrafiche` ON `an_tipianagrafiche`.`id`=`an_tipianagrafiche_anagrafiche`.`idtipoanagrafica` INNER JOIN `an_anagrafiche` ON `an_anagrafiche`.`idanagrafica`=`an_tipianagrafiche_anagrafiche`.`idanagrafica` |where| ORDER BY `optgroup` ASC, `ragione_sociale` ASC';