Ajouter des nouveaux types de documents

[Anto59290]

No description provided.

[alanzirek]

Quelques suggestions d'amélioration :

• suppression du préfixe redondant DOCUMENT_ et du nom de la classe (TypeDocument -> Type) puisque le contexte est déjà clair
• utilisation de underscores dans les valeurs stockées plutôt que des espaces
• placement de AUTRE à la fin de l'énumération

[alanzirek]

Suggested change

	class TypeDocument(models.TextChoices):
	DOCUMENT_ARRETE = "arrete prefectoral ministériel", "Arrêté préfectoral/ministériel"
	DOCUMENT_AUTRE = "autre", "Autre document"
	DOCUMENT_CARTOGRAPHIE = "cartographie", "Cartographie"
	DOCUMENT_CERTIFICAT_PHYTOSANITAIRE = "certificat phytosanitaire", "Certificat phytosanitaire"
	DOCUMENT_COMPTE_RENDU_REUNION = "compte rendu reunion", "Compte rendu de réunion"
	DOCUMENT_COURRIER_OFFICIEL = "courrier officiel", "Courrier officiel"
	DOCUMENT_DSCE = "dsce", "DSCE"
	DOCUMENT_FACTURE = "facture", "Facture"
	DOCUMENT_IMAGE = "image", "Image"
	DOCUMENT_PASSEPORT_PHYTOSANITAIRE = "passeport phytosanitaire", "Passeport phytosanitaire"
	DOCUMENT_RAPPORT_ANALYSE = "rapport analyse", "Rapport d'analyse"
	DOCUMENT_RAPPORT_INSPECTION = "rapport inspection", "Rapport d'inspection"
	DOCUMENT_REGLEMENTATION = "reglementation", "Réglementation"
	DOCUMENT_TRANSPORT = "document de transport", "Document de transport"
	class Type(models.TextChoices):
	ARRETE = "arrete_prefectoral", "Arrêté préfectoral/ministériel"
	CARTOGRAPHIE = "cartographie", "Cartographie"
	CERTIFICAT_PHYTOSANITAIRE = "certificat_phytosanitaire", "Certificat phytosanitaire"
	COMPTE_RENDU = "compte_rendu", "Compte rendu de réunion"
	COURRIER_OFFICIEL = "courrier_officiel", "Courrier officiel"
	DSCE = "dsce", "DSCE"
	FACTURE = "facture", "Facture"
	IMAGE = "image", "Image"
	PASSEPORT_PHYTOSANITAIRE = "passeport_phytosanitaire", "Passeport phytosanitaire"
	RAPPORT_ANALYSE = "rapport_analyse", "Rapport d'analyse"
	RAPPORT_INSPECTION = "rapport_inspection", "Rapport d'inspection"
	REGLEMENTATION = "reglementation", "Réglementation"
	TRANSPORT = "transport", "Document de transport"
	AUTRE = "autre", "Autre document"

[Anto59290]

suppression du préfixe redondant DOCUMENT_ et du nom de la classe puisque le contexte est déjà clair

Pourquoi pas pour les valeurs, mais en général on évite de nommer de chose Type (pour éviter les possible interference avec le mot clé réservé, même si il est en miniscule)

utilisation de underscores dans les valeurs stockées plutôt que des espaces

Pourquoi cette proposition de changement ?

placement de AUTRE à la fin de l'énumération

Idem, pourquoi ne pas respecter l'ordre alphabétique ?
@alanzirek

[alanzirek]

Pourquoi pas pour les valeurs, mais en général on évite de nommer de chose Type (pour éviter les possible interference avec le mot clé réservé, même si il est en miniscule)

Je pense qu'il y a aucun risque d'interférence puisque Type est une classe interne et son accès ne fait via Document.Type.ARRETE non ?
Mais ok aussi avec TypeDocument : Document.TypeDocument.ARRETE

Pourquoi cette proposition de changement ?

Pour garder une cohérence car c'est ce qui est fait sur tous les TextChoice du projet.

Idem, pourquoi ne pas respecter l'ordre alphabétique ?

Pour pouvoir le distinguer de toutes les autres valeurs sur l'interface lorsque la liste est affichée.
Mais peut-être valider ça avec le métier ou design ?