From 36be7f468c2c190be1e7078df7a517e6669ff67f Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=E7=8E=8B=E6=80=A1=E6=A0=8B?= <wangyidong@cnezsoft.com>
Date: Fri, 10 Dec 2021 14:14:25 +0800
Subject: [PATCH]  * code for task #17340.

---
 module/gitlab/js/managegroupmembers.js         | 6 +++++-
 module/gitlab/view/managegroupmembers.html.php | 6 +++---
 2 files changed, 8 insertions(+), 4 deletions(-)

diff --git a/module/gitlab/js/managegroupmembers.js b/module/gitlab/js/managegroupmembers.js
index c67e34038e..a9391db43b 100644
--- a/module/gitlab/js/managegroupmembers.js
+++ b/module/gitlab/js/managegroupmembers.js
@@ -47,15 +47,19 @@ function deleteItem(obj)
 
 $(document).on('change', '[id^="levels"]', function()
 {
+    $tr   = $(this).closest('tr');
     $next = $(this).closest('td').next()
-    if($(this).val() == '50')
+    var ownerLevel = 50;
+    if($(this).val() == ownerLevel)
     {
         $next.prepend('<input type="text" value="" class="form-control disabled" disabled autocomplete="off" />');
         $next.find('[id^="expires"]').addClass('hidden');
+        $tr.find('a[onclick^="deleteItem"]').addClass('disabled');
     }
     else
     {
         $next.find('[id^="expires"]').removeClass('hidden');
         $next.find('input.disabled').remove();
+        $tr.find('a[onclick^="deleteItem"]').removeClass('disabled');
     }
 })
diff --git a/module/gitlab/view/managegroupmembers.html.php b/module/gitlab/view/managegroupmembers.html.php
index 3deab1e37b..08d5891c51 100644
--- a/module/gitlab/view/managegroupmembers.html.php
+++ b/module/gitlab/view/managegroupmembers.html.php
@@ -19,14 +19,14 @@
         </tr>
       </thead>
       <tbody>
+        <?php $ownerLevel = 50;?>
         <?php $i = 0;?>
         <?php foreach($currentMembers as $member):?>
         <tr>
           <td><?php echo html::input("names[$i]", $member->name, "class='form-control' readonly");?></td>
           <td><?php echo html::select("levels[$i]", array(''=>'') + $this->lang->gitlab->accessLevels, $member->access_level, "class='form-control chosen'");?></td>
           <td>
-            <?php /* 50 represents the owner role. */ ?>
-            <?php if($member->access_level == '50'):?>
+            <?php if($member->access_level == $ownerLevel):?>
             <input type="text" value="" class="form-control disabled" disabled autocomplete="off" />
             <?php echo html::input("expires[$i]", $member->expires_at, "class='form-control form-date hidden'");?>
             <?php else:?>
@@ -36,7 +36,7 @@
           </td>
           <td class='c-actions text-center'>
             <?php echo html::a('javascript:;', "<i class='icon-plus'></i>", '', "onclick='addItem(this)' class='btn btn-link'");?>
-            <?php echo html::a('javascript:;', "<i class='icon icon-close'></i>", '', "onclick='deleteItem(this)' class='btn btn-link'");?>
+            <?php echo html::a('javascript:;', "<i class='icon icon-close'></i>", '', "onclick='deleteItem(this)' class='btn btn-link" . ($member->access_level == $ownerLevel ? ' disabled' : '') . "'");?>
           </td>
         </tr>
         <?php $i ++;?>