kallithea Changeset - ce5d4c582a82

Changeset - ce5d4c582a82

Parent rev.

Child rev.

[Not reviewed]

default

0 11 0

Mads Kiilerich - 6 years ago 2019-11-23 23:59:28
mads@kiilerich.com

Grafted from: 998e361b5896

py3: cleanup map usage and avoid py3 ambiguity

Based on 2to3 -f map ... but replace map with something more explicit (unless
born and raised in a lisp world) (but sometimes slightly more verbose).

11 files changed with 14 insertions and 19 deletions:

kallithea/bin/kallithea_api.py

kallithea/controllers/admin/repo_groups.py

kallithea/controllers/api/__init__.py

kallithea/controllers/feed.py

kallithea/lib/helpers.py

kallithea/lib/indexers/daemon.py

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

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

kallithea/model/db.py

kallithea/model/permission.py

kallithea/model/repo.py

0 comments (0 inline, 0 general)

kallithea/bin/kallithea_api.py

➞

Show inline comments

@@ @@ -101,7 +101,7 @@ def main(argv=None): @@
         parser.error('Please specify method name')
     try:
-        margs = dict(map(lambda s: s.split(':', 1), other))
+        margs = dict(s.split(':', 1) for s in other)
     except ValueError:
         sys.stderr.write('Error parsing arguments \n')
         sys.exit()

kallithea/controllers/admin/repo_groups.py

➞

Show inline comments

@@ @@ -25,7 +25,6 @@ Original author and date, and relevant c @@
 :license: GPLv3, see LICENSE.md for more details.
 """
 import itertools
 import logging
 import traceback
@@ @@ -42,7 +41,7 @@ from kallithea.config.routing import url @@
 from kallithea.lib import helpers as h
 from kallithea.lib.auth import HasPermissionAny, HasRepoGroupPermissionLevel, HasRepoGroupPermissionLevelDecorator, LoginRequired
 from kallithea.lib.base import BaseController, render
 from kallithea.lib.utils2 import safe_int
+from kallithea.lib.utils2 import safe_int, safe_unicode
 from kallithea.model.db import RepoGroup, Repository
 from kallithea.model.forms import RepoGroupForm, RepoGroupPermsForm
 from kallithea.model.meta import Session
@@ @@ -120,9 +119,7 @@ class RepoGroupsController(BaseControlle @@
+        )
         for repo_gr in group_iter:
             children_groups = map(h.safe_unicode,
                 itertools.chain((g.name for g in repo_gr.parents),
                                 (x.name for x in [repo_gr])))
             children_groups = [safe_unicode(g.name) for g in repo_gr.parents] + [safe_unicode(repo_gr.name)]
             repo_count = repo_gr.repositories.count()
             repo_groups_data.append({
                 "raw_name": repo_gr.group_name,

kallithea/controllers/api/__init__.py

➞

Show inline comments

@@ @@ -168,7 +168,7 @@ class JSONRPCController(TGController): @@
         # self.kargs and dispatch control to WGIController
         argspec = inspect.getargspec(self._func)
         arglist = argspec[0][1:]
-        defaults = map(type, argspec[3] or [])
+        defaults = [type(arg) for arg in argspec[3] or []]
         default_empty = type(NotImplemented)
         # kw arguments required by this method

kallithea/controllers/feed.py

➞

Show inline comments

@@ @@ -100,7 +100,7 @@ class FeedController(BaseRepoController) @@
             desc_msg.append('\n\n')
             desc_msg.append(raw_diff)
         desc_msg.append('</pre>')
-        return map(safe_unicode, desc_msg)
+        return [safe_unicode(chunk) for chunk in desc_msg]
     def _feed(self, repo_name, kind, feed_factory):
         """Produce a simple feed"""

kallithea/lib/helpers.py

➞

Show inline comments

@@ @@ -379,7 +379,7 @@ def pygmentize_annotation(repo_name, fil @@
             h %= 1
             HSV_tuple = [h, 0.95, 0.95]
             RGB_tuple = hsv_to_rgb(*HSV_tuple)
-            yield map(lambda x: str(int(x * 256)), RGB_tuple)
+            yield [str(int(x * 256)) for x in RGB_tuple]
     cgenerator = gen_color()

kallithea/lib/indexers/daemon.py

➞

Show inline comments

@@ @@ -78,7 +78,7 @@ class WhooshIndexingDaemon(object): @@
         # filter repo list
         if repo_list:
             # Fix non-ascii repo names to unicode
-            repo_list = map(safe_unicode, repo_list)
+            repo_list = set(safe_unicode(repo_name) for repo_name in repo_list)
             self.filtered_repo_paths = {}
             for repo_name, repo in self.repo_paths.items():
                 if repo_name in repo_list:

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

➞

Show inline comments

@@ @@ -47,7 +47,7 @@ class GitInMemoryChangeset(BaseInMemoryC @@
         for node in self.added + self.changed:
             # Compute subdirs if needed
             dirpath, nodename = posixpath.split(node.path)
-            dirnames = map(safe_str, dirpath and dirpath.split('/') or [])
+            dirnames = safe_str(dirpath).split('/') if dirpath else []
             parent = commit_tree
             ancestors = [('', parent)]

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

➞

Show inline comments

@@ @@ -27,7 +27,7 @@ class MercurialChangeset(BaseChangeset): @@
     @LazyProperty
     def tags(self):
-        return map(safe_unicode, self._ctx.tags())
+        return [safe_unicode(tag) for tag in self._ctx.tags()]
     @LazyProperty
     def branch(self):
@@ @@ -95,7 +95,7 @@ class MercurialChangeset(BaseChangeset): @@
     @LazyProperty
     def bookmarks(self):
-        return map(safe_unicode, self._ctx.bookmarks())
+        return [safe_unicode(bookmark) for bookmark in self._ctx.bookmarks()]
     @LazyProperty
     def message(self):

kallithea/model/db.py

➞

Show inline comments

@@ @@ -1163,7 +1163,7 @@ class Repository(Base, BaseDbModel): @@
         # names in the database, but that eventually needs to be converted
         # into a valid system path
         p += self.repo_name.split(Repository.url_sep())
-        return os.path.join(*map(safe_unicode, p))
+        return os.path.join(*(safe_unicode(d) for d in p))
     @property
     def cache_keys(self):
@@ @@ -2511,8 +2511,7 @@ class Gist(Base, BaseDbModel): @@
     def scm_instance(self):
         from kallithea.lib.vcs import get_repo
         base_path = self.base_path()
         return get_repo(os.path.join(*map(safe_str,
                                           [base_path, self.gist_access_id])))
         return get_repo(os.path.join(safe_str(base_path), safe_str(self.gist_access_id)))
 class UserSshKeys(Base, BaseDbModel):

kallithea/model/permission.py

➞

Show inline comments

@@ @@ -73,8 +73,7 @@ class PermissionModel(object): @@
             return '.'.join(perm_name.split('.')[:1])
         perms = UserToPerm.query().filter(UserToPerm.user == user).all()
         defined_perms_groups = map(_get_group,
                                 (x.permission.permission_name for x in perms))
         defined_perms_groups = set(_get_group(x.permission.permission_name) for x in perms)
         log.debug('GOT ALREADY DEFINED:%s', perms)
         DEFAULT_PERMS = Permission.DEFAULT_USER_PERMISSIONS

kallithea/model/repo.py

➞

Show inline comments

@@ @@ -644,7 +644,7 @@ class RepoModel(object): @@
         else:
             _paths = [self.repos_path, new_parent_path, repo_name]
             # we need to make it str for mercurial
-        repo_path = os.path.join(*map(lambda x: safe_str(x), _paths))
+        repo_path = os.path.join(*(safe_str(x) for x in _paths))
         # check if this path is not a repository
         if is_valid_repo(repo_path, self.repos_path):

0 comments (0 inline, 0 general)