From: Tim Otten <totten@civicrm.org>
Date: Tue, 10 Jan 2017 00:44:59 +0000 (-0800)
Subject: Civi/Core - Fix various PHPStorm warnings
X-Git-Url: https://vcs.fsf.org/?a=commitdiff_plain;h=3c250b10bfc577a4f43acff9b596b7d74a17bfda;p=civicrm-core.git

Civi/Core - Fix various PHPStorm warnings
---

diff --git a/Civi/Core/Resolver.php b/Civi/Core/Resolver.php
index 70654bac29..0d71d185e7 100644
--- a/Civi/Core/Resolver.php
+++ b/Civi/Core/Resolver.php
@@ -53,7 +53,7 @@ class Resolver {
    * @param string|array $id
    *   A callback expression; any of the following.
    *
-   * @return array
+   * @return array|callable
    *   A PHP callback. Do not serialize (b/c it may include an object).
    * @throws \RuntimeException
    */
@@ -277,6 +277,7 @@ class ResolverGlobalCallback {
     }
     elseif ($this->mode === 'setter') {
       \CRM_Utils_Array::pathSet($GLOBALS, explode('/', $this->path), $arg1);
+      return NULL;
     }
     else {
       throw new \RuntimeException("Resolver failed: global:// must specify getter or setter mode.");
diff --git a/Civi/Core/SettingsMetadata.php b/Civi/Core/SettingsMetadata.php
index bf84572654..c7ee465233 100644
--- a/Civi/Core/SettingsMetadata.php
+++ b/Civi/Core/SettingsMetadata.php
@@ -112,7 +112,7 @@ class SettingsMetadata {
       $realFolder = realpath($metaDataFolder);
       if (is_dir($realFolder) && !isset($loadedFolders[$realFolder])) {
         $loadedFolders[$realFolder] = TRUE;
-        $settingsMetadata = $settingsMetadata + self::loadSettingsMetaData($metaDataFolder);
+        $settingsMetadata = $settingsMetadata + self::loadSettingsMetadata($metaDataFolder);
       }
     }
     return $settingsMetadata;
diff --git a/Civi/Core/Transaction/Manager.php b/Civi/Core/Transaction/Manager.php
index 9b1d029989..2f40345796 100644
--- a/Civi/Core/Transaction/Manager.php
+++ b/Civi/Core/Transaction/Manager.php
@@ -63,7 +63,7 @@ class Manager {
   }
 
   /**
-   * @param CRM_Core_DAO $dao
+   * @param \CRM_Core_DAO $dao
    *   Handle for the DB connection that will execute transaction statements.
    *   (all we really care about is the query() function)
    */