Merge "Raise a new exception class when we get a page with namespace …

…< 0"

pywikibot/__init__.py

@@ -53,7 +53,7 @@
     SiteDefinitionError, NoSuchSite, UnknownSite, UnknownFamily,
     UnknownExtension,
     NoUsername, UserBlocked,
-    PageRelatedError, IsRedirectPage, IsNotRedirectPage,
+    PageRelatedError, UnsupportedPage, IsRedirectPage, IsNotRedirectPage,
     PageSaveRelatedError, PageNotSaved, OtherPageSaveError,
     LockedPage, CascadeLockedPage, LockedNoPage, NoCreateError,
     EditConflict, PageDeletedConflict, PageCreatedConflict,
@@ -106,7 +106,8 @@
     'SiteDefinitionError', 'NoSuchSite', 'UnknownSite', 'UnknownFamily',
     'UnknownExtension',
     'NoUsername', 'UserBlocked', 'UserActionRefuse',
-    'PageRelatedError', 'IsRedirectPage', 'IsNotRedirectPage',
+    'PageRelatedError', 'UnsupportedPage', 'IsRedirectPage',
+    'IsNotRedirectPage',
     'PageSaveRelatedError', 'PageNotSaved', 'OtherPageSaveError',
     'LockedPage', 'CascadeLockedPage', 'LockedNoPage', 'NoCreateError',
     'EditConflict', 'PageDeletedConflict', 'PageCreatedConflict',

pywikibot/data/api.py

@@ -35,7 +35,7 @@
 from pywikibot.comms import http
 from pywikibot.exceptions import (
     Server504Error, Server414Error, FatalServerError, NoUsername,
-    Error, TimeoutError, InvalidTitle
+    Error, TimeoutError, InvalidTitle, UnsupportedPage
 )
 from pywikibot.tools import (
     MediaWikiVersion, deprecated, itergroup, ip, PY2, getargspec,
@@ -3170,6 +3170,8 @@ def update_page(page, pagedict, props=[]):
         property which would make the value present must be in the props
         parameter.
     @type props: iterable of string
+    @raises InvalidTitle: Page title is invalid
+    @raises UnsupportedPage: Page with namespace < 0 is not supported yet
     """
     if "pageid" in pagedict:
         page._pageid = int(pagedict['pageid'])
@@ -3180,6 +3182,8 @@ def update_page(page, pagedict, props=[]):
         if page.site.sametitle(page.title(), pagedict['title']):
             if 'invalid' in pagedict:
                 raise InvalidTitle('%s: %s' % (page, pagedict['invalidreason']))
+        if int(pagedict['ns']) < 0:
+            raise UnsupportedPage(page)
         raise AssertionError(
             "Page %s has neither 'pageid' nor 'missing' attribute" % pagedict['title'])
     page._contentmodel = pagedict.get('contentmodel')  # can be None

pywikibot/exceptions.py

@@ -24,6 +24,7 @@
 PageRelatedError: any exception which is caused by an operation on a Page.
 
   - NoPage: Page does not exist
+  - UnsupportedPage: Page is not supported due to a namespace restriction
   - IsRedirectPage: Page is a redirect page
   - IsNotRedirectPage: Page is not a redirect page
   - CircularRedirect: Page is a circular redirect
@@ -227,6 +228,16 @@ class NoPage(PageRelatedError):  # noqa
     pass
 
 
+class UnsupportedPage(PageRelatedError):
+
+    """Unsupported page due to namespace restriction."""
+
+    # namespaces < 0 aren't supported (T169213)
+    message = 'Page %s is not supported due to namespace restriction.'
+
+    pass
+
+
 class NoMoveTarget(PageRelatedError):
 
     """Expected move target page not found."""