test: fix report view
* override render_header in report view * cleaner refresh in list view.js
This commit is contained in:
parent
8ab6ba1384
commit
cef1e08df4
2 changed files with 5 additions and 13 deletions
|
|
@ -241,20 +241,8 @@ frappe.views.ListView = class ListView extends frappe.views.BaseList {
|
|||
}
|
||||
|
||||
refresh(refresh_header=false) {
|
||||
this.freeze(true);
|
||||
// fetch data from server
|
||||
return frappe.call(this.get_call_args()).then(r => {
|
||||
// render
|
||||
this.prepare_data(r);
|
||||
this.toggle_result_area();
|
||||
this.before_render();
|
||||
super.refresh().then(() => {
|
||||
this.render_header(refresh_header);
|
||||
this.render();
|
||||
this.after_render();
|
||||
this.freeze(false);
|
||||
if (this.settings.refresh) {
|
||||
this.settings.refresh(this);
|
||||
}
|
||||
});
|
||||
}
|
||||
|
||||
|
|
|
|||
|
|
@ -10,6 +10,10 @@ frappe.views.ReportView = class ReportView extends frappe.views.ListView {
|
|||
return 'Report';
|
||||
}
|
||||
|
||||
render_header() {
|
||||
// Override List View Header
|
||||
}
|
||||
|
||||
setup_defaults() {
|
||||
super.setup_defaults();
|
||||
this.page_title = __('Report:') + ' ' + this.page_title;
|
||||
|
|
|
|||
Loading…
Add table
Reference in a new issue