Skip to content

Commit 544cd26

Browse files
committed
fix currPage always to be options value if setState
1 parent 665161a commit 544cd26

File tree

1 file changed

+8
-11
lines changed

1 file changed

+8
-11
lines changed

src/BootstrapTable.js

Lines changed: 8 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -144,11 +144,15 @@ class BootstrapTable extends Component {
144144

145145
this.store.setData(nextProps.data.slice());
146146

147-
let page;
148-
if (options.page != null) {
147+
// from #481
148+
let page = this.state.currPage;
149+
if (this.props.options.page !== options.page) {
149150
page = options.page;
150-
} else {
151-
page = this.state.currPage;
151+
}
152+
// from #481
153+
let sizePerPage = this.state.sizePerPage;
154+
if (this.props.options.sizePerPage !== options.sizePerPage) {
155+
sizePerPage = options.sizePerPage;
152156
}
153157

154158
if (this.isRemoteDataSource()) {
@@ -157,13 +161,6 @@ class BootstrapTable extends Component {
157161
currPage: page
158162
});
159163
} else {
160-
// #481
161-
// const sizePerPage = options.sizePerPage || this.state.sizePerPage;
162-
let sizePerPage = this.state.sizePerPage;
163-
if (this.props.options.sizePerPage !== options.sizePerPage) {
164-
sizePerPage = options.sizePerPage;
165-
}
166-
167164
// #125
168165
if (!options.page &&
169166
page >= Math.ceil(nextProps.data.length / sizePerPage)) {

0 commit comments

Comments
 (0)