kallithea Changeset - 737c3704b44a

Changeset - 737c3704b44a

Parent rev.

Child rev.

[Not reviewed]

stable

0 4 0

Mads Kiilerich - 10 years ago 2015-11-07 13:16:58
madski@unity3d.com

cleanup: fixes of checking for None

Don't update repoinfo for all repos if an invalid repo is specified.

4 files changed with 4 insertions and 4 deletions:

kallithea/lib/paster_commands/update_repoinfo.py

kallithea/lib/vcs/backends/git/changeset.py

kallithea/lib/vcs/backends/hg/changeset.py

kallithea/model/repo.py

0 comments (0 inline, 0 general)

kallithea/lib/paster_commands/update_repoinfo.py

➞

Show inline comments

@@ @@ -63,7 +63,7 @@ class Command(BasePasterCommand): @@
                                self.options.repo_update_list.split(',')) \
                                if self.options.repo_update_list else None
         if repo_update_list:
+        if repo_update_list is not None:
             repo_list = list(Repository.query()\
                 .filter(Repository.repo_name.in_(repo_update_list)))
         else:

kallithea/lib/vcs/backends/git/changeset.py

➞

Show inline comments

@@ @@ -287,7 +287,7 @@ class GitChangeset(BaseChangeset): @@
         cs_id = safe_str(self.id)
         f_path = safe_str(path)
         if limit:
+        if limit is not None:
             cmd = ['log', '-n', str(safe_int(limit, 0)),
                    '--pretty=format:%H', '-s', cs_id, '--', f_path]

kallithea/lib/vcs/backends/hg/changeset.py

➞

Show inline comments

@@ @@ -256,7 +256,7 @@ class MercurialChangeset(BaseChangeset): @@
         for cs in reversed([x for x in fctx.filelog()]):
             cnt += 1
             hist.append(hex(fctx.filectx(cs).node()))
             if limit and cnt == limit:
+            if limit is not None and cnt == limit:
                 break
         return [self.repository.get_changeset(node) for node in hist]

kallithea/model/repo.py

➞

Show inline comments

@@ @@ -175,7 +175,7 @@ class RepoModel(BaseModel): @@
     @classmethod
     def update_repoinfo(cls, repositories=None):
-        if not repositories:
+        if repositories is None:
             repositories = Repository.getAll()
         for repo in repositories:
             repo.update_changeset_cache()

0 comments (0 inline, 0 general)