Fix Lua error with items on action bar, fixes #276

Author: Snakybo

CHANGELOG.md

@@ -13,6 +13,7 @@ The format of this changelog is based on [Keep a Changelog](https://keepachangel
 ### Fixed
 
 * Fix Moonkin form spell ID
+* Fix Lua error with items on action bars if automatic binding of action bar abilities is enabled {#276}
 
 ### Changed
 
@@ -1756,6 +1757,7 @@ The format of this changelog is based on [Keep a Changelog](https://keepachangel
 [0.4.0]: https://github.com/Snakybo/Clicked/releases/tag/0.4.0
 [0.3.0]: https://github.com/Snakybo/Clicked/releases/tag/0.3.0
 
+[#276]: https://github.com/Snakybo/Clicked/issues/276
 [#270]: https://github.com/Snakybo/Clicked/issues/270
 [#267]: https://github.com/Snakybo/Clicked/issues/267
 [#265]: https://github.com/Snakybo/Clicked/issues/265

Clicked/Core/BindingProcessor.lua

@@ -327,13 +327,22 @@ local function ConstructActions(binding, interactionType, actionBarItems)
 
 		-- Create virtual actions for action bar items
 		for _, item in ipairs(actionBarItems) do
-			if item.key == binding.keybind and (item.type == "SPELL" or item.type == "ITEM") then
-				--- @cast item SpellLibrarySpellResult|SpellLibraryItemResult
-
-				table.insert(actions, {
-					ability = C_Spell.GetSpellName(item.spellId) or C_Item.GetItemNameByID(item.itemId),
-					type = binding.actionType
-				})
+			if item.key == binding.keybind then
+				if item.type == "SPELL" then
+					--- @cast item SpellLibrarySpellResult
+
+					table.insert(actions, {
+						ability = C_Spell.GetSpellName(item.spellId),
+						type = binding.actionType
+					})
+				elseif item.type == "ITEM" then
+					--- @cast item SpellLibraryItemResult
+
+					table.insert(actions, {
+						ability = C_Item.GetItemNameByID(item.itemId),
+						type = binding.actionType
+					})
+				end
 			end
 		end
 	end