fixup! philips-hue: Add batched command handling support

Author: NoahCornell

drivers/SmartThings/philips-hue/src/utils/batched_command_utils.lua

@@ -110,7 +110,6 @@ end
 function batched_command_utils.sort_batch(driver, batch)
   local sorted_batch = KVCounter()
   local misfits = {}
-  local to_match_table = {}
 
   for _, to_inspect in ipairs(batch) do
     -- Check if we can handle this in a batch
@@ -152,9 +151,10 @@ function batched_command_utils.sort_batch(driver, batch)
     local index = #by_command + 1
     -- Finally, group commands with matching bridge and command name by matching arguments
     -- and auxilary command data.
-    for match_idx, _ in ipairs(by_command) do
+    for match_idx, matching_table in ipairs(by_command) do
       -- Grab first command, all the arguments in this table are the same so it doesn't matter.
-      local to_match = to_match_table[match_idx]
+      -- next is a defined function on KVCounter that uses a similar implementation as the default Lua next.
+      local _, to_match = matching_table.next(matching_table, nil)
 
       if utils.deep_table_eq(to_match.capability_command.args, to_inspect.capability_command.args) and
         utils.deep_table_eq(to_match.auxilary_command_data, to_inspect.auxilary_command_data) then
@@ -165,8 +165,6 @@ function batched_command_utils.sort_batch(driver, batch)
     end
     if not by_command[index] then
       table.insert(by_command, index, KVCounter())
-      -- Save this off to grab it easily later
-      to_match_table[index] = to_inspect
     end
     by_command[index][device.id] = to_inspect