You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ofbiz.apache.org by jl...@apache.org on 2014/01/21 22:01:28 UTC
svn commit: r1560184 - in /ofbiz/branches/release11.04: ./
applications/content/webapp/content/WEB-INF/actions/content/
framework/base/src/org/ofbiz/base/util/
framework/webtools/webapp/webtools/WEB-INF/actions/entity/
framework/widget/src/org/ofbiz/wi...
Author: jleroux
Date: Tue Jan 21 21:01:27 2014
New Revision: 1560184
URL: http://svn.apache.org/r1560184
Log:
"Applied fix from trunk for revision: 1560176 " (Only partly some files were not yet concerned)
------------------------------------------------------------------------
r1560176 | jleroux | 2014-01-21 21:53:29 +0100 (mar. 21 janv. 2014) | 20 lignes
A patch from Gareth Carter for "last index for paging is not calculated correctly" https://issues.apache.org/jira/browse/OFBIZ-5497
The last index (used on the last button) is not calculated correctly in ftl or the renderer classes. It currently users floor which works only for list sizes that are not multiples of the view size. When you have a list size which is a multiple of the view size, it does not subtract 1 for zero based index and therefore the last button goes to the last page + 1
Examples
listSize = 1
viewSize = 10
listSize / viewSize = 0.1
floor(0.1) = 0
This one produces the correct value
listSize = 20
viewSize = 10
listSize / viewSize = 2
floor(2) = 2
Last index should be 1 because of zero based index but never occurs
To reproduce, enter a view with a list and paging functionality, the list size must be a multiple of the view size
------------------------------------------------------------------------
Modified:
ofbiz/branches/release11.04/ (props changed)
ofbiz/branches/release11.04/applications/content/webapp/content/WEB-INF/actions/content/GetContentLookupList.groovy
ofbiz/branches/release11.04/framework/base/src/org/ofbiz/base/util/UtilMisc.java
ofbiz/branches/release11.04/framework/webtools/webapp/webtools/WEB-INF/actions/entity/FindGeneric.groovy
ofbiz/branches/release11.04/framework/widget/src/org/ofbiz/widget/form/MacroFormRenderer.java
ofbiz/branches/release11.04/framework/widget/src/org/ofbiz/widget/html/HtmlFormRenderer.java
ofbiz/branches/release11.04/framework/widget/src/org/ofbiz/widget/html/HtmlScreenRenderer.java
ofbiz/branches/release11.04/framework/widget/src/org/ofbiz/widget/screen/MacroScreenRenderer.java
Propchange: ofbiz/branches/release11.04/
------------------------------------------------------------------------------
Merged /ofbiz/trunk:r1560176
Modified: ofbiz/branches/release11.04/applications/content/webapp/content/WEB-INF/actions/content/GetContentLookupList.groovy
URL: http://svn.apache.org/viewvc/ofbiz/branches/release11.04/applications/content/webapp/content/WEB-INF/actions/content/GetContentLookupList.groovy?rev=1560184&r1=1560183&r2=1560184&view=diff
==============================================================================
--- ofbiz/branches/release11.04/applications/content/webapp/content/WEB-INF/actions/content/GetContentLookupList.groovy (original)
+++ ofbiz/branches/release11.04/applications/content/webapp/content/WEB-INF/actions/content/GetContentLookupList.groovy Tue Jan 21 21:01:27 2014
@@ -125,7 +125,7 @@ context.highIndex = highIndex;
context.arraySize = arraySize;
context.resultPartialList = resultPartialList;
-viewIndexLast = (int) (arraySize/viewSize);
+viewIndexLast = UtilMisc.getViewLastIndex(arraySize, viewSize);
context.viewIndexLast = viewIndexLast;
contentAssoc = FastList.newInstance();
context.contentAssoc=resultPartialList;
Modified: ofbiz/branches/release11.04/framework/base/src/org/ofbiz/base/util/UtilMisc.java
URL: http://svn.apache.org/viewvc/ofbiz/branches/release11.04/framework/base/src/org/ofbiz/base/util/UtilMisc.java?rev=1560184&r1=1560183&r2=1560184&view=diff
==============================================================================
--- ofbiz/branches/release11.04/framework/base/src/org/ofbiz/base/util/UtilMisc.java (original)
+++ ofbiz/branches/release11.04/framework/base/src/org/ofbiz/base/util/UtilMisc.java Tue Jan 21 21:01:27 2014
@@ -1058,4 +1058,9 @@ public class UtilMisc {
out.close();
}
}
+
+ public static int getViewLastIndex(int listSize, int viewSize) {
+ return (int)Math.ceil(listSize / viewSize) - 1;
+ }
+
}
Modified: ofbiz/branches/release11.04/framework/webtools/webapp/webtools/WEB-INF/actions/entity/FindGeneric.groovy
URL: http://svn.apache.org/viewvc/ofbiz/branches/release11.04/framework/webtools/webapp/webtools/WEB-INF/actions/entity/FindGeneric.groovy?rev=1560184&r1=1560183&r2=1560184&view=diff
==============================================================================
--- ofbiz/branches/release11.04/framework/webtools/webapp/webtools/WEB-INF/actions/entity/FindGeneric.groovy (original)
+++ ofbiz/branches/release11.04/framework/webtools/webapp/webtools/WEB-INF/actions/entity/FindGeneric.groovy Tue Jan 21 21:01:27 2014
@@ -204,7 +204,7 @@ context.highIndex = highIndex;
context.arraySize = arraySize;
context.resultPartialList = resultPartialList;
-viewIndexLast = (int) (arraySize/viewSize);
+viewIndexLast = UtilMisc.getViewLastIndex(arraySize, viewSize);
context.viewIndexLast = viewIndexLast;
List fieldList = FastList.newInstance();
Modified: ofbiz/branches/release11.04/framework/widget/src/org/ofbiz/widget/form/MacroFormRenderer.java
URL: http://svn.apache.org/viewvc/ofbiz/branches/release11.04/framework/widget/src/org/ofbiz/widget/form/MacroFormRenderer.java?rev=1560184&r1=1560183&r2=1560184&view=diff
==============================================================================
--- ofbiz/branches/release11.04/framework/widget/src/org/ofbiz/widget/form/MacroFormRenderer.java (original)
+++ ofbiz/branches/release11.04/framework/widget/src/org/ofbiz/widget/form/MacroFormRenderer.java Tue Jan 21 21:01:27 2014
@@ -2397,10 +2397,11 @@ public class MacroFormRenderer implement
// Last button
if (highIndex < listSize) {
+ int lastIndex = UtilMisc.getViewLastIndex(listSize, viewSize);
if (ajaxEnabled) {
- ajaxLastUrl = createAjaxParamsFromUpdateAreas(updateAreas, prepLinkText + (listSize / viewSize) + anchor, context);
+ ajaxLastUrl = createAjaxParamsFromUpdateAreas(updateAreas, prepLinkText + lastIndex + anchor, context);
} else {
- linkText = prepLinkText + (listSize / viewSize) + anchor;
+ linkText = prepLinkText + lastIndex + anchor;
lastUrl = rh.makeLink(this.request, this.response, urlPath + linkText);
}
}
Modified: ofbiz/branches/release11.04/framework/widget/src/org/ofbiz/widget/html/HtmlFormRenderer.java
URL: http://svn.apache.org/viewvc/ofbiz/branches/release11.04/framework/widget/src/org/ofbiz/widget/html/HtmlFormRenderer.java?rev=1560184&r1=1560183&r2=1560184&view=diff
==============================================================================
--- ofbiz/branches/release11.04/framework/widget/src/org/ofbiz/widget/html/HtmlFormRenderer.java (original)
+++ ofbiz/branches/release11.04/framework/widget/src/org/ofbiz/widget/html/HtmlFormRenderer.java Tue Jan 21 21:01:27 2014
@@ -2500,11 +2500,12 @@ public class HtmlFormRenderer extends Ht
// Last button
writer.append(" <li class=\"").append(modelForm.getPaginateLastStyle());
if (highIndex < listSize) {
+ int lastIndex = UtilMisc.getViewLastIndex(listSize, viewSize);
writer.append("\"><a href=\"");
if (ajaxEnabled) {
- writer.append("javascript:ajaxUpdateAreas('").append(createAjaxParamsFromUpdateAreas(updateAreas, prepLinkText + (listSize / viewSize) + anchor, context)).append("')");
+ writer.append("javascript:ajaxUpdateAreas('").append(createAjaxParamsFromUpdateAreas(updateAreas, prepLinkText + lastIndex + anchor, context)).append("')");
} else {
- linkText = prepLinkText + (listSize / viewSize) + anchor;
+ linkText = prepLinkText + lastIndex + anchor;
appendOfbizUrl(writer, urlPath + linkText);
}
writer.append("\">").append(modelForm.getPaginateLastLabel(context)).append("</a>");
Modified: ofbiz/branches/release11.04/framework/widget/src/org/ofbiz/widget/html/HtmlScreenRenderer.java
URL: http://svn.apache.org/viewvc/ofbiz/branches/release11.04/framework/widget/src/org/ofbiz/widget/html/HtmlScreenRenderer.java?rev=1560184&r1=1560183&r2=1560184&view=diff
==============================================================================
--- ofbiz/branches/release11.04/framework/widget/src/org/ofbiz/widget/html/HtmlScreenRenderer.java (original)
+++ ofbiz/branches/release11.04/framework/widget/src/org/ofbiz/widget/html/HtmlScreenRenderer.java Tue Jan 21 21:01:27 2014
@@ -338,8 +338,8 @@ public class HtmlScreenRenderer extends
writer.append("<li class=\"").append(modelForm.getPaginateLastStyle());
if (highIndex < listSize) {
writer.append("\"><a href=\"");
- int page = (listSize / viewSize) - 1;
- linkText = prepLinkText + page + anchor;
+ int lastIndex = UtilMisc.getViewLastIndex(listSize, viewSize);
+ linkText = prepLinkText + lastIndex + anchor;
// - make the link
writer.append(rh.makeLink(request, response, linkText));
writer.append("\">").append(modelForm.getPaginateLastLabel(context)).append("</a>");
Modified: ofbiz/branches/release11.04/framework/widget/src/org/ofbiz/widget/screen/MacroScreenRenderer.java
URL: http://svn.apache.org/viewvc/ofbiz/branches/release11.04/framework/widget/src/org/ofbiz/widget/screen/MacroScreenRenderer.java?rev=1560184&r1=1560183&r2=1560184&view=diff
==============================================================================
--- ofbiz/branches/release11.04/framework/widget/src/org/ofbiz/widget/screen/MacroScreenRenderer.java (original)
+++ ofbiz/branches/release11.04/framework/widget/src/org/ofbiz/widget/screen/MacroScreenRenderer.java Tue Jan 21 21:01:27 2014
@@ -761,8 +761,8 @@ public class MacroScreenRenderer impleme
// Last button
String lastLinkUrl = "";
if (highIndex < listSize) {
- int page = (listSize / viewSize);
- linkText = prepLinkText + page + anchor;
+ int lastIndex = UtilMisc.getViewLastIndex(listSize, viewSize);
+ linkText = prepLinkText + lastIndex + anchor;
lastLinkUrl = rh.makeLink(request, response, linkText);
}
String nextLinkUrl = "";