Last modified: 2012-08-06 19:25:37 UTC
Created attachment 10300 [details] patch Totally trivial static function to abstract out __parseArgs[2] and explain. It's possible that we want the flush() functionality to work differently in the future.
Thanks for the patch! Added keywords to ask for code review.
By the way, Adam, you can get a Gerrit account if you want and suggest changes to be merged directly into our Git repository: https://www.mediawiki.org/wiki/Project:Labsconsole_account & https://www.mediawiki.org/wiki/Git/Workflow#Easier_submission_of_code have more information.
What's the status of this bug? And does it actually relate to the VisualEditor extension?
Mass-moving items into VisualEditor product
Gabriel - this still current?
Just noticed that this was assigned to my old (now reactivated with wikimedia address) bugzilla account. It is still current, and still relates to VE since Parsoid is currently treated as a submodule of VE in Bugzilla. We should perhaps create a separate product for it so that we can set up our own set of default assignees etc.
Fixed in commit 1736e52bfb6c6b459b571cd8a8f26c7ee6d84da7. Thanks for your patch, Adam!
Mass-moving bugs into the new 'Parsoid' product.