diff --git a/community/community/widgets/Avatar.html b/community/community/widgets/Avatar.html index 59f59a91..6450ac9f 100644 --- a/community/community/widgets/Avatar.html +++ b/community/community/widgets/Avatar.html @@ -1,5 +1,5 @@ {% set color = member.get_palette() %} - + {% if member.user_image %} diff --git a/community/hooks.py b/community/hooks.py index 488cbe9e..40e0e8ba 100644 --- a/community/hooks.py +++ b/community/hooks.py @@ -141,15 +141,21 @@ website_route_rules = [ {"from_route": "/courses//learn/.", "to_route": "batch/learn"}, {"from_route": "/courses//progress", "to_route": "batch/progress"}, {"from_route": "/courses//join", "to_route": "batch/join"}, - {"from_route": "/user/", "to_route": "profiles/profile"}, ] website_redirects = [ {"source": "/update-profile", "target": "/edit-profile"}, ] -update_website_context = 'community.widgets.update_website_context' - +update_website_context = [ + 'community.widgets.update_website_context', +] +jinja = { + "methods": [ + "community.page_renderers.get_profile_url" + ], + "filters": [] +} ## Specify the additional tabs to be included in the user profile page. ## Each entry must be a subclass of community.community.plugins.ProfileTab # profile_tabs = [] @@ -170,3 +176,12 @@ community_markdown_macro_renderers = { "YouTubeVideo": "community.plugins.youtube_video_renderer", "Video": "community.plugins.video_renderer" } + +# page_renderer to manage profile pages +page_renderer = [ + "community.page_renderers.ProfileRedirectPage", + "community.page_renderers.ProfilePage" +] + +# set this to "/" to have profiles on the top-level +profile_url_prefix = "/users/" diff --git a/community/lms/web_form/profile/profile.js b/community/lms/web_form/profile/profile.js index c9869a5c..8a707633 100644 --- a/community/lms/web_form/profile/profile.js +++ b/community/lms/web_form/profile/profile.js @@ -8,7 +8,7 @@ frappe.ready(function () { frappe.web_form.after_save = () => { setTimeout(() => { - window.location.href = `/user/${frappe.web_form.get_value(["username"])}`; + window.location.href = `/profile_/${frappe.web_form.get_value(["username"])}`; }) } }) diff --git a/community/lms/widgets/CourseTeaser.html b/community/lms/widgets/CourseTeaser.html index 2b7b54ec..280d2783 100644 --- a/community/lms/widgets/CourseTeaser.html +++ b/community/lms/widgets/CourseTeaser.html @@ -12,7 +12,7 @@ {% endif %}
{% with author = course.get_instructor() %} - {{ widgets.Avatar(member=author, avatar_class="avatar-medium") }} {{ author.full_name }} + {{ widgets.Avatar(member=author, avatar_class="avatar-medium") }} {{ author.full_name }} {% endwith %}
diff --git a/community/lms/widgets/InstructorSection.html b/community/lms/widgets/InstructorSection.html index 9c96c294..c478e452 100644 --- a/community/lms/widgets/InstructorSection.html +++ b/community/lms/widgets/InstructorSection.html @@ -1,6 +1,6 @@
{{ widgets.Avatar(member=instructor, avatar_class="avatar-medium") }} - {{ instructor.full_name }} + {{ instructor.full_name }}
Course Creator
diff --git a/community/lms/widgets/MemberCard.html b/community/lms/widgets/MemberCard.html index 77f00556..89879cbb 100644 --- a/community/lms/widgets/MemberCard.html +++ b/community/lms/widgets/MemberCard.html @@ -11,5 +11,5 @@ Created {{ course_count }} {{ suffix }} {% endif %} - + diff --git a/community/lms/widgets/Reviews.html b/community/lms/widgets/Reviews.html index f9ba6524..8fe25df4 100644 --- a/community/lms/widgets/Reviews.html +++ b/community/lms/widgets/Reviews.html @@ -19,7 +19,7 @@
{{ widgets.Avatar(member=review.owner_details, avatar_class="avatar-medium") }} - + {{ review.owner_details.full_name }} diff --git a/community/page_renderers.py b/community/page_renderers.py new file mode 100644 index 00000000..4567d106 --- /dev/null +++ b/community/page_renderers.py @@ -0,0 +1,105 @@ +"""Custom page renderers for Community app. + +Handles rendering of profile pages. +""" +import re +import frappe +from frappe.website.page_renderers.base_renderer import BaseRenderer +from frappe.website.page_renderers.template_page import TemplatePage + +from frappe.website.page_renderers.document_page import DocumentPage +from frappe.website.page_renderers.list_page import ListPage +from frappe.website.page_renderers.not_found_page import NotFoundPage +from frappe.website.page_renderers.print_page import PrintPage +from frappe.website.page_renderers.redirect_page import RedirectPage +from frappe.website.page_renderers.static_page import StaticPage +from frappe.website.page_renderers.template_page import TemplatePage +from frappe.website.page_renderers.web_form import WebFormPage + + +def get_profile_url(username): + """Returns the profile URL given username. + + The default URL prefix for profiles is /users, but tha can be customized. + + This functions looks at the current value from the config and generates + the URL for the profile. + """ + return get_profile_url_prefix() + username + +def get_profile_url_prefix(): + hooks = frappe.get_hooks("profile_url_prefix") or ["/users/"] + return hooks[-1] + +RE_USERNAME = re.compile("[a-zA-Z0-9_]{4,}") + +class ProfileRedirectPage(BaseRenderer): + """Renderer to redirect /profile_/foo to /foo. + + This is useful to redirect to profile pages from javascript as there is no + easy to find the profile prefix. + """ + def can_render(self): + return self.path.startswith("profile_/") + + def render(self): + username = self.path[len("profile_/"):] + frappe.flags.redirect_location = get_profile_url_prefix() + username + return RedirectPage(self.path).render() + +class ProfilePage(BaseRenderer): + def __init__(self, path, http_status_code): + super().__init__(path, http_status_code) + self.renderer = None + + def can_render(self): + if "." in self.path: + return False + + # has prefix and path starts with prefix? + prefix = get_profile_url_prefix().lstrip("/") + if prefix and not self.path.startswith(prefix): + return False + + # not a userpage? + username = self.get_username() + if not RE_USERNAME.match(username): + return False + + # if there is prefix then we can allow all usernames + if prefix: + return True + + # if we are having top-level usernames, then give preference to + # the existing website_route_rules, web pages, web forms etc. + + # Don't handle any of the exsiting website_route_rules + routes = [rule['to_route'] for rule in frappe.get_hooks("website_route_rules")] + if self.path in routes: + return False + + # if any of the existing renders can render, let them do + renderers = [StaticPage, WebFormPage, DocumentPage, TemplatePage, ListPage, PrintPage] + for renderer in renderers: + renderer_instance = renderer(self.path, 200) + if renderer_instance.can_render(): + self.renderer = renderer_instance + return True + + return True + + def get_username(self): + prefix = get_profile_url_prefix().lstrip("/") + return self.path[len(prefix):] + + def render(self): + if self.renderer: + return self.renderer.render() + else: + username = self.get_username() + return render_portal_page("profiles/profile", username=username) + +def render_portal_page(path, **kwargs): + frappe.form_dict.update(kwargs) + page = TemplatePage(path) + return page.render() diff --git a/community/templates/discussions/reply_card.html b/community/templates/discussions/reply_card.html index ff60bc13..4a5439d7 100644 --- a/community/templates/discussions/reply_card.html +++ b/community/templates/discussions/reply_card.html @@ -3,7 +3,7 @@
{% set member = frappe.get_doc("User", reply.owner) %} {{ widgets.Avatar(member=member, avatar_class="avatar-small")}} - + {{ member.full_name }}
just now