Merge pull request #4059 from cmitu/joy2key-fix6

joy2key: match config by Product/Vendor IDs

Author: joolswills

scriptmodules/admin/joy2key/joy2key_sdl.py

@@ -118,15 +118,19 @@ class InputDev(object):
     Class representing a joystick device config.
     Maps the inputs of the device to event names
     name: the device's name
-    guid: the GUID, as retuned by SDL
+    guid: the GUID, as returned by SDL
+    pid: the ProductID of the device
+    vid: the VendorID of the device
     hats - a dictionary of { <HatNo>: list(<HatValue>, <Event>) }
     buttons - a dict of { <ButtonNo>: <Event> }
     axis - a dict of { <AxisNo>: list(<AxisDirection>, <Event>) }
     """
 
-    def __init__(self, _name: str, _guid: str):
+    def __init__(self, _name: str, _vid: int, _pid: int):
         self.name = _name
-        self.guid = _guid
+        self.guid = None
+        self.vid = _vid
+        self.pid = _pid
         self.axis = {}
         self.buttons = {}
         self.hats = {}
@@ -153,7 +157,7 @@ def get_axis_event(self, index: int, value: int) -> list:
             return None
 
     def __str__(self) -> str:
-        return str(f'{self.name}, hats: {self.hats}, buttons: {self.buttons}, axis: {self.axis}')
+        return str(f'{self.name} (P:{self.pid}, V:{self.vid}), hats: {self.hats}, buttons: {self.buttons}, axis: {self.axis}')
 
 
 def generic_event_map(input: str, event_map: dict) -> str:
@@ -227,7 +231,7 @@ def get_all_ra_config(def_buttons: list) -> list:
     """
     ra_config_list = []
     # add a generic mapping at index 0, to be used for un-configured joysticks
-    generic_dev = InputDev("*", "*")
+    generic_dev = InputDev("*", None, None)
     generic_dev.add_mappings(
         {},  # no axis
         {0: 'b', 1: 'a', 3: 'y', 4: 'x'},  # 4 buttons
@@ -236,7 +240,7 @@ def get_all_ra_config(def_buttons: list) -> list:
     ra_config_list.append(generic_dev)
     js_cfg_dir = CONFIG_DIR + '/all/retroarch-joypads/'
 
-    config = ConfigParser(delimiters="=", strict=False, interpolation=None)
+    config = ConfigParser(delimiters="=", strict=False, interpolation=None, converters={'int': (lambda s: s.strip('"'))})
     for file in os.listdir(js_cfg_dir):
         # skip non '.cfg' files
         if not file.endswith('.cfg') or file.startswith('.'):
@@ -247,8 +251,12 @@ def get_all_ra_config(def_buttons: list) -> list:
                 config.clear()
                 # ConfigParser needs a section, make up a section to appease it
                 config.read_string('[device]\n' + cfg_file.read())
+                LOG.debug(f'Parsing config "{file}"')
                 conf_vals = config['device']
                 dev_name = conf_vals['input_device'].strip('"')
+                # fallback to None if there are no PID/VID in the configuration
+                dev_vid = conf_vals.getint('input_vendor_id', None)
+                dev_pid = conf_vals.getint('input_product_id', None)
 
                 # translate the RetroArch inputs from the configuration file
                 axis, buttons, hats = {}, {}, {}
@@ -268,9 +276,10 @@ def get_all_ra_config(def_buttons: list) -> list:
                             axis.setdefault(input_index, []).append((input_value, event_name))
                         else:
                             continue
-                ra_dev_config = InputDev(dev_name, None)
+                ra_dev_config = InputDev(dev_name, dev_vid, dev_pid)
                 ra_dev_config.add_mappings(axis, buttons, hats)
                 ra_config_list.append(ra_dev_config)
+                LOG.debug(f'Added config for "{dev_name}" from "{file}"')
             except Exception as e:
                 LOG.warning(f'Parsing error for {file}: {e}')
                 continue
@@ -370,12 +379,16 @@ def handle_new_input(e: SDL_Event, axis_norm_value: int = 0) -> bool:
                 stick = joystick.SDL_JoystickOpen(event.jdevice.which)
                 name = joystick.SDL_JoystickName(stick).decode('utf-8')
                 guid = create_string_buffer(33)
+                vid = joystick.SDL_JoystickGetVendor(stick)
+                pid = joystick.SDL_JoystickGetProduct(stick)
+
                 _SDL_JoystickGetGUIDString(joystick.SDL_JoystickGetGUID(stick), guid, 33)
-                LOG.debug(f'Joystick #{joystick.SDL_JoystickInstanceID(stick)} {name} added')
+                LOG.debug(f'Joystick #{joystick.SDL_JoystickInstanceID(stick)} {name} (P:{pid}, V:{vid}) added')
                 conf_found = False
-                # try to find a configuration for the joystick
+                # try to find a configuration for the joystick, based on name, GUID and Vendor/Product IDs
                 for key, dev_conf in enumerate(configs):
-                    if dev_conf.name == str(name) or dev_conf.guid == guid.value.decode():
+                    if dev_conf.name == str(name) or dev_conf.guid == guid.value.decode() or \
+                       (dev_conf.pid == str(pid) and dev_conf.vid == str(vid)):
                         # Add the matching joystick configuration to the watched list
                         active_devices[joystick.SDL_JoystickInstanceID(stick)] = (key, stick)
                         LOG.debug(f'Added configuration for known device {configs[key]}')