There only are a dozen of those ( http://tinyurl.com/zyjlyuh ), it would be a waste to write software to deal with that ;-) I'll fix them manually if you don't beat me to it ;-)

@Metamorforme42 this is a tiny part of what would be necessary to help but ... it's a start. Let me know if you agree on this first step or if you think something else should be done first. I should be able to implement that today.

Thanks for a *very* quick resolution :-) IIRC it will be active on wikidata in a week or two.

@Mbch331 right :-) I updated the examples to show the cvs:// form (which is s|^:pserver:|cvs://pserver:| and removing the : after the domain name)

@Lydia_Pintscher I added you to the Subscribers because you participated in https://phabricator.wikimedia.org/T140986. If that is not a good practice, please let me know.

Thanks, I will enjoy it :-)

Project description:

@hashar I'm glad I did not miss anything obvious :-) I'll wait to read @jayvdb advice.

@Aklapper thanks for the link, I'll read it (at least part of it ;-).

start the bot and populate wikidata! :]

@hashar thanks for the help, it is much appreciated :-) From my point of view this task is resolved. Do you see anything else requiring our attention ?

@hashar thanks for your review. Indeed the additional packages are not immediately needed since I'm still to create the tests that need them. I'm asking because they need to be there before the test cases can be submitted for review. The general idea is that the tests should be integration tests when possible to reduce the maintenance burden of a mock.

those are dependencies of your program directly, not of tox/pip right?

@Legoktm done as you suggested at https://gerrit.wikimedia.org/r/#/c/308944/ (thanks a lot for the tip ;-). Is there anything missing here ? For next time I'd be interested in the wikipage I should have read to know that.

was missing

Maybe pywikibot.ItemPage.mergeInto should also have an explicit bot=True argument so that the default is to set the bot flag ?

The mergeInto allows bot=True since it pass all kwargs to mergeItems regardless.

def mergeInto(self, item, **kwargs):
    """
    Merge the item into another item.

What would be the process to ask that the 7za package is installed on the jenkins slaves ?

@valhallasw would you be so kind as to explain the steps to reproduce the problem ? From what https://phabricator.wikimedia.org/diffusion/PWBC/browse/master/pywikibot/page.py reads, it is unclear when self.text can become something that's not a string. Thanks :-)

Proposed patch https://gerrit.wikimedia.org/r/#/c/301664/

https://phabricator.wikimedia.org/diffusion/EWBA/browse/master/repo/includes/ChangeOp/ChangeOpsMerge.php indeed has

public static $conflictTypes = array( 'description', 'sitelink', 'statement' );

@javydb thanks for your review :-) I proposed a simple, backward compatible patch. What do you think ? https://gerrit.wikimedia.org/r/#/c/301094/

Apparently something triggers the tests but I'm not sure why or when.

@aude is there anything else to be done for this patch to benefit wikidata.org ? Or should we just wait ?

I propose the following patch to Gadget-DuplicateReferences.js to handle statements in edit mode differently. If the statement is in edit mode (i.e. the wb-edit class is present), the reference is added but not saved and the statement remains in edit mode.

I tried to reproduce the problem in french but could not. Here is the screenshot that shows it's not case sensitive. Did I miss something ?