Support #11557
[SCP :ID :##5766##] : Validate pending transaction before user deletion
Status: | Work Completed-End life cycle | Start date: | May 04, 2021 | |
---|---|---|---|---|
Priority: | Normal | Due date: | May 04, 2021 | |
Assignee: | Zahir Abd Latif | % Done: | 100% | |
Category: | - | Spent time: | - | |
Target version: | - |
Description
Hi,
Kindly attend below request:-
Request to validate pending transactions before delete Approver.
Scenario: Company with workflow update to non workflow. While there is transactions which pending for approver.
History
#1 Updated by Nurul Athira Abdul Rahim over 3 years ago
- Assignee changed from Nurul Athira Abdul Rahim to Nurul Hasnieza Bt Mohd Zamri
#2 Updated by Nurul Hasnieza Bt Mohd Zamri over 3 years ago
- Status changed from New - Begin Life Cycle to Development / Work In Progress
#3 Updated by Nurul Hasnieza Bt Mohd Zamri over 3 years ago
- Status changed from Development / Work In Progress to Code Review
- Assignee changed from Nurul Hasnieza Bt Mohd Zamri to Najmi Pasarudin
- % Done changed from 0 to 80
Salam Najmi,
please review my code. Thank you.
#4 Updated by Najmi Pasarudin over 3 years ago
- Status changed from Code Review to Development / Work In Progress
- Assignee changed from Najmi Pasarudin to Nurul Hasnieza Bt Mohd Zamri
IBAM
Class: IBSSComUserDeleteServices
Comment:List<BibTransactionHistory> pendingVerifyList = getBibTransactionHistoryDAO().getAllPendingVerifyList(bibUser.getCompanyId());
if(bibUser.getRole().equals("5") && pendingVerifyList.size() > 0) {
throw IBSSErrorHandler.createBusinessException("delete.","authorize.error.exception");
}List<BibTransactionHistory> pendingApprovalList = getBibTransactionHistoryDAO().getAllPendingApprovalList(bibUser.getCompanyId());
if(bibUser.getRole().equals("3") && pendingApprovalList.size() > 0) {
throw IBSSErrorHandler.createBusinessException("delete.","authorize.error.exception");
}
- Use "5".equals(bibUser.getRole()) to avoid null exception. Same to "3".
- Call getAllPendingVerifyList after role checking. No reason to get tnrx list if role is not related. Same for getAllPendingApprovalList
BSNeBiz
Class: IBUserManagementDeleteServices
- Same as IBAM
#5 Updated by Nurul Hasnieza Bt Mohd Zamri over 3 years ago
- Status changed from Development / Work In Progress to Finished Development
#6 Updated by Nurul Hasnieza Bt Mohd Zamri over 3 years ago
- Status changed from Finished Development to Internal Testing
- Assignee changed from Nurul Hasnieza Bt Mohd Zamri to Nurul Athira Abdul Rahim
Athira,
done deployment at sit environment. Kindly retest.
Finding: No validation to delete authorize user when there are still pending verify and pending approval
Solution: Add the validation to delete authorize user
#7 Updated by Nurul Athira Abdul Rahim over 3 years ago
- File 11557 - Validate pending transaction before user deletion.docx added
- Status changed from Internal Testing to System Integration Test
- % Done changed from 80 to 90
Tested and passed.
#8 Updated by Nurul Athira Abdul Rahim over 3 years ago
- File email notification deleted user.jpg added
- File user still exist.jpg added
- Due date set to March 26, 2021
- Assignee changed from Nurul Athira Abdul Rahim to Nurul Hasnieza Bt Mohd Zamri
- % Done changed from 90 to 70
22/03/21 - Tested and failed by Azyan.
Error prompt correctly. However user received email where user had been deleted. Action is made in IBAM and CDB
#9 Updated by Nurul Athira Abdul Rahim over 3 years ago
- Status changed from System Integration Test to Development / Work In Progress
#10 Updated by Nurul Hasnieza Bt Mohd Zamri over 3 years ago
- Status changed from Development / Work In Progress to Internal Testing
- Assignee changed from Nurul Hasnieza Bt Mohd Zamri to Nurul Athira Abdul Rahim
Hi Athira,
the issue has been fixed. Kindly retest.
#11 Updated by Nurul Athira Abdul Rahim over 3 years ago
- File no email out.jpg added
- Status changed from Internal Testing to System Integration Test
- % Done changed from 70 to 90
Tested and passed.
#12 Updated by Nurul Athira Abdul Rahim over 3 years ago
- Status changed from System Integration Test to Pending UAT Deployment
- Assignee changed from Nurul Athira Abdul Rahim to Nurul Hasnieza Bt Mohd Zamri
Tested and Passed on SIT (15/4/2021)
Kindly deploy the fixes to UAT environment.
Thank you
#13 Updated by Nurul Hasnieza Bt Mohd Zamri over 3 years ago
- Status changed from Pending UAT Deployment to Code Review
#14 Updated by Nurul Hasnieza Bt Mohd Zamri over 3 years ago
- Status changed from Code Review to System Integration Test
#15 Updated by Nurul Hasnieza Bt Mohd Zamri over 3 years ago
- Status changed from System Integration Test to User Acceptance Test
- Assignee changed from Nurul Hasnieza Bt Mohd Zamri to Nurul Athira Abdul Rahim
UAT deployment has been done on 16/04/2021. Kindly proceed with user testing.
#16 Updated by Nurul Athira Abdul Rahim over 3 years ago
- Due date changed from March 26, 2021 to May 04, 2021
- Status changed from User Acceptance Test to Pending Prod Deployment
- Assignee changed from Nurul Athira Abdul Rahim to Najmi Pasarudin
- Start date changed from February 24, 2021 to May 04, 2021
Tested and passed.
Kindly prepare for preprod deployment.
Thank you
#17 Updated by Najmi Pasarudin over 3 years ago
- Status changed from Pending Prod Deployment to Pending Review
- Assignee changed from Najmi Pasarudin to Nurul Athira Abdul Rahim
- % Done changed from 90 to 100
Production deployed on 07/05/2021
#18 Updated by Nurul Athira Abdul Rahim about 3 years ago
- Status changed from Pending Review to Work Completed-End life cycle
- Assignee changed from Nurul Athira Abdul Rahim to Zahir Abd Latif
Migration ID 0033