jenkins-bot has submitted this change and it was merged.

Change subject: [bugfix] Avoid KeyError for missing 'warnings' key
......................................................................


[bugfix] Avoid KeyError for missing 'warnings' key

Change-Id: I83d935a86be1e79a3638b81527bb253b7a153906
---
M wikipedia.py
1 file changed, 18 insertions(+), 17 deletions(-)

Approvals:
  Ricordisamoa: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/wikipedia.py b/wikipedia.py
index e825f9f..35438b5 100644
--- a/wikipedia.py
+++ b/wikipedia.py
@@ -7900,23 +7900,24 @@
                 raise Error
 
             # _handle_warnings from core
-            for mod, warning in result['warnings'].items():
-                if mod == 'info':
-                    continue
-                if '*' in warning:
-                    text = warning['*']
-                elif 'html' in warning:
-                    # Bugzilla 49978
-                    text = warning['html']['*']
-                else:
-                    pywikibot.warning(
-                        u'API warning ({0})of unknown format: {1}'.
-                        format(mod, warning))
-                    continue
-                # multiple warnings are in text separated by a newline
-                for single_warning in text.splitlines():
-                    pywikibot.warning(u"API warning (%s): %s"
-                                      % (mod, single_warning))
+            if 'warnings' in result:
+                for mod, warning in result['warnings'].items():
+                    if mod == 'info':
+                        continue
+                    if '*' in warning:
+                        text = warning['*']
+                    elif 'html' in warning:
+                        # Bugzilla 49978
+                        text = warning['html']['*']
+                    else:
+                        pywikibot.warning(
+                            u'API warning ({0})of unknown format: {1}'.
+                            format(mod, warning))
+                        continue
+                    # multiple warnings are in text separated by a newline
+                    for single_warning in text.splitlines():
+                        pywikibot.warning(u"API warning (%s): %s"
+                                          % (mod, single_warning))
 
             for c in result['query']['logevents']:
                 if (not namespace or c['ns'] in namespace) and \

-- 
To view, visit https://gerrit.wikimedia.org/r/218826
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I83d935a86be1e79a3638b81527bb253b7a153906
Gerrit-PatchSet: 1
Gerrit-Project: pywikibot/compat
Gerrit-Branch: master
Gerrit-Owner: Xqt <[email protected]>
Gerrit-Reviewer: John Vandenberg <[email protected]>
Gerrit-Reviewer: Ladsgroup <[email protected]>
Gerrit-Reviewer: Ricordisamoa <[email protected]>
Gerrit-Reviewer: jenkins-bot <>

_______________________________________________
Pywikibot-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/pywikibot-commits

Reply via email to