Catrope has uploaded a new change for review.
https://gerrit.wikimedia.org/r/77362
Change subject: Fix notice caused by not passing the WebRequest object by
reference
......................................................................
Fix notice caused by not passing the WebRequest object by reference
It makes no sense that we should have to pass this by reference but
if we don't do it, we get a PHP notice.
Bug: 52466
Change-Id: Ibc8e33c4776a325cd32b6526dacc200994a73f7f
---
M ApiVisualEditor.php
1 file changed, 2 insertions(+), 1 deletion(-)
git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/VisualEditor
refs/changes/62/77362/1
diff --git a/ApiVisualEditor.php b/ApiVisualEditor.php
index 9a4e4ce..c6ba537 100644
--- a/ApiVisualEditor.php
+++ b/ApiVisualEditor.php
@@ -252,7 +252,8 @@
// HACK: Build a fake EditPage so we can get
checkboxes from it
$article = new Article( $page ); //
Deliberately omitting ,0 so oldid comes from request
$ep = new EditPage( $article );
- $ep->importFormData( $this->getRequest() );
+ $req = $this->getRequest();
+ $ep->importFormData( $req ); // By reference
for some reason (bug 52466)
$tabindex = 0;
$states = array( 'minor' => false, 'watch' =>
false );
$checkboxes = $ep->getCheckboxes( $tabindex,
$states );
--
To view, visit https://gerrit.wikimedia.org/r/77362
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ibc8e33c4776a325cd32b6526dacc200994a73f7f
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/VisualEditor
Gerrit-Branch: master
Gerrit-Owner: Catrope <[email protected]>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
