devs/PCv5
devs
/
PCv5
7
10
Fork 4
Code Issues 25 Pull Requests Releases Wiki Activity

Erreur dans les conversions de routes du forum #37

New Issue
Closed
opened 2019-12-03 18:56:29 +01:00 by Darks · 7 comments
Darks commented 2019-12-03 18:56:29 +01:00
Owner
Copy Link

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 2019-12-03 18:56:29 +01:00
Darks self-assigned this 2019-12-03 18:59:21 +01:00
Lephenixnoir was assigned by Darks 2019-12-03 20:52:28 +01:00
Darks removed their assignment 2019-12-03 20:52:30 +01:00
Darks commented 2019-12-03 20:52:36 +01:00
Author
Owner
Copy Link

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 ?
Lephenixnoir commented 2019-12-04 06:28:26 +01:00
Owner
Copy Link

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 2020-01-27 13:02:31 +01:00
Member
Copy Link

Up

Up
❤️ 1
Lephenixnoir commented 2020-02-09 13:41:13 +01:00
Owner
Copy Link

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 2020-02-09 18:55:42 +01:00
Author
Owner
Copy 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é. 😉

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:
👍 1
Lephenixnoir commented 2020-02-09 20:41:02 +01:00
Owner
Copy Link

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 2020-02-09 22:07:29 +01:00
Author
Owner
Copy Link

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 2020-02-09 22:07:32 +01:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
master
preprod
dev
search
landing_page
account/md_editor
debug_toolbar
new_editor
widgets
api/admin/members
Labels
Clear labels   
Core

Required to release v5

  
bug

Something is not working

  
duplicate

This issue or pull request already exists

  
easy

The fix is considered as easy to make

  
enhancement

New feature

  
help wanted

Need some help

  
invalid

Something is wrong

  
performance

Something is slow, not optimized

  
proposal

Feature to be discussed, RFC

  
question

More information is needed

  
security

This issue is a security breach

  
warning

It works, but for how long?

  
wontfix

This won't be fixed

No Label
Core
bug
duplicate
easy
enhancement
help wanted
invalid
performance
proposal
question
security
warning
wontfix
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
Lephenixnoir
3 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: devs/PCv5#37
No description provided.
Delete Branch "%!s(<nil>)"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?