[IMP] web: list: import attrs

The 'import' attrs is valid according to the rng definition of the
list view. It is used in a few views in Odoo. However, it didn't
actually work (in most cases, it worked by chance because it is
set to "0" in addition to create="0", and create="0" already
disables the import feature).

With this ref, the 'import' attrs is taken into account and we
can now disable the import feature even if the create one is
enabled. Moreover, it is now documented.

1 files changed, 5 insertions, 3 deletions

diff --git a/addons/base_import/static/src/js/import_buttons.js b/addons/base_import/static/src/js/import_buttons.js
index f6d246a8586..89baedc8113 100644
--- a/addons/base_import/static/src/js/import_buttons.js
+++ b/addons/base_import/static/src/js/import_buttons.js
@@ -13,11 +13,13 @@ var ImportViewMixin = {
     /**
      * @override
      * @param {Object} params
-     * @param {boolean} [params.import_enabled=true] determine if the import
-     *   button is visible (in the control panel)
+     * @param {boolean} [params.import_enabled=true] set to false to disable
+     *   the Import feature (no 'Import' button in the control panel). Can also
+     *   be disabled with 'import' attrs set to '0' in the arch.
      */
     init: function (viewInfo, params) {
-        var importEnabled = 'import_enabled' in params ? params.import_enabled : true;
+        var importEnabled = !!JSON.parse(this.arch.attrs.import || '1') &&
+                            ('import_enabled' in params ? params.import_enabled : true);
         this.controllerParams.importEnabled = importEnabled && !config.device.isMobile;
     },
 };