New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Suggested fixes for System Taxonomies #8
Suggested fixes for System Taxonomies #8
Conversation
to ensure that we're using the correct Taxonomy and ObjectTag class types
@@ -52,6 +52,7 @@ | |||
|
|||
AUTHENTICATION_BACKENDS = [ | |||
'rules.permissions.ObjectPermissionBackend', | |||
'django.contrib.auth.backends.ModelBackend', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I had to add this so that Django would let me login to the Admin?
@@ -29,7 +29,7 @@ def setUp(self): | |||
self.taxonomy = Taxonomy.objects.get(name="Life on Earth") | |||
self.system_taxonomy = Taxonomy.objects.get( | |||
name="System defined taxonomy" | |||
).cast() | |||
) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Suggested removing this so that the test_rules are testing with an un-cast system taxonomy, so we know the rules are handling the cast.
@@ -556,7 +558,6 @@ def tag_ref(self, tag_ref: str): | |||
try: | |||
self.tag = self.taxonomy.tag_set.get(pk=tag_ref) | |||
self.value = self.tag.value | |||
return |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: not needed anymore
and fixes nit
5b8a210
to
9a9e038
Compare
No description provided.