Skip to content

fix: 1330 bo filter list ds from organisation#1196

Merged
l-scherer merged 1 commit intomainfrom
fix/1330-bo-list-ds-organisation
Feb 20, 2026
Merged

fix: 1330 bo filter list ds from organisation#1196
l-scherer merged 1 commit intomainfrom
fix/1330-bo-list-ds-organisation

Conversation

@l-scherer
Copy link
Contributor

Ticket(s) lié(s)

1330

Description

Côté BO la liste des demandes de séjours associées à un organisme PersonnePhysique revenait sans aucun filtrage
Correction : restriction du retour sur l'organisme concerné

Screenshot / liens loom

Check-list

  • Ma branche est rebase sur main
  • Des tests ont été écrits pour tous les endpoints créés ou modifiés
  • Refacto "à la volée" des parties sur lesquelles j'ai codée
  • Plus de console.log
  • J'ai ajouté une validation de schéma sur la route que j'ai ajouté ou modifié
  • J'ai converti les fichiers vue en <script lang="ts">
  • Mon code est en Typescript (autant que possible)

Testing instructions

Sur un compte BO qui gère un organisme Personne Physique qui possède des déclarations de séjour
Aller dans le menu "Organisme"
Filtrer sur la personne physique
Cliquer sur le bouton action >
Cliquer sur l'onglet "Déclaration de séjours"
Avant : Affichait toutes les déclarations de séjour du territoire de l'utilisateur connecté
Maintenant : Affiche toutes les déclarations de séjours du territoire pour l'organisme sélectionné

@revu-bot revu-bot bot requested a review from revu-bot February 20, 2026 10:24
@l-scherer l-scherer merged commit 3dba8f9 into main Feb 20, 2026
21 of 28 checks passed
Copy link
Collaborator

@revu-bot revu-bot left a comment

Choose a reason for hiding this comment

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

Summary

This PR fixes a filtering issue in the back-office where the list of "demandes de séjour" (stay requests) for a PersonnePhysique organization was returning unfiltered results. The fix changes the filter type from "number" to "default" for the organismeId parameter.

Key Findings

Severity Issue Impact
CRITICAL Type change may break existing filters Could cause incorrect query behavior or data exposure
IMPORTANT Missing test coverage No verification that the fix actually works

Overall Assessment

The change is minimal (1 line) but affects a critical filtering mechanism. The type change from "number" to "default" suggests the underlying issue may be related to how the query parameter is being processed, but without seeing the processQuery or mapQueryParams implementation, it's difficult to verify this is the correct fix. The PR lacks tests to validate the filtering behavior works correctly for PersonnePhysique organizations.

@sonarqubecloud
Copy link

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants