This feature is in the parse API, allowing you to perform a pre-save transform on the text before parsing it.
This is useful for things such as AJAX diff previews, where currently you have to do a normal page-reloading preview if you want to see PST things such as subst templates and signatures, or make two API requests.
Description
Description
Details
Details
- Reference
- bz51155
| Subject | Repo | Branch | Lines +/- | |
|---|---|---|---|---|
| Add pst option to API difftotext | mediawiki/core | master | +16 -2 |
| Status | Subtype | Assigned | Task | ||
|---|---|---|---|---|---|
| Resolved | Majr | T116178 Use the new 'difftotextpst' option of prop=revisions API in live preview script | |||
| Resolved | Majr | T53155 Add option to ApiQueryRevisions to do the pre-save transform on rvdifftotext before diffing |
Event Timeline
Comment Actions
As stated, this request doesn't make any sense. The content being parsed by rvparse is what is already saved in the database, so it has already had the pre-save transform done on it before it was saved (hence the name "pre-save transform"). There's no reason to do the PST on it again.
But it seems like what you really want is an option to perform the PST on the text passed to rvdifftotext before doing the diff, which could make a sort of sense. But the option shouldn't be named "rvparsepst" or "rvparseonlypst", it should be named something like "rvdifftotextpst" to associate it with what it applies to. I've taken the liberty of changing the bug title accordingly; if you disagree, feel free to revert and we'll go from there.
Actually doing the PST then should be easy enough, you just need to call $difftocontent->preSaveTransform( ... ) in between creating $difftocontent and handing it off to the difference engine. All this is currently around line 600 of ApiQueryRevisions.php.
Comment Actions
Since no one else has done this yet, I had a go at it and submitted the patch through the uploader.
Comment Actions
Change 122327 had a related patch set uploaded (by Anomie):
Add pst option to API difftotext
Comment Actions
This would also be convenient for the live preview in MediaWiki (resources/src/mediawiki.action/mediawiki.action.edit.preview.js), where we currently need two API requests to display the diff (first to PST, second to actually diff).
Comment Actions
Rebased and updated patch to add release notes and get parser options in a better way.
A review before this gets outdated and needs rebasing again would be nice.
Comment Actions
Should go out to WMF wikis with 1.27.0-wmf.4, see https://www.mediawiki.org/wiki/MediaWiki_1.27/Roadmap for the schedule.