jenkins-bot has submitted this change and it was merged.
Change subject: Fix array index for oldid parameter
......................................................................
Fix array index for oldid parameter
This caused an incorrect oldid parameter to be passed to parsoid on html2wt
conversion, resulting in dirty diffs.
Change-Id: If140bbc94f280f9ec5a1c9f6d9c3b4e7a46e1917
---
M includes/libs/virtualrest/ParsoidVirtualRESTService.php
1 file changed, 2 insertions(+), 2 deletions(-)
Approvals:
Alex Monk: Looks good to me, approved
jenkins-bot: Verified
diff --git a/includes/libs/virtualrest/ParsoidVirtualRESTService.php
b/includes/libs/virtualrest/ParsoidVirtualRESTService.php
index 34e43f9..03bdf0d 100644
--- a/includes/libs/virtualrest/ParsoidVirtualRESTService.php
+++ b/includes/libs/virtualrest/ParsoidVirtualRESTService.php
@@ -88,8 +88,8 @@
if ( !isset( $req['body']['html'] ) ) {
throw new Exception( "You must
set an 'html' body key for this request" );
}
- if ( isset( $parts[6] ) ) {
- $req['body']['oldid'] =
$parts[6];
+ if ( isset( $parts[7] ) ) {
+ $req['body']['oldid'] =
$parts[7];
}
} elseif ( $parts[3] == 'wikitext' && $parts[5]
== 'html' ) {
if ( !isset( $req['body']['wikitext'] )
) {
--
To view, visit https://gerrit.wikimedia.org/r/185067
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: If140bbc94f280f9ec5a1c9f6d9c3b4e7a46e1917
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/core
Gerrit-Branch: master
Gerrit-Owner: GWicke <[email protected]>
Gerrit-Reviewer: Aaron Schulz <[email protected]>
Gerrit-Reviewer: Alex Monk <[email protected]>
Gerrit-Reviewer: Catrope <[email protected]>
Gerrit-Reviewer: Esanders <[email protected]>
Gerrit-Reviewer: Jforrester <[email protected]>
Gerrit-Reviewer: Subramanya Sastry <[email protected]>
Gerrit-Reviewer: jenkins-bot <>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits