jenkins-bot submitted this change.

View Change

Approvals: Matěj Suchánek: Looks good to me, but someone else must approve Xqt: Looks good to me, approved jenkins-bot: Verified
[IMPR] Simplify raising HiddenKeyError in LogEntry

Change-Id: Ie81d67603ac9ac1452eeb27dd62201e47bc22505
---
M pywikibot/logentries.py
1 file changed, 13 insertions(+), 7 deletions(-)

diff --git a/pywikibot/logentries.py b/pywikibot/logentries.py
index d6fe6a3..e91168b 100644
--- a/pywikibot/logentries.py
+++ b/pywikibot/logentries.py
@@ -54,13 +54,19 @@
"""
pywikibot.debug('API log entry received:\n{!r}'.format(self),
_logger)
- hidden = {'action', 'logpage', 'ns', 'pageid', 'params', 'title'}
- if ((key in hidden and 'actionhidden' in self)
- or (key == 'comment' and 'commenthidden' in self)
- or (key == 'user' and 'userhidden' in self)):
- raise HiddenKeyError(
- "Log entry ({}) has a hidden '{}' key and you don't have "
- 'permission to view it.'.format(self['type'], key))
+ hidden = {
+ 'actionhidden': [
+ 'action', 'logpage', 'ns', 'pageid', 'params', 'title',
+ ],
+ 'commenthidden': ['comment'],
+ 'userhidden': ['user'],
+ }
+ for hidden_key, hidden_types in hidden.items():
+ if hidden_key in self and key in hidden_types:
+ raise HiddenKeyError(
+ "Log entry ({}) has a hidden '{}' key and you don't have "
+ "permission to view it due to '{}'"
+ .format(self['type'], key, hidden_key))

raise KeyError('Log entry ({}) has no {!r} key'
.format(self['type'], key))

To view, visit change 766191. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: pywikibot/core
Gerrit-Branch: master
Gerrit-Change-Id: Ie81d67603ac9ac1452eeb27dd62201e47bc22505
Gerrit-Change-Number: 766191
Gerrit-PatchSet: 2
Gerrit-Owner: Xqt <info@gno.de>
Gerrit-Reviewer: Matěj Suchánek <matejsuchanek97@gmail.com>
Gerrit-Reviewer: Xqt <info@gno.de>
Gerrit-Reviewer: jenkins-bot
Gerrit-MessageType: merged