Item14202: Bad logic in the link tracking check

foswiki · Nov 11, 2016 · 7c45a41 · 7c45a41
1 parent 64a9660
commit 7c45a41
Showing 1 changed file with 10 additions and 11 deletions.
diff --git a/core/lib/Foswiki/PageCache.pm b/core/lib/Foswiki/PageCache.pm
@@ -470,19 +470,18 @@ sub addTopicRef {
 
     #Foswiki::Func::writeDebug( "addTopicRef $webRef.$topicRef\n" ) if TRACE;
 
-    if ( defined $Foswiki::cfg{Cache}{TrackInternalLinks} ) {
-
-        return if ( $Foswiki::cfg{Cache}{TrackInternalLinks} eq 'off' );
-
-        my $session = $Foswiki::Plugins::SESSION;
-        return
-          unless (
-            ( $Foswiki::cfg{Cache}{TrackInternalLinks} eq 'authenticated' )
-            && $session->inContext('authenticated') );
-    }
+    my $session = $Foswiki::Plugins::SESSION;
 
-    return $this->addDependency( $webRef, $topicRef );
+    return $this->addDependency( $webRef, $topicRef )
+      if ( !defined $Foswiki::cfg{Cache}{TrackInternalLinks}
+        || ( $Foswiki::cfg{Cache}{TrackInternalLinks} eq 'on' )
+        || ( $Foswiki::cfg{Cache}{TrackInternalLinks} eq 'authenticated' )
+        && $session->inContext('authenticated') );
 
+    # If we reach here, either:
+    #   - It is a guest session and TrackInternalLinks was set to authenticated
+    #   - TrackInternalLinks is set to off (or some unexpected value.
+    return;
 }
 
 =begin TML