feat(client/compat): handle string-indexed options

Neither are technically supported, but were possible by design flaw/fluke.
Since somebody at qb decided to utilise this flaw in a recent change, we'll
re-insert the option at the end of the options table.

client/compat/qb-target.lua

@@ -10,7 +10,19 @@ local function convert(options)
     local distance = options.distance
     options = options.options
 
-    for _, v in pairs(options) do
+    -- People may pass options as a hashmap (or mixed, even)
+    for k, v in pairs(options) do
+        if type(k) ~= 'number' then
+            table.insert(options, v)
+        end
+    end
+
+    for id, v in pairs(options) do
+        if type(id) ~= 'number' then
+            options[id] = nil
+            goto continue
+        end
+
         v.onSelect = v.action
         v.distance = v.distance or distance
         v.name = v.name or v.label
@@ -110,6 +122,8 @@ local function convert(options)
         v.citizenid = nil
         v.item = nil
         v.qtarget = true
+
+        ::continue::
     end
 
     return options

client/compat/qtarget.lua

@@ -10,7 +10,19 @@ local function convert(options)
     local distance = options.distance
     options = options.options
 
-    for _, v in pairs(options) do
+    -- People may pass options as a hashmap (or mixed, even)
+    for k, v in pairs(options) do
+        if type(k) ~= 'number' then
+            table.insert(options, v)
+        end
+    end
+
+    for id, v in pairs(options) do
+        if type(id) ~= 'number' then
+            options[id] = nil
+            goto continue
+        end
+
         v.onSelect = v.action
         v.distance = v.distance or distance
         v.name = v.name or v.label
@@ -33,6 +45,8 @@ local function convert(options)
         v.item = nil
         v.required_item = nil
         v.qtarget = true
+
+        ::continue::
     end
 
     return options