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

Screen with pagination on tables lead to multiple db entries when submitting

    Details

      Description

      Submitting a form on a Screen with larger list leads to double DB entries if pagination (next) is used.

      From the cookbook it is known that pagination is automatically added if viewSize and viewIndex is set. Using this "auto pagination" functionalities next button after submitting passes the submit parameters to the next call aswell.

      Following steps do reproduce the problem:

      1. OFBIZ-6547.patch
        0.9 kB
        Gil Portenseigne
      2. DataToReproduceOFBIZ-6547.xml
        4 kB
        Shi Jinghai

        Issue Links

          Activity

          Hide
          gil portenseigne Gil Portenseigne added a comment -

          I just try to reproduce and got the error :

          Found URL parameter [prodCatalogId] passed to secure (https) request-map [...]

          , but in the example with no duplicate entry (surely due to same pk, i did not check).

          To fix this particular form i use the provided patch.

          That i do not know is if it's a wanted behaviour (but which is not good for this kind of screen design) or a bug.

          Show
          gil portenseigne Gil Portenseigne added a comment - I just try to reproduce and got the error : Found URL parameter [prodCatalogId] passed to secure (https) request-map [...] , but in the example with no duplicate entry (surely due to same pk, i did not check). To fix this particular form i use the provided patch. That i do not know is if it's a wanted behaviour (but which is not good for this kind of screen design) or a bug.
          Hide
          adinc Akif Dinc added a comment - - edited

          Unfortunately adding paginate-target did not help here.

           paginate-target="mydevice" 

          I need to say that the Form is included in a screen with include-form within a screenlet. It still passes the previously submited values to the Next-button.

          Show
          adinc Akif Dinc added a comment - - edited Unfortunately adding paginate-target did not help here. paginate-target="mydevice" I need to say that the Form is included in a screen with include-form within a screenlet. It still passes the previously submited values to the Next-button.
          Hide
          shi.jinghai Shi Jinghai added a comment - - edited

          I think this issue is similar to OFBIZ-5634.

          After adding about 20 categories to TestCatalog, I can reproduce this problem as Gil Portenseigne described.

          Show
          shi.jinghai Shi Jinghai added a comment - - edited I think this issue is similar to OFBIZ-5634 . After adding about 20 categories to TestCatalog, I can reproduce this problem as Gil Portenseigne described.
          Hide
          shi.jinghai Shi Jinghai added a comment -

          Test data for the convenience to reproduce OFBIZ-6547.

          Show
          shi.jinghai Shi Jinghai added a comment - Test data for the convenience to reproduce OFBIZ-6547 .
          Hide
          shi.jinghai Shi Jinghai added a comment -

          Thanks Akif Dinc and Gil Portenseigne!

          The patch is in trunk rev. 1692600 and release 13.07 rev. 1692601.

          Show
          shi.jinghai Shi Jinghai added a comment - Thanks Akif Dinc and Gil Portenseigne! The patch is in trunk rev. 1692600 and release 13.07 rev. 1692601.
          Hide
          jacques.le.roux Jacques Le Roux added a comment -

          I confirm I reproduced the same than Gil and Jinghai in R14.12

          Thanks Gil for the patch and Jinghai for the test data (much appreciated)

          I backported in
          R14.12 r1692611
          R12.04 r1692612

          Show
          jacques.le.roux Jacques Le Roux added a comment - I confirm I reproduced the same than Gil and Jinghai in R14.12 Thanks Gil for the patch and Jinghai for the test data (much appreciated ) I backported in R14.12 r1692611 R12.04 r1692612
          Hide
          jacques.le.roux Jacques Le Roux added a comment -

          Akif,

          The case you submitted is fixed, please create a new Jira if you still have another issue, thanks!

          Show
          jacques.le.roux Jacques Le Roux added a comment - Akif, The case you submitted is fixed, please create a new Jira if you still have another issue, thanks!
          Hide
          jacques.le.roux Jacques Le Roux added a comment - - edited

          Jinghai, you need to put in the "Fix Version/s" field the "upcoming branch" and unreleased versions but not trunk and release branches. This information is used in our releases changelogs. Thanks again for you help!

          Show
          jacques.le.roux Jacques Le Roux added a comment - - edited Jinghai, you need to put in the "Fix Version/s" field the "upcoming branch" and unreleased versions but not trunk and release branches. This information is used in our releases changelogs. Thanks again for you help!

            People

            • Assignee:
              shi.jinghai Shi Jinghai
              Reporter:
              adinc Akif Dinc
            • Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development