Overview
Comment: | Vérifions que les comptes existent avant de les utiliser |
---|---|
Downloads: | Tarball | ZIP archive | SQL archive |
Timelines: | family | ancestors | descendants | both | trunk |
Files: | files | file ages | folders |
SHA1: |
a0f927b0301abf5e3385b64dd0b4e390 |
User & Date: | bohwaz on 2015-02-13 03:35:00 |
Other Links: | manifest | tags |
Context
2015-02-14
| ||
01:20 | Suite [16af9ccf928fd8f199465fdd4e8192c83846d301] simplification du report à nouveau (suggéré par @arf) check-in: 7857bf1593 user: bohwaz tags: trunk | |
2015-02-13
| ||
03:35 | Vérifions que les comptes existent avant de les utiliser check-in: a0f927b030 user: bohwaz tags: trunk | |
02:37 | Correction appel namespace check-in: b1ba703d80 user: bohwaz tags: trunk | |
Changes
Modified src/include/lib/Garradin/Compta/Comptes.php from [a07343345f] to [0cb2bda52e].
︙ | ︙ | |||
226 227 228 229 230 231 232 233 234 235 236 237 238 239 | if ($db->simpleQuerySingle('SELECT 1 FROM compta_categories WHERE compte = ? LIMIT 1;', false, $id)) { throw new UserException('Ce compte ne peut être désactivé car des catégories y sont liées.'); } return $db->simpleUpdate('compta_comptes', ['desactive' => 1], 'id = \''.$db->escapeString(trim($id)).'\''); } public function get($id) { $db = DB::getInstance(); return $db->simpleQuerySingle('SELECT * FROM compta_comptes WHERE id = ?;', true, trim($id)); } | > > > > > > > > > > > | 226 227 228 229 230 231 232 233 234 235 236 237 238 239 240 241 242 243 244 245 246 247 248 249 250 | if ($db->simpleQuerySingle('SELECT 1 FROM compta_categories WHERE compte = ? LIMIT 1;', false, $id)) { throw new UserException('Ce compte ne peut être désactivé car des catégories y sont liées.'); } return $db->simpleUpdate('compta_comptes', ['desactive' => 1], 'id = \''.$db->escapeString(trim($id)).'\''); } /** * Renvoie si un compte existe et n'est pas désactivé * @param string $id Numéro de compte * @return boolean TRUE si le compte existe et n'est pas désactivé */ public function isActive($id) { return DB::getInstance()->simpleQuerySingle('SELECT 1 FROM compta_comptes WHERE id = ? AND desactive != 1;', false, $id); } public function get($id) { $db = DB::getInstance(); return $db->simpleQuerySingle('SELECT * FROM compta_comptes WHERE id = ?;', true, trim($id)); } |
︙ | ︙ |
Modified src/include/lib/Garradin/Compta/Exercices.php from [726374475c] to [e2c23e747c].
︙ | ︙ | |||
133 134 135 136 137 138 139 | */ public function doReports($old_id, $date) { $db = DB::getInstance(); $db->exec('BEGIN;'); | < < > > > > > > > > > > > > > > > | 133 134 135 136 137 138 139 140 141 142 143 144 145 146 147 148 149 150 151 152 153 154 155 156 157 158 159 160 161 162 163 164 | */ public function doReports($old_id, $date) { $db = DB::getInstance(); $db->exec('BEGIN;'); $report_crediteur = 110; $report_debiteur = 119; $comptes = new Comptes; if (!$comptes->isActive($report_crediteur)) { throw new UserException('Impossible de faire le report à nouveau : le compte de report créditeur ' . $report_crediteur . ' n\'existe pas ou est désactivé.'); } else if (!$comptes->isActive($report_debiteur)) { throw new UserException('Impossible de faire le report à nouveau : le compte de report débiteur ' . $report_debiteur . ' n\'existe pas ou est désactivé.'); } unset($comptes); $this->solderResultat($old_id, $date); // Récupérer chacun des comptes de bilan et leurs soldes (uniquement les classes 1 à 5) $statement = $db->simpleStatement('SELECT compta_comptes.id AS compte, compta_comptes.position AS position, COALESCE((SELECT SUM(montant) FROM compta_journal WHERE compte_debit = compta_comptes.id AND id_exercice = :id), 0) - COALESCE((SELECT SUM(montant) FROM compta_journal WHERE compte_credit = compta_comptes.id AND id_exercice = :id), 0) AS solde FROM compta_comptes INNER JOIN compta_journal ON compta_comptes.id = compta_journal.compte_debit OR compta_comptes.id = compta_journal.compte_credit |
︙ | ︙ | |||
209 210 211 212 213 214 215 216 217 218 219 220 221 222 223 224 225 226 | */ public function solderResultat($exercice, $date) { $db = DB::getInstance(); $resultat_excedent = 120; $resultat_debiteur = 129; $resultat = $this->getCompteResultat($exercice); $resultat = $resultat['resultat']; if ($resultat != 0) { $journal = new Journal; $journal->add([ 'libelle' => 'Résultat de l\'exercice précédent', 'date' => $date, 'montant' => abs($resultat), | > > > > > > > > > > > > > | | | 222 223 224 225 226 227 228 229 230 231 232 233 234 235 236 237 238 239 240 241 242 243 244 245 246 247 248 249 250 251 252 253 254 255 256 257 258 259 260 261 | */ public function solderResultat($exercice, $date) { $db = DB::getInstance(); $resultat_excedent = 120; $resultat_debiteur = 129; $comptes = new Comptes; if (!$comptes->isActive($resultat_excedent)) { throw new UserException('Impossible de solder l\'exercice : le compte de résultat excédent ' . $resultat_excedent . ' n\'existe pas ou est désactivé.'); } else if (!$comptes->isActive($resultat_debiteur)) { throw new UserException('Impossible de solder l\'exercice : le compte de résultat débiteur ' . $resultat_debiteur . ' n\'existe pas ou est désactivé.'); } unset($comptes); $resultat = $this->getCompteResultat($exercice); $resultat = $resultat['resultat']; if ($resultat != 0) { $journal = new Journal; $journal->add([ 'libelle' => 'Résultat de l\'exercice précédent', 'date' => $date, 'montant' => abs($resultat), 'compte_debit' => $resultat < 0 ? $resultat_debiteur : NULL, 'compte_credit' => $resultat > 0 ? $resultat_excedent : NULL, ]); } return true; } public function delete($id) |
︙ | ︙ |