[Bug 50756] Removing some whitespace is not detected as a change in some circumstances (selser issue?)

2014-06-03 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=50756

Arlo Breault  changed:

   What|Removed |Added

 Status|ASSIGNED|RESOLVED
 CC||abrea...@wikimedia.org
 Resolution|--- |FIXED

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 50756] Removing some whitespace is not detected as a change in some circumstances (selser issue?)

2013-12-19 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=50756

--- Comment #7 from ssas...@wikimedia.org ---
This proved to be a great bug to fix a whole bunch of related things in
Parsoid. It led to about 10-15 patches indirectly.

The final patch that fixes the bug (with full history) is:
https://gerrit.wikimedia.org/r/#/c/97746/ (patch before Parsoid repo split)
https://gerrit.wikimedia.org/r/#/c/102012/ (latest version after migration into
new repo)

This should get merged soon, but the code will not get deployed until January
since we are now entering holiday code freeze time now.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 50756] Removing some whitespace is not detected as a change in some circumstances (selser issue?)

2013-11-25 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=50756

--- Comment #6 from ssas...@wikimedia.org ---
After some more digging and playing around in VE, I was able to reproduce it
with the following snippet: "'''foo''' bar\n\n"

With this example, I can indeed reproduce this on the command line.
Investigating.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 50756] Removing some whitespace is not detected as a change in some circumstances (selser issue?)

2013-11-25 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=50756

--- Comment #5 from Joe Decker  ---
As the person who pointed Oliver at this example originally, I went back to the
version I'd had trouble with, and verified that the problem does not seem to be
occurring now.  You can see my test attempt with this diff:

https://en.wikipedia.org/w/index.php?title=Michael_Lowry_%28actor%29&diff=583277441&oldid=542893928

In summary: I haven't done any wider testing, but at least the example that
caused me to report this appears resolved.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 50756] Removing some whitespace is not detected as a change in some circumstances (selser issue?)

2013-11-25 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=50756

--- Comment #4 from ssas...@wikimedia.org ---
This seems to be working on latest master (see below). I'll test this against
VE in my local mediawiki after installing the cite extension and report back
later today.

--
[subbu@earth lib] echo "foo bar\n\n" > /tmp/wt
[subbu@earth lib] node parse --fetchConfig false --extensions ref,references <
/tmp/wt > /tmp/old.html
[subbu@earth lib] sed 's/foo  /tmp/new.html
[subbu@earth lib] node parse --selser --oldhtmlfile /tmp/old.html --oldtextfile
/tmp/wt < /tmp/new.html
foobar



-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 50756] Removing some whitespace is not detected as a change in some circumstances (selser issue?)

2013-11-25 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=50756

James Forrester  changed:

   What|Removed |Added

   Priority|Unprioritized   |High
 Status|NEW |ASSIGNED
 CC||ssas...@wikimedia.org
  Component|General |serializer
   Assignee|jforres...@wikimedia.org|gwi...@wikimedia.org
Product|VisualEditor|Parsoid
Summary|VisualEditor: removing  |Removing some whitespace is
   |spaces not registering as a |not detected as a change in
   |change - or /working/   |some circumstances (selser
   ||issue?)

--- Comment #3 from James Forrester  ---
Can confirm that the HTML going from VE to Parsoid doesn't include the
whitespace - it's being re-inserted by Parsoid for some reason, possibly due to
a selser bug?

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l