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

Error on the Split Payment button from the Quick Finalize Order screen

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: Release Branch 12.04, Release Branch 13.07, Release Branch 14.12, Trunk
    • Fix Version/s: 14.12.01, 12.04.06, 13.07.03, 16.11.01
    • Component/s: order
    • Labels:
      None

      Description

      Error in trunk:
      ERROR rendering error page [/error/error.jsp], but here is the error text: org.ofbiz.webapp.control.RequestHandlerException: No definition found for view with name [checkoutpayment]

      Error in stable and old:
      org.ofbiz.webapp.control.RequestHandlerException: No definition found for view with name [checkoutpayment]

      This is the same kind of error you got with OFBIZ-6741 before it was fixed

        Activity

        Hide
        vyom0213@gmail.com Vyom Jain added a comment -

        This is an issue even in 09.04 and 10.04 code, so it has been there for sometime now.

        Show
        vyom0213@gmail.com Vyom Jain added a comment - This is an issue even in 09.04 and 10.04 code, so it has been there for sometime now.
        Hide
        lektran Scott Gray added a comment -

        Looks like the view-map was moved to ecommerce in r544053. Probably because the view-map wasn't referenced in the controller.xml for ordermgr. It is actually used though as a view override in component://order/webapp/ordermgr/entry/checkoutoptions.ftl

        Just needs to be restored in the controller and have the screen moved back.

        Show
        lektran Scott Gray added a comment - Looks like the view-map was moved to ecommerce in r544053. Probably because the view-map wasn't referenced in the controller.xml for ordermgr. It is actually used though as a view override in component://order/webapp/ordermgr/entry/checkoutoptions.ftl Just needs to be restored in the controller and have the screen moved back.
        Hide
        jacques.le.roux Jacques Le Roux added a comment -

        Thanks Scott,

        Fixed in
        trunk r1726828
        R15.12 r1726829
        R14.12 r1726830
        R13.07 r1726832 (tree conflict handled by hand)
        R12.04 r1726833 (tree conflict handled by hand)

        Show
        jacques.le.roux Jacques Le Roux added a comment - Thanks Scott, Fixed in trunk r1726828 R15.12 r1726829 R14.12 r1726830 R13.07 r1726832 (tree conflict handled by hand) R12.04 r1726833 (tree conflict handled by hand)
        Hide
        pfm.smits Pierre Smits added a comment -

        So to understand this issue correctly the quick checkout process in the order component is missing a reference to the view-map 'checkoutpayment'?

        Adding the view-map to the controller.xml of the ordermgr component is a good thing, but removing the screen and the associated .ftl from the ecommerce component isn't.
        The front-end side (from a webshop perspective) is often heavily modified, and having the screen and freemarker definition in a backend component shouldn't be happening.

        Show
        pfm.smits Pierre Smits added a comment - So to understand this issue correctly the quick checkout process in the order component is missing a reference to the view-map 'checkoutpayment'? Adding the view-map to the controller.xml of the ordermgr component is a good thing, but removing the screen and the associated .ftl from the ecommerce component isn't. The front-end side (from a webshop perspective) is often heavily modified, and having the screen and freemarker definition in a backend component shouldn't be happening.
        Hide
        jacques.le.roux Jacques Le Roux added a comment -

        That's not our problem OOTB. Everyone can copy and change in custom projects. We want to avoid duplication and separation OOTB. I thought this was obvious for a long time contributor like you

        Show
        jacques.le.roux Jacques Le Roux added a comment - That's not our problem OOTB. Everyone can copy and change in custom projects. We want to avoid duplication and separation OOTB. I thought this was obvious for a long time contributor like you

          People

          • Assignee:
            jacques.le.roux Jacques Le Roux
            Reporter:
            jacques.le.roux Jacques Le Roux
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development