Project

General

Profile

Développement #34559

identifiants de champs avec des majuscules

Added by Frédéric Péters over 5 years ago. Updated over 5 years ago.

Status:
Fermé
Priority:
Normal
Start date:
05 July 2019
Due date:
% Done:

0%

Estimated time:
Patch proposed:
Yes
Planning:
No

Description

File "/usr/lib/python2.7/dist-packages/bijoe/visualization/views.py" in dispatch
  82.         self.visualization = self.visualization(request, cube)

File "/usr/lib/python2.7/dist-packages/bijoe/visualization/views.py" in visualization
  68.         self.form = forms.CubeForm(cube=self.cube, data=request.GET or request.POST)

File "/usr/lib/python2.7/dist-packages/bijoe/visualization/forms.py" in __init__
  189.                 for _id, label in dimension.members:

File "/usr/lib/python2.7/dist-packages/bijoe/engine.py" in members
  91.             cursor.execute(sql)

Exception Type: ProgrammingError at
/visualization/warehouse/demarches_sau_guichet_recette_grandlyon_com/formdata_contactez_nous_concernant_trabool_travail_en_cours/
Exception Value: ERREUR:  la colonne « field_date_derniere_action_GRA » n'existe pas
LINE 1: SELECT "field_date_derniere_action_GRA" AS value, "field_dat...
               ^
HINT:  Peut-être que vous souhaitiez référencer la colonne «
formdata_contactez_nous_concernant_trabool_travail_en_cours.field_date_derniere_action_gra » ou la colonne «
formdata_contactez_nous_concernant_trabool_travail_en_cours.field_date_derniere_action_ar ».

Files


Related issues

Blocked by BiJoe - Développement #35130: respecter la casse des noms de tableFermé30 July 2019

Actions

Associated revisions

Revision 1d3705a7 (diff)
Added by Benjamin Dauvergne over 5 years ago

keep case in tables' and fields' names (#34559)

Revision 0450cf32 (diff)
Added by Benjamin Dauvergne over 5 years ago

keep case of status stable in reference (#34559)

Revision 0afea49d (diff)
Added by Thomas Noël over 5 years ago

Revert "keep case of status stable in reference (#34559)"

This reverts commit 0450cf32ca4a57952a27c7ae6eac3e81a38ba300.

Revision 9a7a280d (diff)
Added by Thomas Noël over 5 years ago

Revert "keep case in tables' and fields' names (#34559)"

This reverts commit 1d3705a7b166034eaa34d896b9b1d174369e337b.

Revision 953ef05f (diff)
Added by Benjamin Dauvergne over 5 years ago

keep case in tables' and fields' names (#34559)

Revision e54a12df (diff)
Added by Benjamin Dauvergne over 5 years ago

keep case of status stable in reference (#34559)

History

#2

Updated by Benjamin Dauvergne over 5 years ago

  • Assignee set to Benjamin Dauvergne
#3

Updated by Benjamin Dauvergne over 5 years ago

Je dois avoir quelque chose dans les yeux parce que pour l'instant je ne vois pas où je ferai un lower dans wcs_olap qui amènerait que la casse ne soit pas conservée lors de la création du champ....

$ git grep lower
wcs_olap/feeder.py:            channel = data.submission.channel.lower()
wcs_olap/feeder.py:                'label': u'fonction %s' % name.lower(),
wcs_olap/feeder.py:                    'label': field.label.lower(),
wcs_olap/feeder.py:                    'label': field.label.lower(),
wcs_olap/feeder.py:                    'label': field.label.lower(),

C'est bizarrement uniquement sur les labels que je change la casse.

#4

Updated by Benjamin Dauvergne over 5 years ago

Ok je comprends c'est dans mes CREATE TABLE que j'oublie les quotes je pense.

#5

Updated by Benjamin Dauvergne over 5 years ago

#6

Updated by Marie Kuntz over 5 years ago

  • Target version set to Wishlist
#7

Updated by Benjamin Dauvergne over 5 years ago

  • Tracker changed from Développement to Bug
#9

Updated by Thomas Noël over 5 years ago

Ok après le merge des deux commits... ça me semble couvrir l'étendue des endroits.

#10

Updated by Thomas Noël over 5 years ago

  • Status changed from Solution proposée to Solution validée
#12

Updated by Benjamin Dauvergne over 5 years ago

  • Status changed from Solution validée to Résolu (à déployer)
commit 1d3705a7b166034eaa34d896b9b1d174369e337b
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date:   Sat Jul 6 14:26:30 2019 +0200

    keep case in tables' and fields' names (#34559)
#13

Updated by Frédéric Péters over 5 years ago

Si tu peux pousser ça rapidement qu'on puisse faire tourner la machinerie au moins une fois sur le SAU, c'est cool.

Ça a été taggué mais pas déployé sur la recette guichet grand lyon, je viens de le faire, j'ai manuellement lancé le /usr/lib/bijoe/import-wcs-data.sh.

Sur ce que pointe Marie dans #34558 ça a l'air ok.

Mais import-wcs-data.sh donne :

2019-07-25 08:31:25,714 DEBUG SQL: CREATE TABLE "formdata_Classement_d_une_voie_routiere"("status_id" smallint REFERENCES status_Classement_d_une_voie_routiere (id), "field_type_demande" smallint REFERENCES "formdata_Classement_d_une_voie_routiere_field_type_demande" (id), "field_compte_de_tiers" smallint REFERENCES "formdata_Classement_d_une_voie_routiere_field_compte_de_tiers" (id), "field_civilite" smallint REFERENCES "formdata_Classement_d_une_voie_routiere_field_civilite" (id), "field_date_depot" varchar, "field_date_AR" varchar, "field_date_AE" varchar, "field_date_cloture" varchar, "function__receiver" smallint REFERENCES role (id)) INHERITS ("formdata") VARS: None
2019-07-25 08:31:25,723 ERROR failed to synchronize with https://demarches.guichet-recette.grandlyon.com/
Traceback (most recent call last):
  File "/usr/lib/python2.7/dist-packages/wcs_olap/cmd.py", line 112, in main2
    feeder.feed()
  File "/usr/lib/python2.7/dist-packages/wcs_olap/feeder.py", line 517, in feed
    formdef_feeder.feed()
  File "/usr/lib/python2.7/dist-packages/wcs_olap/feeder.py", line 999, in feed
    self.do_data_table()
  File "/usr/lib/python2.7/dist-packages/wcs_olap/feeder.py", line 662, in do_data_table
    comment=u'formulaire %s' % self.formdef.schema.name)
  File "/usr/lib/python2.7/dist-packages/wcs_olap/feeder.py", line 352, in create_table
    self.ex(sql)
  File "/usr/lib/python2.7/dist-packages/wcs_olap/feeder.py", line 306, in ex
    self.cur.execute(sql, vars=vars)
ProgrammingError: ERREUR:  la relation « status_classement_d_une_voie_routiere » n'existe pas

Pareil trace sur le SaaS de recette (lui non plus n'avais pas été mis à jour),

2019-07-25 08:34:42,418 ERROR failed to synchronize with https://demarches-arles.test.entrouvert.org/
Traceback (most recent call last):
  File "/usr/lib/python2.7/dist-packages/wcs_olap/cmd.py", line 112, in main2
    feeder.feed()
  File "/usr/lib/python2.7/dist-packages/wcs_olap/feeder.py", line 517, in feed
    formdef_feeder.feed()
  File "/usr/lib/python2.7/dist-packages/wcs_olap/feeder.py", line 999, in feed
    self.do_data_table()
  File "/usr/lib/python2.7/dist-packages/wcs_olap/feeder.py", line 662, in do_data_table
    comment=u'formulaire %s' % self.formdef.schema.name)
  File "/usr/lib/python2.7/dist-packages/wcs_olap/feeder.py", line 352, in create_table
    self.ex(sql)
  File "/usr/lib/python2.7/dist-packages/wcs_olap/feeder.py", line 306, in ex
    self.cur.execute(sql, vars=vars)
ProgrammingError: relation "status_c2_inscription_scolaire_complementaire_cantine_session" does not exist
#14

Updated by Benjamin Dauvergne over 5 years ago

J'ai trouvé la ligne manquante mais j'ai peur d'être aller un poil trop loin encouragé par Thomas, ici on ne parlait que des varname des champs et du nom des champs dans la base SQL, bijoe conservant la casse et pas wcs-olap lors de la création des objets, mais mon dernier patch a aussi ajouté la conservation de la casse sur les noms de table pour le slug des formdefs donc; j'espère juste que ça va marcher dans bijoe sinon il faudra en plus un hotfix sur bijoe assez vite.

#15

Updated by Benjamin Dauvergne over 5 years ago

Ajout des quotes dans la référence à la table des statuts lors de la création de la colonne status.

#16

Updated by Benjamin Dauvergne over 5 years ago

J'ai posé la ligne à la main en recette GNM et relancer l'import pour sau et guichet-recette principal, ça passe, maintenant j'aimerai aller vérifier que tout roule coté bijoe mais plus de compte agent :/ Je cherche.

#17

Updated by Thomas Noël over 5 years ago

Je viens de faire un revert des deux patches ; en attendant le fix dans bijoe, semaine prochaine.

#18

Updated by Benjamin Dauvergne over 5 years ago

#19

Updated by Frédéric Péters over 5 years ago

Je viens de faire un revert des deux patches ; en attendant le fix dans bijoe, semaine prochaine.

Je viens de tagguer/déployer bijoe, si jamais ça dit à quelqu'un de relire ici.

#20

Updated by Frédéric Péters over 5 years ago

  • Status changed from Solution proposée to Résolu (à déployer)

validé/poussé.

commit e54a12df09e46feba7a21c9ee4b320ce1f415bbf
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date:   Thu Jul 25 10:26:47 2019 +0200

    keep case of status stable in reference (#34559)

commit 953ef05f005ee04d2bb26dd61750429aead95b5f
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date:   Sat Jul 6 14:26:30 2019 +0200

    keep case in tables' and fields' names (#34559)
#21

Updated by Frédéric Péters over 5 years ago

  • Status changed from Résolu (à déployer) to Solution déployée

Also available in: Atom PDF