fix currPage always to be options value if setState

AllenFang · Jun 9, 2016 · 544cd26 · 544cd26
1 parent 665161a
commit 544cd26
Showing 1 changed file with 8 additions and 11 deletions.
diff --git a/src/BootstrapTable.js b/src/BootstrapTable.js
@@ -144,11 +144,15 @@ class BootstrapTable extends Component {
 
     this.store.setData(nextProps.data.slice());
 
-    let page;
-    if (options.page != null) {
+    // from #481
+    let page = this.state.currPage;
+    if (this.props.options.page !== options.page) {
       page = options.page;
-    } else {
-      page = this.state.currPage;
+    }
+    // from #481
+    let sizePerPage = this.state.sizePerPage;
+    if (this.props.options.sizePerPage !== options.sizePerPage) {
+      sizePerPage = options.sizePerPage;
     }
 
     if (this.isRemoteDataSource()) {
@@ -157,13 +161,6 @@ class BootstrapTable extends Component {
         currPage: page
       });
     } else {
-      // #481
-      // const sizePerPage = options.sizePerPage || this.state.sizePerPage;
-      let sizePerPage = this.state.sizePerPage;
-      if (this.props.options.sizePerPage !== options.sizePerPage) {
-        sizePerPage = options.sizePerPage;
-      }
-
       // #125
       if (!options.page &&
         page >= Math.ceil(nextProps.data.length / sizePerPage)) {