mirror of https://github.com/zulip/zulip.git
Simplify analytics/activity.html.
For legacy reasons, this template wanted each tab's content as a one-key dictionary, instead of a string. Each tab already has a tuple to allow for fields like title, so this wasn't really giving us any long term flexibility; it was just crufting up the calling code. (imported from commit 2a316107ec223a83efa8735f4810a6fa43107541)
This commit is contained in:
parent
38e479d4a6
commit
59ec080a8d
|
@ -142,7 +142,7 @@ def realm_summary_table(realm_minutes):
|
||||||
'analytics/realm_summary_table.html',
|
'analytics/realm_summary_table.html',
|
||||||
dict(rows=rows, num_active_sites=num_active_sites)
|
dict(rows=rows, num_active_sites=num_active_sites)
|
||||||
)
|
)
|
||||||
return dict(content=content)
|
return content
|
||||||
|
|
||||||
|
|
||||||
def user_activity_intervals():
|
def user_activity_intervals():
|
||||||
|
@ -193,7 +193,7 @@ def user_activity_intervals():
|
||||||
output += "\nTotal Duration in minutes: %s\n" % (total_duration.total_seconds() / 60.,)
|
output += "\nTotal Duration in minutes: %s\n" % (total_duration.total_seconds() / 60.,)
|
||||||
output += "Total Duration amortized to a month: %s" % (total_duration.total_seconds() * 30. / 60.,)
|
output += "Total Duration amortized to a month: %s" % (total_duration.total_seconds() * 30. / 60.,)
|
||||||
content = mark_safe('<pre>' + output + '</pre>')
|
content = mark_safe('<pre>' + output + '</pre>')
|
||||||
return dict(content=content), realm_minutes
|
return content, realm_minutes
|
||||||
|
|
||||||
def sent_messages_report(realm):
|
def sent_messages_report(realm):
|
||||||
title = 'Recently sent messages for ' + realm
|
title = 'Recently sent messages for ' + realm
|
||||||
|
@ -478,7 +478,7 @@ def get_activity(request, realm=REQ(default=None)):
|
||||||
('Durations', duration_content),
|
('Durations', duration_content),
|
||||||
]
|
]
|
||||||
for page in ad_hoc_queries():
|
for page in ad_hoc_queries():
|
||||||
data.append((page['title'], page))
|
data.append((page['title'], page['content']))
|
||||||
|
|
||||||
title = 'Activity'
|
title = 'Activity'
|
||||||
|
|
||||||
|
@ -723,18 +723,16 @@ def get_realm_activity(request, realm):
|
||||||
user_records, content = realm_user_summary_table(all_records)
|
user_records, content = realm_user_summary_table(all_records)
|
||||||
all_user_records.update(user_records)
|
all_user_records.update(user_records)
|
||||||
|
|
||||||
user_content = dict(content=content)
|
data += [(page_title, content)]
|
||||||
|
|
||||||
data += [(page_title, user_content)]
|
|
||||||
|
|
||||||
page_title = 'Clients'
|
page_title = 'Clients'
|
||||||
content = realm_client_table(all_user_records)
|
content = realm_client_table(all_user_records)
|
||||||
data += [(page_title, dict(content=content))]
|
data += [(page_title, content)]
|
||||||
|
|
||||||
|
|
||||||
page_title = 'History'
|
page_title = 'History'
|
||||||
content = sent_messages_report(realm)
|
content = sent_messages_report(realm)
|
||||||
data += [(page_title, dict(content=content))]
|
data += [(page_title, content)]
|
||||||
|
|
||||||
title = realm
|
title = realm
|
||||||
return render_to_response(
|
return render_to_response(
|
||||||
|
@ -751,12 +749,10 @@ def get_user_activity(request, email):
|
||||||
user_summary = get_user_activity_summary(records)
|
user_summary = get_user_activity_summary(records)
|
||||||
content = user_activity_summary_table(user_summary)
|
content = user_activity_summary_table(user_summary)
|
||||||
|
|
||||||
user_content = dict(content=content)
|
data += [('Summary', content)]
|
||||||
data += [('Summary', user_content)]
|
|
||||||
|
|
||||||
content = raw_user_activity_table(records)
|
content = raw_user_activity_table(records)
|
||||||
user_content = dict(content=content)
|
data += [('Info', content)]
|
||||||
data += [('Info', user_content)]
|
|
||||||
|
|
||||||
realm = None
|
realm = None
|
||||||
title = email
|
title = email
|
||||||
|
|
|
@ -39,7 +39,7 @@
|
||||||
{% for name, activity in data %}
|
{% for name, activity in data %}
|
||||||
|
|
||||||
<div class="tab-pane {% if forloop.first %} active {% endif %}" id="{{ name|slugify }}">
|
<div class="tab-pane {% if forloop.first %} active {% endif %}" id="{{ name|slugify }}">
|
||||||
{{ activity.content }}
|
{{ activity }}
|
||||||
</div>
|
</div>
|
||||||
|
|
||||||
{% endfor %}
|
{% endfor %}
|
||||||
|
|
Loading…
Reference in New Issue