Project

General

Profile

Bug #54702

psycopg2.DataError: integer out of range

Added by Frédéric Péters 8 days ago. Updated 7 days ago.

Status:
Résolu (à déployer)
Priority:
Normal
Target version:
-
Start date:
09 Jun 2021
Due date:
% Done:

0%

Estimated time:
Patch proposed:
Yes
Planning:
No

Description

Sur un champ qui a une validation "chiffres" wcs-olap va créer une colonne de type integer; mais on voit que cette validation est également utilisée pour des champs qui ne sont pas des nombres.

Traceback (most recent call last):
  File "/usr/lib/python3/dist-packages/wcs_olap/cmd.py", line 108, in main2
    olap_feeder.feed()
  File "/usr/lib/python3/dist-packages/wcs_olap/feeder.py", line 575, in feed
    formdef_feeder.feed()
  File "/usr/lib/python3/dist-packages/wcs_olap/feeder.py", line 1118, in feed
    self.do_data()
  File "/usr/lib/python3/dist-packages/wcs_olap/feeder.py", line 918, in do_data
    vars=values)
  File "/usr/lib/python3/dist-packages/wcs_olap/feeder.py", line 326, in ex
    self.cur.execute(sql, vars=vars)
psycopg2.DataError: integer out of range

Files

Associated revisions

Revision f8c4a034 (diff)
Added by Benjamin Dauvergne 7 days ago

tests: add invalid integer to wcs fixture (#54702)

Revision 9bd160e0 (diff)
Added by Frédéric Péters 7 days ago

do not store digits value if outside integer range (#54702)

History

#1

Updated by Frédéric Péters 8 days ago

(alternativement on pourrait passer à BIGINT pour juste éloigner le problème).

#3

Updated by Benjamin Dauvergne 8 days ago

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

Ok. Passer par la validation était un moyen simple d'avancer mais c'était peut-être une mauvaise idée, sauf à avoir des validations sémantiquement plus explicites , je me dis que l'idée initiale de passer par extra_class avec une valeur par exemple "stat-integer-quantity" n'était pas si mal.

#4

Updated by Benjamin Dauvergne 7 days ago

  • Assignee set to Benjamin Dauvergne
#5

Updated by Benjamin Dauvergne 7 days ago

  • Status changed from Solution validée to Résolu (à déployer)
commit 9bd160e0dddfb19d203d7a6780259bc003313170
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Wed Jun 9 14:29:31 2021 +0200

    do not store digits value if outside integer range (#54702)

commit f8c4a034642d97299755dde8ae1ddb850e584113
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date:   Fri Jun 11 07:41:08 2021 +0200

    tests: add invalid integer to wcs fixture (#54702)

Also available in: Atom PDF