From f6cd4794eedb989450f830bc4df892e42532170e Mon Sep 17 00:00:00 2001 From: "Alexander A. Klimov" Date: Thu, 17 Dec 2015 16:13:12 +0100 Subject: [PATCH] NavigationItemRenderer::renderTargetAttribute(): don't use the same name for two different variables refs #10330 --- .../Web/Navigation/Renderer/NavigationItemRenderer.php | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/library/Icinga/Web/Navigation/Renderer/NavigationItemRenderer.php b/library/Icinga/Web/Navigation/Renderer/NavigationItemRenderer.php index 4c7cc0fa5..0f22de40b 100644 --- a/library/Icinga/Web/Navigation/Renderer/NavigationItemRenderer.php +++ b/library/Icinga/Web/Navigation/Renderer/NavigationItemRenderer.php @@ -237,13 +237,13 @@ class NavigationItemRenderer return ''; } - $actualTarget = $this->getIcingaLinkTarget($target); - if ($actualTarget !== null) { - return ' data-base-target="' . $actualTarget . '"'; + $icingaTarget = $this->getIcingaLinkTarget($target); + if ($icingaTarget !== null) { + return ' data-base-target="' . $icingaTarget . '"'; } - $actualTarget = $this->getHtmlLinkTarget($target); - return ' target="' . ($actualTarget === null ? $this->view()->escape($target) : $actualTarget) . '"'; + $htmlTarget = $this->getHtmlLinkTarget($target); + return ' target="' . ($htmlTarget === null ? $this->view()->escape($target) : $htmlTarget) . '"'; } /**