Skip to content

Update RemoteControl.cs#61

Merged
ORelio merged 1 commit intoMCCTeam:Indevfrom
medxo:patch-1
Jan 22, 2015
Merged

Update RemoteControl.cs#61
ORelio merged 1 commit intoMCCTeam:Indevfrom
medxo:patch-1

Conversation

@medxo
Copy link
Contributor

@medxo medxo commented Jan 22, 2015

when you make the botowners you remove the spaces so i think you have to remove spaces from here too, then name like "Paladin: medxo" will work too

@ORelio
Copy link
Member

ORelio commented Jan 22, 2015

Very logical suggestion, although I think I was wrong from the beginning when replacing all whitespaces when parsing botowners. I should have first splitted with "," then trimmed each name ie removing only heading and trainling whitespaces. Also, as MCC checks that no parse error occured by checking that the username retrieved from chat message only contains A-Z a-z 0-9 and _ characters, I'm not sure that a title containing a colon and a space would be considered as a valid username. Merging anyway since your suggestion is indeed appropriate :)

ORelio added a commit that referenced this pull request Jan 22, 2015
Update RemoteControl.cs
@ORelio ORelio merged commit 314c556 into MCCTeam:Indev Jan 22, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants