Skip to content

Commit 51d8838

Browse files
authored
Classifier bug fixes (#574)
* Return None if "Latest Pages" block has invalid (i.e. deleted) classifier term * Fix display issue with classifier terms in Wagtail Admin when editing a page.
1 parent 50468fe commit 51d8838

2 files changed

Lines changed: 7 additions & 3 deletions

File tree

coderedcms/blocks/base_blocks.py

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -52,7 +52,10 @@ def to_python(self, value):
5252
Convert the serialized value back into a python object.
5353
"""
5454
if isinstance(value, int):
55-
return self.target_model.objects.get(pk=value)
55+
try:
56+
return self.target_model.objects.get(pk=value)
57+
except self.target_model.DoesNotExist:
58+
return None
5659
return value
5760

5861
def get_prep_value(self, value):

coderedcms/templates/coderedcms/widgets/checkbox_classifiers.html

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@
55
{% if group %}
66
<div class="field-col col4">
77
<ul class="crx-checkbox-group">
8-
<li><label>{{ group }}</label>
8+
<li><label class="w-field__label">{{ group }}</label>
99
{% endif %}
1010
<ul {% if id %}id="{{ id }}_{{ index }}"{% endif %}>
1111
{% for option in options %}
@@ -18,9 +18,10 @@
1818
</div>
1919
{% endif %}
2020
{% if forloop.counter|divisibleby:3 %}
21-
<div class="clearfix"></div>
21+
<div style="clear:both;"></div>
2222
{% endif %}
2323
{% endfor %}
24+
<div style="clear:both;"></div>
2425
</div>
2526
{% endwith %}
2627
{% else %}

0 commit comments

Comments
 (0)