From d1ce056288c97eea18e17f1b950194678618cefc Mon Sep 17 00:00:00 2001 From: Gabriel Rodríguez Alberich Date: Wed, 14 Dec 2022 18:15:46 +0100 Subject: Show follows and following counts on profile page And let their visibility be configured--- users/views/settings/__init__.py | 3 +- users/views/settings/interface.py | 111 +--------------------------------- users/views/settings/profile.py | 11 ++++ users/views/settings/settings_page.py | 108 +++++++++++++++++++++++++++++++++ 4 files changed, 122 insertions(+), 111 deletions(-) create mode 100644 users/views/settings/settings_page.py (limited to 'users/views/settings') diff --git a/users/views/settings/__init__.py b/users/views/settings/__init__.py index f7f4018..f3332d3 100644 --- a/users/views/settings/__init__.py +++ b/users/views/settings/__init__.py @@ -2,9 +2,10 @@ from django.utils.decorators import method_decorator from django.views.generic import RedirectView from users.decorators import identity_required -from users.views.settings.interface import InterfacePage, SettingsPage # noqa +from users.views.settings.interface import InterfacePage # noqa from users.views.settings.profile import ProfilePage # noqa from users.views.settings.security import SecurityPage # noqa +from users.views.settings.settings_page import SettingsPage # noqa @method_decorator(identity_required, name="dispatch") diff --git a/users/views/settings/interface.py b/users/views/settings/interface.py index e8c73a6..c8863b2 100644 --- a/users/views/settings/interface.py +++ b/users/views/settings/interface.py @@ -1,114 +1,5 @@ -from functools import partial -from typing import ClassVar - -from django import forms -from django.core.files import File -from django.shortcuts import redirect -from django.utils.decorators import method_decorator -from django.views.generic import FormView - -from core.models.config import Config, UploadedImage -from users.decorators import identity_required - - -@method_decorator(identity_required, name="dispatch") -class SettingsPage(FormView): - """ - Shows a settings page dynamically created from our settings layout - at the bottom of the page. Don't add this to a URL directly - subclass! - """ - - options_class = Config.IdentityOptions - template_name = "settings/settings.html" - section: ClassVar[str] - options: dict[str, dict[str, str | int]] - layout: dict[str, list[str]] - - def get_form_class(self): - # Create the fields dict from the config object - fields = {} - for key, details in self.options.items(): - field_kwargs = {} - config_field = self.options_class.__fields__[key] - if config_field.type_ is bool: - form_field = partial( - forms.BooleanField, - widget=forms.Select( - choices=[(True, "Enabled"), (False, "Disabled")] - ), - ) - elif config_field.type_ is UploadedImage: - form_field = forms.ImageField - elif config_field.type_ is str: - if details.get("display") == "textarea": - form_field = partial( - forms.CharField, - widget=forms.Textarea, - ) - else: - form_field = forms.CharField - elif config_field.type_ is int: - choices = details.get("choices") - if choices: - field_kwargs["widget"] = forms.Select(choices=choices) - for int_kwarg in {"min_value", "max_value", "step_size"}: - val = details.get(int_kwarg) - if val: - field_kwargs[int_kwarg] = val - form_field = forms.IntegerField - else: - raise ValueError(f"Cannot render settings type {config_field.type_}") - fields[key] = form_field( - label=details["title"], - help_text=details.get("help_text", ""), - required=details.get("required", False), - **field_kwargs, - ) - # Create a form class dynamically (yeah, right?) and return that - return type("SettingsForm", (forms.Form,), fields) - - def load_config(self): - return Config.load_identity(self.request.identity) - - def save_config(self, key, value): - Config.set_identity(self.request.identity, key, value) - - def get_initial(self): - config = self.load_config() - initial = {} - for key in self.options.keys(): - initial[key] = getattr(config, key) - return initial - - def get_context_data(self): - context = super().get_context_data() - context["section"] = self.section - # Gather fields into fieldsets - context["fieldsets"] = {} - for title, fields in self.layout.items(): - context["fieldsets"][title] = [context["form"][field] for field in fields] - return context - - def form_valid(self, form): - # Save each key - for field in form: - if field.field.__class__.__name__ == "ImageField": - # These can be cleared with an extra checkbox - if self.request.POST.get(f"{field.name}__clear"): - self.save_config(field.name, None) - continue - # We shove the preview values in initial_data, so only save file - # fields if they have a File object. - if not isinstance(form.cleaned_data[field.name], File): - continue - self.save_config( - field.name, - form.cleaned_data[field.name], - ) - return redirect(".") - - from activities.models.post import Post +from users.views.settings.settings_page import SettingsPage class InterfacePage(SettingsPage): diff --git a/users/views/settings/profile.py b/users/views/settings/profile.py index 518b916..f01fec4 100644 --- a/users/views/settings/profile.py +++ b/users/views/settings/profile.py @@ -5,6 +5,7 @@ from django.utils.decorators import method_decorator from django.views.generic import FormView from core.files import resize_image +from core.models.config import Config from users.decorators import identity_required @@ -38,6 +39,11 @@ class ProfilePage(FormView): ), required=False, ) + visible_follows = forms.BooleanField( + help_text="Whether or not to show your following and follower counts in your profile.", + widget=forms.Select(choices=[(True, "Visible"), (False, "Hidden")]), + required=False, + ) def get_initial(self): identity = self.request.identity @@ -47,6 +53,7 @@ class ProfilePage(FormView): "icon": identity.icon and identity.icon.url, "image": identity.image and identity.image.url, "discoverable": identity.discoverable, + "visible_follows": identity.config_identity.visible_follows, } def form_valid(self, form): @@ -69,4 +76,8 @@ class ProfilePage(FormView): resize_image(image, size=(1500, 500)), ) identity.save() + # Save profile-specific identity Config + Config.set_identity( + identity, "visible_follows", form.cleaned_data["visible_follows"] + ) return redirect(".") diff --git a/users/views/settings/settings_page.py b/users/views/settings/settings_page.py new file mode 100644 index 0000000..a92d507 --- /dev/null +++ b/users/views/settings/settings_page.py @@ -0,0 +1,108 @@ +from functools import partial +from typing import ClassVar + +from django import forms +from django.core.files import File +from django.shortcuts import redirect +from django.utils.decorators import method_decorator +from django.views.generic import FormView + +from core.models.config import Config, UploadedImage +from users.decorators import identity_required + + +@method_decorator(identity_required, name="dispatch") +class SettingsPage(FormView): + """ + Shows a settings page dynamically created from our settings layout + at the bottom of the page. Don't add this to a URL directly - subclass! + """ + + options_class = Config.IdentityOptions + template_name = "settings/settings.html" + section: ClassVar[str] + options: dict[str, dict[str, str | int]] + layout: dict[str, list[str]] + + def get_form_class(self): + # Create the fields dict from the config object + fields = {} + for key, details in self.options.items(): + field_kwargs = {} + config_field = self.options_class.__fields__[key] + if config_field.type_ is bool: + form_field = partial( + forms.BooleanField, + widget=forms.Select( + choices=[(True, "Enabled"), (False, "Disabled")] + ), + ) + elif config_field.type_ is UploadedImage: + form_field = forms.ImageField + elif config_field.type_ is str: + if details.get("display") == "textarea": + form_field = partial( + forms.CharField, + widget=forms.Textarea, + ) + else: + form_field = forms.CharField + elif config_field.type_ is int: + choices = details.get("choices") + if choices: + field_kwargs["widget"] = forms.Select(choices=choices) + for int_kwarg in {"min_value", "max_value", "step_size"}: + val = details.get(int_kwarg) + if val: + field_kwargs[int_kwarg] = val + form_field = forms.IntegerField + else: + raise ValueError(f"Cannot render settings type {config_field.type_}") + fields[key] = form_field( + label=details["title"], + help_text=details.get("help_text", ""), + required=details.get("required", False), + **field_kwargs, + ) + # Create a form class dynamically (yeah, right?) and return that + return type("SettingsForm", (forms.Form,), fields) + + def load_config(self): + return Config.load_identity(self.request.identity) + + def save_config(self, key, value): + Config.set_identity(self.request.identity, key, value) + + def get_initial(self): + config = self.load_config() + initial = {} + for key in self.options.keys(): + initial[key] = getattr(config, key) + return initial + + def get_context_data(self): + context = super().get_context_data() + context["section"] = self.section + # Gather fields into fieldsets + context["fieldsets"] = {} + for title, fields in self.layout.items(): + context["fieldsets"][title] = [context["form"][field] for field in fields] + return context + + def form_valid(self, form): + # Save each key + for field in form: + if field.field.__class__.__name__ == "ImageField": + # These can be cleared with an extra checkbox + if self.request.POST.get(f"{field.name}__clear"): + self.save_config(field.name, None) + continue + # We shove the preview values in initial_data, so only save file + # fields if they have a File object. + if not isinstance(form.cleaned_data[field.name], File): + continue + self.save_config( + field.name, + form.cleaned_data[field.name], + ) + return redirect(".") -- cgit v1.2.3