Details
-
Story
-
Status: Closed (View Workflow)
-
P3
-
Resolution: Done
-
None
-
-
ACQ Sprint 59
-
3
-
Thunderjet
Description
Currently, the business logic is split between endpoint specific helpers which makes it hard to maintain and reuse.
The agreed solution is to have less granular helpers per business entity/area, for example:
- PurchaseOrderHelper - contains the logic related to Purchase orders
- POLineHelper - contains the logic related to PO lines
- InventoryHelper - contains the logic related to communication with Inventory
Also, error handling methods which present in each endpoint specific helper look the same.
This is because of different Response classes per endpoint on which we invoke static methods such as "respond400WithTextPlain" to build the response.
Possible way to avoid this - by working with Response class directly instead of calling those static methods in which case we may end up with one generic error handling method shared between all endpoints which can reside in the Impl class.
Acceptance criteria:
- All business logic is moved to the helpers outlined above
- Make error handling logic reusable for all endpoints (if possible)
- If 2nd point is achieved, get rid of endpoint specific helpers. Otherwise keep them for handing of error responses and invoking new helpers
TestRail: Results
Attachments
Issue Links
- relates to
-
MODORDERS-199 VendorHelper refactoring in accordance with the general helpers structure
-
- Closed
-
-
UXPROD-1388 Create New Purchase Order Lines
-
- Closed
-