kallithea Changeset - 506dd480e408

Changeset - 506dd480e408

Parent rev.

Child rev.

[Not reviewed]

default

0 3 0

Søren Løvborg - 9 years ago 2016-09-15 15:25:13
sorenl@unity3d.com

db: always do case-insensitive sorting of repository names

We retain the implicit order_by on the follows_repository relationship.
This is probably a bad idea, since it causes sorting even when it's not
needed; but for now, at least, we consistently do case-insensitive sort.

3 files changed with 4 insertions and 6 deletions:

kallithea/controllers/admin/repos.py

kallithea/controllers/home.py

kallithea/model/db.py

0 comments (0 inline, 0 general)

kallithea/controllers/admin/repos.py

➞

Show inline comments

@@ @@ -415,7 +415,7 @@ class ReposController(BaseRepoController @@
             .filter(UserFollowing.user_id == c.default_user_id) \
             .filter(UserFollowing.follows_repository == c.repo_info).scalar()
-        _repos = Repository.query().order_by(Repository.repo_name).all()
+        _repos = Repository.query(sorted=True).all()
         read_access_repos = RepoList(_repos)
         c.repos_list = [(None, _('-- Not a fork --'))]
         c.repos_list += [(x.repo_id, x.repo_name)

kallithea/controllers/home.py

➞

Show inline comments

@@ @@ -72,7 +72,7 @@ class HomeController(BaseController): @@
         #wrapper for conditional cache
         def _c():
             log.debug('generating switcher repo/groups list')
-            all_repos = Repository.query().order_by(Repository.repo_name).all()
+            all_repos = Repository.query(sorted=True).all()
             repo_iter = self.scm_model.get_repos(all_repos)
             all_groups = RepoGroup.query().order_by(RepoGroup.group_name).all()
             repo_groups_iter = self.scm_model.get_repo_groups(all_groups)

kallithea/model/db.py

➞

Show inline comments

@@ @@ -1619,9 +1619,7 @@ class RepoGroup(Base, BaseModel): @@
     @property
     def repositories(self):
         return Repository.query() \
                 .filter(Repository.group == self) \
                 .order_by(Repository.repo_name)
         return Repository.query(sorted=True).filter_by(group=self)
     @property
     def repositories_recursive_count(self):
@@ @@ -2058,7 +2056,7 @@ class UserFollowing(Base, BaseModel): @@
     user = relationship('User', primaryjoin='User.user_id==UserFollowing.user_id')
     follows_user = relationship('User', primaryjoin='User.user_id==UserFollowing.follows_user_id')
-    follows_repository = relationship('Repository', order_by='Repository.repo_name')
+    follows_repository = relationship('Repository', order_by=lambda: func.lower(Repository.repo_name))
     @classmethod
     def get_repo_followers(cls, repo_id):

0 comments (0 inline, 0 general)