Project

General

Profile

Actions

Feature #2266

closed

Add filtering requests of a virtual systems by implementers

Added by Vladimír Kotýnek almost 4 years ago. Updated over 3 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Ondrej Husník
Category:
Virtual systems
Target version:
Start date:
05/21/2020
Due date:
% Done:

100%

Estimated time:
Owner:

Description

On the Virtual systems/Requests page there will be a filter option where you can select implementer or Implementers for whitch the tasks are created.
The most wanted use-case is to implement a filter which will show only those tasks assigned to me.


Related issues

Related to IdStory Identity Manager - Task #2519: Filters: throw better exception, when more than database supported count of parameters is given in filter parameterClosedRadek Tomiška10/15/2020

Actions
Actions #1

Updated by Vít Švanda almost 4 years ago

  • Subject changed from Add filtering requests by Implementers to Add filtering requests of a virtual systems by implementers
  • Assignee changed from Vít Švanda to Ondrej Husník
  • Target version set to 10.4.0
Actions #2

Updated by Vít Švanda almost 4 years ago

  • Target version changed from 10.4.0 to 10.6.0
Actions #3

Updated by Ondrej Husník over 3 years ago

  • Status changed from New to In Progress
Actions #4

Updated by Ondrej Husník over 3 years ago

  • Status changed from In Progress to Needs feedback
  • Assignee changed from Ondrej Husník to Radek Tomiška
  • % Done changed from 0 to 90

I implemented a new filter option for virtual system requests. It searches for requests according to the implementer identity. More than one identity can be selected. Implementers set to virtual system both by identity and by role are looked for.

Please could you do a feedback for me. Especially the JPA predicate form. Thank you.

https://github.com/bcvsolutions/CzechIdMng/commit/683e1c809ddc0349ba56d42c2be0a2d72a22b01b

Actions #5

Updated by Radek Tomiška over 3 years ago

  • Status changed from Needs feedback to In Progress
  • Assignee changed from Radek Tomiška to Ondrej Husník

I did test and codeview, it works and code looks nice, thx!

Review notes:
- Selected implementers are used in "IN" clausule => it can fail, when more than database supported count of parameters will be given (e.g. 2100 for MSSQL in default setting, 1000 for Oracle). I hope no one will use it this way, but exception can be added the same way as here https://github.com/bcvsolutions/CzechIdMng/blob/develop/Realization/backend/core/core-impl/src/main/java/eu/bcvsolutions/idm/core/model/service/impl/DefaultIdmEntityEventService.java#L83 to be more robust. I will add this exception to other places (filters) too.

I did some quick cosmetic changes only:
- CollectionUtils can be used to cover "implementers != null && !implementers.isEmpty()"
- removed redundant "build.add" criteria with just one criteria

Commit:
https://github.com/bcvsolutions/CzechIdMng/commit/4c082fb118cd2a3b61caf2ff3936967a63ae9313

Actions #6

Updated by Radek Tomiška over 3 years ago

  • Related to Task #2519: Filters: throw better exception, when more than database supported count of parameters is given in filter parameter added
Actions #7

Updated by Radek Tomiška over 3 years ago

  • Status changed from In Progress to Resolved
  • % Done changed from 90 to 100

I moved implementation of better exception, when more than database supported count of parameters is given in filter parameter to related #2519.

Actions #8

Updated by Radek Tomiška over 3 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF