Fixed plugins using d_echo (#9498)

Move d_echo to helpers.php and include in autoload
Don't remove from common.php yet to be extra safe.

LibreNMS/Plugins.php

@@ -27,6 +27,7 @@
 namespace LibreNMS;
 
 use App\Models\Plugin;
+use Log;
 
 /**
  * Handles loading of plugins
@@ -179,10 +180,14 @@ class Plugins
         }
 
         foreach (self::$plugins[$hook] as $name) {
-            if (!is_array($params)) {
-                @call_user_func(array($name, $hook));
-            } else {
-                @call_user_func_array(array($name, $hook), $params);
+            try {
+                if (!is_array($params)) {
+                    @call_user_func([$name, $hook]);
+                } else {
+                    @call_user_func_array([$name, $hook], $params);
+                }
+            } catch (\Exception $e) {
+                Log::error($e);
             }
         }
     }

composer.json

@@ -72,7 +72,10 @@
             "LibreNMS\\": "LibreNMS",
             "LibreNMS\\Plugins\\" : "html/plugins",
             "LibreNMS\\Tests\\": "tests"
-        }
+        },
+        "files": [
+            "includes/helpers.php"
+        ]
     },
     "scripts": {
         "pre-update-cmd": "LibreNMS\\ComposerHelper::preUpdate",

includes/common.php

@@ -657,20 +657,23 @@ function is_valid_hostname($hostname)
 /*
  * convenience function - please use this instead of 'if ($debug) { echo ...; }'
  */
-function d_echo($text, $no_debug_text = null)
-{
-    global $debug;
+if (!function_exists('d_echo')) {
+    //TODO remove this after installs have updated, leaving it for for transition
+    function d_echo($text, $no_debug_text = null)
+    {
+        global $debug;
 
-    if (class_exists('\Log')) {
-        \Log::debug(is_string($text) ? rtrim($text) : $text);
-    } elseif ($debug) {
-        print_r($text);
-    }
+        if (class_exists('\Log')) {
+            \Log::debug(is_string($text) ? rtrim($text) : $text);
+        } elseif ($debug) {
+            print_r($text);
+        }
 
-    if (!$debug && $no_debug_text) {
-        echo "$no_debug_text";
+        if (!$debug && $no_debug_text) {
+            echo "$no_debug_text";
+        }
     }
-} // d_echo
+}
 
 /**
  * Output using console color if possible

includes/helpers.php

@@ -0,0 +1,48 @@
+<?php
+/**
+ * helpers.php
+ *
+ * Functions available in both Laravel and Legacy code (must not call any other legacy functions)
+ *
+ * This program is free software: you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation, either version 3 of the License, or
+ * (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program.  If not, see <http://www.gnu.org/licenses/>.
+ *
+ * @package    LibreNMS
+ * @link       http://librenms.org
+ * @copyright  2018 Tony Murray
+ * @author     Tony Murray <murraytony@gmail.com>
+ */
+
+if (!function_exists('d_echo')) {
+    /**
+     * Legacy convenience function - please use this instead of 'if ($debug) { echo ...; }'
+     * Use Log directly in pure Laravel code!
+     *
+     * @param string|array $text The error message or array to print
+     * @param string $no_debug_text Text to print if debug is disabled
+     */
+    function d_echo($text, $no_debug_text = null)
+    {
+        global $debug;
+
+        if (class_exists('\Log')) {
+            \Log::debug(is_string($text) ? rtrim($text) : $text);
+        } elseif ($debug) {
+            print_r($text);
+        }
+
+        if (!$debug && $no_debug_text) {
+            echo "$no_debug_text";
+        }
+    }
+}