[jira] [Comment Edited] (OFBIZ-6463) ordermgr/control/deleteCustRequestParty should delete the record rather than updating the through date

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

[jira] [Comment Edited] (OFBIZ-6463) ordermgr/control/deleteCustRequestParty should delete the record rather than updating the through date

Nicolas Malin (Jira)

    [ https://issues.apache.org/jira/browse/OFBIZ-6463?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14575544#comment-14575544 ]

Adrian Crum edited comment on OFBIZ-6463 at 6/6/15 3:11 AM:
------------------------------------------------------------

The attached patch breaks the design pattern that exists throughout the project. Intersection entities that include a date range are expired, not deleted.

Instead of replacing expiration with deletion, maybe deletion could be ADDED to expiration.


was (Author: [hidden email]):
The attached patch breaks the design pattern that exists throughout the project. Intersection entities that include a date range a re expired, not deleted.

Instead of replacing expiration with deletion, maybe deletion could be ADDED to expiration.

> ordermgr/control/deleteCustRequestParty should delete the record rather than updating the through date
> ------------------------------------------------------------------------------------------------------
>
>                 Key: OFBIZ-6463
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-6463
>             Project: OFBiz
>          Issue Type: Improvement
>          Components: order
>    Affects Versions: Trunk
>            Reporter: Christian Carlow
>            Priority: Minor
>         Attachments: OFBIZ-6463.patch
>
>
> Users expect to be able to remove accidentally added faulty CustRequestParty but the current implementation only updates the through date when the delete button is clicked.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)