From a16f6647473c5ced1acb4b7043a041b30e1d96b7 Mon Sep 17 00:00:00 2001
From: Max Chopart <max.chopart@gmail.com>
Date: Wed, 13 Dec 2023 16:01:05 +0100
Subject: [PATCH] [Upd #18] Updated buttons hiding dependent on user role

---
 js/components/record/Record.js | 44 ++++++++++++++++------------------
 1 file changed, 21 insertions(+), 23 deletions(-)

diff --git a/js/components/record/Record.js b/js/components/record/Record.js
index d62a9afe..63856316 100644
--- a/js/components/record/Record.js
+++ b/js/components/record/Record.js
@@ -106,40 +106,38 @@ class Record extends React.Component {
         const {record, recordSaved, formgen} = this.props;
 
         return <div className="mt-3 text-center">
-            {this._renderRoleSpecificButtons()}
+            <Button className="mx-1" variant='danger' size='sm'
+                    disabled={formgen.status === ACTION_STATUS.PENDING || recordSaved.status === ACTION_STATUS.PENDING
+                        || !this.state.isFormValid || !record.state.isComplete()|| record.phase === RECORD_PHASE.REJECTED}
+                    hidden={record.phase === RECORD_PHASE.COMPLETED ||
+                        EXTENSIONS === EXTENSION_CONSTANTS.OPERATOR}
+                    onClick={this.props.handlers.onReject}>
+                {this.i18n('reject')}{recordSaved.status === ACTION_STATUS.PENDING && <LoaderSmall/>}
+            </Button>
+
+            <Button className="mx-1" variant='success' size='sm'
+                    disabled={formgen.status === ACTION_STATUS.PENDING || recordSaved.status === ACTION_STATUS.PENDING
+                        || !this.state.isFormValid || !record.state.isComplete() || record.phase === RECORD_PHASE.COMPLETED}
+                    hidden={record.phase === RECORD_PHASE.COMPLETED ||
+                        record.phase === RECORD_PHASE.PUBLISHED}
+                    onClick={this.props.handlers.onComplete}>
+                {this.i18n('complete')}{recordSaved.status === ACTION_STATUS.PENDING && <LoaderSmall/>}
+            </Button>
+
             <Button className="mx-1" variant='success' size='sm'
                     disabled={formgen.status === ACTION_STATUS.PENDING || recordSaved.status === ACTION_STATUS.PENDING
                         || !this.state.isFormValid || !record.state.isComplete()}
+                    hidden={(record.phase === RECORD_PHASE.COMPLETED && !this._isAdmin()) ||
+                        record.phase === RECORD_PHASE.PUBLISHED}
                     onClick={this.props.handlers.onSave}>
                 {this.i18n('save')}{recordSaved.status === ACTION_STATUS.PENDING && <LoaderSmall/>}
             </Button>
             <Button className="mx-1" variant='link' size='sm'
+                    hidden={record.phase === RECORD_PHASE.COMPLETED || record.phase === RECORD_PHASE.PUBLISHED}
                     onClick={this.props.handlers.onCancel}>{this.i18n('cancel')}</Button>
         </div>
     }
 
-    _renderRoleSpecificButtons() {
-        const {record, recordSaved, formgen} = this.props;
-
-        return <>
-            {EXTENSIONS === EXTENSION_CONSTANTS.SUPPLIER &&
-                <Button className="mx-1" variant='danger' size='sm'
-                        disabled={formgen.status === ACTION_STATUS.PENDING || recordSaved.status === ACTION_STATUS.PENDING
-                            || !this.state.isFormValid || !record.state.isComplete()|| record.phase === RECORD_PHASE.REJECTED}
-                        onClick={this.props.handlers.onReject}>
-                    {this.i18n('reject')}{recordSaved.status === ACTION_STATUS.PENDING && <LoaderSmall/>}
-                </Button>
-            }
-            {EXTENSIONS === EXTENSION_CONSTANTS.OPERATOR &&
-                <Button className="mx-1" variant='success' size='sm'
-                        disabled={formgen.status === ACTION_STATUS.PENDING || recordSaved.status === ACTION_STATUS.PENDING
-                            || !this.state.isFormValid || !record.state.isComplete() || record.phase === RECORD_PHASE.COMPLETED}
-                        onClick={this.props.handlers.onComplete}>
-                    {this.i18n('complete')}{recordSaved.status === ACTION_STATUS.PENDING && <LoaderSmall/>}
-                </Button>}
-        </>;
-    }
-
     _renderInstitution() {
         const record = this.props.record;
         if (!record.institution) {