Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[ListView] Add test on the values and total records #17

Closed
plepaisant opened this issue Oct 22, 2014 · 0 comments
Closed

[ListView] Add test on the values and total records #17

plepaisant opened this issue Oct 22, 2014 · 0 comments
Assignees

Comments

@plepaisant
Copy link
Contributor

On the search method if the values property is not defined there is no changes triggered and no result displayed.

this.search(criteria, this.model.pageInfo()).then(function success(jsonResponse) {
                    currentView.opts.isReadyModelData = true;
                    currentView.model.setTotalRecords(jsonResponse.totalRecords);
                    currentView.model.reset(jsonResponse.values);
                    //Save the criteria and the pagInfo info the session.
                    currentView.saveCriteria.call(currentView, criteria, currentView.model.pageInfo());
                }).then(null, function error(errorResponse) {
                    currentView.opts.isReadyModelData = true;
                    currentView.render();
                    errorHelper.manageResponseErrors(errorResponse, {
                        isDisplay: true
                    });
                });
@pierr pierr closed this as completed in 774897c Nov 4, 2014
@pierr pierr modified the milestone: v0.1.10 - Bug issues correction Nov 5, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants