View Issue Details

IDProjectCategoryView StatusLast Update
0006171OXID eShop (all versions)1.08. Listmania, Notice list, Gift registrypublic2016-06-09 12:08
Reportermichael_keiluweit 
PrioritylowSeveritytrivialReproducibilityalways
Status resolvedResolutionfixed 
Product Version4.9.4 / 5.2.4 
Target VersionFixed in Version4.10.0 / 5.3.0 
Summary0006171: Wrong list behavior during reaching the last page numbers
DescriptionScenario: A category with 16 pages.


If one of the four first numbers is clicked (highlighted), the pattern of the page numbers is like this:
1 2 3 [4] 5 ... 16

If the page number 5 is clicked, the pattern changes as following:

1 ... 4 [5] 6 ... 16


At the end of the page number list the behavior is nearly the same:

Current:

1 ... [12] 13 14 15 16

Should be:

1 ... 11 [12] 13 ... 16
1 ... 12 [13] 14 15 16


In the attached picture you can see the wrong behavior in the middle screenshot on the right side.
Steps To Reproduce

  • Create 152 articles and assign them to one category

  • Navigate to the category and have a look the page navigation

  • Click at page 4 and see this pattern: 1 2 3 [4] 5 ... 16

  • Click at page 5 and see this pattern: 1 ... 4 [5] 6 ... 16

  • Click at page 12 and see this (wrong) pattern: 1 ... [12] 13 14 15 16





Reproduced in 5.0.14, 5.1.9, 5.2.3 and 5.2.4
Additional InformationSolution

Edit method oxUBase::generatePageNavigation()

Search the line:

} elseif ($pageNavigation->actPage >= $pageNavigation->NrOfPages - $iTmpVal) {

Replace it with:

} elseif ($pageNavigation->actPage >= $pageNavigation->NrOfPages - $iTmpVal + 1) {
TagsSolution Provided
ThemeAzure
BrowserAll
PHP Version5.3
Database Version5.5

Activities

michael_keiluweit

2015-06-16 10:36

administrator  

pagination.png (27,877 bytes)
pagination.png (27,877 bytes)

michael_keiluweit

2016-05-27 10:07

administrator   ~0011607

https://github.com/OXID-eSales/oxideshop_ce/pull/392

anton.fedurtsya

2016-06-09 12:08

administrator   ~0011638

Thank you. Used the solution provided in description. The solution merged to master and 5.3 branch.