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

Mod names should be whitespace-trimmed when being added #102

Closed
Trnrr opened this issue May 30, 2022 · 0 comments
Closed

Mod names should be whitespace-trimmed when being added #102

Trnrr opened this issue May 30, 2022 · 0 comments
Labels
bug Something doesn't work as expected priority I'm working on it, or will be soon

Comments

@Trnrr
Copy link

Trnrr commented May 30, 2022

Mod names can have extra white space and when ferium remove is used on one of these mod names it fails. I understand this is probably working as intended but a solution to this could lead to less confusion. Some fixes could be trimming the white space from the config names or displaying the mod names in "" so we can see any extra white space.

I was running into the problem with the Seamless Loading Screen mod. In ferium list, it displays as "Seamless Leading Screen".

Doing any variation of the below commands didn't work:

  • ferium remove "Seamless Loading Screen"
  • ←[1;31mA mod called Seamless Loading Screen is not present in this profile←[0m
  • ferium remove TyTPFOiF
  • ←[1;31mA mod called TyTPFOiF is not present in this profile←[0m
  • ferium remove Seamless\ Loading\ Screen
  • ←[1;31mA mod called Seamless\ is not present in this profile←[0m

After locating the config file, I noticed the name was "Seamless Loading Screen ". Running the remove command with the extra space at the end removed it.

@theRookieCoder theRookieCoder changed the title Ferium remove fails when extra white space is in modname Mod names should be whitespace-trimmed when being added May 31, 2022
@theRookieCoder theRookieCoder added bug Something doesn't work as expected priority I'm working on it, or will be soon labels May 31, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something doesn't work as expected priority I'm working on it, or will be soon
Development

No branches or pull requests

2 participants