Plugin: add missing substitution, check for supybot.authors.unknown explicitly

This commit is contained in:
James Lu 2019-10-19 12:01:47 -07:00
parent 35a64eca38
commit d3a2964705

View File

@ -170,7 +170,7 @@ class Plugin(callbacks.Plugin):
if author != supybot.authors.unknown: if author != supybot.authors.unknown:
s = _('The %s plugin was written by %s. ' % (cb.name(), author)) s = _('The %s plugin was written by %s. ' % (cb.name(), author))
else: else:
s = _('The %s plugin has not been claimed by an author. ') s = _('The %s plugin has not been claimed by an author. ') % cb.name()
contribs = getattr(module, '__contributors__', {}) contribs = getattr(module, '__contributors__', {})
if contribs: if contribs:
@ -191,8 +191,8 @@ class Plugin(callbacks.Plugin):
contributorNicks = dict((author.nick.lower(), author) for author in contributors.keys()) contributorNicks = dict((author.nick.lower(), author) for author in contributors.keys())
lnick = nick.lower() lnick = nick.lower()
author = getattr(module, '__author__', None) author = getattr(module, '__author__', supybot.authors.unknown)
if author and lnick == author.nick.lower(): if author != supybot.authors.unknown and lnick == author.nick.lower():
# Special case for the plugin author. We remove legacy handling of the case where # Special case for the plugin author. We remove legacy handling of the case where
# someone is listed both as author and contributor, which should never really happen? # someone is listed both as author and contributor, which should never really happen?
return _('%s wrote the %s plugin.') % (author, cb.name()) return _('%s wrote the %s plugin.') % (author, cb.name())