Mercurial > kallithea
changeset 8077:effd091203ae
py3: drop __unicode__ ... and generally move towards just providing a good __repr__
We should never show str() (or similar) to end users.
author | Mads Kiilerich <mads@kiilerich.com> |
---|---|
date | Sat, 28 Dec 2019 23:09:48 +0100 |
parents | bdb79ef2c879 |
children | 45bfab30d433 |
files | kallithea/lib/auth.py kallithea/lib/vcs/backends/base.py kallithea/lib/vcs/nodes.py kallithea/model/db.py |
diffstat | 4 files changed, 49 insertions(+), 58 deletions(-) [+] |
line wrap: on
line diff
--- a/kallithea/lib/auth.py Thu Dec 26 05:17:09 2019 +0100 +++ b/kallithea/lib/auth.py Sat Dec 28 23:09:48 2019 +0100 @@ -543,7 +543,7 @@ if x[1] == 'usergroup.admin'] def __repr__(self): - return "<AuthUser('id:%s[%s]')>" % (self.user_id, self.username) + return "<%s %s: %r>" % (self.__class__.__name__, self.user_id, self.username) def to_cookie(self): """ Serializes this login session to a cookie `dict`. """
--- a/kallithea/lib/vcs/backends/base.py Thu Dec 26 05:17:09 2019 +0100 +++ b/kallithea/lib/vcs/backends/base.py Sat Dec 28 23:09:48 2019 +0100 @@ -365,9 +365,6 @@ def __repr__(self): return self.__str__() - def __unicode__(self): - return u'%s:%s' % (self.revision, self.short_id) - def __eq__(self, other): if type(self) is not type(other): return False
--- a/kallithea/lib/vcs/nodes.py Thu Dec 26 05:17:09 2019 +0100 +++ b/kallithea/lib/vcs/nodes.py Sat Dec 28 23:09:48 2019 +0100 @@ -180,12 +180,6 @@ def __repr__(self): return '<%s %r>' % (self.__class__.__name__, self.path) - def __str__(self): - return self.__repr__() - - def __unicode__(self): - return self.name - def get_parent_path(self): """ Returns node's parent path or empty string if node is root.
--- a/kallithea/model/db.py Thu Dec 26 05:17:09 2019 +0100 +++ b/kallithea/model/db.py Sat Dec 28 23:09:48 2019 +0100 @@ -167,12 +167,6 @@ Session().delete(obj) def __repr__(self): - if hasattr(self, '__unicode__'): - # python repr needs to return str - try: - return safe_str(self.__unicode__()) - except UnicodeDecodeError: - pass return '<DB:%s>' % (self.__class__.__name__) @@ -239,10 +233,10 @@ % (list(self.SETTINGS_TYPES), val)) self._app_settings_type = val - def __unicode__(self): - return u"<%s('%s:%s[%s]')>" % ( + def __repr__(self): + return "<%s %s.%s=%r>" % ( self.__class__.__name__, - self.app_settings_name, self.app_settings_value, self.app_settings_type + self.app_settings_name, self.app_settings_type, self.app_settings_value ) @classmethod @@ -398,8 +392,9 @@ new_ui.ui_value = val def __repr__(self): - return '<%s[%s]%s=>%s]>' % (self.__class__.__name__, self.ui_section, - self.ui_key, self.ui_value) + return '<%s %s.%s=%r>' % ( + self.__class__.__name__, + self.ui_section, self.ui_key, self.ui_value) class User(Base, BaseDbModel): @@ -535,9 +530,8 @@ except Exception: log.error(traceback.format_exc()) - def __unicode__(self): - return u"<%s('id:%s:%s')>" % (self.__class__.__name__, - self.user_id, self.username) + def __repr__(self): + return "<%s %s: %r')>" % (self.__class__.__name__, self.user_id, self.username) @classmethod def guess_instance(cls, value): @@ -776,9 +770,8 @@ ip_range=self._get_ip_range(self.ip_addr) ) - def __unicode__(self): - return u"<%s('user_id:%s=>%s')>" % (self.__class__.__name__, - self.user_id, self.ip_addr) + def __repr__(self): + return "<%s %s: %s>" % (self.__class__.__name__, self.user_id, self.ip_addr) class UserLog(Base, BaseDbModel): @@ -796,10 +789,10 @@ action = Column(UnicodeText(), nullable=False) action_date = Column(DateTime(timezone=False), nullable=False) - def __unicode__(self): - return u"<%s('id:%s:%s')>" % (self.__class__.__name__, - self.repository_name, - self.action) + def __repr__(self): + return "<%s %r: %r')>" % (self.__class__.__name__, + self.repository_name, + self.action) @property def action_as_day(self): @@ -849,10 +842,10 @@ except Exception: log.error(traceback.format_exc()) - def __unicode__(self): - return u"<%s('id:%s:%s')>" % (self.__class__.__name__, - self.users_group_id, - self.users_group_name) + def __repr__(self): + return "<%s %s: %r')>" % (self.__class__.__name__, + self.users_group_id, + self.users_group_name) @classmethod def guess_instance(cls, value): @@ -1013,9 +1006,9 @@ primaryjoin='PullRequest.other_repo_id==Repository.repo_id', cascade="all, delete-orphan") - def __unicode__(self): - return u"<%s('%s:%s')>" % (self.__class__.__name__, self.repo_id, - safe_unicode(self.repo_name)) + def __repr__(self): + return "<%s %s: %r>" % (self.__class__.__name__, + self.repo_id, self.repo_name) @hybrid_property def landing_rev(self): @@ -1488,9 +1481,9 @@ self.group_name = group_name self.parent_group = parent_group - def __unicode__(self): - return u"<%s('id:%s:%s')>" % (self.__class__.__name__, self.group_id, - self.group_name) + def __repr__(self): + return "<%s %s: %s>" % (self.__class__.__name__, + self.group_id, self.group_name) @classmethod def _generate_choice(cls, repo_group): @@ -1735,8 +1728,8 @@ permission_id = Column(Integer(), primary_key=True) permission_name = Column(String(255), nullable=False) - def __unicode__(self): - return u"<%s('%s:%s')>" % ( + def __repr__(self): + return "<%s %s: %r>" % ( self.__class__.__name__, self.permission_id, self.permission_name ) @@ -1801,8 +1794,9 @@ Session().add(n) return n - def __unicode__(self): - return u'<%s => %s >' % (self.user, self.repository) + def __repr__(self): + return '<%s %s at %s: %s>' % ( + self.__class__.__name__, self.user, self.repository, self.permission) class UserUserGroupToPerm(Base, BaseDbModel): @@ -1830,8 +1824,9 @@ Session().add(n) return n - def __unicode__(self): - return u'<%s => %s >' % (self.user, self.user_group) + def __repr__(self): + return '<%s %s at %s: %s>' % ( + self.__class__.__name__, self.user, self.user_group, self.permission) class UserToPerm(Base, BaseDbModel): @@ -1848,8 +1843,9 @@ user = relationship('User') permission = relationship('Permission') - def __unicode__(self): - return u'<%s => %s >' % (self.user, self.permission) + def __repr__(self): + return '<%s %s: %s>' % ( + self.__class__.__name__, self.user, self.permission) class UserGroupRepoToPerm(Base, BaseDbModel): @@ -1877,8 +1873,9 @@ Session().add(n) return n - def __unicode__(self): - return u'<UserGroupRepoToPerm:%s => %s >' % (self.users_group, self.repository) + def __repr__(self): + return '<%s %s at %s: %s>' % ( + self.__class__.__name__, self.users_group, self.repository, self.permission) class UserGroupUserGroupToPerm(Base, BaseDbModel): @@ -1906,8 +1903,9 @@ Session().add(n) return n - def __unicode__(self): - return u'<UserGroupUserGroup:%s => %s >' % (self.target_user_group, self.user_group) + def __repr__(self): + return '<%s %s at %s: %s>' % ( + self.__class__.__name__, self.user_group, self.target_user_group, self.permission) class UserGroupToPerm(Base, BaseDbModel): @@ -2039,8 +2037,8 @@ self.cache_args = repo_name self.cache_active = False - def __unicode__(self): - return u"<%s('%s:%s[%s]')>" % ( + def __repr__(self): + return "<%s %s: %s=%s" % ( self.__class__.__name__, self.cache_id, self.cache_key, self.cache_active) @@ -2229,8 +2227,8 @@ comment = relationship('ChangesetComment') pull_request = relationship('PullRequest') - def __unicode__(self): - return u"<%s('%s:%s')>" % ( + def __repr__(self): + return "<%s %r by %r>" % ( self.__class__.__name__, self.status, self.author ) @@ -2450,7 +2448,9 @@ return (self.gist_expires != -1) & (time.time() > self.gist_expires) def __repr__(self): - return '<Gist:[%s]%s>' % (self.gist_type, self.gist_access_id) + return "<%s %s %s>" % ( + self.__class__.__name__, + self.gist_type, self.gist_access_id) @classmethod def guess_instance(cls, value):