Update for #179

amonapp · Sep 17, 2017 · 8590325 · 8590325
1 parent 999df9b
commit 8590325
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -5,6 +5,7 @@
 
 ### Bugfixes
 
+- [#179](https://github.com/amonapp/amon/issues/179): Tags page error when creating a tag without a group
 - [#162](https://github.com/amonapp/amon/issues/162): Email sending not fully functional - replace legacy mailer with Django
 
 

diff --git a/amon/apps/api/views/tags.py b/amon/apps/api/views/tags.py
@@ -20,11 +20,12 @@ def get(self, request):
             group = tag.get('group')
             if group:
                 data['group'] = dict_from_cursor(data=group, keys=['name', '_id'])
+            else:
+                data['group'] = {}
 
             filtered_tags.append(data)
 
-
-        filtered_tags.sort(key=lambda e: e.get('group', {}).get('name'))
+        filtered_tags.sort(key=lambda e: e.get('group', {}).get('name', ''))
 
         status = settings.API_RESULTS['ok']
Original file line number	Diff line number	Diff line change
Expand Up		@@ -5,6 +5,7 @@

		### Bugfixes

		- [#179](https://github.com/amonapp/amon/issues/179): Tags page error when creating a tag without a group
		- [#162](https://github.com/amonapp/amon/issues/162): Email sending not fully functional - replace legacy mailer with Django


Expand Down