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

Page bottom navigation problem with grid and include-grid

    Details

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

      Description

      This ticket is in reference of issue OFBIZ-7020. In the fix we need to manually specify the "NO_PAGINATOR" in golbalContext to stop inner grid pagination. Under this ticket we looking for a framework level fix.

      1. OFBIZ-7648.patch
        3 kB
        Rohit Koushal

        Activity

        Hide
        rohit.koushal Rohit Koushal added a comment -

        This patch will fix the problem in pagination when grid is used under another grid.

        Show
        rohit.koushal Rohit Koushal added a comment - This patch will fix the problem in pagination when grid is used under another grid.
        Hide
        rishisolankii Rishi Solanki added a comment - - edited

        Rohit Koushal I see you have put the PAGINATOR_NUMBER in the page context as well. Also removed the NO_PAGINATOR usage from globalContext in PartyForms.

        Does it mean putting the values in the global context is of no use? Thanks!

        Show
        rishisolankii Rishi Solanki added a comment - - edited Rohit Koushal I see you have put the PAGINATOR_NUMBER in the page context as well. Also removed the NO_PAGINATOR usage from globalContext in PartyForms. Does it mean putting the values in the global context is of no use? Thanks!
        Hide
        rohit.koushal Rohit Koushal added a comment -

        Hi Rishi Solanki,

        It is not like that, fix provided under the OFBIZ-7020 is just a temporary fix in which I have set NO_PAGINATOR in the golbalContext to resolve the problem. But Now in the provided fix, I made changes at the framework level. So we don't have to explicitly set the NO_PAGINATOR in the golbalContext on any screen/grid .

        Show
        rohit.koushal Rohit Koushal added a comment - Hi Rishi Solanki , It is not like that, fix provided under the OFBIZ-7020 is just a temporary fix in which I have set NO_PAGINATOR in the golbalContext to resolve the problem. But Now in the provided fix, I made changes at the framework level. So we don't have to explicitly set the NO_PAGINATOR in the golbalContext on any screen/grid .
        Hide
        arunpati Arun Patidar added a comment -

        Committed changes in trunk at rev:1768150

        Thanks Rohit Koushal and Rishi Solanki for the contribution.

        Show
        arunpati Arun Patidar added a comment - Committed changes in trunk at rev:1768150 Thanks Rohit Koushal and Rishi Solanki for the contribution.

          People

          • Assignee:
            arunpati Arun Patidar
            Reporter:
            rohit.koushal Rohit Koushal
          • Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development