[Bps-public-commit] RT-Extension-MergeUsers branch master updated. 1.08-2-g2169ab7

BPS Git Server git at git.bestpractical.com
Fri Feb 9 15:42:35 UTC 2024


This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "RT-Extension-MergeUsers".

The branch, master has been updated
       via  2169ab73f44ab8e575b553141e053bda99b61062 (commit)
      from  cb4f0cb8aa907e5d487f6fb5639f8924ff41a839 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit 2169ab73f44ab8e575b553141e053bda99b61062
Author: sunnavy <sunnavy at bestpractical.com>
Date:   Fri Feb 9 10:35:11 2024 -0500

    Fix uninitialized warnings cased by undefined FIELD
    
    FIELD is an optional argument, e.g. if FUNCTION is provided already.

diff --git a/lib/RT/Extension/MergeUsers.pm b/lib/RT/Extension/MergeUsers.pm
index 9002cdd..f207078 100644
--- a/lib/RT/Extension/MergeUsers.pm
+++ b/lib/RT/Extension/MergeUsers.pm
@@ -724,7 +724,7 @@ sub TweakRoleLimitArgs {
     *Limit = sub {
         my $self = shift;
         my %args = @_;
-        if (   $args{FIELD} eq 'Owner'
+        if (   ( $args{FIELD} // '' ) eq 'Owner'
             && ( $args{OPERATOR} || '=' ) =~ /^!?=$/
             && $args{VALUE} =~ /^(\d+)$/ )
         {

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

Summary of changes:
 lib/RT/Extension/MergeUsers.pm | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)


hooks/post-receive
-- 
RT-Extension-MergeUsers


More information about the Bps-public-commit mailing list