[Rt-commit] rt branch, 4.4/numeric-eq-warning-in-circular-links-check, created. rt-4.4.0rc2-16-gb65e1e2

Shawn Moore shawn at bestpractical.com
Tue Dec 8 12:46:46 EST 2015


The branch, 4.4/numeric-eq-warning-in-circular-links-check has been created
        at  b65e1e2b1060c33169e0d93862b9220cfa51f983 (commit)

- Log -----------------------------------------------------------------
commit b65e1e2b1060c33169e0d93862b9220cfa51f983
Author: sunnavy <sunnavy at bestpractical.com>
Date:   Thu Nov 5 02:03:30 2015 +0800

    use eq instead because link values could contain non-numeric chars
    
    besides tickets, we have article, asset and even external tickets.
    Fixes: I#31399

diff --git a/lib/RT/Record.pm b/lib/RT/Record.pm
index ae0262d..40e8d8e 100644
--- a/lib/RT/Record.pm
+++ b/lib/RT/Record.pm
@@ -1412,7 +1412,7 @@ sub _AddLink {
             LinkType  => $args{'Type'},
             Direction => $direction eq 'Target' ? 'Base' : 'Target',
         );
-        if ( grep { $_->id == ( $direction eq 'Target' ? $args{'Base'} : $args{'Target'} ) } @tickets ) {
+        if ( grep { $_->id eq ( $direction eq 'Target' ? $args{'Base'} : $args{'Target'} ) } @tickets ) {
             return ( 0, $self->loc("Refused to add link which would create a circular relationship") );
         }
     }

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


More information about the rt-commit mailing list