OFBiz
  1. OFBiz
  2. OFBIZ-1945

Create/use order for POS save / load sale

    Details

    • Type: Improvement Improvement
    • Status: In Progress
    • Priority: Major Major
    • Resolution: Unresolved
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: specialpurpose/pos
    • Labels:
      None

      Description

      Currently, the "load sale" / "save sale" pos functions save a shopping list. I'd like to change that so that orders are created and updated. This would increase the security of the solution and allow work orders to be created, etc.

      I'm about 90% there, but I'm hitting a wall. I can currently save off a created order, restore the order, update payments, etc. This includes saving discounts, item configurations, etc. However, there's a problem in saving the new items to the order. I'm probably overlooking something obvious, so if someone wants to double check me, I'd appreciate it.

      1. ofbiz-1945.patch
        22 kB
        Chris Lombardi

        Issue Links

          Activity

          Hide
          Chris Lombardi added a comment -

          Currently doesn't save the added order items.

          Show
          Chris Lombardi added a comment - Currently doesn't save the added order items.
          Hide
          BJ Freeman added a comment -

          Does this save on the POS only or get replicated up the line so another store can pull it down.

          Show
          BJ Freeman added a comment - Does this save on the POS only or get replicated up the line so another store can pull it down.
          Hide
          Chris Lombardi added a comment -

          Synchronization issues aside, this should work such that any POS has access to the list of not completed orders (ORDERHEADER). Changing the behavior is as easy as changing the query that fetches the list of orders. I haven't tried this, but given an appropriate query, it should allow pulling up orders placed through other channels. Restricting the list, i.e. having orders available at any POS to a certain user id, should also be easy.

          It should also allow for taking advantage of ofbiz infrastructure because this more closely adheres to the "normal" ofbiz order flow. I was thinking of routing product requests to specific stations, etc. I'm sure others will have more and better ideas.

          Show
          Chris Lombardi added a comment - Synchronization issues aside, this should work such that any POS has access to the list of not completed orders (ORDERHEADER). Changing the behavior is as easy as changing the query that fetches the list of orders. I haven't tried this, but given an appropriate query, it should allow pulling up orders placed through other channels. Restricting the list, i.e. having orders available at any POS to a certain user id, should also be easy. It should also allow for taking advantage of ofbiz infrastructure because this more closely adheres to the "normal" ofbiz order flow. I was thinking of routing product requests to specific stations, etc. I'm sure others will have more and better ideas.
          Hide
          Jacques Le Roux added a comment -

          An updated patch (merged with révision 739595) with some more changes :
          . Use of FastMapt instead of HashMap in some places in POS
          . A total replacement of doubles by BigDecimals in POS (with some implied adds in UtilFormatOut.java)

          Still have to look at

          • if (!m_trans.addListToCart(sale, m_pos, addToCart)) {
            + if (!m_trans.restoreOrder(sale, m_pos, addToCart)) {

          Should not be commited yet...

          Show
          Jacques Le Roux added a comment - An updated patch (merged with révision 739595) with some more changes : . Use of FastMapt instead of HashMap in some places in POS . A total replacement of doubles by BigDecimals in POS (with some implied adds in UtilFormatOut.java) Still have to look at if (!m_trans.addListToCart(sale, m_pos, addToCart)) { + if (!m_trans.restoreOrder(sale, m_pos, addToCart)) { Should not be commited yet...
          Hide
          Jacques Le Roux added a comment - - edited

          Hi Chris,

          I have commited your patch in r745257 with some modifications, but it's still not used since it miss some features:

          • AutoSave (automatically save current transaction)
          • Add a saved sale to the current sale
          • Remove a saved sale
          • Replace current sale by a saved sale and remove simultaneously the saved sale

          I will try to do that later if nobody beat me on it...

          Show
          Jacques Le Roux added a comment - - edited Hi Chris, I have commited your patch in r745257 with some modifications, but it's still not used since it miss some features: AutoSave (automatically save current transaction) Add a saved sale to the current sale Remove a saved sale Replace current sale by a saved sale and remove simultaneously the saved sale I will try to do that later if nobody beat me on it...

            People

            • Assignee:
              Jacques Le Roux
              Reporter:
              Chris Lombardi
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:

                Development