diff options
| author | Aaron Bohy <aab@odoo.com> | 2019-07-16 13:11:13 +0000 |
|---|---|---|
| committer | Aaron Bohy <aab@odoo.com> | 2019-07-17 06:54:49 +0000 |
| commit | e2893ce3863ddf858ebee00096e06bbf010ddbc2 (patch) | |
| tree | f8648bfb9dc8d3d6d380b1a33dc1c8e74d352153 /addons/web/static/src/js/views | |
| parent | 00c966b69a8d24fbaafdabfd7ee32d747c8b3783 (diff) | |
[REF] web: factorize no content helper
Different templates were defined for graph, pivot and cohort views
for the no content helper. Those templates were similar (only the
message sometimes slighlty changed). This rev. creates one generic
template used by those three views.
closes odoo/odoo#34916
Signed-off-by: Mathieu Duckerts-Antoine <Polymorphe57@users.noreply.github.com>
Diffstat (limited to 'addons/web/static/src/js/views')
| -rw-r--r-- | addons/web/static/src/js/views/graph/graph_renderer.js | 10 | ||||
| -rw-r--r-- | addons/web/static/src/js/views/pivot/pivot_renderer.js | 5 |
2 files changed, 7 insertions, 8 deletions
diff --git a/addons/web/static/src/js/views/graph/graph_renderer.js b/addons/web/static/src/js/views/graph/graph_renderer.js index 02fdb102358..3cc8ce80eba 100644 --- a/addons/web/static/src/js/views/graph/graph_renderer.js +++ b/addons/web/static/src/js/views/graph/graph_renderer.js @@ -621,11 +621,7 @@ return AbstractRenderer.extend({ } var dataPoints = this._filterDataPoints(); if (!dataPoints.length && this.state.mode !== 'pie') { - this.$el.append(qweb.render('GraphView.error', { - title: _t("No data to display"), - description: _t("Try to add some records, or make sure that " + - "there is no active filter in the search bar."), - })); + this.$el.append(qweb.render('View.NoContentHelper')); } else if (this.isInDOM) { // only render the graph if the widget is already in the DOM (this // happens typically after an update), otherwise, it will be @@ -758,7 +754,7 @@ return AbstractRenderer.extend({ }); if (someNegative && !allNegative) { this.$el.empty(); - this.$el.append(qweb.render('GraphView.error', { + this.$el.append(qweb.render('View.NoContentHelper', { title: _t("Invalid data"), description: _t("Pie chart cannot mix positive and negative numbers. " + "Try to change your domain to only display positive results"), @@ -767,7 +763,7 @@ return AbstractRenderer.extend({ } if (allZero && !this.isEmbedded && this.state.origins.length === 1) { this.$el.empty(); - this.$el.append(qweb.render('GraphView.error', { + this.$el.append(qweb.render('View.NoContentHelper', { title: _t("Invalid data"), description: _t("Pie chart cannot display all zero numbers.. " + "Try to change your domain to display positive results"), diff --git a/addons/web/static/src/js/views/pivot/pivot_renderer.js b/addons/web/static/src/js/views/pivot/pivot_renderer.js index 29c3424e141..f2d2d24bea8 100644 --- a/addons/web/static/src/js/views/pivot/pivot_renderer.js +++ b/addons/web/static/src/js/views/pivot/pivot_renderer.js @@ -77,7 +77,10 @@ var PivotRenderer = AbstractRenderer.extend({ _render: function () { if (!this._hasContent()) { // display the nocontent helper - this._replaceElement(QWeb.render('PivotView.nodata')); + this._replaceElement(QWeb.render('View.NoContentHelper', { + description: _t("Try to add some records, or make sure that there is at least " + + "one measure and no active filter in the search bar."), + })); return this._super.apply(this, arguments); } |
