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

RequestHandlerException on request link of Quote Price page

    Details

      Description

      Steps to reproduce:
      1. Go to Order component (https://demo-trunk.ofbiz.apache.org/ordermgr/control/main)
      2. Click on Quotes from submenu.(https://demo-trunk.ofbiz.apache.org/ordermgr/control/FindQuote)
      3. Provide search preferences and click on Find button.
      4. Click on Quote Id link (https://demo-trunk.ofbiz.apache.org/ordermgr/control/ViewQuote?quoteId=CQ0001).
      5. Click on Quote Price subtab (https://demo-trunk.ofbiz.apache.org/ordermgr/control/ManageQuotePrices?quoteId=CQ0001)
      6. Click on link under View Request (https://demo-trunk.ofbiz.apache.org/workeffort/control/requestitem)
      It gives a RequestHandlerException.

      1. OFBIZ-9384.patch
        1 kB
        Aditya Sharma
      2. screenshot-1.png
        24 kB
        Aditya Sharma

        Activity

        Hide
        aditya.sharma Aditya Sharma added a comment -

        Problem:
        1. QuotePrices tab redirects to ManageQuotePrices.
        2. ManageQuotePrices screen includes ManageQuotePrices form of QuoteForms.xml to list quote price records.
        3.View Request field contains link with request for workeffort controller while request belongs to ordermgr
        Solution:
        Changed link URL with the request for ordrmgr. Additionally added a check to show link only when custRequestId and custRequestItemSeqId exist.

        Show
        aditya.sharma Aditya Sharma added a comment - Problem: 1. QuotePrices tab redirects to ManageQuotePrices. 2. ManageQuotePrices screen includes ManageQuotePrices form of QuoteForms.xml to list quote price records. 3.View Request field contains link with request for workeffort controller while request belongs to ordermgr Solution: Changed link URL with the request for ordrmgr. Additionally added a check to show link only when custRequestId and custRequestItemSeqId exist.
        Hide
        jacques.le.roux Jacques Le Roux added a comment -

        Thanks Aditya,

        Your modified patch is in
        trunk r1798808
        R16.11 r1798809
        R15.12 r1798810
        R14.12 r1798811

        I simply changed the link from inter-app to simple (inter-app works also but is not right)

        Show
        jacques.le.roux Jacques Le Roux added a comment - Thanks Aditya, Your modified patch is in trunk r1798808 R16.11 r1798809 R15.12 r1798810 R14.12 r1798811 I simply changed the link from inter-app to simple (inter-app works also but is not right)
        Hide
        aditya.sharma Aditya Sharma added a comment -

        Thanks, Jacques

        Show
        aditya.sharma Aditya Sharma added a comment - Thanks, Jacques

          People

          • Assignee:
            jacques.le.roux Jacques Le Roux
            Reporter:
            aditya.sharma Aditya Sharma
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development