The new get_readonly_fields() method of the ModelAdmin breaks with many to many fields
This breaks if a ManyToMany field is specified as readonly.
This is because the contents() function that is called in the fieldset.html template tries to display the value of the field but gets a ValueError if it is a ManyToManyField because they can't be used: "[Model name] instance needs to have a primary key value before a many-to-many relationship can be used"
Just need to add ValueError to the catch as seen in the patch I attached, called catch_value_error.diff.
Hope it gets merged before beta 2.
Component: |
Uncategorized → Contrib apps
|
Has patch: |
set
|
Version: |
1.1 → 1.2-beta
|
Component: |
Contrib apps → django.contrib.admin
|
milestone: |
→ 1.2
|
Triage Stage: |
Unreviewed → Accepted
|
Resolution: |
→ fixed
|
Status: |
new → closed
|
Very simple fix, just catch one more Exception