Skip to content
This repository was archived by the owner on Mar 15, 2018. It is now read-only.

Commit 8dc6faf

Browse files
committed
Revert "limit personas displayed on profile page to 60 (bug 860407)"
This reverts commit 3f2f1f7.
1 parent fa199ca commit 8dc6faf

File tree

2 files changed

+4
-10
lines changed

2 files changed

+4
-10
lines changed

apps/users/templates/users/profile.html

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -64,11 +64,7 @@ <h2>{{ _("Add-ons I've created") }}</h2>
6464

6565
{% if personas %}
6666
<div id="{{ 'my-themes' if addons.object_list else 'my-submissions' }}" class="island c">
67-
{% if limited_personas %}
68-
<h2>{{ _("My most popular 60 Themes") }}</h2>
69-
{% else %}
70-
<h2>{{ _("Themes I've created") }}</h2>
71-
{% endif %}
67+
<h2>{{ _("Themes I've created") }}</h2>
7268
{{ personas|impala_persona_grid(cols=5, pagesize=15) }}
7369
</div>
7470
{% endif %}

apps/users/views.py

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -581,9 +581,8 @@ def profile(request, user):
581581
items = user.apps_listed
582582
else:
583583
items = user.addons_listed.exclude(type=amo.ADDON_PERSONA)
584-
personas = user.addons_listed.filter(type=amo.ADDON_PERSONA)
585-
#XXX hack to protect performance until we have proper incremental loading
586-
personas = personas.order_by('-average_daily_users')[:60]
584+
personas = (user.addons_listed.filter(type=amo.ADDON_PERSONA)
585+
.order_by('-average_daily_users'))
587586
addons = amo.utils.paginate(request,
588587
items.order_by('-weekly_downloads'))
589588
else:
@@ -603,8 +602,7 @@ def get_addons(reviews):
603602
data = {'profile': user, 'own_coll': own_coll, 'reviews': reviews,
604603
'fav_coll': fav_coll, 'edit_any_user': edit_any_user,
605604
'addons': addons, 'own_profile': own_profile,
606-
'personas': personas,
607-
'limited_personas': len(personas) == 60}
605+
'personas': personas}
608606
if not own_profile:
609607
data['abuse_form'] = AbuseForm(request=request)
610608

0 commit comments

Comments
 (0)