Suppression de paramètres obsolètes

[bruhnild]

Les paramètres suivants sont toujours présents dans le fichier base.py, mais ne sont plus utilisés aujourd'hui.

Ils étaient employés sur Rando V2, mais sont désormais obsolètes :

• SPLIT_TREKS_CATEGORIES_BY_PRACTICE
• SPLIT_TREKS_CATEGORIES_BY_ACCESSIBILITY
• SPLIT_TREKS_CATEGORIES_BY_ITINERANCY
• HIDE_PUBLISHED_TREKS_IN_TOPOLOGIES
• TREK_WITH_POIS_PICTURES

Il faudrait supprimer ces paramètres du code de Geotrek-admin, ainsi que toutes les références associées.

Une PR est en cours pour les retirer également de la documentation.

[submarcos]

Idealement on ne devrait supprimer dans une PR à part que les settings plus itilisés dans le code.
Il ya parmi ces réglages certains qui sont encore utilisés dans le code, piur ceux là il convient d'abord d'assurer qu'on les supprime, puis de les supprimer + arranger la doc en meme temps

• SPLIT_TREKS_CATEGORIES_BY_PRACTICE: dans les tests mais ne teste rien car plus de code specifique
• SPLIT_TREKS_CATEGORIES_BY_ACCESSIBILITY: plus utilisé du tout
• SPLIT_TREKS_CATEGORIES_BY_ITINERANCY: dans les tests mais ne teste rien car plus de code specifique
• HIDE_PUBLISHED_TREKS_IN_TOPOLOGIES: toujors utilisé dans le code, permet de forcer à "vide" la liste des itinéraires publiés liés à une topologie
• TREK_WITH_POIS_PICTURES: plus utilisé du tout

[babastienne]

HIDE_PUBLISHED_TREKS_IN_TOPOLOGIES: toujors utilisé dans le code, permet de forcer à "vide" la liste des itinéraires publiés liés à une topologie

Oui ça permet de créer un attribut published_treks mais j'ai l'impression que cet attribut n'est utilisé nul part, non ?

---

Pour moi si c'est présent dans du code mais qui ne sert à rien on fait d'une pierre deux coups : on vire les settings et le code mort associé.

Pour la doc on a déjà viré les références à ces settings puisque de de toute manière on a ré-organisé la doc complètement et que pour moi maintenir une doc avec des contenus qui n'ont pas de fonction ça ne pas de sens.

[submarcos]

Oui ça permet de créer un attribut published_treks mais j'ai l'impression que cet attribut n'est utilisé nul part, non ?

Bonne question

Pour moi l'idée c'est de ne pas virer des settings dans une PR de doc si on ne supprime pas le code associé. Autant faire ça en meme temps, et donc une PR dédiée de suppression code / test / doc