SlashCommandEducator should ignore method calls

[Zabuzard]

Sometimes people write messages like

.toString()

Currently, SlashCommandEducator picks that up and thinks it is a command, for example .close. In this case, it is obviously a false positive.

To prevent this, we should additionally check whether the message ends with ). If so, we can just ignore it. Just adding the check here in SlashCommandEducator.java:

And additionally adding ".toString()" to the tests in SlashCommandEducatorTest.java here:

[Taz03]

i write method calls without the braces 💀

[marko-radosavljevic]

How often do people even try to .close or something similar? Instructions how to close are already in every thread. I assume there are probably more people trying to .toString() or != 0 than people trying to write commands?

Also, when you type .close or !close, discord suggests you correct slash command in drop-up menu.

I agree with the original intent, to educate people on slash command usage, but not if we annoy more people than we help. ^^

[Zabuzard]

We help more than we annoy. I see a legit hint every 2 days and an incorrect hint every 2 weeks only.

The false-positive identification is already really accurate:

• starts with .!?
• no spaces in the message (this one covers most)
• very short message (covers the rest)
• [and with this issue: doesnt end with )]

[tmcdonnell2]

Hello, would I be able to take this task?