CRM-13823 - CRM Core - remove incorrect calls to severityMap()
authorJon goldberg <jon@palantetech.coop>
Wed, 8 Jul 2015 16:08:51 +0000 (12:08 -0400)
committerJon goldberg <jon@palantetech.coop>
Tue, 21 Jul 2015 20:38:19 +0000 (16:38 -0400)
CRM/Utils/Check.php

index 47ab63200d4ca9139fb6b74779310fe056f1edba..cd2e18d6079fd83203aa0d82b5f81991ec2707a0 100644 (file)
@@ -83,10 +83,9 @@ class CRM_Utils_Check {
         }
         $statusMessages = array();
         $statusType = 'alert';
-
         foreach ($messages as $message) {
-          if ($filter === TRUE || call_user_func($filter, $message->getSeverity()) >= 3) {
-            $statusType = (call_user_func($filter, $message->getSeverity()) >= 4) ? 'error' : $statusType;
+          if ($filter === TRUE || $message->getSeverity() >= 3) {
+            $statusType = $message->getSeverity() >= 4 ? 'error' : $statusType;
             $statusMessage = $message->getMessage();
             $statusMessages[] = $statusTitle = $message->getTitle();
           }
@@ -118,7 +117,8 @@ class CRM_Utils_Check {
     if ($aSeverity == $bSeverity) {
       return strcmp($a->getName(), $b->getName());
     }
-    return (self::severityMap($aSeverity) < self::severityMap($bSeverity));
+    // The Message constructor guarantees that these will always be integers.
+    return ($aSeverity < $bSeverity);
   }
 
   /**
@@ -215,7 +215,6 @@ class CRM_Utils_Check {
         }
       }
     }
-
     uasort($messages, array(__CLASS__, 'severitySort'));
 
     return $messages;