Skip to content
Snippets Groups Projects
Commit c87df89b authored by stvn's avatar stvn
Browse files

Merge PR #24055 regisb/fix-lti-provider-admin

* Commits:
  Fix creation of LTIProviderConfig object in admin
parents d7e6e01b 288ccc67
No related branches found
No related tags found
No related merge requests found
......@@ -223,9 +223,9 @@ class ProviderConfig(ConfigurationModel):
abstract = True
def clean(self):
""" Ensure that either `icon_class` or `icon_image` is set """
""" Ensure that at most `icon_class` or `icon_image` is set """
super(ProviderConfig, self).clean()
if bool(self.icon_class) == bool(self.icon_image):
if bool(self.icon_class) and bool(self.icon_image):
raise ValidationError('Either an icon class or an icon image must be given (but not both)')
@property
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment