From c1bb17709a656ad09d397950ffaf3a7034b7d6a0 Mon Sep 17 00:00:00 2001
From: Johannes Meyer <johannes.meyer@icinga.com>
Date: Mon, 10 Aug 2020 10:29:54 +0200
Subject: [PATCH] Navigation\Action: Don't try to resolve non existing urls

fixes #4211
---
 modules/monitoring/library/Monitoring/Web/Navigation/Action.php | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/modules/monitoring/library/Monitoring/Web/Navigation/Action.php b/modules/monitoring/library/Monitoring/Web/Navigation/Action.php
index 4360c122c..7e4ffe341 100644
--- a/modules/monitoring/library/Monitoring/Web/Navigation/Action.php
+++ b/modules/monitoring/library/Monitoring/Web/Navigation/Action.php
@@ -102,7 +102,7 @@ class Action extends NavigationItem
     public function getUrl()
     {
         $url = parent::getUrl();
-        if (! $this->resolved && $url === null) {
+        if (! $this->resolved && $url === null && $this->rawUrl !== null) {
             $this->setUrl(Url::fromPath(Macro::resolveMacros($this->rawUrl, $this->getObject())));
             $this->resolved = true;
             return parent::getUrl();