Erreur dans les conversions de routes du forum #37

Closed
opened 4 years ago by Darks · 7 comments
Darks commented 4 years ago
Owner

Les routes du forum sont mal interprétées.

Exemple, je créé le topic /forum/news/calc/2. Je peux aussi y accéder par /forum/news/projects/2. Comportement attendu → 404.

De plus, url_for('forum_topic', topic.forum, topic.id) retourne quelque chose de la forme /forum/news/calc/<Topic: #2>, ce qui casse les liens. Comportement attendu : /forum/news/calc/2-slug-du-topic.

Les routes du forum sont mal interprétées. Exemple, je créé le topic `/forum/news/calc/2`. Je peux aussi y accéder par `/forum/news/projects/2`. Comportement attendu → 404. De plus, `url_for('forum_topic', topic.forum, topic.id)` retourne quelque chose de la forme `/forum/news/calc/<Topic: #2>`, ce qui casse les liens. Comportement attendu : `/forum/news/calc/2-slug-du-topic`.
Darks added the bug label 4 years ago
Darks self-assigned this 4 years ago
Lephenixnoir was assigned by Darks 4 years ago
Darks removed their assignment 4 years ago
Darks commented 4 years ago
Poster
Owner

J'ai corrigé le point n°2 dans le commit b5e875e136

Pour le reste, @Lephenixnoir tu pourra essayer de regarder ?

J'ai corrigé le point n°2 dans le commit b5e875e136 Pour le reste, @Lephenixnoir tu pourra essayer de regarder ?
Owner

J'ai pas l'environnement de dev ici, mais je devrais pouvoir le faire.

J'ai pas l'environnement de dev ici, mais je devrais pouvoir le faire.
Eragon commented 3 years ago
Collaborator

Up

Up
Owner

J'ai attaqué sérieusement ce bug aujourd'hui. Pas encore gagné mais c'est un début.

Déjà le problème d'origine selon lequel les topics seraient accessibles sous d'autres noms de forum n'a plus l'air de se poser.

Je confirmerai plus en détail quand j'arriverai à créer des nouveaux topics, ce qui nécessite pour l'instant de se connecter donc de configurer LDAP... ce qui est un peu la merde. @Darks détails désirés.

J'ai attaqué sérieusement ce bug aujourd'hui. Pas encore gagné mais c'est un début. Déjà le problème d'origine selon lequel les topics seraient accessibles sous d'autres noms de forum n'a plus l'air de se poser. Je confirmerai plus en détail quand j'arriverai à créer des nouveaux topics, ce qui nécessite pour l'instant de se connecter donc de configurer LDAP... ce qui est un peu la merde. @Darks détails désirés.
Darks commented 3 years ago
Poster
Owner

Méthode si tu veux pas te faire chier : ajouter USE_LDAP = False à la classe LocalConfig

Méthode si tu veux (un peu) te faire chier : https://gitea.planet-casio.com/devs/VPS-config/src/branch/master/ldap

Je précise que si t'utilise pas systemd faut adapter le script que je t'ai linké. 😉

Méthode si tu veux pas te faire chier : ajouter `USE_LDAP = False` à la classe `LocalConfig` Méthode si tu veux (un peu) te faire chier : https://gitea.planet-casio.com/devs/VPS-config/src/branch/master/ldap Je précise que si t'utilise pas systemd faut adapter le script que je t'ai linké. :wink:
Owner

Merci ! J'ai configuré le LDAP, ça ne coûte rien.

Je suppose que j'ai mis trop longtemps à étudier la question... en tous cas je ne peux pas reproduire le bug. @Darks s'il te reste un moment pour vérifier que j'ai bien cherché, sinon on peut fermer l'isue.

Merci ! J'ai configuré le LDAP, ça ne coûte rien. Je suppose que j'ai mis trop longtemps à étudier la question... en tous cas je ne peux pas reproduire le bug. @Darks s'il te reste un moment pour vérifier que j'ai bien cherché, sinon on peut fermer l'isue.
Darks commented 3 years ago
Poster
Owner

Je dois t'avouer que je ne sais plus d'où venait exactement le bug. Y'a du y'avoir un fix entre temps. M'enfin si ça marche, je ferme.

Merci pour ton aide 🙂

Je dois t'avouer que je ne sais plus d'où venait exactement le bug. Y'a du y'avoir un fix entre temps. M'enfin si ça marche, je ferme. Merci pour ton aide 🙂
Darks closed this issue 3 years ago
Sign in to join this conversation.
No Milestone
No Assignees
3 Participants
Notifications
Due Date

No due date set.

Dependencies

No dependencies set.

Reference: devs/PCv5#37
Loading…
There is no content yet.