Stigmj added a comment.
I will admit, after thinking a bit more, I'll concede it's not necessarily
wrong behaviour, but the code doesn't cover all cases anyway. These may be more
of a "cosmetic" bug character:
Running with master:
$ git checkout master
Already on 'master'
Your branch is up-to-date with 'origin/master'.
$ python pwb.py fixing_redirects.py -simulate -page:"Bruker:Stigmj/Sandkasse"
Retrieving 1 pages from wikipedia:no.
>>> Bruker:Stigmj/Sandkasse <<<
WARNING: Source section #Se_også and target section [[no:Alpin
kombinasjon#Superkombinasjon]] found. Skipping.
WARNING: Source section #Se_også and target section [[no:Alpin
kombinasjon#Superkombinasjon]] found. Skipping.
WARNING: Source section #Se_også and target section [[no:Alpin
kombinasjon#Superkombinasjon]] found. Skipping.
WARNING: Source section #Se_også and target section [[no:Alpin
kombinasjon#Superkombinasjon]] found. Skipping.
@@ -1,14 +1,14 @@
- * [[Evgeni Plushenko]]
+ * [[Jevgenij Pljusjtsjenko|Evgeni Plushenko]]
- * [[Evgeni Plushenko|test]]
+ * [[jevgenij Pljusjtsjenko|test]]
- * [[Evgeni Plushenko|Jevgenij Pljusjtsjenko]]
+ * [[Jevgenij Pljusjtsjenko]]
- * [[Evgeni Plushenko|Jevgenij Pljusjtsjenkovitsj]]
+ * [[Jevgenij Pljusjtsjenko]]vitsj
- * [[Evgeni Plushenko|jevgenij Pljusjtsjenko]]
+ * [[jevgenij Pljusjtsjenko]]
- * [[Evgeni Plushenko|jevgenij Pljusjtsjenkovitsj]]
+ * [[jevgenij Pljusjtsjenko]]vitsj
- * [[Evgeni Plushenko]]:Jau
- * [[Evgeni Plushenko]]:hau
+ * [[Jevgenij Pljusjtsjenko|Evgeni Plushenko]]:Jau
+ * [[Jevgenij Pljusjtsjenko|Evgeni Plushenko]]:hau
- * [[Evgeni Plushenko|:breakit]]
+ * [[jevgenij Pljusjtsjenko|breakit]]
- * [[superkombinasjon]]
- * [[Superkombinasjon]]
+ * [[alpin kombinasjon#Superkombinasjon|superkombinasjon]]
+ * [[Alpin kombinasjon#Superkombinasjon|Superkombinasjon]]
- * [[superkombinasjon|kombinasjon]]
+ * [[alpin kombinasjon#Superkombinasjon|kombinasjon]]
- * [[Superkombinasjon]]test
+ * [[Alpin kombinasjon#Superkombinasjon|Superkombinasjontest]]
- * [[superkombinasjon|kombinasjon]]stest
+ * [[alpin kombinasjon#Superkombinasjon|kombinasjonstest]]
@@ -21 +21 @@
- * [[ishockey-VM 2011 (elitedivisjonen)|ishockey-VM 2011]]
+ * [[ishockey-VM 2011 (toppdivisjonen)|ishockey-VM 2011]]
Now running with old version:
$ git checkout 8a7c42f5~1
Note: checking out '8a7c42f5~1'.
You are in 'detached HEAD' state. You can look around, make experimental
changes and commit them, and you can discard any commits you make in this
state without impacting any branches by performing another checkout.
If you want to create a new branch to retain commits you create, you may
do so (now or later) by using -b with the checkout command again. Example:
git checkout -b new_branch_name
HEAD is now at e56b4be... Merge "flake8-docstrings 0.2.2-0.2.4 fail"
$ python pwb.py fixing_redirects.py -simulate -page:"Bruker:Stigmj/Sandkasse"
Retrieving 1 pages from wikipedia:no.
>>> Bruker:Stigmj/Sandkasse <<<
@@ -1,18 +1,18 @@
- * [[Evgeni Plushenko]]
+ * [[Jevgenij Pljusjtsjenko|Evgeni Plushenko]]
- * [[Evgeni Plushenko|test]]
+ * [[Jevgenij Pljusjtsjenko|test]]
- * [[Evgeni Plushenko|Jevgenij Pljusjtsjenko]]
+ * [[Jevgenij Pljusjtsjenko]]
- * [[Evgeni Plushenko|Jevgenij Pljusjtsjenkovitsj]]
+ * [[Jevgenij Pljusjtsjenko]]vitsj
- * [[Evgeni Plushenko|jevgenij Pljusjtsjenko]]
+ * [[jevgenij Pljusjtsjenko]]
- * [[Evgeni Plushenko|jevgenij Pljusjtsjenkovitsj]]
+ * [[jevgenij Pljusjtsjenko]]vitsj
- * [[Evgeni Plushenko]]:Jau
- * [[Evgeni Plushenko]]:hau
+ * [[Jevgenij Pljusjtsjenko|Evgeni Plushenko]]:Jau
+ * [[Jevgenij Pljusjtsjenko|Evgeni Plushenko]]:hau
- * [[Evgeni Plushenko|:breakit]]
+ * [[Jevgenij Pljusjtsjenko|:breakit]]
- * [[superkombinasjon]]
- * [[Superkombinasjon]]
+ * [[Alpin kombinasjon|superkombinasjon]]
+ * [[Alpin kombinasjon|Superkombinasjon]]
- * [[superkombinasjon|kombinasjon]]
+ * [[Alpin kombinasjon|kombinasjon]]
- * [[Superkombinasjon]]test
+ * [[Alpin kombinasjon|Superkombinasjontest]]
- * [[superkombinasjon|kombinasjon]]stest
+ * [[Alpin kombinasjon|kombinasjonstest]]
- * [[Superkombinasjon#Se_også]]
+ * [[Alpin kombinasjon#Se_også|Superkombinasjon]]
- * [[Superkombinasjon#Se_også|testing]]
+ * [[Alpin kombinasjon#Se_også|testing]]
- * [[Superkombinasjon#Se_også|Testing]]
+ * [[Alpin kombinasjon#Se_også|Testing]]
- * [[Superkombinasjon#Se_også|testing]]inging
+ * [[Alpin kombinasjon#Se_også|testinginging]]
@@ -21 +21 @@
- * [[ishockey-VM 2011 (elitedivisjonen)|ishockey-VM 2011]]
+ * [[Ishockey-VM 2011 (toppdivisjonen)|ishockey-VM 2011]]
The discrepancies are shown best here:
$ master
@@ -21 +21 @@
- * [[ishockey-VM 2011 (elitedivisjonen)|ishockey-VM 2011]]
+ * [[ishockey-VM 2011 (toppdivisjonen)|ishockey-VM 2011]]
$ 8a7c42f5~1
@@ -21 +21 @@
- * [[ishockey-VM 2011 (elitedivisjonen)|ishockey-VM 2011]]
+ * [[Ishockey-VM 2011 (toppdivisjonen)|ishockey-VM 2011]]
I can't see any rationale for having the first letter of the targetPage be
lower case as long as you have a linktext there as well. I may be wrong,
though..
The other strange behaviour is here:
$ master
- * [[Evgeni Plushenko|:breakit]]
+ * [[jevgenij Pljusjtsjenko|breakit]]
$ 8a7c42f5~1
- * [[Evgeni Plushenko|:breakit]]
+ * [[Jevgenij Pljusjtsjenko|:breakit]]
In the example above, the new version strips the ":" from the linktext as well.
Why?
TASK DETAIL
https://phabricator.wikimedia.org/T123892
EMAIL PREFERENCES
https://phabricator.wikimedia.org/settings/panel/emailpreferences/
To: Stigmj
Cc: jayvdb, XZise, Xqt, gerritbot, Aklapper, StudiesWorld, Stigmj,
pywikibot-bugs-list
_______________________________________________
pywikibot-bugs mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/pywikibot-bugs
