You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airavata.apache.org by ma...@apache.org on 2021/02/23 17:36:55 UTC

[airavata-django-portal] branch develop updated: AIRAVATA-3412 Only update admins to MANAGE_SHARING if owner

This is an automated email from the ASF dual-hosted git repository.

machristie pushed a commit to branch develop
in repository https://gitbox.apache.org/repos/asf/airavata-django-portal.git


The following commit(s) were added to refs/heads/develop by this push:
     new a14f6ad  AIRAVATA-3412 Only update admins to MANAGE_SHARING if owner
     new 64f3ce1  Merge branch 'airavata-3412' into develop
a14f6ad is described below

commit a14f6ad55b957fa2931d741b81806b9f6be8cf68
Author: Marcus Christie <ma...@apache.org>
AuthorDate: Tue Feb 23 12:36:16 2021 -0500

    AIRAVATA-3412 Only update admins to MANAGE_SHARING if owner
---
 django_airavata/static/common/js/components/ShareButton.vue | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/django_airavata/static/common/js/components/ShareButton.vue b/django_airavata/static/common/js/components/ShareButton.vue
index 1bee9e9..c7d2d33 100644
--- a/django_airavata/static/common/js/components/ShareButton.vue
+++ b/django_airavata/static/common/js/components/ShareButton.vue
@@ -244,7 +244,11 @@ export default {
           this.localSharedEntity.addGroup({ group: this.readOnlyAdminsGroup });
           this.emitUnsavedEvent();
         }
-        if (this.localSharedEntity.entityId && this.autoAddAdminGroups) {
+        if (
+          this.localSharedEntity.entityId &&
+          this.autoAddAdminGroups &&
+          this.localSharedEntity.isOwner
+        ) {
           // AIRAVATA-3297 Admins group used to get WRITE permission, but the
           // new default is MANAGE_SHARING so update if necessary
           // Since autoAddAdminGroups is true, there should already be an adminsGroupPermission