Fix admin errors on profile and procedure pages#1893
Open
zspirit wants to merge 4 commits intoMTES-MCT:mainfrom
Open
Fix admin errors on profile and procedure pages#1893zspirit wants to merge 4 commits intoMTES-MCT:mainfrom
zspirit wants to merge 4 commits intoMTES-MCT:mainfrom
Conversation
Author
|
cc @celine-m-s @rik pour review |
Quand DJANGO_SETTINGS_MODULE pointe sur core.settings.dev (via .env), le CompressedManifestStaticFilesStorage attend un collectstatic préalable, ce qui fait échouer les tests admin. Fixes MTES-MCT#1847
Le champ other_poste utilisait les choices de RoleType y compris pour les profils côté collectivité, qui saisissent du texte libre dans Nuxt. Le formulaire admin refusait ces valeurs à la validation. Fixes MTES-MCT#1793
zspirit
force-pushed
the
fix/admin-issues-1847-1793
branch
from
43d8e0a to
0a54a35
Compare
…nt (MTES-MCT#1714) Le fallback de is_intercommunal et is_sectoriel_consolide comptait les COMD, contrairement à l'annotation with_perimetre_counts qui les exclut déjà.
…ES-MCT#1788) Remplace le lookup statique par un calcul basé sur type_document. Affiche le vrai type (PLUi, PLUiH, etc.) et gère plusieurs procédures en cours.
|
Collaborator
|
Bonsoir @zspirit , |
This was
linked to
issues
Mar 23, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Cinq bugs admin/backend corrigés.
Tests admin cassés (#1847)
Le
.envde dev définitDJANGO_SETTINGS_MODULE=core.settings.dev, ce qui écrase la config pytest (core.settings.test). Les tests tournent donc avecCompressedManifestStaticFilesStoragequi attend un manifest généré parcollectstatic— absent en local, d'où le crash sur toutes les pages admin.Corrigé via une fixture autouse dans
conftest.pyqui forceStaticFilesStorage.Champ "Autre poste" bloqué pour les collectivités (#1793)
other_posteest unArrayFielddont lebase_fieldachoices=RoleType. Côté PPA c'est cohérent (sélection dans une liste), mais côté collectivité les utilisateurs saisissent du texte libre dans Nuxt. L'admin Django refusait ces valeurs parce que la validation choices s'applique à deux niveaux : formulaire et modèle (full_clean).Corrigé avec un
ProfileAdminFormqui redéfinitother_posteenSimpleArrayField(CharField())et l'exclut de la validation modèle.Communes déléguées comptées dans le fallback périmètre (#1714)
is_intercommunaletis_sectoriel_consolideutilisent un fallbackself.perimetre.count()quand l'annotation n'est pas disponible (vue change admin). Ce fallback ne filtrait pas les COMD, contrairement à l'annotationwith_perimetre_countsqui fait déjàcommune__nouvelle=None.Corrigé en ajoutant
.filter(nouvelle=None)au fallback.Libellé code état simplifié statique (#1787, #1788)
libelle_code_etat_simplifieutilisait un dictionnaire statique qui ne distinguait pas PLU/PLUi/PLUiH/etc. (tous mappés sur le code 3). Le libellé affichait toujours "PLU" même pour un PLUi sectoriel.Remplacé par un calcul dynamique basé sur
type_documentqui affiche le vrai type de document et gère les cas avec plusieurs procédures en cours.