Project

General

Profile

Actions

Feature #2700

closed

Bulk move users to a different work position and change their contract expiration

Added by Alena Peterová about 3 years ago. Updated almost 3 years ago.

Status:
Closed
Priority:
High
Assignee:
Ondrej Husník
Category:
Bulk operations
Target version:
Start date:
03/02/2021
Due date:
% Done:

100%

Estimated time:
16.00 h
Owner:

Description

As discussed, it would be nice to support these two different bulk operations together (for performance reasons, for UX reasons), so the idea is:
  • Select identities
  • Select if the work position should be changed - if so, select its new value
  • Select if the contract validity should be changed - if so, select its new value
  • Run
  • All contracts of the identities will be changed according to the selection. (Note: moving to the new work position must recalculate the automatic roles by tree nodes)

The permissions should behave as in #2664 - if an identity has multiple contracts and the current user can't edit all, then the change should take place only for the editable contracts.


Related issues

Related to IdStory Identity Manager - Defect #2771: Contracts controlled by time slices shouldn't be updated through form projectionClosedRadek Tomiška04/20/2021

Actions
Related to IdStory Identity Manager - Defect #2753: Contract slices: Contract guarantee can be added / deleted when contract is controlled by slicesClosedOndrej Husník04/14/2021

Actions
Related to IdStory Identity Manager - Defect #2990: Some bulk actions started by managers process more users/contracts than they shouldClosedTomáš Doischer11/02/2021

Actions
Actions #2

Updated by Radek Tomiška about 3 years ago

  • Assignee changed from Radek Tomiška to Ondrej Husník
  • Estimated time set to 16.00 h
Actions #3

Updated by Ondrej Husník about 3 years ago

  • Status changed from New to In Progress
Actions #4

Updated by Ondrej Husník about 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

This bulk action implements setting of the new work position and validity dates for user's contracts. There is small divergence in using this BA from regular usage on the contract detail. The thing is that if no value is selected, corresponding item is not changed unlike setting empty value which can be performed on standard detail. Therefore this BA is not able to set unlimited contract validity or no contract position. This limitation will be removed in future.

Please could you do me a feedback?
https://github.com/bcvsolutions/CzechIdMng/commit/1bb49e316051b12d8e7c375998b1895af6a564f3
some doc:
https://wiki.czechidm.com/tutorial/adm/identities_bulk_actions#change_contract_validity_and_position

Actions #5

Updated by Radek Tomiška almost 3 years ago

  • Status changed from Needs feedback to Resolved
  • Assignee changed from Radek Tomiška to Ondrej Husník
  • % Done changed from 90 to 100

I did test and code review, it works, thx.

Review notes:
- we are not using abbreviations
- unnecessary tree node loading removed
- placeholder for inputs added

Commit:
https://github.com/bcvsolutions/CzechIdMng/commit/29d81d2457e5c118a5c1d076fd1dea7a6bca9f5e

Actions #6

Updated by Radek Tomiška almost 3 years ago

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

I found an issue: Bulk action is process even for contracts, which are controlled by contract time slices. Contract controlled by cntract slices cannot be changed directly.

Actions #7

Updated by Radek Tomiška almost 3 years ago

  • Related to Defect #2771: Contracts controlled by time slices shouldn't be updated through form projection added
Actions #8

Updated by Radek Tomiška almost 3 years ago

  • Related to Defect #2753: Contract slices: Contract guarantee can be added / deleted when contract is controlled by slices added
Actions #9

Updated by Ondrej Husník almost 3 years ago

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

Beside main task of this ticket there was also implemented BE parts of preventing from contract modification when contract is controlled by time slice.
Please, could you provide me feedback?

https://github.com/bcvsolutions/CzechIdMng/commit/b1577cc2cd2bde95c992a885dd01bf8921382d3a

Actions #10

Updated by Radek Tomiška almost 3 years ago

  • Status changed from Needs feedback to Resolved
  • Assignee changed from Radek Tomiška to Ondrej Husník
  • % Done changed from 80 to 100

I did test and review, it works, thx. This check will help on every place in application, nice.

Review notes:
- count method is used to prevent load all slices for check
- event content cannot be null

Commit:
https://github.com/bcvsolutions/CzechIdMng/commit/7251036d9fe204a9b2003a4db061ed2e1e9e0594

Actions #11

Updated by Radek Tomiška almost 3 years ago

  • Status changed from Resolved to Closed
Actions #12

Updated by Radek Tomiška over 2 years ago

  • Related to Defect #2990: Some bulk actions started by managers process more users/contracts than they should added
Actions

Also available in: Atom PDF