Skip to content

Correction bug messages qui disparaissent#209

Open
A-Louarn wants to merge 9 commits intoForumHFR:masterfrom
A-Louarn:master
Open

Correction bug messages qui disparaissent#209
A-Louarn wants to merge 9 commits intoForumHFR:masterfrom
A-Louarn:master

Conversation

@A-Louarn
Copy link
Copy Markdown

J'ai commencé à regarder pour corriger les posts qui disparaissent (souvent après un message de la modération).

J'ai corrigé le problème mais le rendu des posts est assez moche, et je ne suis pas certain de comment le réparer.

Le crawler de post a été passé des regexps à un parser HTML (JSoup) qui devrait être plus simple à maintenir et poser moins de problème avec les agencements non standards du forums.

Refactoring: HTML parser instead of regexp. Should be easier to
maintain.
Bugfix: disappearing messages after moderation message
@Ayuget
Copy link
Copy Markdown
Collaborator

Ayuget commented Sep 30, 2019

Merci, je vais étudier ça le week-end prochain en rentrant de vacances.

Je n'avais pas utilisé jSoup volontairement car c'était vraiment moins performant que les regex a l'époque. Mais t'as raison il faudrait que je refasse des bench

@Ayuget
Copy link
Copy Markdown
Collaborator

Ayuget commented Sep 30, 2019

Car c'est sûr qu'utiliser des Regex c'est en soit tout pourri. Sur desktop je ne me serais jamais posé la question.

De mémoire quand j'ai commencé à coder l'app il y a 4 ans, c'était assez flagrant la différence de perf entre jSoup et les regex

@A-Louarn
Copy link
Copy Markdown
Author

Je n'ai pas l'impression d'avoir des ralentissements notables, mais il est possible que ce soit le cas.
Si c'est trop génant, il est possible d'utiliser un parser SAX au lieu de DOM, ça devrait donner un boost de performance à moindres frais (et AMHA être presque aussi rapide que les regexp).

Dans tous les cas, ça serait une bonne idée de faire des bench, oui. Si tu sais comment faire, ça m'intéresse.

@Ayuget
Copy link
Copy Markdown
Collaborator

Ayuget commented Sep 30, 2019

Tu peux tester ça avec JMH c'est pas super compliqué à mettre en place.

Merci pour ton travail en tout cas ça fait super plaisir !!!

@A-Louarn
Copy link
Copy Markdown
Author

Je regarderai JMH quand j'aurai du temps.

De rien, c'est toujours sympa de bosser sur une appli que j'utilise tous les jours 😄

@A-Louarn
Copy link
Copy Markdown
Author

Les quotes & spoilers sont fixées 😉

J'ai eu un bug marrant en corrigeant les quotes : celles créées avec un "[quote]" n'étaient pas fermées et donc tout le resté de la page était dans un bloc de quote 😄

Reste plus qu'à supprimer les signatures des messages et ce devrait être prêt pour la prod.

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