Maniphest T116178

Use the new 'difftotextpst' option of prop=revisions API in live preview script
Closed, ResolvedPublic
Actions

Assigned To

Authored By

	matmarex
	Oct 21 2015, 2:31 PM

Tags

Referenced Files

None

Subscribers

Tokens

"Pterodactyl" token, awarded by matmarex.

Description

Use the new 'difftotextpst' option (implemented in T53155) of prop=revisions API in live preview script (resources/src/mediawiki.action/mediawiki.action.edit.preview.js). We currently need two API requests to display the diff (first to PST, second to actually diff).

Details

	Subject	Repo	Branch	Lines +/-
	Use the new 'difftotextpst' option of prop=revisions API in live preview script	mediawiki/core	master	+52 -46

Customize query in gerrit

Related Objects
Search...

		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

matmarex created this task.Oct 21 2015, 2:31 PM

matmarex raised the priority of this task from to Medium.

matmarex updated the task description. (Show Details)

matmarex added projects: MediaWiki-Page-editing, good first task.

matmarex subscribed.

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptOct 21 2015, 2:31 PM

matmarex added a subtask: T53155: Add option to ApiQueryRevisions to do the pre-save transform on rvdifftotext before diffing.Oct 21 2015, 2:31 PM

Anomie closed subtask T53155: Add option to ApiQueryRevisions to do the pre-save transform on rvdifftotext before diffing as Resolved.Oct 21 2015, 2:37 PM

Change 247849 had a related patch set uploaded (by Majr):
Use the new 'difftotextpst' option of prop=revisions API in live preview script

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

gerritbot added a project: Patch-For-Review.Oct 21 2015, 3:08 PM

Majr claimed this task.Oct 21 2015, 3:11 PM

TheDJ closed this task as Resolved.Oct 25 2015, 11:42 AM

TheDJ removed a project: Patch-For-Review.

TheDJ set Security to None.

Change 247849 merged by jenkins-bot:
Use the new 'difftotextpst' option of prop=revisions API in live preview script

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

ReleaseTaggerBot added projects: MW-1.27-release-notes, MW-1.27-release (WMF-deploy-2015-10-27_(1.27.0-wmf.4)).Oct 25 2015, 12:00 PM

matmarex awarded a token.Oct 26 2015, 12:17 PM

matmarex mentioned this in T119376: Maniphest burnup report incorrectly counts "All Time" opened and closed tasks (due to lack of transactions from Bugzilla/RT times).Nov 23 2015, 11:38 AM