Skip to content

Commit 3dbbe80

Browse files
Pek5892Pek5892
committed
refactor: ottimizzazione query delete con funzione delete()
1 parent ea69fa7 commit 3dbbe80

51 files changed

Lines changed: 97 additions & 144 deletions

File tree

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

‎actions.php‎

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -555,7 +555,10 @@
555555

556556
// Eliminazione
557557
elseif (!empty($customs)) {
558-
$dbo->query('DELETE FROM `zz_field_record` WHERE `id_record` = '.prepare($id_record).' AND `id_field` IN ('.implode(',', array_map('prepare', array_column($customs, 'id'))).')');
558+
$dbo->delete('zz_field_record', [
559+
'id_record' => $id_record,
560+
'id_field' => array_column($customs, 'id')
561+
]);
559562
}
560563
}
561564
}

‎modules/aggiornamenti/src/Controlli/ColonneDuplicateViste.php‎

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -134,7 +134,7 @@ public function execute($record, $params = [])
134134

135135
// Elimina tutte tranne la prima
136136
for ($i = 1; $i < count($viste); ++$i) {
137-
$database->query('DELETE FROM `zz_views` WHERE `id` = '.prepare($viste[$i]['id']));
137+
$database->delete('zz_views', ['id' => $viste[$i]['id']]);
138138
}
139139
}
140140
} elseif (str_starts_with((string) $record_id, 'record_lang_')) {
@@ -151,7 +151,7 @@ public function execute($record, $params = [])
151151
ORDER BY `id` DESC
152152
');
153153
for ($i = 1; $i < count($duplicati); ++$i) {
154-
$database->query('DELETE FROM `zz_views_lang` WHERE `id` = '.prepare($duplicati[$i]['id']));
154+
$database->delete('zz_views_lang', ['id' => $duplicati[$i]['id']]);
155155
}
156156
}
157157
}
@@ -193,7 +193,7 @@ public function solveGlobal($params = [])
193193
if (count($group) > 1) {
194194
// Mantieni il primo record (più recente per ID) ed elimina gli altri
195195
for ($i = 1; $i < count($group); ++$i) {
196-
$database->query('DELETE FROM `zz_views` WHERE `id` = '.prepare($group[$i]['id']));
196+
$database->delete('zz_views', ['id' => $group[$i]['id']]);
197197
$results['name_'.$group[$i]['id']] = true;
198198
}
199199
}
@@ -222,7 +222,7 @@ public function solveGlobal($params = [])
222222
if (count($group) > 1) {
223223
// Mantieni il primo record (più recente per ID) ed elimina gli altri
224224
for ($i = 1; $i < count($group); ++$i) {
225-
$database->query('DELETE FROM `zz_views_lang` WHERE `id` = '.prepare($group[$i]['id']));
225+
$database->delete('zz_views_lang', ['id' => $group[$i]['id']]);
226226
$results['record_lang_'.$group[$i]['id']] = true;
227227
}
228228
}

‎modules/aggiornamenti/src/Controlli/PluginDuplicati.php‎

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -166,7 +166,7 @@ public function execute($record, $params = [])
166166

167167
// Elimina tutti tranne il primo
168168
for ($i = 1; $i < \count($plugins); ++$i) {
169-
$database->query('DELETE FROM `zz_plugins` WHERE `id` = '.prepare($plugins[$i]['id']));
169+
$database->delete('zz_plugins', ['id' => $plugins[$i]['id']]);
170170
}
171171
}
172172
} elseif (str_starts_with((string) $record_id, 'title_diversi_')) {
@@ -186,7 +186,7 @@ public function execute($record, $params = [])
186186
ORDER BY `id` DESC
187187
');
188188
for ($i = 1; $i < \count($duplicati); ++$i) {
189-
$database->query('DELETE FROM `zz_plugins_lang` WHERE `id` = '.prepare($duplicati[$i]['id']));
189+
$database->delete('zz_plugins_lang', ['id' => $duplicati[$i]['id']]);
190190
}
191191
}
192192
}
@@ -228,7 +228,7 @@ public function solveGlobal($params = [])
228228
if (\count($group) > 1) {
229229
// Mantieni il primo record (più recente per ID) ed elimina gli altri
230230
for ($i = 1; $i < \count($group); ++$i) {
231-
$database->query('DELETE FROM `zz_plugins` WHERE `id` = '.prepare($group[$i]['id']));
231+
$database->delete('zz_plugins', ['id' => $group[$i]['id']]);
232232
$results['name_'.$group[$i]['id']] = true;
233233
}
234234
}
@@ -257,7 +257,7 @@ public function solveGlobal($params = [])
257257
if (count($group) > 1) {
258258
// Mantieni il primo record (più recente per ID) ed elimina gli altri
259259
for ($i = 1; $i < count($group); ++$i) {
260-
$database->query('DELETE FROM `zz_plugins_lang` WHERE `id` = '.prepare($group[$i]['id']));
260+
$database->delete('zz_plugins_lang', ['id' => $group[$i]['id']]);
261261
$results['record_lang_'.$group[$i]['id']] = true;
262262
}
263263
}

‎modules/ammortamenti/actions.php‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -61,7 +61,7 @@
6161
}
6262

6363
// Elimino le righe di ammortamento precedenti
64-
$dbo->query('DELETE FROM `co_righe_ammortamenti` WHERE `id_riga` = '.prepare($id_record));
64+
$dbo->delete('co_righe_ammortamenti', ['id_riga' => $id_record]);
6565
foreach ($anni as $i => $anno) {
6666
$perc = $percentuale[$i];
6767

‎modules/articoli/actions.php‎

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -361,8 +361,7 @@
361361
// Leggo info prodotto per descrizione mg_movimenti
362362
$rs = $dbo->fetchArray('SELECT lotto, serial, altro FROM mg_prodotti WHERE id='.prepare($idprodotto));
363363

364-
$query = 'DELETE FROM mg_prodotti WHERE id='.prepare($idprodotto);
365-
if ($dbo->query($query)) {
364+
if ($dbo->delete('mg_prodotti', ['id' => $idprodotto])) {
366365
// Movimento il magazzino se l'ho specificato nelle impostazioni
367366
if (setting("Movimenta il magazzino durante l'inserimento o eliminazione dei lotti/serial number")) {
368367
$articolo->movimenta(-1, tr('Eliminazione dal magazzino del prodotto con serial _SERIAL_', [
@@ -385,8 +384,7 @@
385384
// Aggiorno la quantità dell'articolo
386385
$dbo->query('UPDATE `mg_articoli` SET `qta`=`qta`-'.$qta.' WHERE `id`='.prepare($idarticolo));
387386

388-
$query = 'DELETE FROM mg_movimenti WHERE id='.prepare($idmovimento);
389-
if ($dbo->query($query)) {
387+
if ($dbo->delete('mg_movimenti', ['id' => $idmovimento])) {
390388
flash()->info(tr('Movimento rimosso!'));
391389
}
392390
break;

‎modules/articoli/bulk.php‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -161,8 +161,8 @@
161161
if (!empty($elementi)) {
162162
$dbo->query('UPDATE `mg_articoli` SET `deleted_at` = NOW() WHERE `id` = '.prepare($id).Modules::getAdditionalsQuery($id_module));
163163
} else {
164-
$dbo->query('DELETE FROM `mg_prezzi_articoli` WHERE `id_articolo` = '.prepare($id));
165-
$dbo->query('DELETE FROM `mg_articoli` WHERE `id` = '.prepare($id).Modules::getAdditionalsQuery($id_module));
164+
$dbo->delete('mg_prezzi_articoli', ['id_articolo' => $id]);
165+
$dbo->delete('mg_articoli', ['id' => $id]);
166166
}
167167
}
168168

‎modules/automezzi/actions.php‎

Lines changed: 8 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -133,6 +133,7 @@
133133
]);
134134

135135
flash()->info(tr('Viaggio aggiunto al registro!'));
136+
136137
break;
137138

138139
// Modifica viaggio
@@ -158,17 +159,17 @@
158159
], ['id' => $idviaggio]);
159160

160161
flash()->info(tr('Viaggio aggiornato!'));
162+
161163
break;
162164

163165
// Eliminazione viaggio
164166
case 'delviaggio':
165167
$idviaggio = post('id');
166168

167-
$query = 'DELETE FROM an_automezzi_viaggi WHERE id='.prepare($idviaggio);
168-
169-
if ($dbo->query($query)) {
169+
if ($dbo->delete('an_automezzi_viaggi', ['id' => $idviaggio])) {
170170
flash()->info(tr('Viaggio rimosso dal registro!'));
171171
}
172+
172173
break;
173174

174175
// Aggiunta rifornimento
@@ -197,6 +198,7 @@
197198
]);
198199

199200
flash()->info(tr('Rifornimento aggiunto!'));
201+
200202
break;
201203

202204
// Modifica rifornimento
@@ -224,17 +226,17 @@
224226
], ['id' => $idrifornimento]);
225227

226228
flash()->info(tr('Rifornimento aggiornato!'));
229+
227230
break;
228231

229232
// Eliminazione rifornimento
230233
case 'delrifornimento':
231234
$idrifornimento = post('id');
232235

233-
$query = 'DELETE FROM an_automezzi_rifornimenti WHERE id='.prepare($idrifornimento);
234-
235-
if ($dbo->query($query)) {
236+
if ($dbo->delete('an_automezzi_rifornimenti', ['id' => $idrifornimento])) {
236237
flash()->info(tr('Rifornimento rimosso!'));
237238
}
239+
238240
break;
239241

240242
// Firma viaggio

‎modules/beni/actions.php‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -72,7 +72,7 @@
7272
$documenti = $dbo->fetchNum('SELECT `id` FROM `dt_ddt` WHERE `idaspettobeni`='.prepare($id_record).' UNION SELECT `id` FROM `co_documenti` WHERE `idaspettobeni`='.prepare($id_record));
7373

7474
if ((!empty($id_record)) && empty($documenti)) {
75-
$dbo->query('DELETE FROM `dt_aspettobeni` WHERE `id`='.prepare($id_record));
75+
$dbo->delete('dt_aspettobeni', ['id' => $id_record]);
7676
flash()->info(tr('Aspetto beni eliminato con successo.'));
7777
} else {
7878
flash()->error(tr('Sono presenti dei documenti collegati a questo aspetto beni.'));

‎modules/categorie/actions.php‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -144,7 +144,7 @@
144144
}
145145

146146
if ($dbo->fetchNum('SELECT * FROM `mg_articoli` WHERE (`id_categoria`='.prepare($id).' OR `id_sottocategoria`='.prepare($id).' OR `id_sottocategoria` IN (SELECT `id` FROM `zz_categorie` WHERE `parent`='.prepare($id).')) AND `deleted_at` IS NULL') == 0) {
147-
$dbo->query('DELETE FROM `zz_categorie` WHERE `id`='.prepare($id));
147+
$dbo->delete('zz_categorie', ['id' => $id]);
148148

149149
flash()->info(tr('Tipologia di _TYPE_ eliminata con successo!', [
150150
'_TYPE_' => 'categoria',

‎modules/causali/actions.php‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -76,7 +76,7 @@
7676
$documenti = $dbo->fetchNum('SELECT `id` FROM `dt_ddt` WHERE `idcausalet`='.prepare($id_record).' UNION SELECT `id` FROM `co_documenti` WHERE `idcausalet`='.prepare($id_record));
7777

7878
if ((!empty($id_record)) && empty($documenti)) {
79-
$dbo->query('DELETE FROM `dt_causalet` WHERE `id`='.prepare($id_record));
79+
$dbo->delete('dt_causalet', ['id' => $id_record]);
8080
} else {
8181
$dbo->update('dt_causalet', [
8282
'deleted_at' => date('Y-m-d H:i:s'),

0 commit comments

Comments
 (0)