Hide "Protected" privacy level from users

docs/privacy.rst

@@ -2,7 +2,7 @@ Privacy Levels
 ==============
 
 Read the Docs supports 3 different privacy levels on 2 different objects;
-Public, Protected, Private on Projects and Versions.
+Public, Private on Projects and Versions.
 
 Understanding the Privacy Levels
 --------------------------------
@@ -12,8 +12,6 @@ Understanding the Privacy Levels
 +============+============+===========+===========+=============+
 | Private    | No         | No        | No        | Yes         |
 +------------+------------+-----------+-----------+-------------+
-| Protected  | Yes        | No        | No        | Yes         |
-+------------+------------+-----------+-----------+-------------+
 | Public     | Yes        | Yes       | Yes       | Yes         |
 +------------+------------+-----------+-----------+-------------+
 
@@ -27,18 +25,6 @@ Public
 This is the easiest and most obvious. It is also the default.
 It means that everything is available to be seen by everyone.
 
-Protected
-~~~~~~~~~
-
-Protected means that your object won't show up in Listing Pages,
-but Detail pages still work. For example, a Project that is Protected will
-not show on the homepage Recently Updated list, however,
-if you link directly to the project, you will get a 200 and the page will display.
-
-Protected Versions are similar, they won't show up in your version listings,
-but will be available once linked to.
-
-
 Private
 ~~~~~~~
 

readthedocs/builds/forms.py

@@ -6,10 +6,11 @@
 from django.utils.translation import ugettext_lazy as _
 
 from readthedocs.builds.models import Version
+from readthedocs.core.mixins import HideProtectedLevelMixin
 from readthedocs.core.utils import trigger_build
 
 
-class VersionForm(forms.ModelForm):
+class VersionForm(HideProtectedLevelMixin, forms.ModelForm):
 
     class Meta:
         model = Version

readthedocs/core/mixins.py

@@ -4,8 +4,11 @@
 
 from django.contrib.auth.decorators import login_required
 from django.utils.decorators import method_decorator
+from django.utils.translation import ugettext_lazy as _
 from vanilla import ListView
 
+from readthedocs.projects.constants import PRIVACY_CHOICES, PROTECTED
+
 
 class ListViewWithForm(ListView):
 
@@ -22,3 +25,29 @@ class LoginRequiredMixin:
     @method_decorator(login_required)
     def dispatch(self, *args, **kwargs):
         return super().dispatch(*args, **kwargs)
+
+
+class HideProtectedLevelMixin:
+
+    """
+    Hide ``protected`` privacy level from Form.
+
+    Remove Protected for now since it cause confusions to users.
+
+    If the current ``privacy_level`` is ``protected`` we show it (so users keep
+    seeing consistency values), and hide it otherwise (so it can't be selected).
+
+    There is a better way to manage this by using Version states See:
+    https://github.com/rtfd/readthedocs.org/issues/5321
+    """
+
+    def __init__(self, *args, **kwargs):
+        super().__init__(*args, **kwargs)
+
+        if any([
+            not self.instance,
+            self.instance and self.instance.privacy_level != PROTECTED,
+        ]):
+            privacy_level = list(PRIVACY_CHOICES)
+            privacy_level.remove((PROTECTED, _('Protected')))
+            self.fields['privacy_level'].choices = privacy_level

readthedocs/projects/forms.py

@@ -14,6 +14,7 @@
 from guardian.shortcuts import assign
 from textclassifier.validators import ClassifierValidator
 
+from readthedocs.core.mixins import HideProtectedLevelMixin
 from readthedocs.core.utils import slugify, trigger_build
 from readthedocs.core.utils.extend import SettingsOverrideObject
 from readthedocs.integrations.models import Integration
@@ -189,7 +190,7 @@ def clean_tags(self):
         return tags
 
 
-class ProjectAdvancedForm(ProjectTriggerBuildMixin, ProjectForm):
+class ProjectAdvancedForm(HideProtectedLevelMixin, ProjectTriggerBuildMixin, ProjectForm):
 
     """Advanced project option form."""
 

readthedocs/projects/models.py

@@ -329,8 +329,7 @@ class Project(models.Model):
         choices=constants.PRIVACY_CHOICES,
         default=settings.DEFAULT_PRIVACY_LEVEL,
         help_text=_(
-            'Level of privacy that you want on the repository. '
-            'Protected means public but not in listings.',
+            'Level of privacy that you want on the repository.',
         ),
     )
     version_privacy_level = models.CharField(