[Bps-public-commit] RT-Extension-MandatoryOnTransition branch, master, updated. 0.18-1-gc20612b

? sunnavy sunnavy at bestpractical.com
Tue Jul 2 14:01:37 EDT 2019


The branch, master has been updated
       via  c20612be0158141454103d764943444463dfb1e3 (commit)
      from  40d25932e9c95b5d550dfbaedd2edd5c9f7b6c55 (commit)

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

- Log -----------------------------------------------------------------
commit c20612be0158141454103d764943444463dfb1e3
Author: Craig Kaiser <craig at bestpractical.com>
Date:   Tue Jul 2 13:48:07 2019 -0400

    Bail on RequiredFields if no status or queue change occurring
    
    Only want to check for mandatory fields when there is a queue or status
    change taking place. If no change is occuring then we need to bail or we
    could return incorrect mandatory messages.

diff --git a/lib/RT/Extension/MandatoryOnTransition.pm b/lib/RT/Extension/MandatoryOnTransition.pm
index 97bf52c..54e63d5 100644
--- a/lib/RT/Extension/MandatoryOnTransition.pm
+++ b/lib/RT/Extension/MandatoryOnTransition.pm
@@ -341,7 +341,7 @@ sub RequiredFields {
 
     my ($from_queue, $to_queue) = ($args{Queue}, $args{ToQueue} || $args{Queue});
 
-    return ([], [], []) unless ($from and $to) or ($from_queue and $to_queue );
+    return ([], [], []) unless ($from and $to and $from ne $to) or ($from_queue and $to_queue and $from_queue ne $to_queue);
 
     my %config = ();
     %config = $self->Config($args{Queue});

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


More information about the Bps-public-commit mailing list