From 7d7ee440fbfe15b430c683f558e1c7cfb3a093b7 Mon Sep 17 00:00:00 2001
From: Carlos Lancha <carlos.lancha@liferay.com>
Date: Wed, 14 Mar 2018 16:06:10 +0100
Subject: [PATCH 1/2] Check if item href is not null for quick items in
 management toolbar | Fixes #733

---
 packages/clay-management-toolbar/src/ClayManagementToolbar.soy | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/packages/clay-management-toolbar/src/ClayManagementToolbar.soy b/packages/clay-management-toolbar/src/ClayManagementToolbar.soy
index 4dd493293a..ea0dfb0747 100644
--- a/packages/clay-management-toolbar/src/ClayManagementToolbar.soy
+++ b/packages/clay-management-toolbar/src/ClayManagementToolbar.soy
@@ -237,7 +237,7 @@
 						{foreach $item in $actionItems}
 							{if $item.quickAction and $item.icon and $spritemap}
 								<li class="nav-item navbar-breakpoint-down-d-none" data-onclick="{$_handleQuickActionClicked}">
-									{if $item.href}
+									{if isNonnull($item.href)}
 										{call ClayLink.render}
 											{param elementClasses: 'quick-action-item' /}
 											{param href: $item.href /}

From c2ebcabcd3a7e7d304248808537ef113ef638acc Mon Sep 17 00:00:00 2001
From: Carlos Lancha <carlos.lancha@liferay.com>
Date: Wed, 14 Mar 2018 16:06:26 +0100
Subject: [PATCH 2/2] =?UTF-8?q?Update=20soy=20tests=20|=C2=A0Fixes=20#733?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 .../expected/ClayManagementToolbar.active     |  2 +-
 .../input/ClayManagementToolbar.active        | 24 ++++++++++++++++++-
 2 files changed, 24 insertions(+), 2 deletions(-)

diff --git a/packages/clay-isomorphic/fixtures/expected/ClayManagementToolbar.active b/packages/clay-isomorphic/fixtures/expected/ClayManagementToolbar.active
index 997277c3ae..35f403220d 100644
--- a/packages/clay-isomorphic/fixtures/expected/ClayManagementToolbar.active
+++ b/packages/clay-isomorphic/fixtures/expected/ClayManagementToolbar.active
@@ -1 +1 @@
-<nav class="management-bar management-bar-primary navbar navbar-expand-md navbar-nowrap"><div class="container"><ul class="navbar-nav navbar-nav-expand"><li class="nav-item"><div class="custom-control custom-checkbox"><label><input checked="checked"class="custom-control-input"ref="input" type="checkbox" /><span class="custom-control-label"><span class="custom-control-label-text sr-only">Select Page</span></span></label></div></li><li class="nav-item"><span class="navbar-text">1 of 3 <span class="navbar-breakpoint-down-d-none">items selected</span></span></li><li class="nav-item nav-item-shrink"><button class="btn nav-btn btn-link"aria-label="Select all"type="null">Select all</button></li></ul><ul class="navbar-nav"><li class="nav-item"><div class="dropdown dropdown-action"><button aria-expanded="false"aria-haspopup="true" aria-label="toggle" class="dropdown-toggle btn nav-link nav-link-monospaced btn-unstyled"data-onclick="toggle"type="button"><svg aria-hidden="true" class="lexicon-icon lexicon-icon-ellipsis-v"focusable="false"><title>ellipsis-v</title><use xlink:href="spritemap.svg#ellipsis-v" /></svg></button><div class="dropdown-menu" ref="menu"><ul class="list-unstyled"></ul></div></div></li></ul></div></nav>
\ No newline at end of file
+<nav class="management-bar management-bar-primary navbar navbar-expand-md navbar-nowrap"><div class="container"><ul class="navbar-nav navbar-nav-expand"><li class="nav-item"><div class="custom-control custom-checkbox"><label><input checked="checked"class="custom-control-input"ref="input" type="checkbox" /><span class="custom-control-label"><span class="custom-control-label-text sr-only">Select Page</span></span></label></div></li><li class="nav-item"><span class="navbar-text">1 of 3 <span class="navbar-breakpoint-down-d-none">items selected</span></span></li><li class="nav-item nav-item-shrink"><button class="btn nav-btn btn-link"aria-label="Select all"type="null">Select all</button></li></ul><ul class="navbar-nav"><li class="nav-item navbar-breakpoint-down-d-none" data-onclick="null"><button class="btn nav-link nav-link-monospaced quick-action-item btn-unstyled"aria-label="icon"type="null"><svg aria-hidden="true" class="lexicon-icon lexicon-icon-icon"focusable="false"><title>icon</title><use xlink:href="spritemap.svg#icon" /></svg></button></li><li class="nav-item navbar-breakpoint-down-d-none" data-onclick="null"><button class="btn nav-link nav-link-monospaced quick-action-item btn-unstyled"aria-label="icon"type="null"><svg aria-hidden="true" class="lexicon-icon lexicon-icon-icon"focusable="false"><title>icon</title><use xlink:href="spritemap.svg#icon" /></svg></button></li><li class="nav-item"><div class="dropdown dropdown-action"><button aria-expanded="false"aria-haspopup="true" aria-label="toggle" class="dropdown-toggle btn nav-link nav-link-monospaced btn-unstyled"data-onclick="toggle"type="button"><svg aria-hidden="true" class="lexicon-icon lexicon-icon-ellipsis-v"focusable="false"><title>ellipsis-v</title><use xlink:href="spritemap.svg#ellipsis-v" /></svg></button><div class="dropdown-menu" ref="menu"><ul class="list-unstyled"><li data-onclick="null"><a class=" dropdown-item">action 1</a></li><li data-onclick="null"><a class=" dropdown-item">action 2</a></li><li data-onclick="null"><a class=" dropdown-item"></a></li><li data-onclick="null"><a class=" dropdown-item"></a></li><li data-onclick="null"><a class=" dropdown-item">action 5</a></li><li data-onclick="null"><a class=" dropdown-item">action 6</a></li></ul></div></div></li></ul></div></nav>
\ No newline at end of file
diff --git a/packages/clay-isomorphic/fixtures/input/ClayManagementToolbar.active b/packages/clay-isomorphic/fixtures/input/ClayManagementToolbar.active
index 66fe6c7fab..eb03cdccf2 100644
--- a/packages/clay-isomorphic/fixtures/input/ClayManagementToolbar.active
+++ b/packages/clay-isomorphic/fixtures/input/ClayManagementToolbar.active
@@ -2,7 +2,29 @@
     "spritemap": "spritemap.svg",
     "totalItems": 3,
     "actionItems": [
-
+        {
+            "label": "action 1",
+        },
+        {
+            "label": "action 2",
+            "quickAction": true,
+        },
+        {
+            "icon": "icon",
+        },
+        {
+            "icon": "icon",
+            "quickAction": true,
+        },
+        {
+            "icon": "icon",
+            "label": "action 5",
+        },
+        {
+            "icon": "icon",
+            "label": "action 6",
+            "quickAction": true
+        }
     ],
     "handleActionClicked_": "",
     "elementClasses": "clay-management-toolbar-1 clay-management-toolbar-2",