fix: Added support for django42

Author: zubairshakoorarbisoft

analytics_data_api/urls.py

@@ -1,11 +1,11 @@
-from django.conf.urls import include, url
+from django.urls import include, re_path
 from rest_framework.urlpatterns import format_suffix_patterns
 
 app_name = 'analytics_data_api'
 
 urlpatterns = [
-    url(r'^v0/', include('analytics_data_api.v0.urls')),
-    url(r'^v1/', include('analytics_data_api.v1.urls')),
+    re_path(r'^v0/', include('analytics_data_api.v0.urls')),
+    re_path(r'^v1/', include('analytics_data_api.v1.urls')),
 ]
 
 urlpatterns = format_suffix_patterns(urlpatterns)

analytics_data_api/v0/__init__.py

@@ -1 +0,0 @@
-default_app_config = 'analytics_data_api.v0.apps.ApiAppConfig'

analytics_data_api/v0/models.py

@@ -135,7 +135,7 @@ class Meta(BaseCourseModel.Meta):
 
     module_id = models.CharField(db_index=True, max_length=255)
     part_id = models.CharField(db_index=True, max_length=255)
-    correct = models.NullBooleanField()
+    correct = models.BooleanField(default=None, null=True)
     value_id = models.CharField(db_index=True, max_length=255, null=True)
     answer_value = models.TextField(null=True, db_column='answer_value_text')
     variant = models.IntegerField(null=True)

analytics_data_api/v0/urls/__init__.py

@@ -1,21 +1,20 @@
-from django.conf.urls import include, url
-from django.urls import reverse_lazy
+from django.urls import include, path, reverse_lazy
 from django.views.generic import RedirectView
 
 app_name = 'analytics_data_api.v0'
 
 COURSE_ID_PATTERN = r'(?P<course_id>[^/+]+[/+][^/+]+[/+][^/]+)'
 
 urlpatterns = [
-    url(r'^courses/', include('analytics_data_api.v0.urls.courses')),
-    url(r'^problems/', include('analytics_data_api.v0.urls.problems')),
-    url(r'^videos/', include('analytics_data_api.v0.urls.videos')),
-    url('^', include('analytics_data_api.v0.urls.learners')),
-    url('^', include('analytics_data_api.v0.urls.course_summaries')),
-    url('^', include('analytics_data_api.v0.urls.programs')),
+    path('courses/', include('analytics_data_api.v0.urls.courses')),
+    path('problems/', include('analytics_data_api.v0.urls.problems')),
+    path('videos/', include('analytics_data_api.v0.urls.videos')),
+    path('', include('analytics_data_api.v0.urls.learners')),
+    path('', include('analytics_data_api.v0.urls.course_summaries')),
+    path('', include('analytics_data_api.v0.urls.programs')),
 
     # pylint: disable=no-value-for-parameter
-    url(r'^authenticated/$', RedirectView.as_view(url=reverse_lazy('authenticated')), name='authenticated'),
-    url(r'^health/$', RedirectView.as_view(url=reverse_lazy('health')), name='health'),
-    url(r'^status/$', RedirectView.as_view(url=reverse_lazy('status')), name='status'),
+    path('authenticated/', RedirectView.as_view(url=reverse_lazy('authenticated')), name='authenticated'),
+    path('health/', RedirectView.as_view(url=reverse_lazy('health')), name='health'),
+    path('status/', RedirectView.as_view(url=reverse_lazy('status')), name='status'),
 ]

analytics_data_api/v0/urls/course_summaries.py

@@ -1,9 +1,9 @@
-from django.conf.urls import url
+from django.urls import re_path
 
 from analytics_data_api.v0.views import course_summaries as views
 
 app_name = 'course_summaries'
 
 urlpatterns = [
-    url(r'^course_summaries/$', views.CourseSummariesView.as_view(), name='course_summaries'),
+    re_path(r'^course_summaries/$', views.CourseSummariesView.as_view(), name='course_summaries'),
 ]

analytics_data_api/v0/urls/courses.py

@@ -1,4 +1,4 @@
-from django.conf.urls import url
+from django.urls import re_path
 
 from analytics_data_api.v0.urls import COURSE_ID_PATTERN
 from analytics_data_api.v0.views import courses as views
@@ -25,4 +25,4 @@
 
 for path, view, name in COURSE_URLS:
     regex = fr'^{COURSE_ID_PATTERN}/{path}/$'
-    urlpatterns.append(url(regex, view.as_view(), name=name))
+    urlpatterns.append(re_path(regex, view.as_view(), name=name))

analytics_data_api/v0/urls/learners.py

@@ -1,4 +1,4 @@
-from django.conf.urls import url
+from django.urls import re_path
 
 from analytics_data_api.constants.learner import UUID_REGEX_PATTERN
 from analytics_data_api.v0.views import learners as views
@@ -8,6 +8,6 @@
 USERNAME_PATTERN = r'(?P<username>[\w.+-]+)'
 
 urlpatterns = [
-    url(fr'^enterprise/(?P<enterprise_customer>{UUID_REGEX_PATTERN})/engagements/$',
-        views.EnterpriseLearnerEngagementView.as_view(), name='engagements'),
+    re_path(fr'^enterprise/(?P<enterprise_customer>{UUID_REGEX_PATTERN})/engagements/$',
+            views.EnterpriseLearnerEngagementView.as_view(), name='engagements'),
 ]

analytics_data_api/v0/urls/problems.py

@@ -1,6 +1,6 @@
 import re
 
-from django.conf.urls import url
+from django.urls import re_path
 
 from analytics_data_api.v0.views import problems as views
 
@@ -12,9 +12,10 @@
 ]
 
 urlpatterns = [
-    url(r'^(?P<module_id>.+)/sequential_open_distribution/$',
-        views.SequentialOpenDistributionView.as_view(), name='sequential_open_distribution'),
+    re_path(r'^(?P<module_id>.+)/sequential_open_distribution/$',
+            views.SequentialOpenDistributionView.as_view(),
+            name='sequential_open_distribution'),
 ]
 
 for path, view, name in PROBLEM_URLS:
-    urlpatterns.append(url(r'^(?P<problem_id>.+)/' + re.escape(path) + r'/$', view.as_view(), name=name))
+    urlpatterns.append(re_path(r'^(?P<problem_id>.+)/' + re.escape(path) + r'/$', view.as_view(), name=name))

analytics_data_api/v0/urls/programs.py

@@ -1,9 +1,9 @@
-from django.conf.urls import url
+from django.urls import path
 
 from analytics_data_api.v0.views import programs as views
 
 app_name = 'programs'
 
 urlpatterns = [
-    url(r'^programs/$', views.ProgramsView.as_view(), name='programs'),
+    path('programs/', views.ProgramsView.as_view(), name='programs'),
 ]

analytics_data_api/v0/urls/videos.py

@@ -1,6 +1,6 @@
 import re
 
-from django.conf.urls import url
+from django.urls import re_path
 
 from analytics_data_api.v0.views import videos as views
 
@@ -13,4 +13,4 @@
 urlpatterns = []
 
 for path, view, name in VIDEO_URLS:
-    urlpatterns.append(url(r'^(?P<video_id>.+)/' + re.escape(path) + r'/$', view.as_view(), name=name))
+    urlpatterns.append(re_path(r'^(?P<video_id>.+)/' + re.escape(path) + r'/$', view.as_view(), name=name))