Overview
Comment: | Autres occurrences de fautes sur la condition de suppression |
---|---|
Downloads: | Tarball | ZIP archive | SQL archive |
Timelines: | family | ancestors | descendants | both | dev |
Files: | files | file ages | folders |
SHA1: |
25dbb07ee330880a70c6377f44213447 |
User & Date: | bohwaz on 2018-10-27 20:04:41 |
Other Links: | branch diff | manifest | tags |
Context
2018-10-27
| ||
22:05 | Ne vider le cache statique que lors d'événements type mise à jour check-in: 52e2bcee25 user: bohwaz tags: dev | |
20:04 | Autres occurrences de fautes sur la condition de suppression check-in: 25dbb07ee3 user: bohwaz tags: dev | |
20:03 | Correction faute d'inattention : ne pas compte_debit OR compte_credit (bug à la suppression de compte encore lié à des écritures) check-in: 2ab615a5de user: bohwaz tags: dev | |
Changes
Modified src/include/lib/Garradin/Compta/Comptes.php from [e84f27a354] to [88f96751dc].
︙ | ︙ | |||
176 177 178 179 180 181 182 | public function canDelete($id) { $db = DB::getInstance(); $id = trim($id); if ($db->firstColumn('SELECT 1 FROM compta_journal | | | 176 177 178 179 180 181 182 183 184 185 186 187 188 189 190 | public function canDelete($id) { $db = DB::getInstance(); $id = trim($id); if ($db->firstColumn('SELECT 1 FROM compta_journal WHERE compte_debit = ? OR compte_credit = ? LIMIT 1;', $id, $id)) { return false; } if ($db->test('compta_categories', $db->where('compte', $id))) { return false; |
︙ | ︙ | |||
202 203 204 205 206 207 208 | { $db = DB::getInstance(); $id = trim($id); if ($db->firstColumn('SELECT 1 FROM compta_journal WHERE id_exercice = (SELECT id FROM compta_exercices WHERE cloture = 0 LIMIT 1) | | | 202 203 204 205 206 207 208 209 210 211 212 213 214 215 216 | { $db = DB::getInstance(); $id = trim($id); if ($db->firstColumn('SELECT 1 FROM compta_journal WHERE id_exercice = (SELECT id FROM compta_exercices WHERE cloture = 0 LIMIT 1) AND (compte_debit = ? OR compte_credit = ?) LIMIT 1;', $id, $id)) { $code = 1; return false; } if ($db->test('compta_categories', $db->where('compte', $id))) { |
︙ | ︙ |
Modified src/include/lib/Garradin/Compta/Comptes_Bancaires.php from [5768c2e53a] to [c2ddfadd03].
︙ | ︙ | |||
80 81 82 83 84 85 86 | { throw new UserException('Ce compte n\'est pas un compte bancaire.'); } // Ne pas supprimer/désactiver un compte qui est utilisé dans l'exercice courant if ($db->firstColumn('SELECT 1 FROM compta_journal WHERE id_exercice = (SELECT id FROM compta_exercices WHERE cloture = 0 LIMIT 1) | | | 80 81 82 83 84 85 86 87 88 89 90 91 92 93 94 | { throw new UserException('Ce compte n\'est pas un compte bancaire.'); } // Ne pas supprimer/désactiver un compte qui est utilisé dans l'exercice courant if ($db->firstColumn('SELECT 1 FROM compta_journal WHERE id_exercice = (SELECT id FROM compta_exercices WHERE cloture = 0 LIMIT 1) AND (compte_debit = ? OR compte_credit = ?) LIMIT 1;', $id, $id)) { throw new UserException('Ce compte ne peut être supprimé car des écritures y sont liées sur l\'exercice courant. ' . 'Il faut supprimer ou ré-attribuer ces écritures avant de pouvoir supprimer le compte.'); } // Il n'est pas possible de supprimer ou désactiver un compte qui est lié à des catégories if ($db->test('compta_categories', $db->where('compte', $id))) |
︙ | ︙ |