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

Multiline selection errors with test command ("unexpected argument") #12

Closed
Jiogo18 opened this issue Jan 21, 2024 · 1 comment
Closed
Labels
bug Something isn't working

Comments

@Jiogo18
Copy link

Jiogo18 commented Jan 21, 2024

Fish version: 3.7.0
Natural Selection version: latest commit (2023-11-27)

Selecting multiple lines tries to execute the lines and gives an error.

How to reproduce

image

After selecting 2 chars or more on the left:
image

Source

if test -z (commandline --current-selection)

Here, the second line selected is not used as an argument of test -z because of the \n.
It does something like this
image

Solution

I don't know fish enough. In the idea, it should be something like if test -z "$(commandline --current-selection)".
I don't know if other functions are affected by this.

@daleeidd daleeidd added the bug Something isn't working label Jan 22, 2024
@daleeidd
Copy link
Owner

Thank you for the detailed report. All fixed with latest commit.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants