Merge pull request #67 from NETWAYS/fix/validhtml

Fix invalid return value for getPreviewHTML

Author: martialblog

library/Grafana/ProvidedHook/Icingadb/IcingaDbGrapher.php

@@ -338,7 +338,7 @@ private function getMyPreviewHtml($serviceName, $hostName, HtmlDocument $preview
      * @throws ConfigurationError
      * @throws \Icinga\Exception\ProgrammingError
      */
-    public function getPreviewHtml(Model $object, $report = false)
+    public function getPreviewHtml(Model $object, $report = false): ValidHtml
     {
         $this->object = $object;
         // Use the cachetime based on the check intervals
@@ -378,7 +378,7 @@ public function getPreviewHtml(Model $object, $report = false)
         $customvars = $this->getDb()->fetchPairs($varsFlat->assembleSelect());
 
         if ($object->perfdata_enabled == "n" || (( isset($customvars[$this->custvardisable]) && json_decode(strtolower($customvars[$this->custvardisable])) !== false))) {
-            return '';
+            return HtmlString::create('');
         }
 
         if (array_key_exists($this->custvarconfig, $customvars)