Updates from review

Author: stsewd

readthedocs/organizations/views/private.py

@@ -221,7 +221,7 @@ def _get_csv_data(self):
     def get_context_data(self, **kwargs):
         organization = self.get_organization()
         context = super().get_context_data(**kwargs)
-        context['enabled'] = self._is_enabled(organization)
+        context['enabled'] = self._is_feature_enabled(organization)
         context['days_limit'] = self._get_retention_days_limit(organization)
         context['filter'] = self.filter
         context['AuditLog'] = AuditLog
@@ -242,7 +242,7 @@ def _get_start_date(self):
     def _get_queryset(self):
         """Return the queryset without filters."""
         organization = self.get_organization()
-        if not self._is_enabled(organization):
+        if not self._is_feature_enabled(organization):
             return AuditLog.objects.none()
         start_date = self._get_start_date()
         queryset = AuditLog.objects.filter(
@@ -282,7 +282,7 @@ def _get_retention_days_limit(self, organization):
             default=settings.RTD_AUDITLOGS_DEFAULT_RETENTION_DAYS,
         )
 
-    def _is_enabled(self, organization):
+    def _is_feature_enabled(self, organization):
         return PlanFeature.objects.has_feature(
             organization,
             type=self.feature_type,

readthedocs/settings/base.py

@@ -683,6 +683,8 @@ def DOCKER_LIMITS(self):
     DEFAULT_PRIVACY_LEVEL = 'public'
     DEFAULT_VERSION_PRIVACY_LEVEL = 'public'
     ALLOW_ADMIN = True
+    # Enable all features by default,
+    # otherwise the organization subscription is checked.
     RTD_ALL_FEATURES_ENABLED = True
 
     # Organization settings