Modify Enketo submission and formList urls

Modify Enketo submission and formList urls by changing the name of the
"form_pk" arguement to "xform_pk"

onadata/apps/api/tests/viewsets/test_xform_list_viewset.py

@@ -1005,7 +1005,7 @@ def test_retrieve_form_using_pk(self):
         request = self.factory.get(
             f'/enketo/{self.xform.pk}/formList')
         response = self.view(
-            request, form_pk=self.xform.pk)
+            request, xform_pk=self.xform.pk)
         self.assertEqual(response.status_code, 401)
 
         # Set require auth to false for form owner
@@ -1030,7 +1030,7 @@ def test_retrieve_form_using_pk(self):
             f'/enketo/{self.xform.pk}/formList')
         request.META.update(auth(request.META, response))
         response = self.view(
-            request, form_pk=self.xform.pk)
+            request, xform_pk=self.xform.pk)
         self.assertEqual(response.status_code, 200)
         self.assertEqual(len(response.data), 1)
         self.assertEqual(
@@ -1041,7 +1041,7 @@ def test_retrieve_form_using_pk(self):
         request = self.factory.get(
             f'/enketo/{self.xform.pk}/formList')
         response = self.view(
-            request, form_pk=self.xform.pk)
+            request, xform_pk=self.xform.pk)
         self.assertEqual(response.status_code, 200)
         self.assertEqual(len(response.data), 1)
         self.assertEqual(

onadata/apps/api/viewsets/xform_list_viewset.py

@@ -73,7 +73,7 @@ def get_renderers(self):
 
     def filter_queryset(self, queryset):
         username = self.kwargs.get('username')
-        form_pk = self.kwargs.get('form_pk')
+        form_pk = self.kwargs.get('xform_pk')
         if (not username and not form_pk) and \
                 self.request.user.is_anonymous:
             # raises a permission denied exception, forces authentication
@@ -84,8 +84,9 @@ def filter_queryset(self, queryset):
             profile = get_object_or_404(
                 UserProfile, user__username=username)
         elif form_pk:
-            form = get_object_or_404(XForm, pk=form_pk)
-            profile = form.user.profile
+            queryset = queryset.filter(pk=form_pk)
+            if queryset.first():
+                profile = queryset.first().user.profile
 
         if profile:
             if profile.require_auth and self.request.user.is_anonymous:

onadata/apps/main/urls.py

@@ -224,7 +224,7 @@
     re_path(r'^(?P<username>\w+)/formList$',
         XFormListViewSet.as_view({'get': 'list', 'head': 'list'}),
         name='form-list'),
-    re_path(r'^enketo/(?P<form_pk>\w+)/formList$',
+    re_path(r'^enketo/(?P<xform_pk>\w+)/formList$',
         XFormListViewSet.as_view({'get': 'list', 'head': 'list'}),
         name='form-list'),
     re_path(r'^(?P<username>\w+)/(?P<xform_pk>\d+)/formList$',
@@ -258,7 +258,7 @@
     re_path(r'^(?P<username>\w+)/submission$',
         XFormSubmissionViewSet.as_view({'post': 'create', 'head': 'create'}),
         name='submissions'),
-    re_path(r'^enketo/(?P<form_pk>\w+)/submission$',
+    re_path(r'^enketo/(?P<xform_pk>\w+)/submission$',
         XFormSubmissionViewSet.as_view({'post': 'create', 'head': 'create'}),
         name='submissions'),
     re_path(r'^(?P<username>\w+)/(?P<xform_pk>\d+)/submission$',

onadata/libs/filters.py

@@ -25,10 +25,6 @@ def filter_queryset(self, request, queryset, view):
             view.lookup_field, view.kwargs.get('xform_pk'))
         lookup_field = view.lookup_field
 
-        if view.kwargs.get('form_pk'):
-            form_id = view.kwargs.get('form_pk')
-            lookup_field = 'pk'
-
         queryset = queryset.filter(deleted_at=None)
         if request.user.is_anonymous:
             return queryset