Make lowerCase conversion optional

[josephrexme]

This forces every input to be converted to lower case but I need the correct letter casing from the input. Can we take this out completely or make it optional like unicodePunctuation

rivescript-js/src/brain.coffee

Line 588 in e734896

msg = msg.toLowerCase()

[josephrexme]

Is there any risk posed by upper case letters? Why can't there be case sensitive inputs?

[dcsan]

I think we discussed this originally and @kirsle felt it would be too demanding on users to have case sensitive required. we had problems around having to artificially type things like i am here with I in lowercase.

#143

[josephrexme]

@dcsan with the changes on my PR referenced here #246 , we wouldn't be forcing case sensitive inputs on users. It only gets added when the author wants it. I.e new Rivescript({caseSensitive: true}) without that, everything converts to lower case. I just can't tell why the tests are failing on that. I'd write a test for it but it is quite hard to understand how the tests are arranged per feature and the test errors are so long with no detail or proper trace

[josephrexme]

@kirsle I got the tests passing on the PR for this. Is it something that will be considered?