Item13117: CHECK is no longer an array.

SMELL: I thought the "also" setting was supposed to trigger a 2nd checker. If A changes, also check B. However the one place this is used - AuthScripts, doesn't seem to behave that way. Changing the LoginManager setting doesn't trigger the AuthScripts checker.
foswiki · Jan 14, 2015 · 1fbf9c9 · 1fbf9c9
1 parent 8edbfc5
commit 1fbf9c9
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/core/lib/Foswiki/Configure/Value.pm b/core/lib/Foswiki/Configure/Value.pm
@@ -476,13 +476,13 @@ sub find_also_dependencies {
     foreach my $slave ( split( /[\s,]+/, $this->{CHECK_ON_CHANGE} ) ) {
         my $vob = $root->getValueObject($slave);
         next unless ($vob);
-        my $check = $vob->{CHECK}->[0];
+        my $check = $vob->{CHECK};
         if ($check) {
             $check->{also} ||= [];
             push( @{ $check->{also} }, $slave );
         }
         else {
-            $vob->{CHECK}->[0] = { also => [$slave] };
+            $vob->{CHECK} = { also => [$slave] };
         }
     }
 }