Revert "ENT-960 Fix catalogs creation bug due to request data format"

This reverts commit c2abe452.

course_discovery/apps/api/v1/tests/test_views/test_catalogs.py

@@ -129,10 +129,11 @@ class CatalogViewSetTests(ElasticsearchTestMixin, SerializationMixin, OAuth2Mixi
         data = {
             'name': 'Test Catalog',
             'query': '*:*',
-            'viewers': str(viewers),
+            'viewers': ','.join(viewers)
         }
 
-        response = self.client.post(self.catalog_list_url, data, format='json')
+        # NOTE: We explicitly avoid using the JSON data type so that we properly test string parsing.
+        response = self.client.post(self.catalog_list_url, data)
         self.assertEqual(response.status_code, 201)
 
         catalog = Catalog.objects.latest()
@@ -144,10 +145,10 @@ class CatalogViewSetTests(ElasticsearchTestMixin, SerializationMixin, OAuth2Mixi
         """ Verify no users are created if an error occurs while processing a create request. """
         # The missing name and query fields should trigger an error
         data = {
-            'viewers': str(['new-guy'])
+            'viewers': ['new-guy']
         }
         original_user_count = User.objects.count()
-        response = self.client.post(self.catalog_list_url, data, format='json')
+        response = self.client.post(self.catalog_list_url, data)
 
         self.assertEqual(response.status_code, 400)
         self.assertEqual(User.objects.count(), original_user_count)

course_discovery/apps/api/v1/views/catalogs.py

-import ast
 import datetime
 
 from django.db import transaction
@@ -33,13 +32,13 @@ class CatalogViewSet(viewsets.ModelViewSet):
     @transaction.atomic
     def create(self, request, *args, **kwargs):
         """ Create a new catalog. """
-        data = dict(request.data.copy())
+        data = request.data.copy()
         usernames = request.data.get('viewers', ())
 
         # Add support for parsing a comma-separated list from Swagger
         if isinstance(usernames, str):
-            usernames = ast.literal_eval(usernames)
-            data['viewers'] = usernames
+            usernames = usernames.split(',')
+            data.setlist('viewers', usernames)
 
         # Ensure the users exist
         for username in usernames: