Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

state set console command does not accept blanks within a string, instead it is treated as a second value (ack flag) #319

Closed
Pony5dotzero opened this issue Jan 27, 2019 · 14 comments · Fixed by ioBroker/ioBroker#275
Labels

Comments

@Pony5dotzero
Copy link

Example:
pi@ioBroker-RasPi:/opt/iobroker $ iobroker state set lgtv.0.popup "Hallo"
Set "lgtv.0.popup" with value: Hallo

pi@ioBroker-RasPi:/opt/iobroker $ iobroker state set lgtv.0.popup "Hallo Welt"
Set "lgtv.0.popup" with value: Hallo and ack flag Welt

Even if send with double apostrophe like iobroker state set ""Hallo Welt"" it shows:
Set "lgtv.0.popup" with value: Hallo and ack flag Welt

@Apollon77
Copy link
Collaborator

It seems that yargs is not parsing Strings correctly, there are several issues fixed and some still open :-( Maybe we can workaround somehow or need to "minimize" the problem to send in a bug report with them

@Apollon77
Copy link
Collaborator

For now I would expect it to work when no spaces are in the parameter. @Pony5dotzero could you please verify this?

@AlCalzone
Copy link
Collaborator

Which version of yargs is the js-controller 1.4.2 using? I can't reproduce it with a simple test script. But that might also be because I'm using git bash on windows.

@Pony5dotzero
Copy link
Author

Pony5dotzero commented Jan 27, 2019

Yes, if you use a dot instead of a blank it works as expected. e.g. "Hallo.Welt"
Im using the iobroker raspi image from 14.4.2018 as far as i remember.

@Apollon77
Copy link
Collaborator

Apollon77 commented Jan 27, 2019

According to 1.4.2-tag it should be "yargs": "^11.0.0" (also in 1.3.x controller!)

@Apollon77
Copy link
Collaborator

PS: We have two forum threads with such problems 1x message, 1x set State, two users

@AlCalzone
Copy link
Collaborator

@Apollon77 it would be good to know which os the users have.

@Pony5dotzero
Copy link
Author

iobroker Raspbian image based on debian stretch

@Obsthaendler
Copy link

iobroker Raspbian image based on debian stretch

I have Raspbian Stretch (Version 9) with Kernel 3.13.79-v7+

@Apollon77
Copy link
Collaborator

relatesd to yargs/yargs-parser#180

@volkerverkamp
Copy link

I am suffering from this issue as well - any new status on this?

@foxriver76
Copy link
Collaborator

I've pushed a fix to the installer and fixer, will most likely be in the next fixer version

@foxriver76
Copy link
Collaborator

I guess OS is linux where the problem occurs? For @AlCalzone it worked on Windows

@AlCalzone
Copy link
Collaborator

I guess OS is linux where the problem occurs? For @AlCalzone it worked on Windows

I think so, yes

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants