refactor: ottimizzazione funzioni delete()

Author: Pek5892

editor.php

@@ -89,7 +89,7 @@
 
     $advanced_sessions = setting('Attiva notifica di presenza utenti sul record');
     if (!empty($advanced_sessions)) {
-        $dbo->query('DELETE FROM zz_semaphores WHERE id_utente='.prepare(auth_osm()->getUser()['id']).' AND posizione='.prepare($id_module.', '.$id_record));
+        $dbo->delete('zz_semaphores', ['id_utente' => auth_osm()->getUser()['id'], 'posizione' => $id_module.', '.$id_record]);
 
         $dbo->query('INSERT INTO zz_semaphores (id_utente, posizione, updated) VALUES ('.prepare(auth_osm()->getUser()['id']).', '.prepare($id_module.', '.$id_record).', NOW())');
 

modules/articoli/src/Import/CSV.php

@@ -348,15 +348,15 @@ public function init()
 
                     if ($direzione) {
                         // Elimina solo i prezzi nella direzione specificata
-                        $database->query('DELETE FROM mg_prezzi_articoli WHERE id_articolo = '.prepare($articolo->id).' AND id_anagrafica = '.prepare($anagrafica->id).' AND dir = '.prepare($direzione));
+                        $database->delete('mg_prezzi_articoli', ['id_articolo' => $articolo->id, 'id_anagrafica' => $anagrafica->id, 'dir' => $direzione]);
                     } else {
                         // Se la direzione non è specificata, elimina tutti i prezzi per questa combinazione
-                        $database->query('DELETE FROM mg_prezzi_articoli WHERE id_articolo = '.prepare($articolo->id).' AND id_anagrafica = '.prepare($anagrafica->id));
+                        $database->delete('mg_prezzi_articoli', ['id_articolo' => $articolo->id, 'id_anagrafica' => $anagrafica->id]);
                     }
                 }
 
                 if (!empty($record['codice_fornitore']) && !empty($record['descrizione_fornitore'])) {
-                    $database->query('DELETE FROM mg_fornitore_articolo WHERE id_articolo = '.prepare($articolo->id).' AND id_fornitore = '.prepare($anagrafica->id));
+                    $database->delete('mg_fornitore_articolo', ['id_articolo' => $articolo->id, 'id_fornitore' => $anagrafica->id]);
                 }
             }
 

modules/automezzi/actions.php

@@ -296,7 +296,7 @@
         break;
 
     case 'delete':
-        $dbo->query('DELETE FROM `an_sedi` WHERE `id`='.prepare($id_record));
+        $dbo->delete('an_sedi', ['id' => $id_record]);
 
         flash()->info(tr('Automezzo eliminato e articoli riportati in magazzino!'));
 

modules/contratti/bulk.php

@@ -168,7 +168,7 @@
                 $new_idcontratto = $new_contratto->id;
 
                 // Correzioni dei prezzi per gli interventi
-                $dbo->query('DELETE FROM co_contratti_tipiintervento WHERE idcontratto='.prepare($new_idcontratto));
+                $dbo->delete('co_contratti_tipiintervento', ['idcontratto' => $new_idcontratto]);
                 $dbo->query('INSERT INTO co_contratti_tipiintervento(idcontratto, idtipointervento, costo_ore, costo_km, costo_dirittochiamata, costo_ore_tecnico, costo_km_tecnico, costo_dirittochiamata_tecnico) SELECT '.prepare($new_idcontratto).', idtipointervento, costo_ore, costo_km, costo_dirittochiamata, costo_ore_tecnico, costo_km_tecnico, costo_dirittochiamata_tecnico FROM co_contratti_tipiintervento AS z WHERE idcontratto='.prepare($contratto->id));
                 $new_contratto->save();
 

modules/utenti/actions.php

@@ -226,7 +226,7 @@
             $group->delete();
             User::where('idgruppo', $id_record)->delete();
             UserTokens::whereIn('id_utente', User::where('idgruppo', $id_record)->pluck('id'))->delete();
-            $dbo->query('DELETE FROM `zz_permissions` WHERE `idgruppo`='.prepare($id_record));
+            $dbo->delete('zz_permissions', ['idgruppo' => $id_record]);
             flash()->info(tr('Gruppo e relativi utenti eliminati!'));
         } else {
             flash()->error(tr('Questo gruppo non si può eliminare!'));

modules/utenti/src/API/v1/Logout.php

@@ -32,7 +32,7 @@ public function create($request)
 
         if (!empty($request['token']) && !empty($user)) {
             // Cancellazione della chiave
-            $database->query('DELETE FROM `zz_tokens` WHERE `token` = '.prepare($request['token']).' AND `id_utente` = '.prepare($user['id']));
+            $database->delete('zz_tokens', ['token' => $request['token'], 'id_utente' => $user['id']]);
         } else {
             $response = [
                 'status' => API\Response::getStatus()['unauthorized']['code'],

plugins/automezzi_manutenzioni/actions.php

@@ -68,7 +68,7 @@
     case 'delscadenza':
         $idscadenza = post('id');
 
-        $dbo->query('DELETE FROM an_automezzi_scadenze WHERE id = '.prepare($idscadenza));
+        $dbo->delete('an_automezzi_scadenze', ['id' => $idscadenza]);
 
         flash()->info(tr('Scadenza eliminata correttamente!'));