Skip to content

#patch (2631) [ACTION] Empêcher la création et la validation de métriques vides#1430

Merged
superfeedboy merged 6 commits intodevelopfrom
issue/2631
Mar 25, 2026
Merged

#patch (2631) [ACTION] Empêcher la création et la validation de métriques vides#1430
superfeedboy merged 6 commits intodevelopfrom
issue/2631

Conversation

@ch-benard
Copy link
Copy Markdown
Contributor

@ch-benard ch-benard commented Mar 3, 2026

🛠 Description de la PR

Problème

Lorsqu'un utilisateur saisissait des indicateurs pour une année puis supprimait toutes les valeurs,
une ligne vide restait dans la table action_metrics. Cette ligne vide causait une erreur de validation
lors de la modification des dates de début/fin de l'action si l'année ne correspondait plus à la période
d'exécution.

Exemple : action du 01/01/2026 au 26/02/2026, avec une ligne vide pour 2025 → erreur "L'année 2025 n'est pas valide"

Solution

  • Prévention : Filtrage des années vides avant insertion en base de données
  • Tolérance : Validation qui ignore les années sans données réelles
  • Nettoyage automatique : Les lignes vides existantes sont supprimées lors de la prochaine mise à jour (via resetAsideData)

Changements

  1. Création d'une fonction utilitaire hasMetricValues() pour détecter les métriques vides
  2. Modification de insertMetrics() pour ne plus insérer les années sans données
  3. Modification du validateur pour ignorer les années vides lors de la vérification des dates
  4. Amélioration du message d'erreur pour plus de clarté

@ch-benard ch-benard requested a review from superfeedboy March 3, 2026 14:53
@ch-benard ch-benard self-assigned this Mar 3, 2026
@tristanrobert
Copy link
Copy Markdown
Contributor

tristanrobert commented Mar 3, 2026

Snyk checks have passed. No issues have been found so far.

Status Scan Engine Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

Copy link
Copy Markdown
Contributor

@superfeedboy superfeedboy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Proposition de simplification et correction de nommage de fonction fléchée exportée

Comment thread packages/api/server/services/action/_common/hasMetricValues.ts Outdated
Comment thread packages/api/server/models/actionModel/create/insertMetrics.ts Outdated
@sonarqubecloud
Copy link
Copy Markdown

@superfeedboy superfeedboy merged commit 834fccd into develop Mar 25, 2026
7 checks passed
@superfeedboy superfeedboy deleted the issue/2631 branch March 25, 2026 10:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants