[Bps-public-commit] RT-Extension-MergeUsers branch, master, updated. 1.06-1-gccb8e95

? sunnavy sunnavy at bestpractical.com
Tue Oct 13 16:22:01 EDT 2020


The branch, master has been updated
       via  ccb8e956304ee59a07ed61e747c8e3019cf8b49d (commit)
      from  e900bc02055990e64bea128634289777ec3fcf09 (commit)

Summary of changes:
 lib/RT/Extension/MergeUsers.pm | 3 ---
 1 file changed, 3 deletions(-)

- Log -----------------------------------------------------------------
commit ccb8e956304ee59a07ed61e747c8e3019cf8b49d
Author: sunnavy <sunnavy at bestpractical.com>
Date:   Wed Oct 14 04:11:45 2020 +0800

    Fall back to the original value in case it's not a user
    
    As watchers could be groups too, if we can't find user from the value,
    the value could be a group, in which case we shouldn't touch it.

diff --git a/lib/RT/Extension/MergeUsers.pm b/lib/RT/Extension/MergeUsers.pm
index 2bd3abd..848214d 100644
--- a/lib/RT/Extension/MergeUsers.pm
+++ b/lib/RT/Extension/MergeUsers.pm
@@ -655,9 +655,6 @@ sub TweakRoleLimitArgs {
                 $args{VALUE} = $o->id;
             }
         }
-        else {
-            $args{VALUE} = 0;
-        }
     }
     return %args;
 }

-----------------------------------------------------------------------


More information about the Bps-public-commit mailing list