Merge pull request #1156 from LD4P/paging_rounding

Force rounding up for last page of paging.
LD4P · Aug 2, 2019 · cb0d861 · cb0d861
2 parents f4cedf4 + cb97234
commit cb0d861
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/src/components/search/SearchResultsPaging.jsx b/src/components/search/SearchResultsPaging.jsx
@@ -42,7 +42,7 @@ const SearchResultsPaging = (props) => {
         newCurrentPage = currentPage + 1
         break
       case '»':
-        newCurrentPage = Math.round(props.totalResults / Config.searchResultsPerPage)
+        newCurrentPage = Math.ceil(props.totalResults / Config.searchResultsPerPage)
         break
       case undefined: // this is required to capture clicks on disabled buttons
         return
@@ -55,7 +55,7 @@ const SearchResultsPaging = (props) => {
     props.fetchSearchResults(props.queryString, queryFrom)
   }
 
-  const lastPage = () => Math.round(props.totalResults / Config.searchResultsPerPage)
+  const lastPage = () => Math.ceil(props.totalResults / Config.searchResultsPerPage)
   const pageButton = (key, active) => <Pagination.Item key={ key } active={ active }>{key}</Pagination.Item>
   const pageButtons = () => Array.from({ length: lastPage() }, (_, index) => pageButton(index + 1, index + 1 === currentPage))