Commit d0a42848 by Clinton Blackburn

Corrected read-only field display for CourseEnrollment admin

Read-only fields are read-only for existing models. These values can be set when creating new models.

ECOM-2172
parent d0b6d181
...@@ -137,7 +137,12 @@ class CourseEnrollmentAdmin(admin.ModelAdmin): ...@@ -137,7 +137,12 @@ class CourseEnrollmentAdmin(admin.ModelAdmin):
list_display = ('id', 'course_id', 'mode', 'user', 'is_active',) list_display = ('id', 'course_id', 'mode', 'user', 'is_active',)
list_filter = ('mode', 'is_active',) list_filter = ('mode', 'is_active',)
search_fields = ('course_id', 'mode', 'user__username',) search_fields = ('course_id', 'mode', 'user__username',)
readonly_fields = ('course_id', 'mode', 'user',)
def get_readonly_fields(self, request, obj=None):
# The course_id, mode, and user fields should not be editable for an existing enrollment.
if obj:
return self.readonly_fields + ('course_id', 'mode', 'user',)
return self.readonly_fields
class Meta(object): # pylint: disable=missing-docstring class Meta(object): # pylint: disable=missing-docstring
model = CourseEnrollment model = CourseEnrollment
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment