Project

General

Profile

Actions
Copy link

Task #2519

closed

Filters: throw better exception, when more than database supported count of parameters is given in filter parameter

Added by Radek Tomiška over 3 years ago. Updated over 3 years ago.

Status:
Closed
Priority:
Low
Assignee:
Radek Tomiška
Category:
Filters
Target version:
10.6.0
Start date:
10/15/2020
Due date:
% Done:

100%

Estimated time:
Owner:

Description

Throw better exception, when more than database supported count of parameters is given in filter parameter used in 'IN' clausules.
Add application configuration to count of supported parameters.

Related issues

Related to IdStory Identity Manager - Feature #2266: Add filtering requests of a virtual systems by implementersClosedOndrej Husník05/21/2020

Actions
Related to IdStory Identity Manager - Defect #2488: Recalculation of a large number of automatic roles on MS SQL Server ends in errorClosedRadek Tomiška09/15/2020

Actions
Actions
Copy link
 #1

Updated by Radek Tomiška over 3 years ago

  • Related to Feature #2266: Add filtering requests of a virtual systems by implementers added
Actions
Copy link
 #2

Updated by Radek Tomiška over 3 years ago

  • Related to Defect #2488: Recalculation of a large number of automatic roles on MS SQL Server ends in error added
Actions
Copy link
 #3

Updated by Radek Tomiška over 3 years ago

  • Status changed from New to In Progress
Actions
Copy link
 #4

Updated by Radek Tomiška over 3 years ago

  • Status changed from In Progress to Needs feedback
  • Assignee changed from Radek Tomiška to Vít Švanda
  • % Done changed from 0 to 90

Better exception added, configurable maximum added, changelog added, doc added.

Commit:
https://github.com/bcvsolutions/CzechIdMng/commit/57da7fbeeea1501f4f1b2e1691afb6771be523a6

Doc:
https://wiki.czechidm.com/devel/documentation/application_configuration/dev/backend#entity_filters

Could you provide me a feedback, please?

Actions
Copy link
 #5

Updated by Vít Švanda over 3 years ago

  • Status changed from Needs feedback to In Progress
  • Assignee changed from Vít Švanda to Radek Tomiška

I did reivew and tested it.

It works fine on places where you implemented check, but in product exists more places use a IN (DefaultIdentityIdentifiersFilter ...). I have idea made this check for complete predicate. This feature will cover future using of IN too.

Actions
Copy link
 #6

Updated by Radek Tomiška over 3 years ago

  • Status changed from In Progress to Needs feedback
  • Assignee changed from Radek Tomiška to Vít Švanda

Good idea, thx! Implemented in commit:
https://github.com/bcvsolutions/CzechIdMng/commit/b8db1b72d6d346bf2624a7960a84529ee539631a

Could you please provide me a feedback again?

Actions
Copy link
 #7

Updated by Vít Švanda over 3 years ago

  • Status changed from Needs feedback to Resolved
  • Assignee changed from Vít Švanda to Radek Tomiška
  • % Done changed from 90 to 100

I did reivew and tested it. Works awesome on all places. Thanks for that.

Actions
Copy link
 #8

Updated by Radek Tomiška over 3 years ago

  • Status changed from Resolved to Closed
Actions
Copy link

Also available in: Atom PDF