taiga
/
taiga-back
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Improving HighLightedContentSerializer and making more consistent how the assigned_to extra info is shown

remotes/origin/issue/4795/notification_even_they_are_disabled
Alejandro Alonso 2016-07-20 08:38:50 +02:00
parent 2c5716a097
commit 25c74cfb07
3 changed files with 71 additions and 81 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

30
taiga/users/serializers.py
View File

@ -26,7 +26,7 @@ from taiga.base.utils.thumbnails import get_thumbnail_url
from taiga.projects.models import Project from taiga.projects.models import Project
from .services import get_user_photo_url, get_big_photo_url, get_user_big_photo_url from .services import get_user_photo_url, get_big_photo_url, get_user_big_photo_url
from taiga.users.gravatar import get_user_gravatar_id from taiga.users.gravatar import get_user_gravatar_id
from taiga.users.models import User
from collections import namedtuple from collections import namedtuple
@ -182,10 +182,8 @@ class HighLightedContentSerializer(serializers.LightSerializer):
project_is_private = MethodField() project_is_private = MethodField()
project_blocked_code = Field() project_blocked_code = Field()
assigned_to_username = Field() assigned_to = Field(attr="assigned_to_id")
assigned_to_full_name = Field() assigned_to_extra_info = MethodField()
assigned_to_photo = MethodField()
assigned_to_gravatar_id = MethodField()
is_watcher = MethodField() is_watcher = MethodField()
total_watchers = Field() total_watchers = Field()
@ -235,23 +233,11 @@ class HighLightedContentSerializer(serializers.LightSerializer):
return get_thumbnail_url(logo, settings.THN_LOGO_SMALL) return get_thumbnail_url(logo, settings.THN_LOGO_SMALL)
return None return None
def get_assigned_to_photo(self, obj): def get_assigned_to_extra_info(self, obj):
type = getattr(obj, "type", "") assigned_to = None
if type == "project": if obj.assigned_to_extra_info is not None:
return None assigned_to = User(**obj.assigned_to_extra_info)
return UserBasicInfoSerializer(assigned_to).data
UserData = namedtuple("UserData", ["photo", "email"])
user_data = UserData(photo=obj.assigned_to_photo, email=obj.assigned_to_email or "")
return get_user_photo_url(user_data)
def get_assigned_to_gravatar_id(self, obj):
type = getattr(obj, "type", "")
if type == "project":
return None
UserData = namedtuple("UserData", ["photo", "email"])
user_data = UserData(photo=obj.assigned_to_photo, email=obj.assigned_to_email or "")
return get_user_gravatar_id(user_data)
def get_tags_colors(self, obj): def get_tags_colors(self, obj):
tags = getattr(obj, "tags", []) tags = getattr(obj, "tags", [])

35
taiga/users/services.py
View File

@ -54,7 +54,7 @@ def get_user_by_username_or_email(username_or_email):
return user return user
def get_and_validate_user(*, username:str, password:str) -> bool: def get_and_validate_user(*, username: str, password: str) -> bool:
""" """
Check if user with username/email exists and specified Check if user with username/email exists and specified
password matchs well with existing user password. password matchs well with existing user password.
@ -115,17 +115,17 @@ def get_visible_project_ids(from_user, by_user):
# Authenticated # Authenticated
if by_user.is_authenticated(): if by_user.is_authenticated():
#Calculating the projects wich from_user user is member # Calculating the projects wich from_user user is member
by_user_project_ids = by_user.memberships.values_list("project__id", flat=True) by_user_project_ids = by_user.memberships.values_list("project__id", flat=True)
#Adding to the condition two OR situations: # Adding to the condition two OR situations:
#- The from user has a role that allows access to the project # - The from user has a role that allows access to the project
#- The to user is the owner # - The to user is the owner
member_perm_conditions |= \ member_perm_conditions |= \
Q(project__id__in=by_user_project_ids, role__permissions__contains=required_permissions) |\ Q(project__id__in=by_user_project_ids, role__permissions__contains=required_permissions) |\
Q(project__id__in=by_user_project_ids, is_admin=True) Q(project__id__in=by_user_project_ids, is_admin=True)
Membership = apps.get_model('projects', 'Membership') Membership = apps.get_model('projects', 'Membership')
#Calculating the user memberships adding the permission filter for the by user # Calculating the user memberships adding the permission filter for the by user
memberships_qs = Membership.objects.filter(member_perm_conditions, user=from_user) memberships_qs = Membership.objects.filter(member_perm_conditions, user=from_user)
project_ids = memberships_qs.values_list("project__id", flat=True) project_ids = memberships_qs.values_list("project__id", flat=True)
return project_ids return project_ids
@ -137,8 +137,8 @@ def get_stats_for_user(from_user, by_user):
total_num_projects = len(project_ids) total_num_projects = len(project_ids)
roles = [_(r) for r in from_user.memberships.filter(project__id__in=project_ids).values_list( role_names = from_user.memberships.filter(project__id__in=project_ids).values_list("role__name", flat=True)
"role__name", flat=True)] roles = [_(r) for r in role_names]
roles = list(set(roles)) roles = list(set(roles))
User = apps.get_model('users', 'User') User = apps.get_model('users', 'User')
@ -210,9 +210,9 @@ def get_watched_content_for_user(user):
list.append(object_id) list.append(object_id)
user_watches[ct_model] = list user_watches[ct_model] = list
#Now for projects, # Now for projects,
projects_watched = get_projects_watched(user) projects_watched = get_projects_watched(user)
project_content_type_model=ContentType.objects.get(app_label="projects", model="project").model project_content_type_model = ContentType.objects.get(app_label="projects", model="project").model
user_watches[project_content_type_model] = projects_watched.values_list("id", flat=True) user_watches[project_content_type_model] = projects_watched.values_list("id", flat=True)
return user_watches return user_watches
@ -297,7 +297,7 @@ def _build_sql_for_type(for_user, type, table_name, action_table, ref_column="re
WHERE {action_table}.user_id = {for_user_id} WHERE {action_table}.user_id = {for_user_id}
""" """
sql = sql.format(for_user_id=for_user.id, type=type, table_name=table_name, sql = sql.format(for_user_id=for_user.id, type=type, table_name=table_name,
action_table=action_table, ref_column = ref_column, action_table=action_table, ref_column=ref_column,
project_column=project_column, assigned_to_column=assigned_to_column, project_column=project_column, assigned_to_column=assigned_to_column,
slug_column=slug_column, subject_column=subject_column) slug_column=slug_column, subject_column=subject_column)
@ -306,7 +306,6 @@ def _build_sql_for_type(for_user, type, table_name, action_table, ref_column="re
def get_watched_list(for_user, from_user, type=None, q=None): def get_watched_list(for_user, from_user, type=None, q=None):
filters_sql = "" filters_sql = ""
and_needed = False
if type: if type:
filters_sql += " AND type = %(type)s " filters_sql += " AND type = %(type)s "
@ -322,7 +321,9 @@ def get_watched_list(for_user, from_user, type=None, q=None):
SELECT entities.*, SELECT entities.*,
projects_project.name as project_name, projects_project.description as description, projects_project.slug as project_slug, projects_project.is_private as project_is_private, projects_project.name as project_name, projects_project.description as description, projects_project.slug as project_slug, projects_project.is_private as project_is_private,
projects_project.blocked_code as project_blocked_code, projects_project.tags_colors, projects_project.logo, projects_project.blocked_code as project_blocked_code, projects_project.tags_colors, projects_project.logo,
users_user.username assigned_to_username, users_user.full_name assigned_to_full_name, users_user.photo assigned_to_photo, users_user.email assigned_to_email users_user.id as assigned_to_id,
row_to_json(users_user) as assigned_to_extra_info
FROM ( FROM (
{userstories_sql} {userstories_sql}
UNION UNION
@ -401,7 +402,6 @@ def get_watched_list(for_user, from_user, type=None, q=None):
def get_liked_list(for_user, from_user, type=None, q=None): def get_liked_list(for_user, from_user, type=None, q=None):
filters_sql = "" filters_sql = ""
and_needed = False
if type: if type:
filters_sql += " AND type = %(type)s " filters_sql += " AND type = %(type)s "
@ -417,7 +417,8 @@ def get_liked_list(for_user, from_user, type=None, q=None):
SELECT entities.*, SELECT entities.*,
projects_project.name as project_name, projects_project.description as description, projects_project.slug as project_slug, projects_project.is_private as project_is_private, projects_project.name as project_name, projects_project.description as description, projects_project.slug as project_slug, projects_project.is_private as project_is_private,
projects_project.blocked_code as project_blocked_code, projects_project.tags_colors, projects_project.logo, projects_project.blocked_code as project_blocked_code, projects_project.tags_colors, projects_project.logo,
users_user.username assigned_to_username, users_user.full_name assigned_to_full_name, users_user.photo assigned_to_photo, users_user.email assigned_to_email users_user.id as assigned_to_id,
row_to_json(users_user) as assigned_to_extra_info
FROM ( FROM (
{projects_sql} {projects_sql}
) as entities ) as entities
@ -484,7 +485,6 @@ def get_liked_list(for_user, from_user, type=None, q=None):
def get_voted_list(for_user, from_user, type=None, q=None): def get_voted_list(for_user, from_user, type=None, q=None):
filters_sql = "" filters_sql = ""
and_needed = False
if type: if type:
filters_sql += " AND type = %(type)s " filters_sql += " AND type = %(type)s "
@ -500,7 +500,8 @@ def get_voted_list(for_user, from_user, type=None, q=None):
SELECT entities.*, SELECT entities.*,
projects_project.name as project_name, projects_project.description as description, projects_project.slug as project_slug, projects_project.is_private as project_is_private, projects_project.name as project_name, projects_project.description as description, projects_project.slug as project_slug, projects_project.is_private as project_is_private,
projects_project.blocked_code as project_blocked_code, projects_project.tags_colors, projects_project.logo, projects_project.blocked_code as project_blocked_code, projects_project.tags_colors, projects_project.logo,
users_user.username assigned_to_username, users_user.full_name assigned_to_full_name, users_user.photo assigned_to_photo, users_user.email assigned_to_email users_user.id as assigned_to_id,
row_to_json(users_user) as assigned_to_extra_info
FROM ( FROM (
{userstories_sql} {userstories_sql}
UNION UNION

27
tests/integration/test_users.py
View File

@ -558,9 +558,8 @@ def test_get_watched_list_valid_info_for_project():
assert project_watch_info["project_slug"] == None assert project_watch_info["project_slug"] == None
assert project_watch_info["project_is_private"] == None assert project_watch_info["project_is_private"] == None
assert project_watch_info["project_blocked_code"] == None assert project_watch_info["project_blocked_code"] == None
assert project_watch_info["assigned_to_username"] == None assert project_watch_info["assigned_to"] == None
assert project_watch_info["assigned_to_full_name"] == None assert project_watch_info["assigned_to_extra_info"] == None
assert project_watch_info["assigned_to_photo"] == None
def test_get_watched_list_for_project_with_ignored_notify_level(): def test_get_watched_list_for_project_with_ignored_notify_level():
@ -613,9 +612,8 @@ def test_get_liked_list_valid_info():
assert project_like_info["project_slug"] == None assert project_like_info["project_slug"] == None
assert project_like_info["project_is_private"] == None assert project_like_info["project_is_private"] == None
assert project_like_info["project_blocked_code"] == None assert project_like_info["project_blocked_code"] == None
assert project_like_info["assigned_to_username"] == None assert project_like_info["assigned_to"] == None
assert project_like_info["assigned_to_full_name"] == None assert project_like_info["assigned_to_extra_info"] == None
assert project_like_info["assigned_to_photo"] == None
def test_get_watched_list_valid_info_for_not_project_types(): def test_get_watched_list_valid_info_for_not_project_types():
@ -667,9 +665,11 @@ def test_get_watched_list_valid_info_for_not_project_types():
assert instance_watch_info["project_slug"] == instance.project.slug assert instance_watch_info["project_slug"] == instance.project.slug
assert instance_watch_info["project_is_private"] == instance.project.is_private assert instance_watch_info["project_is_private"] == instance.project.is_private
assert instance_watch_info["project_blocked_code"] == instance.project.blocked_code assert instance_watch_info["project_blocked_code"] == instance.project.blocked_code
assert instance_watch_info["assigned_to_username"] == instance.assigned_to.username assert instance_watch_info["assigned_to"] != None
assert instance_watch_info["assigned_to_full_name"] == instance.assigned_to.full_name assert instance_watch_info["assigned_to_extra_info"]["username"] == instance.assigned_to.username
assert instance_watch_info["assigned_to_photo"] != "" assert instance_watch_info["assigned_to_extra_info"]["full_name_display"] == instance.assigned_to.get_full_name()
assert instance_watch_info["assigned_to_extra_info"]["photo"] == None
assert instance_watch_info["assigned_to_extra_info"]["gravatar_id"] != None
def test_get_voted_list_valid_info(): def test_get_voted_list_valid_info():
@ -724,9 +724,12 @@ def test_get_voted_list_valid_info():
assert instance_vote_info["project_slug"] == instance.project.slug assert instance_vote_info["project_slug"] == instance.project.slug
assert instance_vote_info["project_is_private"] == instance.project.is_private assert instance_vote_info["project_is_private"] == instance.project.is_private
assert instance_vote_info["project_blocked_code"] == instance.project.blocked_code assert instance_vote_info["project_blocked_code"] == instance.project.blocked_code
assert instance_vote_info["assigned_to_username"] == instance.assigned_to.username assert instance_vote_info["assigned_to"] != None
assert instance_vote_info["assigned_to_full_name"] == instance.assigned_to.full_name assert instance_vote_info["assigned_to_extra_info"]["username"] == instance.assigned_to.username
assert instance_vote_info["assigned_to_photo"] != "" assert instance_vote_info["assigned_to_extra_info"]["full_name_display"] == instance.assigned_to.get_full_name()
assert instance_vote_info["assigned_to_extra_info"]["photo"] == None
assert instance_vote_info["assigned_to_extra_info"]["gravatar_id"] != None
def test_get_watched_list_with_liked_and_voted_objects(client): def test_get_watched_list_with_liked_and_voted_objects(client):