homebridge-harmony icon indicating copy to clipboard operation
homebridge-harmony copied to clipboard

[BUG] Device Command with "/" in command name not creating switch

Open tmurfin opened this issue 3 years ago • 10 comments

Describe the bug I have a device that is giving me the command InputDisc/Phono but it says "No function list found on startup when i try to make a switch to be able to select the input, because the "/" is a modifier to make toggle commands, so I can not select this because the "/" is a modifier to make toggle commands, so I can not select this input on my device.

To Reproduce Steps to reproduce the behavior

Expected behavior A clear and concise description of what you expected to happen.

Screenshots Screen Shot 2021-03-01 at 11 03 30 PM Screen Shot 2021-03-01 at 11 03 10 PM

IOS (please complete the following information):

  • Version
  • Device:

Logs Your log in debug mode (-D option)

Config { "name": "Harmony Hub", "hubIP": "", "hubName": "Harmony Hub", "TVAccessory": false, "switchAccessories": false, "publishGeneralMuteSwitch": false, "publishGeneralVolumeSlider": false, "publishGeneralVolumeSwitches": false, "linkVolumeControlToTV": false, "publishSwitchActivitiesAsIndividualAccessories": true, "devicesToPublishAsAccessoriesSwitch": [ "NAD Amp|Power;PowerOn/PowerOff", "NAD Amp|Mute;Mute", "NAD Amp|Volume Up;VolumeUp", "NAD Amp|Volume Down;VolumeDown", "NAD Amp|Media Player;InputMp", "NAD Amp|Video;InputVideo", "NAD Amp|Turntable;InputDisc/Phono" ], "publishDevicesAsIndividualAccessories": false, "showCommandsAtStartup": true, "platform": "HarmonyHubWebSocket" },

Additional context Add any other context about the problem here.

tmurfin avatar Mar 02 '21 05:03 tmurfin

Hi,

thanks for reporting. Will try to sanitise names in order to be compliant. Meanwhile, can't you rename it in harmony app ?

nicoduj avatar Mar 02 '21 10:03 nicoduj

I was not able to rename it, I had to create a new command in harmony hub from IR as a work around.

tmurfin avatar Mar 02 '21 14:03 tmurfin

Ok right, if it is a pre learned you can't renamed . Will fix that asap

nicoduj avatar Mar 02 '21 14:03 nicoduj

this one involves a lot of changes (probably going to json config instead of string based). Will postpone sorry.

nicoduj avatar Apr 20 '21 08:04 nicoduj

Hey all, was just wondering if there were any updates regarding this issue 😊

SSutherland94 avatar Jul 07 '21 02:07 SSutherland94

Hi, like I said it involves a lot of changes in the plugin, and I don't have a lot of time to spend on it sorry.

nicoduj avatar Jul 07 '21 06:07 nicoduj

Sure would be nice if this could be fixed. I have a number of devices that suffer from this problem.

F-Kafka avatar Feb 25 '22 16:02 F-Kafka

Adding to this request, have the same issue for both zones of my Onkyo Amp.

JSVarga avatar Jun 10 '22 00:06 JSVarga

Running into the same problem with an Onkyo Amp.

Could this be worked around by adding a \ in front of the /?

Morac2 avatar Sep 02 '23 15:09 Morac2

I changed the check in the code from / to \ and now I can add switches for commands with / in it.

That effectively changes the state control from slash to backslash, but I don’t care about that.

Morac2 avatar Sep 03 '23 23:09 Morac2