Support #11557

[SCP :ID :##5766##] : Validate pending transaction before user deletion

Added by Zahir Abd Latif over 3 years ago. Updated about 3 years ago.

Status:Work Completed-End life cycleStart date:May 04, 2021
Priority:NormalDue 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.

11557 - Validate pending transaction before user deletion.docx (3.26 MB) Nurul Athira Abdul Rahim, March 10, 2021 19:39

email notification deleted user.jpg (120 KB) Nurul Athira Abdul Rahim, March 25, 2021 12:54

user still exist.jpg (333 KB) Nurul Athira Abdul Rahim, March 25, 2021 12:54

no email out.jpg (1.6 MB) Nurul Athira Abdul Rahim, March 30, 2021 16:05

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

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");
}

Comment:
  1. Use "5".equals(bibUser.getRole()) to avoid null exception. Same to "3".
  2. Call getAllPendingVerifyList after role checking. No reason to get tnrx list if role is not related. Same for getAllPendingApprovalList

BSNeBiz
Class: IBUserManagementDeleteServices

Comment:
  1. 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

Tested and passed.

#8 Updated by Nurul Athira Abdul Rahim over 3 years ago

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

Also available in: Atom PDF