From c1f8c4abdad5c3a604a02f64cc46fc96e6fd5a57 Mon Sep 17 00:00:00 2001
From: hote <hote@odoo.com>
Date: Sun, 8 Jan 2023 15:53:33 +0000
Subject: [PATCH] [IMP] web: add disabled props to CopyButton and CopyClipboard
 field

The CopyClipboardButton Component acts as a button. As such it is
interesting to add the possibility to make it disabled when needed.
This commit adds the possibility to add the attribute `disabled="1"`
to the xml code and let it function as one should expect.

See task 2672713 for a practical use.

closes odoo/odoo#111724

Signed-off-by: Luca Vitali <luvi@odoo.com>
---
 .../fields/copy_clipboard/copy_button.js      |  1 +
 .../fields/copy_clipboard/copy_button.xml     |  1 +
 .../copy_clipboard/copy_clipboard_field.js    | 17 +++++-
 .../copy_clipboard/copy_clipboard_field.xml   |  2 +-
 .../fields/copy_clipboard_field_tests.js      | 60 ++++++++++++++++++-
 5 files changed, 74 insertions(+), 7 deletions(-)

diff --git a/addons/web/static/src/views/fields/copy_clipboard/copy_button.js b/addons/web/static/src/views/fields/copy_clipboard/copy_button.js
index f0484dcc4da..c797d0eb364 100644
--- a/addons/web/static/src/views/fields/copy_clipboard/copy_button.js
+++ b/addons/web/static/src/views/fields/copy_clipboard/copy_button.js
@@ -42,6 +42,7 @@ CopyButton.template = "web.CopyButton";
 CopyButton.props = {
     className: { type: String, optional: true },
     copyText: { type: String, optional: true },
+    disabled: { type: Boolean, optional: true },
     successText: { type: String, optional: true },
     content: { type: [String, Object], optional: true },
 };
diff --git a/addons/web/static/src/views/fields/copy_clipboard/copy_button.xml b/addons/web/static/src/views/fields/copy_clipboard/copy_button.xml
index dcf05f77db0..f96bcee6cc2 100644
--- a/addons/web/static/src/views/fields/copy_clipboard/copy_button.xml
+++ b/addons/web/static/src/views/fields/copy_clipboard/copy_button.xml
@@ -5,6 +5,7 @@
         <button
             class="text-nowrap"
             t-ref="button"
+            t-att-disabled="props.disabled"
             t-attf-class="btn btn-primary o_clipboard_button {{ props.className || '' }}"
             t-on-click.stop="onClick"
         >
diff --git a/addons/web/static/src/views/fields/copy_clipboard/copy_clipboard_field.js b/addons/web/static/src/views/fields/copy_clipboard/copy_clipboard_field.js
index 020c7ec0d95..c2f77211a0e 100644
--- a/addons/web/static/src/views/fields/copy_clipboard/copy_clipboard_field.js
+++ b/addons/web/static/src/views/fields/copy_clipboard/copy_clipboard_field.js
@@ -1,7 +1,10 @@
 /** @odoo-module **/
 
 import { _lt } from "@web/core/l10n/translation";
+import { evaluateExpr } from "@web/core/py_js/py";
+import { evalDomain } from "@web/views/utils";
 import { registry } from "@web/core/registry";
+import { omit } from "@web/core/utils/objects";
 
 import { CopyButton } from "./copy_button";
 import { UrlField } from "../url/url_field";
@@ -20,19 +23,27 @@ class CopyClipboardField extends Component {
         return `o_btn_${this.props.type}_copy btn-sm`;
     }
     get fieldProps() {
-        const fieldProps = {...this.props};
-        delete fieldProps.string;
-        return fieldProps;
+        return omit(this.props, "string", "disabledExpr");
+    }
+    get disabled() {
+        const context = this.props.record.evalContext;
+        const evaluated = this.props.disabledExpr ? evaluateExpr(this.props.disabledExpr) : false;
+        if (evaluated instanceof Array) {
+            return evalDomain(evaluated, context);
+        }
+        return Boolean(evaluated);
     }
 }
 CopyClipboardField.template = "web.CopyClipboardField";
 CopyClipboardField.props = {
     ...standardFieldProps,
     string: { type: String, optional: true },
+    disabledExpr: { type: String, optional: true },
 };
 CopyClipboardField.extractProps = ({ attrs }) => {
     return {
         string: attrs.string,
+        disabledExpr: attrs.disabled,
     };
 };
 
diff --git a/addons/web/static/src/views/fields/copy_clipboard/copy_clipboard_field.xml b/addons/web/static/src/views/fields/copy_clipboard/copy_clipboard_field.xml
index 6d789ce5954..48ee7ce4063 100644
--- a/addons/web/static/src/views/fields/copy_clipboard/copy_clipboard_field.xml
+++ b/addons/web/static/src/views/fields/copy_clipboard/copy_clipboard_field.xml
@@ -9,7 +9,7 @@
     </t>
 
     <t t-name="web.CopyClipboardButtonField" owl="1">
-        <CopyButton t-if="props.value" className="copyButtonClassName" content="props.value" copyText="copyText" successText="successText"/>
+        <CopyButton t-if="props.value" className="copyButtonClassName" content="props.value" copyText="copyText" disabled="disabled" successText="successText"/>
     </t>
 
 </templates>
diff --git a/addons/web/static/tests/views/fields/copy_clipboard_field_tests.js b/addons/web/static/tests/views/fields/copy_clipboard_field_tests.js
index 89daeb46345..21b5ef13e46 100644
--- a/addons/web/static/tests/views/fields/copy_clipboard_field_tests.js
+++ b/addons/web/static/tests/views/fields/copy_clipboard_field_tests.js
@@ -2,7 +2,13 @@
 
 import { browser } from "@web/core/browser/browser";
 import { registry } from "@web/core/registry";
-import { click, getFixture, nextTick, patchWithCleanup } from "@web/../tests/helpers/utils";
+import {
+    click,
+    getFixture,
+    editInput,
+    nextTick,
+    patchWithCleanup,
+} from "@web/../tests/helpers/utils";
 import { makeView, setupViewRegistries } from "@web/../tests/views/helpers";
 
 const serviceRegistry = registry.category("services");
@@ -222,9 +228,9 @@ QUnit.module("Fields", (hooks) => {
         );
     });
 
-    QUnit.module("CopyToClipboardButtonField");
+    QUnit.module("CopyClipboardButtonField");
 
-    QUnit.test("CopyToClipboardButtonField in form view", async function (assert) {
+    QUnit.test("CopyClipboardButtonField in form view", async function (assert) {
         patchWithCleanup(browser, {
             navigator: {
                 clipboard: {
@@ -266,4 +272,52 @@ Ho-ho-hoooo Merry Christmas`,
             "yop",
         ]);
     });
+
+    QUnit.test("CopyClipboardButtonField can be disabled", async function (assert) {
+        patchWithCleanup(browser, {
+            navigator: {
+                clipboard: {
+                    writeText: (text) => {
+                        assert.step(text);
+                        return Promise.resolve();
+                    },
+                },
+            },
+        });
+
+        await makeView({
+            serverData,
+            type: "form",
+            resModel: "partner",
+            arch: `
+                <form>
+                    <sheet>
+                        <div>
+                            <field name="text_field" disabled="1" widget="CopyClipboardButton"/>
+                            <field name="char_field" disabled="[('char_field', '=', 'yop')]" widget="CopyClipboardButton"/>
+                            <field name="char_field" widget="char"/>
+                        </div>
+                    </sheet>
+                </form>`,
+            resId: 1,
+        });
+
+        assert.containsOnce(
+            target,
+            ".o_clipboard_button.o_btn_text_copy[disabled]",
+            "The inner button should be disabled."
+        );
+        assert.containsOnce(
+            target,
+            ".o_clipboard_button.o_btn_char_copy[disabled]",
+            "The inner button should be disabled."
+        );
+
+        await editInput(target, ".o_input", "yip");
+        assert.containsNone(
+            target,
+            ".o_clipboard_button.o_btn_char_copy[disabled]",
+            "The inner button should not be disabled."
+        );
+    });
 });