(ofbiz-framework) branch trunk updated: Fixed: pagination, jump to page with AJAX enabled is broken (OFBIZ-12817 - #637)

[jleroux]

This is an automated email from the ASF dual-hosted git repository.

jleroux pushed a commit to branch trunk
in repository https://gitbox.apache.org/repos/asf/ofbiz-framework.git


The following commit(s) were added to refs/heads/trunk by this push:
     new d0373f1774 Fixed: pagination, jump to page with AJAX enabled is broken 
(OFBIZ-12817 - #637)
d0373f1774 is described below

commit d0373f177417ef867ca8ab16c6260a8fa471ddce
Author: Florian Motteau <florian.mott...@nereide.fr>
AuthorDate: Fri Feb 16 08:44:12 2024 +0100

    Fixed: pagination, jump to page with AJAX enabled is broken (OFBIZ-12817 - 
#637)
    
    Jump to page with AJAX enabled fetches the wrong page.
    We need to decrement the value entered by the user by one (page numbers are 
zero-indexed)
    
    Thanks: Néréide Team :)
---
 .../java/org/apache/ofbiz/widget/renderer/macro/MacroFormRenderer.java  | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git 
a/framework/widget/src/main/java/org/apache/ofbiz/widget/renderer/macro/MacroFormRenderer.java
 
b/framework/widget/src/main/java/org/apache/ofbiz/widget/renderer/macro/MacroFormRenderer.java
index 16fc404bf2..fdf6f6a818 100644
--- 
a/framework/widget/src/main/java/org/apache/ofbiz/widget/renderer/macro/MacroFormRenderer.java
+++ 
b/framework/widget/src/main/java/org/apache/ofbiz/widget/renderer/macro/MacroFormRenderer.java
@@ -1916,7 +1916,7 @@ public final class MacroFormRenderer implements 
FormStringRenderer {
         if (listSize > 0 && this.javaScriptEnabled) {
             if (ajaxEnabled) {
                 ajaxSelectUrl = 
MacroCommonRenderer.createAjaxParamsFromUpdateAreas(updateAreas, null, 
modelForm,
-                        prepLinkText + "' + this.value + '", context);
+                        prepLinkText + "' + (this.value - 1) + '", context);
             } else {
                 linkText = prepLinkText;
                 if (linkText.startsWith("/")) {