jayvdb created this task.
jayvdb claimed this task.
jayvdb added subscribers: pywikipedia-bugs, valhallasw, siebrand, Nemo_bis, jayvdb.
jayvdb added projects: Pywikibot-i18n, i18n.
TASK DESCRIPTION
JSON files have been added to pywikibot/i18n, which now has python and JSON files with the same messages. The JSON files are not used yet, as the code changes to enable JSON have exposed packaging problems that are the subject of RFC https://www.mediawiki.org/wiki/Requests_for_comment/pywikibot_2.0_packaging
We need syntax validation of these JSON files for gerrit submissions, as message changes typically need to be approved quickly (and without errors) so they can be merged and the core & compat i18n submodule updated, before the messages can be used in code changes.
TASK DETAIL
https://phabricator.wikimedia.org/T85335
REPLY HANDLER ACTIONS
Reply to comment or attach files, or !close, !claim, !unsubscribe or !assign <username>.
EMAIL PREFERENCES
https://phabricator.wikimedia.org/settings/panel/emailpreferences/
To: jayvdb
Cc: Aklapper, valhallasw, siebrand, Nemo_bis, jayvdb, Gryllida, Shizhao, Arrbee, pywikipedia-bugs
jayvdb created this task.
jayvdb claimed this task.
jayvdb added subscribers: jayvdb, Legoktm, hashar.
jayvdb added projects: Continuous-Integration, pywikibot-core.
Restricted Application added subscribers: Aklapper, pywikipedia-bugs.
TASK DESCRIPTION
As jenkins will no longer automatically run rules in tox.ini, we need to hardwire pep8 and pep257 into jenkins, like the mediawiki lint tests.
pep8 runs without arguments, using the config in tox.ini for the exclude and ignore list.
pep257 doesn't have an exclude parameter (yet), but looks like they are interested in that feature:
https://github.com/GreenSteam/pep257/pull/22#issuecomment-70471875
While waiting for feedback on adding exclude to pep257, it would be nice if we didnt need an exclude list .. ;-)
date.py is being ignored by flake8, but is processed by pep257, so I've tackled that.
https://gerrit.wikimedia.org/r/#/c/185815/
pep257 is validating ez_setup.py , which I've tried to fix at the source of the problem:
https://bitbucket.org/pypa/setuptools/pull-request/117/pep8-and-pep257-comp…
pep257 doesnt ignore items with '# noqa' , so it is emitting errors for interwiki.py (easy) and pywikibot/exceptions.py (messy).
Also if the job is hardwired into jenkins, and we can only maintain one match list in tox.ini , we cant reimplement the 'mandatory docstring' list without some serious cleanup first.
TASK DETAIL
https://phabricator.wikimedia.org/T87169
REPLY HANDLER ACTIONS
Reply to comment or attach files, or !close, !claim, !unsubscribe or !assign <username>.
EMAIL PREFERENCES
https://phabricator.wikimedia.org/settings/panel/emailpreferences/
To: jayvdb
Cc: hashar, Legoktm, jayvdb, Aklapper, greg, pywikipedia-bugs
XZise created this task.
XZise added a subscriber: XZise.
XZise added a project: pywikibot-core.
TASK DESCRIPTION
Similar to T74847 the wikibase settings are hardcoded into the family files. There is [[https://en.wikipedia.org/w/api.php?action=query&meta=wikibase|`action=query…]] which would allow to make dynamically if the API plays along. In T74847 a very nasty problem surfaced which made it at least very hard to determine the Site object parameter (basically which family and code it uses) based on the result (see T85153).
So part of this task is also to determine if this API call has the same problem so other wikibase installations in the wild should be queried (or more exact: installations which use another wikibase). At least from the result of the English Wikipedia it should be easy to get a Site object because it's possible to construct the URL similar to the interwiki map URL for which we already have an implementation.
TASK DETAIL
https://phabricator.wikimedia.org/T85331
REPLY HANDLER ACTIONS
Reply to comment or attach files, or !close, !claim, !unsubscribe or !assign <username>.
EMAIL PREFERENCES
https://phabricator.wikimedia.org/settings/panel/emailpreferences/
To: XZise
Cc: Aklapper, XZise, jayvdb, pywikipedia-bugs
XZise created this task.
XZise added subscribers: XZise, jayvdb.
XZise added projects: pywikibot-core, Pywikibot-compat-to-core.
Restricted Application added subscribers: Aklapper, pywikipedia-bugs.
TASK DESCRIPTION
Hi, the code should use `unicode_literals` throughout to avoid using `u'…'` which is one the reasons why Python 3 versions before 3.3 aren't supported. @jayvdb said somewhere (determine), that he'd prefer to do that in one go. I suggest to atleast do it in all new code, so that changing `u'…'` into `'…'` won't break git blame for those files when it's introduced later. It is possible to use `u'…'` on conjunction with `unicode_literals`, but that could confuse newcomers into thinking that `u'…'` is necessary.
TASK DETAIL
https://phabricator.wikimedia.org/T89589
REPLY HANDLER ACTIONS
Reply to comment or attach files, or !close, !claim, !unsubscribe or !assign <username>.
EMAIL PREFERENCES
https://phabricator.wikimedia.org/settings/panel/emailpreferences/
To: XZise
Cc: jayvdb, Aklapper, XZise, droid, pywikipedia-bugs
jayvdb created this task.
jayvdb claimed this task.
jayvdb added a subscriber: jayvdb.
jayvdb added a project: Pywikibot-Other-scripts.
Restricted Application added subscribers: Aklapper, pywikipedia-bugs.
TASK DESCRIPTION
This script describes itself as
"Script to copy images to Wikimedia Commons, or to another wiki."
However, it will only transfer files to Wikimedia Commons in compat, or the shared file repo in core.
In compat, it doesnt transfer files when fileIsOnCommons() is true.
In core, it doesnt transfer files when fileIsShared is true.
The 'fileIsonCommons' check was added in February 2007
https://mediawiki.org/wiki/Special:Code/pywikipedia/3286
I am unable to see why that was necessary, as it did already have a check whether the page is on the target site, and it would sys.exit if it was on the target site!
http://git.wikimedia.org/blob/pywikibot%2Fcompat.git/261583094f7912cc7bab81…
TASK DETAIL
https://phabricator.wikimedia.org/T87082
REPLY HANDLER ACTIONS
Reply to comment or attach files, or !close, !claim, !unsubscribe or !assign <username>.
EMAIL PREFERENCES
https://phabricator.wikimedia.org/settings/panel/emailpreferences/
To: jayvdb
Cc: pywikipedia-bugs, jayvdb, Aklapper
XZise created this task.
XZise added a subscriber: XZise.
XZise added projects: pywikibot-core, Pywikibot-tests.
Restricted Application added subscribers: Aklapper, pywikipedia-bugs.
TASK DESCRIPTION
Hi, I ran a test on Travis and [[https://travis-ci.org/xZise/pywikibot-core/jobs/51820643|one 2.6 test]] failed and it looks like that one dependency (or more like the dependency of one depencendcy) is using a package which is not compatible with Python 2.6 because it's using sets via curly brackets (`s = {42, 74, 47}`).
Unfortunately that package is not a direct dependency but it looks like that is just compatible with Python 2.7+.
TASK DETAIL
https://phabricator.wikimedia.org/T90441
REPLY HANDLER ACTIONS
Reply to comment or attach files, or !close, !claim, !unsubscribe or !assign <username>.
EMAIL PREFERENCES
https://phabricator.wikimedia.org/settings/panel/emailpreferences/
To: XZise
Cc: pywikipedia-bugs, Aklapper, XZise, jayvdb
jayvdb created this task.
jayvdb added a subscriber: jayvdb.
jayvdb added a project: pywikibot-core.
TASK DESCRIPTION
There is an open pull request for a py3 problem, but our unit tests must not hit this section of code, or something
https://github.com/invisibleroads/socketIO-client/pull/60
However our unit tests fail on py 2.6
https://travis-ci.org/wikimedia/pywikibot-core/jobs/45687549
```
test_RC_pagegenerator_result (tests.pagegenerators_tests.LiveRCPageGeneratorTestCase) ... /home/travis/build/wikimedia/pywikibot-core/websocket_client-0.23.0-py2.6.egg/websocket/_core.py:223: DeprecationWarning: BaseException.message has been deprecated as of Python 2.6
message = getattr(exception, 'message', '')
No handlers could be found for logger "socketIO_client"
No output has been received in the last 10 minutes, this potentially indicates a stalled build or something wrong with the build itself.
The build has been terminated
```
TASK DETAIL
https://phabricator.wikimedia.org/T85716
REPLY HANDLER ACTIONS
Reply to comment or attach files, or !close, !claim, !unsubscribe or !assign <username>.
EMAIL PREFERENCES
https://phabricator.wikimedia.org/settings/panel/emailpreferences/
To: jayvdb
Cc: Aklapper, jayvdb, pywikipedia-bugs
jayvdb created this task.
jayvdb assigned this task to valhallasw.
jayvdb added a subscriber: jayvdb.
jayvdb added projects: pywikibot-core, Pywikibot-pagegenerators.
TASK DESCRIPTION
In the event of a problem in socketio_client, the log messages are discarded, with the logging layer emitting one notice to indicate that this has happened:
```
No handlers could be found for logger "socketIO_client"
```
When a logger was added for T85717 , we see that the problem was a critical disconnection, with an informative message:
```
WARNING:socketIO_client:[connection error] connection closed ()
```
I'm not sure that adding a socketIO_client" logger is the right thing to do. Ideally, pywikibot is catching exceptions and these messages are logged using the "pywiki-rcstream" logger. But we need logging soon, if we're to diagnose problems that occur.
As pywikibot is not currently catching this exception (which I believe is coming from websocket), we need to investigate that. Communication of important connection status changes might be happening via other mechanisms, like the generic handlers or by callbacks.
TASK DETAIL
https://phabricator.wikimedia.org/T86506
REPLY HANDLER ACTIONS
Reply to comment or attach files, or !close, !claim, !unsubscribe or !assign <username>.
EMAIL PREFERENCES
https://phabricator.wikimedia.org/settings/panel/emailpreferences/
To: valhallasw, jayvdb
Cc: Aklapper, jayvdb, pywikipedia-bugs