From f3a811b2b233ad2aaeaa623baaeefe5637b468bf Mon Sep 17 00:00:00 2001 From: Aaron Bohy Date: Wed, 22 Feb 2017 14:21:37 +0100 Subject: [PATCH] [FIX] web_settings_dashboard: remove useless imports It also allows this module to be loaded in the test environment, where the module 'web.web_client' is missing. Also fixes some linting warnings. --- .../static/src/js/dashboard.js | 12 +++++------- 1 file changed, 5 insertions(+), 7 deletions(-) diff --git a/addons/web_settings_dashboard/static/src/js/dashboard.js b/addons/web_settings_dashboard/static/src/js/dashboard.js index cfd37bb50f0..27a552fc0d0 100644 --- a/addons/web_settings_dashboard/static/src/js/dashboard.js +++ b/addons/web_settings_dashboard/static/src/js/dashboard.js @@ -2,13 +2,11 @@ odoo.define('web_settings_dashboard', function (require) { "use strict"; var core = require('web.core'); -var Widget = require('web.Widget'); -var Model = require('web.Model'); -var session = require('web.session'); -var PlannerCommon = require('web.planner.common'); var framework = require('web.framework'); -var webclient = require('web.web_client'); +var Model = require('web.Model'); +var PlannerCommon = require('web.planner.common'); var PlannerDialog = PlannerCommon.PlannerDialog; +var Widget = require('web.Widget'); var QWeb = core.qweb; var _t = core._t; @@ -16,7 +14,7 @@ var _t = core._t; var Dashboard = Widget.extend({ template: 'DashboardMain', - init: function(parent, data){ + init: function(){ this.all_dashboards = ['apps', 'invitations', 'planner', 'share']; return this._super.apply(this, arguments); }, @@ -28,7 +26,7 @@ var Dashboard = Widget.extend({ load: function(dashboards){ var self = this; var loading_done = new $.Deferred(); - session.rpc("/web_settings_dashboard/data", {}).then(function (data) { + this.performRPC("/web_settings_dashboard/data", {}).then(function (data) { // Load each dashboard var all_dashboards_defs = []; _.each(dashboards, function(dashboard) {