jenkins-bot has submitted this change. ( https://gerrit.wikimedia.org/r/c/pywikibot/core/+/894208 )
Change subject: [bugfix] load page info when creating a page if not updated previously
......................................................................
[bugfix] load page info when creating a page if not updated previously
If pageid was retrieved and the page was created after that,
the new pageid is not updated and remains 0. Therefore call
site.loadpageinfo in such case.
Bug: T330980
Change-Id: I6a867da59dda24e0387db556edda85a92d570cca
---
M pywikibot/page/_basepage.py
1 file changed, 18 insertions(+), 0 deletions(-)
Approvals:
Bináris: Looks good to me, but someone else must approve
Xqt: Looks good to me, approved
jenkins-bot: Verified
diff --git a/pywikibot/page/_basepage.py b/pywikibot/page/_basepage.py
index 2dc891a..8a7374e 100644
--- a/pywikibot/page/_basepage.py
+++ b/pywikibot/page/_basepage.py
@@ -1291,6 +1291,10 @@
if not quiet:
pywikibot.warning(f'Page {link} not saved')
raise PageSaveRelatedError(self)
+
+ if not self.pageid:
+ self.site.loadpageinfo(self)
+
if not quiet:
pywikibot.info(f'Page {link} saved')
--
To view, visit https://gerrit.wikimedia.org/r/c/pywikibot/core/+/894208
To unsubscribe, or for help writing mail filters, visit https://gerrit.wikimedia.org/r/settings
Gerrit-Project: pywikibot/core
Gerrit-Branch: master
Gerrit-Change-Id: I6a867da59dda24e0387db556edda85a92d570cca
Gerrit-Change-Number: 894208
Gerrit-PatchSet: 1
Gerrit-Owner: Xqt <info(a)gno.de>
Gerrit-Reviewer: Bináris <wikiposta(a)gmail.com>
Gerrit-Reviewer: Xqt <info(a)gno.de>
Gerrit-Reviewer: jenkins-bot
Gerrit-MessageType: merged
jenkins-bot has submitted this change. ( https://gerrit.wikimedia.org/r/c/pywikibot/core/+/894206 )
Change subject: [tests] Skip test on proofwiki on github action
......................................................................
[tests] Skip test on proofwiki on github action
Access to proofwiki is forbidden (status 403) if called from
github action. Skip this test then instead of expect the failure.
Bug: T331223
Change-Id: Ice47cf3ae7883bfa4012e7b8fea4b7a2008d9dd7
---
M tests/site_detect_tests.py
1 file changed, 16 insertions(+), 4 deletions(-)
Approvals:
Xqt: Looks good to me, approved
jenkins-bot: Verified
diff --git a/tests/site_detect_tests.py b/tests/site_detect_tests.py
index 918d76f..6173ff7 100755
--- a/tests/site_detect_tests.py
+++ b/tests/site_detect_tests.py
@@ -18,7 +18,7 @@
from pywikibot.site_detect import MWSite
from tests.aspects import PatchingTestCase, TestCase
-from tests.utils import DrySite, expected_failure_if, skipping
+from tests.utils import DrySite, skipping
class SiteDetectionTestCase(TestCase):
@@ -58,7 +58,6 @@
"""Test detection of MediaWiki sites."""
standard_version_sites = (
- # 'http://www.proofwiki.org/wiki/$1',
'http://www.ck-wissen.de/ckwiki/index.php?title=$1',
'http://en.citizendium.org/wiki/$1',
# Server that hosts www.wikichristian.org is unreliable - it
@@ -113,10 +112,10 @@
with self.subTest(url=urlparse(url).netloc):
self.assertSite(url)
- @expected_failure_if(os.getenv('GITHUB_ACTIONS')) # T331223
def test_proofreadwiki(self):
"""Test detection of proofwiki.org site."""
- # test is failing on github, see T331223
+ if os.getenv('GITHUB_ACTIONS'):
+ self.skipTest('Skip test on github due to T331223')
self.assertSite('http://www.proofwiki.org/wiki/$1')
def test_non_standard_version_sites(self):
--
To view, visit https://gerrit.wikimedia.org/r/c/pywikibot/core/+/894206
To unsubscribe, or for help writing mail filters, visit https://gerrit.wikimedia.org/r/settings
Gerrit-Project: pywikibot/core
Gerrit-Branch: master
Gerrit-Change-Id: Ice47cf3ae7883bfa4012e7b8fea4b7a2008d9dd7
Gerrit-Change-Number: 894206
Gerrit-PatchSet: 1
Gerrit-Owner: Xqt <info(a)gno.de>
Gerrit-Reviewer: Xqt <info(a)gno.de>
Gerrit-Reviewer: jenkins-bot
Gerrit-MessageType: merged