jeff/umap
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Remove the limit of visible maps in user’s view

Browse source 
It should not have any performance issue given that we paginate over the list anyway.

Fix #1025
This commit is contained in:
David Larlet 2023-05-12 13:51:39 -04:00
parent 7dda4a6da0
commit da1e25be2a
1 changed files with 1 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
umap/views.py
View file

@ -160,11 +160,9 @@ class UserMaps(DetailView, PaginatorMixin):
maps = manager.filter(Q(owner=self.object) | Q(editors=self.object)) maps = manager.filter(Q(owner=self.object) | Q(editors=self.object))
if owner: if owner:
per_page = settings.UMAP_MAPS_PER_PAGE_OWNER per_page = settings.UMAP_MAPS_PER_PAGE_OWNER
limit = 100
else: else:
per_page = settings.UMAP_MAPS_PER_PAGE per_page = settings.UMAP_MAPS_PER_PAGE
limit = 50 maps = maps.distinct().order_by("-modified_at")
maps = maps.distinct().order_by("-modified_at")[:limit]
maps = self.paginate(maps, per_page) maps = self.paginate(maps, per_page)
kwargs.update({"maps": maps}) kwargs.update({"maps": maps})
return super(UserMaps, self).get_context_data(**kwargs) return super(UserMaps, self).get_context_data(**kwargs)