Bug - Fixed Uneffect "Burning, Man" in relay browser not working

Grotfang

Developer
Clicking on the left pane effect (underlined turn number) leads to not removing the effect, and the following print out in the CLI:

Code:
[COLOR="DarkOrange"]> uneffect Burning, Man[/COLOR]

[COLOR="Red"]Ambiguous effect name: Burning[/COLOR]
burning ears
burning hands
burning heart
burning soul
burning, man
[COLOR="Red"]Ambiguous effect name: Man[/COLOR]
burning, man
man's worst enemy
shamed & manipulated
tin, man
weather, man
 

bumcheekcity

Active member
Code:
> uneffect Burning, Man

Ambiguous     effect name: Burning
burning ears
burning hands
burning     heart
burning soul
burning, man
Ambiguous     effect name: Man
burning, man
man's worst enemy
shamed &     manipulated
tin, man
weather, man

>     uneffect Burning\, Man

Unknown effect:     Burning\
Ambiguous effect name: Man
burning,     man
man's worst enemy
shamed & manipulated
tin, man
weather,     man

> uneffect "Burning, Man"

Unknown     effect: "Burning
Unknown effect: Man"
 

Fluxxdog

Active member
Update:
Code:
> uneffect Burning, Man

Ambiguous     effect name: Burning
burning ears
burning hands
burning     heart
burning soul
burning, man
Ambiguous     effect name: Man
burning, man
man's worst enemy
shamed &     manipulated
things man was not meant to eat
tin, man
triangle, man
weather,     man
you read the manual

> uneffect Burning     Man

Burning, Man cannot be removed with an available item or     skill...
Using soft green whatever...
Burning, Man removed.
First ambiguity was from clicking in the relay, same as OP.
PS: Glad I did a search for this before posting a new bug.
 

Fluxxdog

Active member
I think the surprise is the link from the relay browser made by mafia is what causes the problem. Since a delimiter is part of the command, it uses that. Gonna have the same adjustment as the use where it tries matching exact before splitting the string?
 

Veracity

Developer
Staff member
Revision 13243. Tested with "Ninja, Please" - which you're unlikely to want to uneffect, but it's a lot easier to test with than Burning, Man. There are actually 53 different status effects with commas in their name.
 
Top