Possible multiplication of $fResult in $result
authorHarm Hermsen <mail@hhermsen.nl>
Wed, 17 Sep 2014 13:34:03 +0000 (15:34 +0200)
committerTim Otten <totten@civicrm.org>
Fri, 19 Sep 2014 16:33:31 +0000 (12:33 -0400)
commit1cba072e4ebd965fe92413b32fa1f787af4483f1
tree46e1ec5b74d427a71c7c839f22cabad71e071e05
parent8b3edc18ff94ddbff03b61762defb51e2729045f
Possible multiplication of $fResult in $result

The old structure allowed for possible multiplication of $fResult in $result when a specific $fnSuffix hook was implemented in one module, but not in another.
We only want to add $fResult to $result if $fnName exists, so moved the array_merge() into that if-statement. Furthermore, if $numParams is not case-specified in the switch, at the array_merge(), $fResult could hold the result from a previous module (because in the default case $fResult is not being set), so we have to initialize $fResult to an empty array in the foreach().
CRM/Utils/Hook.php