Bug #16744: Probative value - Identifier filter issue by hazco75 · Pull Request #3907 · ProgrammeVitam/vitam-ui

hazco75 · 2026-08-12T12:11:45Z

Description

Problème de filtre sur la colonne Identifiant dans l'app "Relevés de valeur probante"

Summary by CodeRabbit

Bug Fixes
- Corrected sorting for the identifier column in the probative value list.

coderabbitai · 2026-08-12T12:12:16Z

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: c3f02c9b-5fb8-4faa-b847-44e60bc3267b

📥 Commits

Reviewing files that changed from the base of the PR and between 9e0e03c and fa453a5.

📒 Files selected for processing (1)

ui/ui-frontend/projects/referential/src/app/probative-value/probative-value-list/probative-value-list.component.html

📝 Walkthrough

Walkthrough

The identifier column in the probative value list now sorts by evId instead of #id.

Changes

Probative value sorting

Layer / File(s)	Summary
Update identifier sorting key `ui/ui-frontend/projects/referential/src/app/probative-value/probative-value-list/probative-value-list.component.html`	The identifier column sorting configuration now uses `evId` instead of `#id`.

Estimated code review effort: 1 (Trivial) | ~2 minutes

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Description check	⚠️ Warning	The description states the filter issue but omits the required change type, documentation, tests, migration, checklist, and contributor sections.	Complete the template with the change type, documentation impact, test method and results, migration status, checklist, and contributor.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly identifies the bug and the affected Probative value identifier filter.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

✨ Finishing Touches

🧪 Generate unit tests (beta)

Create PR with unit tests
Commit unit tests in branch bug_16744_probative_value

_{Comment @coderabbitai help to get the list of available commands.}

vitam-prg · 2026-08-12T12:18:23Z

Checkmarx One – Scan Summary & Details – ea593fdd-2dc4-41a5-9b58-153a91838084

New Issues (11)

Checkmarx found the following issues in this Pull Request

#	Issue	Source File / Package	Checkmarx Insight
1	CVE-2026-73088	Npm-browserslist-4.28.6	details Recommended version: 4.28.7 Description: Browserslist is a configuration tool for sharing target browsers and Node.js versions between front-end tools. Prior to 4.28.7, normalizeStats() in... Attack Vector: NETWORK Attack Complexity: LOW Vulnerable Package
2	CVE-2026-73089	Npm-browserslist-4.28.6	details Recommended version: 4.28.7 Description: Browserslist is a configuration tool for sharing target browsers and Node.js versions between front-end tools. Prior to 4.28.7, index.js retains ev... Attack Vector: NETWORK Attack Complexity: LOW Vulnerable Package
3	Parameter_Tampering	api/api-iam/iam/src/main/java/fr/gouv/vitamui/iam/server/rest/CasController.java: 239	details Method getUser at line 239 of /api/api-iam/iam/src/main/java/fr/gouv/vitamui/iam/server/rest/CasController.java gets user input from element embe... Attack Vector
4	Parameter_Tampering	api/api-iam/iam/src/main/java/fr/gouv/vitamui/iam/server/rest/TenantController.java: 130	details Method create at line 130 of /api/api-iam/iam/src/main/java/fr/gouv/vitamui/iam/server/rest/TenantController.java gets user input from element dt... Attack Vector
5	Parameter_Tampering	api/api-iam/iam/src/main/java/fr/gouv/vitamui/iam/server/rest/CasController.java: 301	details Method logout at line 301 of /api/api-iam/iam/src/main/java/fr/gouv/vitamui/iam/server/rest/CasController.java gets user input from element authT... Attack Vector
6	Parameter_Tampering	api/api-iam/iam/src/main/java/fr/gouv/vitamui/iam/server/rest/CasController.java: 197	details Method changePassword at line 197 of /api/api-iam/iam/src/main/java/fr/gouv/vitamui/iam/server/rest/CasController.java gets user input from eleme... Attack Vector
7	Parameter_Tampering	api/api-iam/iam/src/main/java/fr/gouv/vitamui/iam/server/rest/TenantController.java: 130	details Method create at line 130 of /api/api-iam/iam/src/main/java/fr/gouv/vitamui/iam/server/rest/TenantController.java gets user input from element dt... Attack Vector
8	Parameter_Tampering	api/api-iam/iam/src/main/java/fr/gouv/vitamui/iam/server/rest/CasController.java: 301	details Method logout at line 301 of /api/api-iam/iam/src/main/java/fr/gouv/vitamui/iam/server/rest/CasController.java gets user input from element authT... Attack Vector
9	Parameter_Tampering	api/api-iam/iam/src/main/java/fr/gouv/vitamui/iam/server/rest/TenantController.java: 130	details Method create at line 130 of /api/api-iam/iam/src/main/java/fr/gouv/vitamui/iam/server/rest/TenantController.java gets user input from element dt... Attack Vector
10	Parameter_Tampering	api/api-iam/iam/src/main/java/fr/gouv/vitamui/iam/server/rest/CasController.java: 221	details Method getUsersByEmail at line 221 of /api/api-iam/iam/src/main/java/fr/gouv/vitamui/iam/server/rest/CasController.java gets user input from elem... Attack Vector
11	Parameter_Tampering	api/api-iam/iam/src/main/java/fr/gouv/vitamui/iam/server/rest/TenantController.java: 130	details Method create at line 130 of /api/api-iam/iam/src/main/java/fr/gouv/vitamui/iam/server/rest/TenantController.java gets user input from element dt... Attack Vector

Fixed Issues (308)

Great job! The following issues were fixed in this Pull Request

Severity	Issue	Source File / Package
	~~CVE-2026-67318~~	Npm-axios-1.18.1
	~~Privacy_Violation~~	api/api-iam/iam-security/src/main/java/fr/gouv/vitamui/iam/security/service/SecurityService.java: 85
	~~Privacy_Violation~~	api/api-iam/iam-security/src/main/java/fr/gouv/vitamui/iam/security/service/SecurityService.java: 85
	~~Privacy_Violation~~	api/api-iam/iam-security/src/main/java/fr/gouv/vitamui/iam/security/service/SecurityService.java: 85
	~~Log_Forging~~	api/api-iam/iam/src/main/java/fr/gouv/vitamui/iam/server/rest/CasController.java: 266
	~~Log_Forging~~	api/api-collect/collect/src/main/java/fr/gouv/vitamui/collect/server/rest/ProjectController.java: 216
	~~Log_Forging~~	api/api-collect/collect/src/main/java/fr/gouv/vitamui/collect/server/rest/ProjectController.java: 217
	~~Log_Forging~~	api/api-collect/collect/src/main/java/fr/gouv/vitamui/collect/server/rest/ProjectController.java: 216
	~~Log_Forging~~	api/api-collect/collect/src/main/java/fr/gouv/vitamui/collect/server/rest/ProjectController.java: 216
	~~Log_Forging~~	api/api-iam/iam/src/main/java/fr/gouv/vitamui/iam/server/rest/TenantController.java: 130
	~~Log_Forging~~	api/api-archive-search/archive-search/src/main/java/fr/gouv/vitamui/archives/search/server/rest/ArchivesSearchController.java: 459
	~~Log_Forging~~	api/api-archive-search/archive-search/src/main/java/fr/gouv/vitamui/archives/search/server/rest/ArchivesSearchController.java: 467
	~~Log_Forging~~	api/api-archive-search/archive-search/src/main/java/fr/gouv/vitamui/archives/search/server/rest/ArchivesSearchController.java: 467
	~~Log_Forging~~	api/api-archive-search/archive-search/src/main/java/fr/gouv/vitamui/archives/search/server/rest/ArchivesSearchController.java: 459
	~~Log_Forging~~	api/api-archive-search/archive-search/src/main/java/fr/gouv/vitamui/archives/search/server/rest/ArchivesSearchController.java: 467
	~~Log_Forging~~	api/api-archive-search/archive-search/src/main/java/fr/gouv/vitamui/archives/search/server/rest/ArchivesSearchController.java: 459
	~~Log_Forging~~	api/api-archive-search/archive-search/src/main/java/fr/gouv/vitamui/archives/search/server/rest/ArchivesSearchController.java: 467
	~~Log_Forging~~	api/api-archive-search/archive-search/src/main/java/fr/gouv/vitamui/archives/search/server/rest/ArchivesSearchController.java: 467
	~~Log_Forging~~	api/api-archive-search/archive-search/src/main/java/fr/gouv/vitamui/archives/search/server/rest/ArchivesSearchController.java: 459
	~~Log_Forging~~	api/api-archive-search/archive-search/src/main/java/fr/gouv/vitamui/archives/search/server/rest/ArchivesSearchController.java: 459
	~~Log_Forging~~	api/api-archive-search/archive-search/src/main/java/fr/gouv/vitamui/archives/search/server/rest/ArchivesSearchController.java: 467
	~~Log_Forging~~	api/api-archive-search/archive-search/src/main/java/fr/gouv/vitamui/archives/search/server/rest/ArchivesSearchController.java: 459
	~~Log_Forging~~	api/api-archive-search/archive-search/src/main/java/fr/gouv/vitamui/archives/search/server/rest/ArchivesSearchController.java: 467
	~~Log_Forging~~	api/api-archive-search/archive-search/src/main/java/fr/gouv/vitamui/archives/search/server/rest/ArchivesSearchController.java: 459
	~~Log_Forging~~	api/api-archive-search/archive-search/src/main/java/fr/gouv/vitamui/archives/search/server/rest/ArchivesSearchController.java: 459
	~~Log_Forging~~	api/api-archive-search/archive-search/src/main/java/fr/gouv/vitamui/archives/search/server/rest/ArchivesSearchController.java: 467
	~~Log_Forging~~	api/api-archive-search/archive-search/src/main/java/fr/gouv/vitamui/archives/search/server/rest/ArchivesSearchController.java: 268
	~~Log_Forging~~	api/api-archive-search/archive-search/src/main/java/fr/gouv/vitamui/archives/search/server/rest/ArchivesSearchController.java: 268
	~~Log_Forging~~	api/api-archive-search/archive-search/src/main/java/fr/gouv/vitamui/archives/search/server/rest/ArchivesSearchController.java: 268
	~~Log_Forging~~	api/api-archive-search/archive-search/src/main/java/fr/gouv/vitamui/archives/search/server/rest/ArchivesSearchController.java: 268
	~~Log_Forging~~	api/api-archive-search/archive-search/src/main/java/fr/gouv/vitamui/archives/search/server/rest/ArchivesSearchController.java: 268
	~~Log_Forging~~	api/api-collect/collect/src/main/java/fr/gouv/vitamui/collect/server/rest/TransactionArchiveUnitController.java: 163
	~~Log_Forging~~	api/api-iam/iam/src/main/java/fr/gouv/vitamui/iam/server/rest/UserController.java: 148
	~~Log_Forging~~	api/api-iam/iam/src/main/java/fr/gouv/vitamui/iam/server/rest/UserController.java: 148
	~~Log_Forging~~	api/api-archive-search/archive-search/src/main/java/fr/gouv/vitamui/archives/search/server/rest/ArchivesSearchController.java: 217
	~~Log_Forging~~	api/api-collect/collect/src/main/java/fr/gouv/vitamui/collect/server/rest/TransactionController.java: 268
	~~Log_Forging~~	api/api-archive-search/archive-search/src/main/java/fr/gouv/vitamui/archives/search/server/rest/ArchivesSearchController.java: 217
	~~Log_Forging~~	api/api-archive-search/archive-search/src/main/java/fr/gouv/vitamui/archives/search/server/rest/ArchivesSearchController.java: 179
	~~Log_Forging~~	api/api-collect/collect/src/main/java/fr/gouv/vitamui/collect/server/rest/TransactionController.java: 268
	~~Log_Forging~~	api/api-archive-search/archive-search/src/main/java/fr/gouv/vitamui/archives/search/server/rest/ArchivesSearchController.java: 217
	~~Log_Forging~~	api/api-archive-search/archive-search/src/main/java/fr/gouv/vitamui/archives/search/server/rest/ArchivesSearchController.java: 180
	~~Log_Forging~~	api/api-collect/collect/src/main/java/fr/gouv/vitamui/collect/server/rest/ProjectObjectGroupController.java: 151
	~~Log_Forging~~	api/api-archive-search/archive-search/src/main/java/fr/gouv/vitamui/archives/search/server/rest/ArchivesSearchController.java: 268
	~~Log_Forging~~	api/api-archive-search/archive-search/src/main/java/fr/gouv/vitamui/archives/search/server/rest/ArchivesSearchController.java: 287
	~~Log_Forging~~	api/api-archive-search/archive-search/src/main/java/fr/gouv/vitamui/archives/search/server/rest/ArchivesSearchController.java: 179
	~~Log_Forging~~	api/api-archive-search/archive-search/src/main/java/fr/gouv/vitamui/archives/search/server/rest/ArchivesSearchController.java: 287
	~~Log_Forging~~	api/api-archive-search/archive-search/src/main/java/fr/gouv/vitamui/archives/search/server/rest/ArchivesSearchController.java: 248
	~~Log_Forging~~	api/api-archive-search/archive-search/src/main/java/fr/gouv/vitamui/archives/search/server/rest/ArchivesSearchController.java: 287
	~~Log_Forging~~	api/api-archive-search/archive-search/src/main/java/fr/gouv/vitamui/archives/search/server/rest/ArchivesSearchController.java: 287
	~~Log_Forging~~	api/api-archive-search/archive-search/src/main/java/fr/gouv/vitamui/archives/search/server/rest/ArchivesSearchController.java: 287
	~~Log_Forging~~	api/api-archive-search/archive-search/src/main/java/fr/gouv/vitamui/archives/search/server/rest/ArchivesSearchController.java: 258
	~~Log_Forging~~	api/api-iam/iam/src/main/java/fr/gouv/vitamui/iam/server/rest/UserController.java: 148
	~~Log_Forging~~	api/api-archive-search/archive-search/src/main/java/fr/gouv/vitamui/archives/search/server/rest/ArchivesSearchController.java: 287
	~~Log_Forging~~	api/api-archive-search/archive-search/src/main/java/fr/gouv/vitamui/archives/search/server/rest/ArchivesSearchController.java: 287
	~~Log_Forging~~	api/api-archive-search/archive-search/src/main/java/fr/gouv/vitamui/archives/search/server/rest/ArchivesSearchController.java: 287
	~~Log_Forging~~	api/api-archive-search/archive-search/src/main/java/fr/gouv/vitamui/archives/search/server/rest/ArchivesSearchController.java: 287
	~~Log_Forging~~	api/api-archive-search/archive-search/src/main/java/fr/gouv/vitamui/archives/search/server/rest/ArchivesSearchController.java: 192

More results are available on the CxOne platform

Use @Checkmarx to interact with Checkmarx PR Assistant.
Examples:
@Checkmarx how are you able to help me?
@Checkmarx rescan this PR

Bug #16744: Probative value - Identifier filter issue

fa453a5

hazco75 added this to the IT 174 milestone Aug 12, 2026

hazco75 added bug Something isn't working small pr embarquant peu de changements et à review rapide, ne nécessitant qu'un reviewer labels Aug 12, 2026

marob approved these changes Aug 12, 2026

View reviewed changes

mkhediri approved these changes Aug 14, 2026

View reviewed changes

hazco75 merged commit fe410d9 into develop Aug 14, 2026
21 checks passed

hazco75 deleted the bug_16744_probative_value branch August 14, 2026 12:57

Navigation Menu

Uh oh!

Bug #16744: Probative value - Identifier filter issue - #3907

Bug #16744: Probative value - Identifier filter issue#3907
hazco75 merged 1 commit into
developfrom
bug_16744_probative_value

hazco75 commented Aug 12, 2026 •

edited by coderabbitai Bot

Loading

Uh oh!

coderabbitai Bot commented Aug 12, 2026 •

edited

Loading

Walkthrough

Changes

❌ Failed checks (1 warning)

Uh oh!

vitam-prg commented Aug 12, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

Uh oh!

Conversation

hazco75 commented Aug 12, 2026 • edited by coderabbitai Bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Description

Summary by CodeRabbit

Uh oh!

coderabbitai Bot commented Aug 12, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Walkthrough

Changes

❌ Failed checks (1 warning)

Uh oh!

vitam-prg commented Aug 12, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

hazco75 commented Aug 12, 2026 •

edited by coderabbitai Bot

Loading

coderabbitai Bot commented Aug 12, 2026 •

edited

Loading