languagetool-org / languagetool

Style and Grammar Checker for 25+ Languages
https://languagetool.org
GNU Lesser General Public License v2.1
11.88k stars 1.37k forks source link

Sentence splitting issues #10683

Open PavanLuca opened 1 month ago

PavanLuca commented 1 month ago

It didn't underline the whole sentence and changed the day by itself. Schermata del 2024-06-26 12-29-47

Here's the text: It's yet the second time I send a message in Italian there, the first time I hoped that no one saw it but, as I said, there anyone can see deleted messages for 24 hours before it disappears so anyone could see my message in Italian (March the 27th 👇).

danielnaber commented 1 month ago

@jaumeortola You made an "exclude words in parentheses" commit not long ago, could this cause it? (seems to be quite an edge case, though)

jaumeortola commented 1 month ago

My commit should have changed only the conditions for the rule to match, not the length of the underlined text. Anyway, I will check it.

jaumeortola commented 1 month ago

The problem is that we are trying to exclude non-words at the end of the sentence, and 27th 👇). are considered non-words. I don't know what is the purpose of this logic. Do you know it, @SteVio89? We could just underline from the start to the end of the sentence.