[Feature] - Ajoute un meilleur mapping Cim-10 / Cim-11#44
Conversation
There was a problem hiding this comment.
Pull request overview
Cette PR semble viser à améliorer le mapping CIM-10/CIM-11, mais les changements inclus portent principalement sur du nettoyage de code et l’ignorance d’un fichier CSV généré.
Changes:
- Suppression d’un bloc de debug (
dump()/exit) dans la commande d’import CIM-11. - Ajout de
missing-mapping.csvau.gitignoreet correction de l’entrée*.sql.
Reviewed changes
Copilot reviewed 1 out of 3 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| src/Command/Cim11Import.php | Retire un code de debug bloquant dans la résolution du code CIM-10. |
| .gitignore | Ignore un fichier CSV local (missing-mapping.csv) et corrige l’entrée *.sql. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| private function getCim10Code(string $code): ?string | ||
| { | ||
| if ('SD82' === $code) { | ||
| dump($this->cim11Mapping[$code]); | ||
| exit; | ||
| } | ||
|
|
||
| if (isset($this->cim11Mapping[$code])) { | ||
| return $this->cim11Mapping[$code]; | ||
| } |
There was a problem hiding this comment.
Le titre de la PR indique un « meilleur mapping CIM-10/CIM-11 », mais les changements visibles ici sont surtout un nettoyage (suppression de dump/exit) et un ajustement du .gitignore. Si l’amélioration de mapping est attendue, il manque probablement la modification du fichier de mapping (ex: mapping-cim-11.csv) ou il faudrait ajuster le titre/la description pour refléter le contenu réel.
Xusifob
force-pushed
the
bugfix/cim-11-import
branch
from
6c8130a to
eb95615
Compare
* [Feature] - Update the cim-11 import * fixes * fixes
No description provided.