Opened 9 years ago

Closed 9 years ago

#6238 closed (fixed)

String size limit too low for model instances in databrowse

Reported by: Nick Efford Owned by: Nick Efford
Component: contrib.databrowse Version: master
Severity: Keywords:
Cc: Triage Stage: Unreviewed
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: UI/UX:

Description

The databrowse app imposes a hard-coded size limit of 30 characters on the string representations of model instances. This can cause problems if you have a model with __unicode__ method that creates lengthy strings.

The attached patch raises the limit to 100 characters. It also makes this value a module-level constant called STRING_SIZE_LIMIT, to improve code clarity.

Attachments (1)

databrowse.diff (724 bytes) - added by Nick Efford 9 years ago.
Patch to raise the string size limit in the databrowse app

Download all attachments as: .zip

Change History (2)

Changed 9 years ago by Nick Efford

Attachment: databrowse.diff added

Patch to raise the string size limit in the databrowse app

comment:1 Changed 9 years ago by Malcolm Tredinnick

Resolution: fixed
Status: newclosed

(In [6948]) Fixed #6238 -- Bumped up the length of data displayed in databrowse slightly. Patch from Nick Efford.

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