From 96b78adf45be2c27ae514d1a3793a9433ef8d71f Mon Sep 17 00:00:00 2001
From: Christopher Speller <crspeller@gmail.com>
Date: Thu, 14 Nov 2019 10:32:08 -0800
Subject: [PATCH] Adding ability to disable attachment buttons and fields.
 (#3541)

* Adding ability to disable attachment buttons and fields.

* Revert package lock

* Use changeOpacity
---
 .../autocomplete_selector/autocomplete_selector.js    |  8 +++++++-
 .../action_button/action_button.js                    | 11 +++++++++--
 .../message_attachments/action_menu/action_menu.js    |  3 +++
 .../action_menu/action_menu.test.js                   | 10 ++++++++++
 .../message_attachments/attachment_actions.js         |  2 ++
 5 files changed, 31 insertions(+), 3 deletions(-)

diff --git a/app/components/autocomplete_selector/autocomplete_selector.js b/app/components/autocomplete_selector/autocomplete_selector.js
index 83abb8488..c5b8c1499 100644
--- a/app/components/autocomplete_selector/autocomplete_selector.js
+++ b/app/components/autocomplete_selector/autocomplete_selector.js
@@ -36,6 +36,7 @@ export default class AutocompleteSelector extends PureComponent {
         errorText: PropTypes.node,
         roundedBorders: PropTypes.bool,
         isLandscape: PropTypes.bool.isRequired,
+        disabled: PropTypes.bool,
     };
 
     static contextTypes = {
@@ -119,6 +120,7 @@ export default class AutocompleteSelector extends PureComponent {
             showRequiredAsterisk,
             roundedBorders,
             isLandscape,
+            disabled,
         } = this.props;
         const {selectedText} = this.state;
         const style = getStyleSheet(theme);
@@ -188,9 +190,10 @@ export default class AutocompleteSelector extends PureComponent {
                     {labelContent}
                 </View>
                 <TouchableWithFeedback
-                    style={style.flex}
+                    style={disabled ? style.disabled : null}
                     onPress={this.goToSelectorScreen}
                     type={'opacity'}
+                    disabled={disabled}
                 >
                     <View style={inputStyle}>
                         <Text
@@ -284,5 +287,8 @@ const getStyleSheet = makeStyleSheetFromTheme((theme) => {
             color: theme.errorTextColor,
             fontSize: 14,
         },
+        disabled: {
+            opacity: 0.5,
+        },
     };
 });
diff --git a/app/components/message_attachments/action_button/action_button.js b/app/components/message_attachments/action_button/action_button.js
index a604f6995..760a101d8 100644
--- a/app/components/message_attachments/action_button/action_button.js
+++ b/app/components/message_attachments/action_button/action_button.js
@@ -6,7 +6,7 @@ import PropTypes from 'prop-types';
 import Button from 'react-native-button';
 
 import {preventDoubleTap} from 'app/utils/tap';
-import {makeStyleSheetFromTheme} from 'app/utils/theme';
+import {makeStyleSheetFromTheme, changeOpacity} from 'app/utils/theme';
 import ActionButtonText from './action_button_text';
 
 export default class ActionButton extends PureComponent {
@@ -19,6 +19,7 @@ export default class ActionButton extends PureComponent {
         postId: PropTypes.string.isRequired,
         theme: PropTypes.object.isRequired,
         cookie: PropTypes.string.isRequired,
+        disabled: PropTypes.bool,
     };
 
     handleActionPress = preventDoubleTap(() => {
@@ -27,13 +28,15 @@ export default class ActionButton extends PureComponent {
     }, 4000);
 
     render() {
-        const {name, theme} = this.props;
+        const {name, theme, disabled} = this.props;
         const style = getStyleSheet(theme);
 
         return (
             <Button
                 containerStyle={style.button}
+                disabledContainerStyle={style.buttonDisabled}
                 onPress={this.handleActionPress}
+                disabled={disabled}
             >
                 <ActionButtonText
                     message={name}
@@ -49,6 +52,7 @@ const getStyleSheet = makeStyleSheetFromTheme((theme) => {
         button: {
             borderRadius: 2,
             backgroundColor: theme.buttonBg,
+            opacity: 1,
             alignItems: 'center',
             marginBottom: 2,
             marginRight: 5,
@@ -56,6 +60,9 @@ const getStyleSheet = makeStyleSheetFromTheme((theme) => {
             paddingHorizontal: 10,
             paddingVertical: 7,
         },
+        buttonDisabled: {
+            backgroundColor: changeOpacity(theme.buttonBg, 0.3),
+        },
         text: {
             color: theme.buttonColor,
             fontSize: 12,
diff --git a/app/components/message_attachments/action_menu/action_menu.js b/app/components/message_attachments/action_menu/action_menu.js
index 3ea8d46fd..8157556d5 100644
--- a/app/components/message_attachments/action_menu/action_menu.js
+++ b/app/components/message_attachments/action_menu/action_menu.js
@@ -18,6 +18,7 @@ export default class ActionMenu extends PureComponent {
         options: PropTypes.arrayOf(PropTypes.object),
         postId: PropTypes.string.isRequired,
         selected: PropTypes.object,
+        disabled: PropTypes.bool,
     };
 
     constructor(props) {
@@ -62,6 +63,7 @@ export default class ActionMenu extends PureComponent {
             name,
             dataSource,
             options,
+            disabled,
         } = this.props;
         const {selected} = this.state;
 
@@ -72,6 +74,7 @@ export default class ActionMenu extends PureComponent {
                 options={options}
                 selected={selected}
                 onSelected={this.handleSelect}
+                disabled={disabled}
             />
         );
     }
diff --git a/app/components/message_attachments/action_menu/action_menu.test.js b/app/components/message_attachments/action_menu/action_menu.test.js
index 4037f7070..517f15f05 100644
--- a/app/components/message_attachments/action_menu/action_menu.test.js
+++ b/app/components/message_attachments/action_menu/action_menu.test.js
@@ -52,4 +52,14 @@ describe('ActionMenu', () => {
 
         expect(wrapper.state('selected')).toBe(props.selected);
     });
+
+    test('disabled works', () => {
+        const props = {
+            ...baseProps,
+            disabled: true,
+        };
+        const wrapper = shallow(<ActionMenu {...props}/>);
+
+        expect(wrapper.props().disabled).toBe(true);
+    });
 });
diff --git a/app/components/message_attachments/attachment_actions.js b/app/components/message_attachments/attachment_actions.js
index 9f4203fbd..c526aec85 100644
--- a/app/components/message_attachments/attachment_actions.js
+++ b/app/components/message_attachments/attachment_actions.js
@@ -41,6 +41,7 @@ export default class AttachmentActions extends PureComponent {
                         defaultOption={action.default_option}
                         options={action.options}
                         postId={postId}
+                        disabled={action.disabled}
                     />
                 );
                 break;
@@ -53,6 +54,7 @@ export default class AttachmentActions extends PureComponent {
                         cookie={action.cookie}
                         name={action.name}
                         postId={postId}
+                        disabled={action.disabled}
                     />
                 );
                 break;