Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Gestion des labels dans subdivision fiscale #168

Closed

Conversation

jusabatier
Copy link
Collaborator

Suite à https://github.com/GFI-Informatique/cadastrapp/issues/156

Cette modification permet d'utiliser les labels fournis par le cadastre dans la fiche de la parcelle, onglet Subdivision fiscale au niveau de "nature de la culture".

Ceci augmente la complexité de la requete via des "union", mais je n'ai pas trouvé de moyen plus simple pour gérer le fait que certaines fois les clefs correspondant aux labels soient "NULL" et donc non comparable.

Merci de me dire si vous voyez des simplifications/optimisations possibles afin que je les fasse.

@MaelREBOUX
Copy link
Member

J'aurais plutôt penché pour modifier les vues en base afin d'avoir la valeur décodée dans les tables.
Est-ce possible partout et dans ce cas précis ?

@landryb
Copy link
Member

landryb commented Aug 22, 2016

J'aurais plutôt penché pour modifier les vues en base afin d'avoir la valeur décodée dans les tables.

Disons qu'il faut qu'on garde un minimum de cohérence, sinon ca va rapidement devenir le b****l. On a déja des tables de correspondance pour certains champs dans commun/tables, ca me paraissait 'plus propre' de continuer sur le meme modele. Que t'apporte le fait d'avoir la valeur décodée dans la table ? C'est juste une jointure a ajouter dans la requete.... @pierrejego une préférence ?

@MaelREBOUX
Copy link
Member

+1 pour la cohérence et on est justement au tournant et on doit faire un choix

  • soit on se dépatouille pour avoir les valeurs dans les tables à la place de codes degré de difficulté ??)
  • soit le code devra tjs faire des appels aux tables de correspondances

Ce qu'il fait avoir à l'esprit c'est que les 2 parties (webapp et addon) doivent avoir les valeurs en clair.
Maintenant : comment on fait ?

cf #156 #189 #192

@MaelREBOUX
Copy link
Member

@pierrejego #262

@MaelREBOUX MaelREBOUX reopened this Oct 4, 2016
@MaelREBOUX MaelREBOUX added this to the bug fixes milestone Oct 4, 2016
@jusabatier jusabatier closed this Oct 4, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants