Changeset - 862e55888ff8
[Not reviewed]
default
0 3 0
domruf - 8 years ago 2017-08-06 12:36:57
dominikruf@gmail.com
js: don't show number of members of groups when doing autocomplete

This is done to make the database queries cheaper.
3 files changed with 2 insertions and 7 deletions:
0 comments (0 inline, 0 general)
kallithea/model/repo.py
Show inline comments
 
@@ -140,14 +140,12 @@ class RepoModel(object):
 
        user_groups = UserGroup.query() \
 
            .filter(UserGroup.users_group_active == True) \
 
            .order_by(UserGroup.users_group_name) \
 
            .options(subqueryload(UserGroup.members)) \
 
            .all()
 
        user_groups = UserGroupList(user_groups, perm_level='read')
 
        return [
 
            {
 
                'id': gr.users_group_id,
 
                'grname': gr.users_group_name,
 
                'grmembers': len(gr.members),
 
            } for gr in user_groups]
 

	
 
    @classmethod
kallithea/public/js/base.js
Show inline comments
 
@@ -1109,11 +1109,9 @@ var autocompleteFormatter = function (oR
 
    // group
 
    if (oResultData.grname) {
 
        return autocompleteGravatar(
 
            "{0}: {1} ({2} {3})".format(
 
            "{0}: {1}".format(
 
                _TM['Group'],
 
                autocompleteHighlightMatch(oResultData.grname, query),
 
                oResultData.grmembers,
 
                _TM['members']),
 
                autocompleteHighlightMatch(oResultData.grname, query)),
 
            null, null, true);
 
    }
 

	
kallithea/templates/base/root.html
Show inline comments
 
@@ -31,7 +31,6 @@
 
                'Stop following this repository': ${h.jshtml(_('Stop following this repository'))},
 
                'Start following this repository': ${h.jshtml(_('Start following this repository'))},
 
                'Group': ${h.jshtml(_('Group'))},
 
                'members': ${h.jshtml(_('members'))},
 
                'Loading ...': ${h.jshtml(_('Loading ...'))},
 
                'loading ...': ${h.jshtml(_('loading ...'))},
 
                'Search truncated': ${h.jshtml(_('Search truncated'))},
0 comments (0 inline, 0 general)