Page MenuHomePhabricator

Add serialize/deserialize methods to ve.dm.Transaction and ve.dm.IndexValueStore
Closed, ResolvedPublic1 Story Points

Description

Code in ve.dm.Change can already serialize them, but they should handle their own serialization. This would make code cleaner and also help with ad-hoc testing.

For ve.dm.IndexValueStore, a value serializer/deserializer is needed in order to accomplish this.

Event Timeline

dchan created this task.Mar 28 2017, 3:12 PM
Restricted Application added a subscriber: Aklapper. · View Herald TranscriptMar 28 2017, 3:12 PM

Change 344812 had a related patch set uploaded (by Divec):
[VisualEditor/VisualEditor@master] Serialize/deserialize for Transaction and IndexValueStore

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

Jdforrester-WMF triaged this task as Normal priority.
Jdforrester-WMF set the point value for this task to 1.

Change 344812 merged by jenkins-bot:
[VisualEditor/VisualEditor@master] Serialize/deserialize for Transaction and IndexValueStore

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

Change 345794 had a related patch set uploaded (by Jforrester):
[mediawiki/extensions/VisualEditor@master] Update VE core submodule to master (a6c25d08e)

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

Change 345794 merged by jenkins-bot:
[mediawiki/extensions/VisualEditor@master] Update VE core submodule to master (a6c25d08e)

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

Jdforrester-WMF closed this task as Resolved.Apr 1 2017, 3:18 PM
Jdforrester-WMF edited projects, added Technical-Debt; removed Patch-For-Review.
Restricted Application added a project: User-Ryasmeen. · View Herald TranscriptApr 1 2017, 3:18 PM