2019-05-02 18:17:27 -04:00
|
|
|
# frozen_string_literal: true
|
|
|
|
|
2013-02-05 14:16:51 -05:00
|
|
|
class CurrentUserSerializer < BasicUserSerializer
|
|
|
|
|
2013-02-25 11:42:20 -05:00
|
|
|
attributes :name,
|
|
|
|
:unread_notifications,
|
|
|
|
:unread_private_messages,
|
2020-03-31 19:09:20 -04:00
|
|
|
:unread_high_priority_notifications,
|
2016-11-08 03:12:40 -05:00
|
|
|
:read_first_notification?,
|
2013-02-25 11:42:20 -05:00
|
|
|
:admin?,
|
|
|
|
:notification_channel_position,
|
2013-02-14 12:10:53 -05:00
|
|
|
:moderator?,
|
2013-05-02 03:22:27 -04:00
|
|
|
:staff?,
|
2015-01-21 15:58:04 -05:00
|
|
|
:title,
|
2020-05-14 18:42:00 -04:00
|
|
|
:any_posts,
|
2013-04-27 20:37:53 -04:00
|
|
|
:enable_quoting,
|
2019-05-31 01:43:17 -04:00
|
|
|
:enable_defer,
|
2013-04-27 20:37:53 -04:00
|
|
|
:external_links_in_new_tab,
|
2013-06-15 02:58:24 -04:00
|
|
|
:dynamic_favicon,
|
2013-06-11 12:30:38 -04:00
|
|
|
:trust_level,
|
2017-08-28 12:07:30 -04:00
|
|
|
:can_send_private_email_messages,
|
2021-03-31 20:22:40 -04:00
|
|
|
:can_send_private_messages,
|
2013-11-06 12:56:26 -05:00
|
|
|
:can_edit,
|
2014-01-21 12:42:20 -05:00
|
|
|
:can_invite_to_forum,
|
2014-02-13 11:42:35 -05:00
|
|
|
:no_password,
|
2014-03-07 12:58:53 -05:00
|
|
|
:can_delete_account,
|
|
|
|
:should_be_redirected_to_top,
|
2015-09-21 14:28:20 -04:00
|
|
|
:redirected_to_top,
|
2014-06-17 21:21:40 -04:00
|
|
|
:custom_fields,
|
2014-06-18 14:04:10 -04:00
|
|
|
:muted_category_ids,
|
2022-02-16 18:42:02 -05:00
|
|
|
:indirectly_muted_category_ids,
|
2021-05-05 19:14:07 -04:00
|
|
|
:regular_category_ids,
|
|
|
|
:tracked_category_ids,
|
|
|
|
:watched_first_post_category_ids,
|
|
|
|
:watched_category_ids,
|
2019-12-09 17:50:05 -05:00
|
|
|
:muted_tag_ids,
|
2021-05-05 19:14:07 -04:00
|
|
|
:watched_tags,
|
|
|
|
:watching_first_post_tags,
|
|
|
|
:tracked_tags,
|
|
|
|
:muted_tags,
|
|
|
|
:regular_tags,
|
2015-04-07 04:02:10 -04:00
|
|
|
:dismissed_banner_key,
|
2015-04-10 16:29:13 -04:00
|
|
|
:is_anonymous,
|
2019-01-03 12:03:01 -05:00
|
|
|
:reviewable_count,
|
2021-08-26 12:16:00 -04:00
|
|
|
:read_faq?,
|
2016-06-01 16:47:42 -04:00
|
|
|
:automatically_unpin_topics,
|
2016-08-18 21:58:20 -04:00
|
|
|
:mailing_list_mode,
|
DEV: Topic tracking state improvements (#13218)
I merged this PR in yesterday, finally thinking this was done https://github.com/discourse/discourse/pull/12958 but then a wild performance regression occurred. These are the problem methods:
https://github.com/discourse/discourse/blob/1aa20bd681e634f7fff22953ed62d90c2573b331/app/serializers/topic_tracking_state_serializer.rb#L13-L21
Turns out date comparison is super expensive on the backend _as well as_ the frontend.
The fix was to just move the `treat_as_new_topic_start_date` into the SQL query rather than using the slower `UserOption#treat_as_new_topic_start_date` method in ruby. After this change, 1% of the total time is spent with the `created_in_new_period` comparison instead of ~20%.
----
History:
Original PR which had to be reverted **https://github.com/discourse/discourse/pull/12555**. See the description there for what this PR is achieving, plus below.
The issue with the original PR is addressed in https://github.com/discourse/discourse/pull/12958/commits/92ef54f4020111ffacb0f2a27da5d5c2855f9d5d
If you went to the `x unread` link for a tag Chrome would freeze up and possibly crash, or eventually unfreeze after nearly 10 mins. Other routes for unread/new were similarly slow. From profiling the issue was the `sync` function of `topic-tracking-state.js`, which calls down to `isNew` which in turn calls `moment`, a change I had made in the PR above. The time it takes locally with ~1400 topics in the tracking state is 2.3 seconds.
To solve this issue, I have moved these calculations for "created in new period" and "unread not too old" into the tracking state serializer.
When I was looking at the profiler I also noticed this issue which was just compounding the problem. Every time we modify topic tracking state we recalculate the sidebar tracking/everything/tag counts. However this calls `forEachTracked` and `countTags` which can be quite expensive as they go through the whole tracking state (and were also calling the removed moment functions).
I added some logs and this was being called 30 times when navigating to a new /unread route because `sync` is being called from `build-topic-route` (one for each topic loaded due to pagination). So I just added a debounce here and it makes things even faster.
Finally, I changed topic tracking state to use a Map so our counts of the state keys is faster (Maps have .size whereas objects you have to do Object.keys(obj) which is O(n).)
<!-- NOTE: All pull requests should have tests (rspec in Ruby, qunit in JavaScript). If your code does not include test coverage, please include an explanation of why it was omitted. -->
2021-06-01 19:06:29 -04:00
|
|
|
:treat_as_new_topic_start_date,
|
2016-09-18 00:30:15 -04:00
|
|
|
:previous_visit_at,
|
2017-08-11 22:05:16 -04:00
|
|
|
:seen_notification_id,
|
|
|
|
:primary_group_id,
|
2021-07-08 03:46:21 -04:00
|
|
|
:flair_group_id,
|
2018-02-08 12:56:10 -05:00
|
|
|
:can_create_topic,
|
2020-08-19 10:41:40 -04:00
|
|
|
:can_create_group,
|
2018-06-13 14:57:32 -04:00
|
|
|
:link_posting_access,
|
2018-07-27 02:41:07 -04:00
|
|
|
:external_id,
|
2018-10-10 13:00:08 -04:00
|
|
|
:top_category_ids,
|
2018-12-18 02:41:42 -05:00
|
|
|
:hide_profile_and_presence,
|
2019-03-20 00:40:25 -04:00
|
|
|
:groups,
|
2019-04-05 10:13:36 -04:00
|
|
|
:second_factor_enabled,
|
2019-04-11 19:02:18 -04:00
|
|
|
:ignored_users,
|
2019-12-03 02:31:16 -05:00
|
|
|
:title_count_mode,
|
2019-12-09 14:15:47 -05:00
|
|
|
:timezone,
|
2020-08-14 09:40:56 -04:00
|
|
|
:featured_topic,
|
2020-12-18 10:03:51 -05:00
|
|
|
:skip_new_user_tips,
|
|
|
|
:do_not_disturb_until,
|
2021-03-19 09:19:15 -04:00
|
|
|
:has_topic_draft,
|
2021-05-05 19:14:07 -04:00
|
|
|
:can_review,
|
2021-07-27 07:05:33 -04:00
|
|
|
:draft_count,
|
2021-10-05 23:11:52 -04:00
|
|
|
:default_calendar,
|
2022-03-08 12:44:18 -05:00
|
|
|
:bookmark_auto_delete_preference,
|
2021-08-26 12:16:00 -04:00
|
|
|
:pending_posts_count
|
|
|
|
|
|
|
|
delegate :user_stat, to: :object, private: true
|
|
|
|
delegate :any_posts, :draft_count, :pending_posts_count, :read_faq?, to: :user_stat
|
2018-02-08 12:56:10 -05:00
|
|
|
|
2018-12-18 02:41:42 -05:00
|
|
|
def groups
|
2021-05-12 06:06:39 -04:00
|
|
|
owned_group_ids = GroupUser.where(user_id: id, owner: true).pluck(:group_id).to_set
|
2021-09-07 00:30:40 -04:00
|
|
|
object.visible_groups.pluck(:id, :name, :has_messages).map do |id, name, has_messages|
|
|
|
|
group = { id: id, name: name, has_messages: has_messages }
|
2021-05-12 06:06:39 -04:00
|
|
|
group[:owner] = true if owned_group_ids.include?(id)
|
|
|
|
group
|
|
|
|
end
|
2018-12-18 02:41:42 -05:00
|
|
|
end
|
2018-12-10 10:23:29 -05:00
|
|
|
|
2018-06-13 14:57:32 -04:00
|
|
|
def link_posting_access
|
|
|
|
scope.link_posting_access
|
2018-02-08 12:56:10 -05:00
|
|
|
end
|
2017-12-20 19:23:45 -05:00
|
|
|
|
|
|
|
def can_create_topic
|
|
|
|
scope.can_create_topic?(nil)
|
|
|
|
end
|
2013-02-05 14:16:51 -05:00
|
|
|
|
2020-08-19 10:41:40 -04:00
|
|
|
def can_create_group
|
|
|
|
scope.can_create_group?
|
|
|
|
end
|
|
|
|
|
|
|
|
def include_can_create_group?
|
|
|
|
scope.can_create_group?
|
|
|
|
end
|
|
|
|
|
2018-10-10 13:00:08 -04:00
|
|
|
def hide_profile_and_presence
|
|
|
|
object.user_option.hide_profile_and_presence
|
|
|
|
end
|
|
|
|
|
2016-02-16 23:46:19 -05:00
|
|
|
def enable_quoting
|
|
|
|
object.user_option.enable_quoting
|
|
|
|
end
|
|
|
|
|
2019-05-31 01:43:17 -04:00
|
|
|
def enable_defer
|
|
|
|
object.user_option.enable_defer
|
|
|
|
end
|
|
|
|
|
2016-02-16 23:46:19 -05:00
|
|
|
def external_links_in_new_tab
|
|
|
|
object.user_option.external_links_in_new_tab
|
|
|
|
end
|
|
|
|
|
|
|
|
def dynamic_favicon
|
|
|
|
object.user_option.dynamic_favicon
|
|
|
|
end
|
|
|
|
|
2019-04-11 19:02:18 -04:00
|
|
|
def title_count_mode
|
|
|
|
object.user_option.title_count_mode
|
|
|
|
end
|
|
|
|
|
2016-02-16 23:46:19 -05:00
|
|
|
def automatically_unpin_topics
|
|
|
|
object.user_option.automatically_unpin_topics
|
|
|
|
end
|
|
|
|
|
2016-02-18 00:57:22 -05:00
|
|
|
def should_be_redirected_to_top
|
|
|
|
object.user_option.should_be_redirected_to_top
|
|
|
|
end
|
|
|
|
|
|
|
|
def redirected_to_top
|
|
|
|
object.user_option.redirected_to_top
|
|
|
|
end
|
|
|
|
|
2019-12-03 02:31:16 -05:00
|
|
|
def timezone
|
|
|
|
object.user_option.timezone
|
|
|
|
end
|
|
|
|
|
2021-10-05 23:11:52 -04:00
|
|
|
def default_calendar
|
|
|
|
object.user_option.default_calendar
|
|
|
|
end
|
|
|
|
|
2022-03-08 12:44:18 -05:00
|
|
|
def bookmark_auto_delete_preference
|
|
|
|
object.user_option.bookmark_auto_delete_preference
|
|
|
|
end
|
|
|
|
|
2017-08-28 12:07:30 -04:00
|
|
|
def can_send_private_email_messages
|
2018-08-26 21:38:11 -04:00
|
|
|
scope.can_send_private_messages_to_email?
|
2017-08-28 12:07:30 -04:00
|
|
|
end
|
|
|
|
|
2021-03-31 20:22:40 -04:00
|
|
|
def can_send_private_messages
|
|
|
|
scope.can_send_private_message?(Discourse.system_user)
|
|
|
|
end
|
|
|
|
|
2013-06-11 12:30:38 -04:00
|
|
|
def can_edit
|
|
|
|
true
|
|
|
|
end
|
|
|
|
|
2013-11-06 12:56:26 -05:00
|
|
|
def can_invite_to_forum
|
2020-05-06 16:57:26 -04:00
|
|
|
scope.can_invite_to_forum?
|
2013-11-06 12:56:26 -05:00
|
|
|
end
|
|
|
|
|
|
|
|
def include_can_invite_to_forum?
|
|
|
|
scope.can_invite_to_forum?
|
|
|
|
end
|
|
|
|
|
2014-01-21 12:42:20 -05:00
|
|
|
def no_password
|
|
|
|
true
|
|
|
|
end
|
|
|
|
|
|
|
|
def include_no_password?
|
|
|
|
!object.has_password?
|
|
|
|
end
|
|
|
|
|
2014-02-13 11:42:35 -05:00
|
|
|
def include_can_delete_account?
|
2014-02-13 15:51:19 -05:00
|
|
|
scope.can_delete_user?(object)
|
2014-02-13 11:42:35 -05:00
|
|
|
end
|
|
|
|
|
|
|
|
def can_delete_account
|
2014-02-13 15:51:19 -05:00
|
|
|
true
|
2014-02-13 11:42:35 -05:00
|
|
|
end
|
|
|
|
|
2015-09-21 14:28:20 -04:00
|
|
|
def include_redirected_to_top?
|
2016-02-18 00:57:22 -05:00
|
|
|
object.user_option.redirected_to_top.present?
|
2014-03-07 12:58:53 -05:00
|
|
|
end
|
|
|
|
|
2014-06-05 16:16:30 -04:00
|
|
|
def custom_fields
|
2014-06-10 21:57:22 -04:00
|
|
|
fields = nil
|
|
|
|
if SiteSetting.public_user_custom_fields.present?
|
|
|
|
fields = SiteSetting.public_user_custom_fields.split('|')
|
|
|
|
end
|
|
|
|
DiscoursePluginRegistry.serialized_current_user_fields.each do |f|
|
|
|
|
fields ||= []
|
|
|
|
fields << f
|
|
|
|
end
|
|
|
|
|
|
|
|
if fields.present?
|
2017-11-14 19:55:37 -05:00
|
|
|
User.custom_fields_for_ids([object.id], fields)[object.id] || {}
|
2014-06-10 21:57:22 -04:00
|
|
|
else
|
|
|
|
{}
|
|
|
|
end
|
2014-06-05 16:16:30 -04:00
|
|
|
end
|
|
|
|
|
2014-06-17 21:21:40 -04:00
|
|
|
def muted_category_ids
|
2021-05-05 19:14:07 -04:00
|
|
|
categories_with_notification_level(:muted)
|
2018-07-27 02:41:07 -04:00
|
|
|
end
|
|
|
|
|
2022-02-16 18:42:02 -05:00
|
|
|
def indirectly_muted_category_ids
|
|
|
|
CategoryUser.indirectly_muted_category_ids(object)
|
|
|
|
end
|
|
|
|
|
2021-05-05 19:14:07 -04:00
|
|
|
def regular_category_ids
|
|
|
|
categories_with_notification_level(:regular)
|
|
|
|
end
|
|
|
|
|
|
|
|
def tracked_category_ids
|
|
|
|
categories_with_notification_level(:tracking)
|
|
|
|
end
|
|
|
|
|
|
|
|
def watched_category_ids
|
|
|
|
categories_with_notification_level(:watching)
|
|
|
|
end
|
|
|
|
|
|
|
|
def watched_first_post_category_ids
|
|
|
|
categories_with_notification_level(:watching_first_post)
|
|
|
|
end
|
|
|
|
|
|
|
|
# this is a weird outlier that is used for topic tracking state which
|
|
|
|
# needs the actual ids, which is why it is duplicated with muted_tags
|
2019-12-09 17:50:05 -05:00
|
|
|
def muted_tag_ids
|
|
|
|
TagUser.lookup(object, :muted).pluck(:tag_id)
|
|
|
|
end
|
|
|
|
|
2021-05-05 19:14:07 -04:00
|
|
|
def muted_tags
|
|
|
|
tags_with_notification_level(:muted)
|
|
|
|
end
|
|
|
|
|
|
|
|
def tracked_tags
|
2021-05-13 19:45:14 -04:00
|
|
|
tags_with_notification_level(:tracking)
|
2021-05-05 19:14:07 -04:00
|
|
|
end
|
|
|
|
|
|
|
|
def watching_first_post_tags
|
|
|
|
tags_with_notification_level(:watching_first_post)
|
|
|
|
end
|
|
|
|
|
|
|
|
def watched_tags
|
|
|
|
tags_with_notification_level(:watching)
|
|
|
|
end
|
|
|
|
|
|
|
|
def regular_tags
|
|
|
|
tags_with_notification_level(:regular)
|
|
|
|
end
|
|
|
|
|
2019-04-05 10:13:36 -04:00
|
|
|
def ignored_users
|
|
|
|
IgnoredUser.where(user: object.id).joins(:ignored_user).pluck(:username)
|
|
|
|
end
|
|
|
|
|
2018-07-27 02:41:07 -04:00
|
|
|
def top_category_ids
|
2018-07-30 06:36:36 -04:00
|
|
|
omitted_notification_levels = [CategoryUser.notification_levels[:muted], CategoryUser.notification_levels[:regular]]
|
2018-07-27 02:41:07 -04:00
|
|
|
CategoryUser.where(user_id: object.id)
|
2018-07-30 06:36:36 -04:00
|
|
|
.where.not(notification_level: omitted_notification_levels)
|
2018-07-27 02:41:07 -04:00
|
|
|
.order("
|
|
|
|
CASE
|
|
|
|
WHEN notification_level = 3 THEN 1
|
|
|
|
WHEN notification_level = 2 THEN 2
|
|
|
|
WHEN notification_level = 4 THEN 3
|
|
|
|
END")
|
2014-06-17 21:21:40 -04:00
|
|
|
.pluck(:category_id)
|
2018-07-30 23:42:30 -04:00
|
|
|
.slice(0, SiteSetting.header_dropdown_category_count)
|
2014-06-17 21:21:40 -04:00
|
|
|
end
|
|
|
|
|
2014-06-18 14:04:10 -04:00
|
|
|
def dismissed_banner_key
|
|
|
|
object.user_profile.dismissed_banner_key
|
|
|
|
end
|
|
|
|
|
2015-04-07 04:02:10 -04:00
|
|
|
def is_anonymous
|
2015-04-07 22:29:43 -04:00
|
|
|
object.anonymous?
|
2015-04-07 04:02:10 -04:00
|
|
|
end
|
|
|
|
|
2019-01-03 12:03:01 -05:00
|
|
|
def reviewable_count
|
|
|
|
Reviewable.list_for(object).count
|
2015-04-21 13:59:57 -04:00
|
|
|
end
|
|
|
|
|
2021-03-19 15:20:41 -04:00
|
|
|
def can_review
|
|
|
|
scope.can_see_review_queue?
|
|
|
|
end
|
|
|
|
|
2016-06-01 16:47:42 -04:00
|
|
|
def mailing_list_mode
|
|
|
|
object.user_option.mailing_list_mode
|
|
|
|
end
|
|
|
|
|
DEV: Topic tracking state improvements (#13218)
I merged this PR in yesterday, finally thinking this was done https://github.com/discourse/discourse/pull/12958 but then a wild performance regression occurred. These are the problem methods:
https://github.com/discourse/discourse/blob/1aa20bd681e634f7fff22953ed62d90c2573b331/app/serializers/topic_tracking_state_serializer.rb#L13-L21
Turns out date comparison is super expensive on the backend _as well as_ the frontend.
The fix was to just move the `treat_as_new_topic_start_date` into the SQL query rather than using the slower `UserOption#treat_as_new_topic_start_date` method in ruby. After this change, 1% of the total time is spent with the `created_in_new_period` comparison instead of ~20%.
----
History:
Original PR which had to be reverted **https://github.com/discourse/discourse/pull/12555**. See the description there for what this PR is achieving, plus below.
The issue with the original PR is addressed in https://github.com/discourse/discourse/pull/12958/commits/92ef54f4020111ffacb0f2a27da5d5c2855f9d5d
If you went to the `x unread` link for a tag Chrome would freeze up and possibly crash, or eventually unfreeze after nearly 10 mins. Other routes for unread/new were similarly slow. From profiling the issue was the `sync` function of `topic-tracking-state.js`, which calls down to `isNew` which in turn calls `moment`, a change I had made in the PR above. The time it takes locally with ~1400 topics in the tracking state is 2.3 seconds.
To solve this issue, I have moved these calculations for "created in new period" and "unread not too old" into the tracking state serializer.
When I was looking at the profiler I also noticed this issue which was just compounding the problem. Every time we modify topic tracking state we recalculate the sidebar tracking/everything/tag counts. However this calls `forEachTracked` and `countTags` which can be quite expensive as they go through the whole tracking state (and were also calling the removed moment functions).
I added some logs and this was being called 30 times when navigating to a new /unread route because `sync` is being called from `build-topic-route` (one for each topic loaded due to pagination). So I just added a debounce here and it makes things even faster.
Finally, I changed topic tracking state to use a Map so our counts of the state keys is faster (Maps have .size whereas objects you have to do Object.keys(obj) which is O(n).)
<!-- NOTE: All pull requests should have tests (rspec in Ruby, qunit in JavaScript). If your code does not include test coverage, please include an explanation of why it was omitted. -->
2021-06-01 19:06:29 -04:00
|
|
|
def treat_as_new_topic_start_date
|
|
|
|
object.user_option.treat_as_new_topic_start_date
|
|
|
|
end
|
|
|
|
|
2020-08-14 09:40:56 -04:00
|
|
|
def skip_new_user_tips
|
|
|
|
object.user_option.skip_new_user_tips
|
|
|
|
end
|
|
|
|
|
2017-08-11 22:05:16 -04:00
|
|
|
def include_primary_group_id?
|
|
|
|
object.primary_group_id.present?
|
|
|
|
end
|
|
|
|
|
2018-03-07 22:54:31 -05:00
|
|
|
def external_id
|
|
|
|
object&.single_sign_on_record&.external_id
|
|
|
|
end
|
|
|
|
|
|
|
|
def include_external_id?
|
2021-02-08 05:04:33 -05:00
|
|
|
SiteSetting.enable_discourse_connect
|
2018-03-07 22:54:31 -05:00
|
|
|
end
|
2019-03-20 00:40:25 -04:00
|
|
|
|
|
|
|
def second_factor_enabled
|
2019-11-08 00:11:53 -05:00
|
|
|
object.totp_enabled? || object.security_keys_enabled?
|
2019-03-20 00:40:25 -04:00
|
|
|
end
|
2019-12-09 14:15:47 -05:00
|
|
|
|
|
|
|
def featured_topic
|
|
|
|
object.user_profile.featured_topic
|
|
|
|
end
|
2021-03-19 09:19:15 -04:00
|
|
|
|
|
|
|
def has_topic_draft
|
|
|
|
true
|
|
|
|
end
|
|
|
|
|
|
|
|
def include_has_topic_draft?
|
|
|
|
Draft.has_topic_draft(object)
|
|
|
|
end
|
2013-02-05 14:16:51 -05:00
|
|
|
end
|