Last modified: 2013-11-05 16:06:17 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T50570, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 48570 - Spaces added before colons and links in some cases
Spaces added before colons and links in some cases
Status: RESOLVED FIXED
Product: Parsoid
Classification: Unclassified
General (Other open bugs)
unspecified
All All
: High major
: ---
Assigned To: Gabriel Wicke
:
: 50637 51024 (view as bug list)
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2013-05-17 07:43 UTC by Ltrlg
Modified: 2013-11-05 16:06 UTC (History)
10 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description Ltrlg 2013-05-17 07:43:19 UTC
See https://fr.wikipedia.org/w/index.php?title=COE&diff=prev&oldid=93166267 for example.

I only removed the link, but VisualEditor added a space before the colons.

Another example: https://fr.wikipedia.org/w/index.php?title=MSS&diff=prev&oldid=93166411

There was already two spaces, but VisualEditor added a third.
Comment 1 Elitre 2013-06-23 10:33:54 UTC
The same happens when you edit the first word of a sequence "word + word + wikilink", it will add an unnecessary space before the link. Examples,
http://it.wikipedia.org/w/index.php?title=Amazon.com&diff=prev&oldid=59604951 - 
http://it.wikipedia.org/w/index.php?title=Google&diff=59622535&oldid=59377529 (<<talmente popolare ch in  [[lingua inglese|inglese]] >>).
This last one is a deliberate vandalism from me to check the issue, I later rollbacked.
Comment 2 James Forrester 2013-06-23 19:59:15 UTC
(In reply to comment #0)
> See https://fr.wikipedia.org/w/index.php?title=COE&diff=prev&oldid=93166267
> for example.
> 
> I only removed the link, but VisualEditor added a space before the colons.
> 
> Another example:
> https://fr.wikipedia.org/w/index.php?title=MSS&diff=prev&oldid=93166411
> 
> There was already two spaces, but VisualEditor added a third.

These have been now fixed - sorry for the difficulty.

(In reply to comment #1)
> The same happens when you edit the first word of a sequence "word + word +
> wikilink", it will add an unnecessary space before the link. Examples,
> http://it.wikipedia.org/w/index.php?title=Amazon.
> com&diff=prev&oldid=59604951 - 
> http://it.wikipedia.org/w/index.php?title=Google&diff=59622535&oldid=59377529
> (<<talmente popolare ch in  [[lingua inglese|inglese]] >>).
> This last one is a deliberate vandalism from me to check the issue, I later
> rollbacked.

I *think* this is also fixed but I'm not absolutely sure; please re-open if you can still reproduce this.
Comment 3 Elitre 2013-08-02 11:11:22 UTC
We unfortunately can:
http://en.wikipedia.org/w/index.php?title=Little_Mosque_on_the_Prairie&diff=prev&oldid=566820635

It seems to be triggered by removal of '.
Another user verified this (not actually saving, only preview mode) in http://en.wikipedia.org/wiki/Indian_Bank , removing the ' after "Binny&Co" would cause a whitespace to be added right before the next wikilink.
Comment 4 Chris McKenna 2013-08-02 11:23:39 UTC
It's not just ' either, also seen when removing - and " in my sandbox https://en.wikipedia.org/w/index.php?title=User%3AThryduulf%2Fsandbox&diff=566841354&oldid=566840716

spaces were added before internal and external links, except in the first instance where a space was added before a colon instead.
Comment 5 Chris McKenna 2013-08-02 11:26:11 UTC
Could the fix to bug 2035 be related to this?
Comment 6 Chris McKenna 2013-08-02 11:26:46 UTC
Ignore last comment, should be: Could the fix to bug 52035 be related to this?
Comment 7 Ed Sanders 2013-08-19 16:51:15 UTC
This looks like 1. a Parsoid issue, 2. fixed in master. Reassigning to Parsoid to confirm.
Comment 9 Ltrlg 2013-09-17 20:09:57 UTC
See also bug 51024
Comment 10 Chris McKenna 2013-09-17 22:01:06 UTC
*** Bug 51024 has been marked as a duplicate of this bug. ***
Comment 11 Chris McKenna 2013-09-17 22:02:30 UTC
Bug 51024 has several more examples of this bug occurring if anyone is looking for them.
Comment 12 ssastry 2013-09-17 23:42:48 UTC
Confirmed on simple test case. Investigating.
---------------------------------------

[subbu@earth tests] echo "foo : bar" > /tmp/wt
[subbu@earth tests] node parse < /tmp/wt > /tmp/old.html
[subbu@earth tests] cat /tmp/old.html | sed 's/bar/bars/g;' > /tmp/new.html
[subbu@earth tests] node parse --html2wt --selser --oldtextfile /tmp/wt --oldhtmlfile /tmp/old.html < /tmp/new.html
foo  : bars
Comment 13 Gerrit Notification Bot 2013-09-18 01:04:12 UTC
Change 84701 had a related patch set uploaded by Subramanya Sastry:
(Bug 48570) Fix subtle selser bug handling separator-only nodes

https://gerrit.wikimedia.org/r/84701
Comment 14 Roan Kattouw 2013-09-19 18:02:26 UTC
*** Bug 50637 has been marked as a duplicate of this bug. ***
Comment 15 Gerrit Notification Bot 2013-09-20 16:56:05 UTC
Change 84701 merged by jenkins-bot:
(Bug 48570) Fix subtle selser bug handling separator-only nodes

https://gerrit.wikimedia.org/r/84701

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links