Opened 8 years ago

Closed 8 years ago

#4518 closed (fixed)

Null string in DecimalField results in error: decimal.py _raise_error, line 2267

Reported by: Richard House <Richard.House@…> Owned by: adrian
Component: Core (Other) Version: master
Severity: Keywords: DecimalField
Cc: Triage Stage: Unreviewed
Has patch: no Needs documentation:
Needs tests: Patch needs improvement:
Easy pickings: UI/UX:

Description

I am running the django web server/sqlite on WinXP with Python 2.4 and the latest django SVN version. In keeping up with the FloatField to DecimalField changes I came across an error in both the admin pages and user pages with one of my apps.
In the debug page (excellent feature), an InvalidOperation was being raised at location "C:\Python24\lib\decimal.py in _raise_error, line 2267".

I traced the cause to some of my DecimalField values had empty string values at some stage of the process. I have blank=True and null=True on that field. A change to typecast_decimal function at line 96 of C:
\Python24\Lib\site-packages\django\db\backends\util.py solved the problem.

def typecast_decimal(s):
    if s is None:
        return None
    return decimal.Decimal(s)

became:

def typecast_decimal(s):
    if s is None or s == '':  ### Change
        return None
    return decimal.Decimal(s)

Suggested for a SVN change.

Change History (1)

comment:1 Changed 8 years ago by mtredinnick

  • Resolution set to fixed
  • Status changed from new to closed

(In [5450]) Fixed #4518 -- Added handling of empty strings in typecast_decimal() for
SQLite's benefit. Thanks, Richard House.

Note: See TracTickets for help on using tickets.
Back to Top