Opened 16 years ago

Closed 16 years ago

Last modified 13 years ago

#7885 closed (fixed)

Prevent duplicates in fields and fieldsets declarations

Reported by: Julien Phalip Owned by: wamberg
Component: contrib.admin Version: dev
Severity: Keywords:
Cc: Triage Stage: Ready for checkin
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

The patch I did for #4305 was only fixing the issue for 'fieldsets', so I closed it in favour of this ticket, which also takes care of 'fields'.

The attached patch prevents the multiple declarations of fields in the 'fields' and 'fieldsets' attributes. This is to prevent some confusion, having a same field multiple times in a form not making much sense and potentially leading to some unexpected behaviour when saving. Patch also contains tests.

Attachments (2)

duplicate_field_declarations.diff (2.5 KB ) - added by Julien Phalip 16 years ago.
Patch + tests
duplicate_field_declarations_2.diff (2.4 KB ) - added by wamberg 16 years ago.
Added python2.3 support to previous patch.

Download all attachments as: .zip

Change History (7)

by Julien Phalip, 16 years ago

Patch + tests

comment:1 by James Turk, 16 years ago

Triage Stage: UnreviewedAccepted

comment:2 by wamberg, 16 years ago

Owner: changed from nobody to wamberg

by wamberg, 16 years ago

Added python2.3 support to previous patch.

comment:3 by wamberg, 16 years ago

Triage Stage: AcceptedReady for checkin

Ready for checkin with Malcolm's blessing at DC Sprint

comment:4 by Brian Rosner, 16 years ago

Resolution: fixed
Status: newclosed

(In [8173]) Fixed #7885 -- Prevent duplicates in fields and fieldsets declarations. Thanks julien and wamberg for the patches.

comment:5 by Jacob, 13 years ago

milestone: 1.0 beta

Milestone 1.0 beta deleted

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