Maniphest T110692

Roundtripping diffs with batching enabled
Closed, ResolvedPublic
Actions

Assigned To

Authored By

	ssastry
	Aug 28 2015, 4:02 PM

Tags

Referenced Files

None

Subscribers

Description

Without batching:

[subbu@earth api] echo '[[Archivo:Rainstick.ogg|center|323x323px]]' | node parse --useBatchAPI false --wt2wt --prefix eswiki
[[Archivo:Rainstick.ogg|center|323x323px]]

With batching:

[subbu@earth api] echo '[[Archivo:Rainstick.ogg|center|323x323px]]' | node parse --useBatchAPI true --wt2wt --prefix eswiki
[[Archivo:Rainstick.ogg|center]]

The parse output is obviously different in the two cases. Needs investigation and fixing.
Found in RT-testing. There a few other similar roundtripping diffs http://parsoid-tests.wikimedia.org/regressions/between/4ded41e0ff3d2c12f82b4a95adf8a6d1807cf427/15cbff66ee349997a89aeb266a63a0665a915f35

Related Objects

Mentioned In: T111089: Parsoid Batching API: Imageinfo returns strings for dimensions instead of numbers which the mediawiki API does
rGPAR4072a42387c4: T110692: Massage batching API imageinfo width/height to numbers

Event Timeline

ssastry created this task.Aug 28 2015, 4:02 PM

ssastry raised the priority of this task from to High.

ssastry updated the task description. (Show Details)

ssastry added a project: Parsoid.

ssastry added subscribers: ssastry, tstarling.

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptAug 28 2015, 4:02 PM

ssastry updated the task description. (Show Details)Aug 28 2015, 9:32 PM

ssastry set Security to None.

Change 234714 had a related patch set uploaded (by Subramanya Sastry):
T110692: Massage batching API imageinfo width/height to numbers

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

gerritbot added a project: Patch-For-Review.Aug 29 2015, 4:00 AM

Change 234714 merged by jenkins-bot:
T110692: Massage batching API imageinfo width/height to numbers

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

tstarling mentioned this in rGPAR4072a42387c4: T110692: Massage batching API imageinfo width/height to numbers.Aug 31 2015, 10:37 PM

ssastry closed this task as Resolved.Sep 1 2015, 6:40 PM

ssastry claimed this task.

ssastry removed a project: Patch-For-Review.

ssastry removed a subscriber: gerritbot.

ssastry mentioned this in T111089: Parsoid Batching API: Imageinfo returns strings for dimensions instead of numbers which the mediawiki API does.Sep 1 2015, 6:44 PM