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

FindService.performFindList can't be swapped for performFind of grids/forms

    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 is a container issue to have grids/forms render the same way regardless of which service is chosen.

      OFBIZ-6419 will resolve the issue that causes the listSize to be set to the default viewSize.

      FormRender.renderItemRows has a TODO note mentioning that the code needs to be redisigned because it doesn't skip previously iterated rows won't be rendered. Such redisign will likely be involved with the patches to resolve this issue.

        Issue Links

          Activity

          Hide
          ofbizzer Christian Carlow added a comment -

          This issue was encountered while trying to insert group/summary records in the iterator for OFBIZ-6419 which couldn't be done because EntityListIterator doesn't support insertRow.

          Show
          ofbizzer Christian Carlow added a comment - This issue was encountered while trying to insert group/summary records in the iterator for OFBIZ-6419 which couldn't be done because EntityListIterator doesn't support insertRow.
          Hide
          ofbizzer Christian Carlow added a comment - - edited

          Just making a note here in case forgotten later. Once the results of performFindList are displayed, it is necessary to assign override-list-size attribute so that FormRenderer.renderItemRows sets the lowIndex correctly, otherwise pages > 0 are rendered as empty.

          Show
          ofbizzer Christian Carlow added a comment - - edited Just making a note here in case forgotten later. Once the results of performFindList are displayed, it is necessary to assign override-list-size attribute so that FormRenderer.renderItemRows sets the lowIndex correctly, otherwise pages > 0 are rendered as empty.
          Hide
          jacques.le.roux Jacques Le Roux added a comment -

          WIth OFBIZ-6425 and OFBIZ-6422 implemented this is done

          Show
          jacques.le.roux Jacques Le Roux added a comment - WIth OFBIZ-6425 and OFBIZ-6422 implemented this is done

            People

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

              Dates

              • Created:
                Updated:
                Resolved:

                Development