search

demarches-simplifiees / demarches-simplifiees.fr

Dématérialiser et simplifier les démarches administratives
https://www.demarches-simplifiees.fr
GNU Affero General Public License v3.0
193 stars 88 forks source link

ETQ admin/instructeur, je souhaite récupérer la valeur d'un type de champ decimal même si la valeur est précédé et ou suivi d'espace blanc #10554

Closed mfo closed 3 months ago

mfo commented 3 months ago

see: le mail de l'educnat/maël :/

codecov[bot] commented 3 months ago

Codecov Report

Attention: Patch coverage is 88.88889% with 1 line in your changes missing coverage. Please review.

Project coverage is 81.08%. Comparing base (828c539) to head (cfda807). Report is 2 commits behind head on main.

Files Patch % Lines
...maintenance/fix_decimal_number_with_spaces_task.rb 88.88% 1 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #10554 +/- ## ======================================= Coverage 81.08% 81.08% ======================================= Files 1228 1229 +1 Lines 26002 26011 +9 Branches 4701 4703 +2 ======================================= + Hits 21084 21092 +8 - Misses 4918 4919 +1 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

mfo commented 3 months ago

Je me demande pourquoi ne pas faire le meme type de nettoyage a la normalisation plutôt que fallback a nil ? Je pense surtout aux nombres c/c dans le champ qui pourraient contenir des espaces (ptet dans une autre pr pour pas retarder ce fix)

je pensais faire une autre PR pr la normalisation (a appliquer sur les integer aussi), jvoulais pas trop flouter le bazarre. mais je peux fusionner les deux