Closes #395 Fix indexed passed when replacing a break (for only one p…

…age) with the actual page

CHANGELOG.md

@@ -2,6 +2,10 @@
 
 - Removed support for depecrated `extraAriaContext` (please use `ariaLabelBuilder` instead)
 
+## >= 8.0.1
+
+- Fix the indexed passed when replacing a break (containing only one page) with the actual page (see: https://github.com/AdeleD/react-paginate/pull/395)
+
 ## >= 8.0.0
 
 - Remove button role for links with an href (see: https://github.com/AdeleD/react-paginate/pull/390)

__tests__/PaginationBoxView-test.js

@@ -2127,4 +2127,55 @@ describe('Test custom props', () => {
       ).toBe('break');
     });
   });
+
+  describe('Prevent breaks for one page', () => {
+    it('test clicks on previous and next buttons', () => {
+      const pagination = ReactTestUtils.renderIntoDocument(
+        <PaginationBoxView
+          pageRangeDisplayed={5}
+          pageCount={12}
+          renderOnZeroPageCount={null}
+          marginPagesDisplayed={1}
+        />
+      );
+
+      const elmts = ReactTestUtils.scryRenderedDOMComponentsWithTag(
+        pagination,
+        'a'
+      );
+      const previous = elmts[0];
+      const next = elmts[elmts.length - 1];
+
+      ReactTestUtils.Simulate.click(next);
+      expect(
+        ReactDOM.findDOMNode(pagination).querySelectorAll('.selected a').length
+      ).toBe(1);
+      expect(
+        ReactDOM.findDOMNode(pagination).querySelector('.selected a').textContent
+      ).toBe('2');
+
+      // Click to go to page 8.
+      for (let i = 1; i < 7; i++) {
+        ReactTestUtils.Simulate.click(next);
+        expect(
+          ReactDOM.findDOMNode(pagination).querySelectorAll('.selected a').length
+        ).toBe(1);
+        expect(
+          ReactDOM.findDOMNode(pagination).querySelector('.selected a').textContent
+        ).toBe(`${2+i}`);
+      }
+      expect(
+        ReactDOM.findDOMNode(pagination).querySelector('.selected a').textContent
+      ).toBe('8');
+
+      ReactTestUtils.Simulate.click(previous);
+
+      expect(
+        ReactDOM.findDOMNode(pagination).querySelectorAll('.selected a').length
+      ).toBe(1);
+      expect(
+        ReactDOM.findDOMNode(pagination).querySelector('.selected a').textContent
+      ).toBe('7');
+    });
+  });
 });

package.json

@@ -1,6 +1,6 @@
 {
   "name": "react-paginate",
-  "version": "8.0.0",
+  "version": "8.0.1",
   "description": "A ReactJS component that creates a pagination.",
   "main": "./dist/react-paginate.js",
   "repository": {

react_components/PaginationBoxView.js

@@ -445,8 +445,8 @@ export default class PaginationBoxView extends Component {
         ) {
           actualPageElement = {
             type: 'page',
-            index: i,
-            display: createPageView(i),
+            index: pageElement.index,
+            display: createPageView(pageElement.index),
           };
         }
         // We add the displayed elements in the same pass, to avoid another iteration.