Uploaded image for project: 'OFBiz'
  1. OFBiz
  2. OFBIZ-6854

Convert some order entites CRUD service from simple to entity-auto

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Minor
    • Resolution: Done
    • Affects Version/s: Trunk
    • Fix Version/s: 16.11.01
    • Component/s: order
    • Labels:
      None

      Description

      I converted following services from simple to entity-auto :

      • createOrderNotificationLog
      • createOrderItemBilling
      • createOrderAdjustment
      • updateOrderAdjustment
      • createOrderAdjustmentBilling
      • createOrderShipment
      • updateOrderShipment
      • deleteOrderShipment
      • createCommunicationEventOrder
      • removeCommunicationEventOrder
      • createOrderItemShipGroup
      • createOrderContactMech
      • removeOrderContactMech
      • createOrderTerm
      • removeOrderTerm
      • createOrderRequirementCommitment

      And from java to entity-auto for createOrderPaymentPreference

        Activity

        Hide
        deepak.dixit Deepak Dixit added a comment -

        I think we can close this issue.

        Show
        deepak.dixit Deepak Dixit added a comment - I think we can close this issue.
        Hide
        jacques.le.roux Jacques Le Roux added a comment -

        Oops, I did not see the elephant in the room: they are permission issues. So I guess the message to the user will be less clear. Not sure it's a real issue but we might consider it...

        Show
        jacques.le.roux Jacques Le Roux added a comment - Oops, I did not see the elephant in the room: they are permission issues. So I guess the message to the user will be less clear. Not sure it's a real issue but we might consider it...
        Hide
        jacques.le.roux Jacques Le Roux added a comment -

        If we agree, we should apply the same idea each time it's necessary

        Show
        jacques.le.roux Jacques Le Roux added a comment - If we agree, we should apply the same idea each time it's necessary
        Hide
        jacques.le.roux Jacques Le Roux added a comment -

        Before closing we might consider removing the OrderSecurityError* properties which were removed with r1726542 and are now useless

        Show
        jacques.le.roux Jacques Le Roux added a comment - Before closing we might consider removing the OrderSecurityError* properties which were removed with r1726542 and are now useless
        Hide
        soledad Nicolas Malin added a comment -

        Done at revision 1726542 on trunk.

        Show
        soledad Nicolas Malin added a comment - Done at revision 1726542 on trunk.

          People

          • Assignee:
            soledad Nicolas Malin
            Reporter:
            soledad Nicolas Malin
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development