Skip to content

OFBIZ-13437 Change 'Remove' action to 'Reject' on the Find Requirement screen#1358

Open
toaditi wants to merge 1 commit into
apache:trunkfrom
toaditi:ofbiz-13437-reject-requirement
Open

OFBIZ-13437 Change 'Remove' action to 'Reject' on the Find Requirement screen#1358
toaditi wants to merge 1 commit into
apache:trunkfrom
toaditi:ofbiz-13437-reject-requirement

Conversation

@toaditi

@toaditi toaditi commented Jun 16, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Replaces the hard-delete 'Remove' button on the Find Requirement screen with a 'Reject' action that soft-deletes by setting statusId = REQ_REJECTED
  • The REQ_REJECTED status and all valid transitions already exist in seed data; the existing SECA auto-logs a RequirementStatus history entry on every statusId change via updateRequirement
  • Preserves the full audit trail instead of cascading hard-deleting the requirement and its related records

Changes

  • RequirementForms.xml: replaced deleteLink/CommonRemove/deleteRequirement with rejectLink/FormFieldTitle_rejectButton/rejectRequirement, passing statusId=REQ_REJECTED as a fixed parameter
  • controller.xml: replaced deleteRequirement request-map (invoked deleteRequirementAndRelated) with rejectRequirement (invokes existing updateRequirement)

Test Plan

  • Navigate to Order Manager → Requirements → Find Requirements
  • Confirm the 'Remove' button is replaced by a 'Reject' button
  • Click 'Reject' on a requirement and verify its status changes to 'Rejected' without deleting the record
  • Verify the RequirementStatus history entry is created automatically

…t screen

Replace the hard-delete 'Remove' button with a 'Reject' action that sets
the requirement status to REQ_REJECTED, preserving the audit trail.
The REQ_REJECTED status and valid transitions already exist in seed data,
and the existing SECA auto-logs a RequirementStatus history entry on
any statusId change via updateRequirement.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant