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

Export RP en CSV #378

Closed
pierrejego opened this issue Jan 10, 2018 · 6 comments
Closed

Export RP en CSV #378

pierrejego opened this issue Jan 10, 2018 · 6 comments
Assignees
Milestone

Comments

@pierrejego
Copy link
Member

Il est demandé le développement d’un nouvelle fonction permettant à l’utilisateur de pouvoir demander le téléchargement de 3 fichiers CSV correspondants aux 3 blocs d’information du relevé de propriété :
propriétaires
propriétés bâties
propriétés non bâties

Ces 3 fichiers seront générés côté serveur et encapsulé dans un ZIP pour effectuer 1 seul téléchargement.

La commande permettant cet export sera rajoutée dans le traitement des sélections de la recherche avancée.

@pierrejego pierrejego self-assigned this Jan 10, 2018
@pierrejego pierrejego added this to the v 1.6 milestone Jan 10, 2018
@pierrejego
Copy link
Member Author

@MaelREBOUX et @jusabatier

J'ai commencé les exports, mais comme je le disais à Catherine, est-ce qu'il faut rajouter deux fichiers pour la partie imposition ?
Et est-ce que dans l'export, il faut garder les valeurs d'imposition par C,GC,TSE etc ainsi que les exonérations ? Et si oui, est-ce que je rajoute des colonnes pour chacun, ou est-ce que je duplique les lignes comme dans le RP ?

Deuxième question, j'ai commencé à utiliser la lib openCSV. Elle est sous licence apache2, c'est bien compatible avec geOrchestra ? http://opencsv.sourceforge.net/license.html

@landryb
Copy link
Member

landryb commented Jan 10, 2018

C'est pas plus simple d'avoir une api par fonction, et un menu déroulant permettant d'exporter qu'un seul des fichiers, plutot que de les bundler ensemble dans une fonction/zip ? je pense au cas ou la base est grosse, si tu fais 5 requetes 'lourdes' à la suite puis tu zippe le tout avant d'y renvoyer, ca va prendre plus de temps (vu de l'utilisateur) que 5 requetes distinctes, potentiellement faisables en parallèle...

@pierrejego
Copy link
Member Author

Non c'est pas plus simple, la je vais reprendre le même objet que pour générer le PDF du RP. Je vais juste faire des csv et un zip au lieu du PDF. ça devrait prendre presque le même temps que le RP actuelle (mais je crois que chez toi c'est long ....)

@MaelREBOUX
Copy link
Member

Point tel fait avec @pierrejego ce matin.

Les attributs des bases d'impositions seront mis dans des colonnes distinctes pour éviter de multiplier les linges par 3.

Il faut régler la question de compatibilité de licence également.

pierrejego pushed a commit that referenced this issue Feb 1, 2018
pierrejego pushed a commit that referenced this issue Feb 1, 2018
pierrejego pushed a commit that referenced this issue Feb 1, 2018
pierrejego pushed a commit that referenced this issue Feb 6, 2018
@MaelREBOUX
Copy link
Member

Testé sur master hier.
Données OK d’après ce que j’en vois.

Séparateur virgule alors qu’on point-virgule pour export lots, parcelles, proprios -> point-virgule pour homogénéiser ?

pierrejego pushed a commit that referenced this issue Feb 16, 2018
@MaelREBOUX
Copy link
Member

Point-virgule OK. Je ferme.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants