Trésorerie : prérequis base de données à la migration Doctrine - #2389
Open
vgreb wants to merge 1 commit into
Open
Trésorerie : prérequis base de données à la migration Doctrine#2389vgreb wants to merge 1 commit into
vgreb wants to merge 1 commit into
Conversation
Prérequis de schéma et de fixtures à la migration de la comptabilité vers Doctrine. Aucune modification de code applicatif. Là où Ting écrivait des identifiants bruts sans vérifier leur existence, Doctrine résout une association vers une entité ou vers `null`. Deux incohérences que Ting masquait deviennent donc bloquantes. **`compta.idmode_regl`** était `NOT NULL`, avec `0` comme sentinelle « pas de règlement » — une valeur qui ne correspond à aucune ligne de `compta_reglement` (ids 1 à 9). La colonne passe nullable et les `0` deviennent `NULL`, ce que la sentinelle signifiait déjà. **Le seed des comptes** ne créait que les ids 1 à 4, alors que les importeurs Crédit Mutuel référencent `ComptaCompte::COURANT_CMUT` (5) et `LIVRET_CMUT` (6) depuis longtemps. À l'import, Ting écrivait un `idcompte` pointant dans le vide ; Doctrine y verrait `null`, en violation de `compta.idcompte NOT NULL`. Les deux comptes sont ajoutés aux fixtures.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Prérequis de schéma et de fixtures à la migration de la comptabilité vers Doctrine. Aucune modification de code applicatif.
Là où Ting écrivait des identifiants bruts sans vérifier leur existence, Doctrine résout une association vers une entité ou vers
null. Deux incohérences que Ting masquait deviennent donc bloquantes.compta.idmode_reglétaitNOT NULL, avec0comme sentinelle « pas de règlement » — une valeur qui ne correspond à aucune ligne decompta_reglement(ids 1 à 9). La colonne passe nullable et les0deviennentNULL, ce que la sentinelle signifiait déjà.Le seed des comptes ne créait que les ids 1 à 4, alors que les importeurs Crédit Mutuel référencent
ComptaCompte::COURANT_CMUT(5) etLIVRET_CMUT(6) depuis longtemps. À l'import, Ting écrivait unidcomptepointant dans le vide ; Doctrine y verraitnull, en violation decompta.idcompte NOT NULL. Les deux comptes sont ajoutés aux fixtures.