Skip to content

fix filters in PricelistsSearcher - #51

Merged
zikani03 merged 1 commit into
openimis:developfrom
Y-Note-SAS:feature-35459
Aug 12, 2026
Merged

fix filters in PricelistsSearcher#51
zikani03 merged 1 commit into
openimis:developfrom
Y-Note-SAS:feature-35459

Conversation

@Blue-B-code

@Blue-B-code Blue-B-code commented Feb 1, 2026

Copy link
Copy Markdown
Contributor

Description

This PR fixes an issue in the medical pricelist module where the frontend could generate GraphQL queries containing multiple location_Uuid arguments (for example when both region and district filters were selected).

Because the backend only supports a single location_Uuid, such queries resulted in the GraphQL error:

There can only be one argument named "location_Uuid".

The fix introduces a centralized filter-building logic that guarantees only one location_Uuid is sent per request, following a clear business rule:
district selection always takes precedence over region selection.

The user interface behavior was also adjusted to prevent conflicting selections and ensure predictable filtering behavior, without requiring any backend changes.

Type of Change

  • Bug fix

Related Issue(s) / Task(s)

Demo

before
Capture d’écran du 2026-02-01 20-22-36
after
Capture d’écran du 2026-02-01 20-23-42

Checklist

  • Unit tests added/modified N/A
  • I18n / translation handled N/A

@Blue-B-code Blue-B-code reopened this Jul 21, 2026
@zikani03
zikani03 marked this pull request as draft July 30, 2026 10:09
@Blue-B-code
Blue-B-code marked this pull request as ready for review July 31, 2026 14:57
@mngoe
mngoe requested a review from zikani03 August 3, 2026 11:31
@zikani03
zikani03 merged commit 9a36e49 into openimis:develop Aug 12, 2026
1 of 3 checks passed
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.

4 participants