[MediaWiki-commits] [Gerrit] mediawiki/core[master]: SpecialBooksources: Fix submitting the form from a subpage link
jenkins-bot has submitted this change and it was merged. Change subject: SpecialBooksources: Fix submitting the form from a subpage link .. SpecialBooksources: Fix submitting the form from a subpage link Visiting Special:Booksources/ and then trying to enter a new ISBN in the form field didn't work because the $par would take precedence over the query parameter. Removing the subpage from the HTMLForm context apparently fixes this. Change-Id: Ia007648e23beadcc017e117e59ce3df1b329dcdc --- M includes/specials/SpecialBooksources.php 1 file changed, 3 insertions(+), 1 deletion(-) Approvals: Bartosz Dziewoński: Looks good to me, approved jenkins-bot: Verified diff --git a/includes/specials/SpecialBooksources.php b/includes/specials/SpecialBooksources.php index 2fef725..72e0b88 100644 --- a/includes/specials/SpecialBooksources.php +++ b/includes/specials/SpecialBooksources.php @@ -133,7 +133,9 @@ ], ]; - HTMLForm::factory( 'ooui', $formDescriptor, $this->getContext() ) + $context = new DerivativeContext( $this->getContext() ); + $context->setTitle( $this->getPageTitle() ); + HTMLForm::factory( 'ooui', $formDescriptor, $context ) ->setWrapperLegendMsg( 'booksources-search-legend' ) ->setSubmitTextMsg( 'booksources-search' ) ->setMethod( 'get' ) -- To view, visit https://gerrit.wikimedia.org/r/316203 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia007648e23beadcc017e117e59ce3df1b329dcdc Gerrit-PatchSet: 1 Gerrit-Project: mediawiki/core Gerrit-Branch: master Gerrit-Owner: LegoktmGerrit-Reviewer: Bartosz Dziewoński Gerrit-Reviewer: Florianschmidtwelzow Gerrit-Reviewer: jenkins-bot <> ___ MediaWiki-commits mailing list MediaWiki-commits@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
[MediaWiki-commits] [Gerrit] mediawiki/core[master]: SpecialBooksources: Fix submitting the form from a subpage link
Legoktm has uploaded a new change for review. https://gerrit.wikimedia.org/r/316203 Change subject: SpecialBooksources: Fix submitting the form from a subpage link .. SpecialBooksources: Fix submitting the form from a subpage link Visiting Special:Booksources/ and then trying to enter a new ISBN in the form field didn't work because the $par would take precedence over the query parameter. Removing the subpage from the HTMLForm context apparently fixes this. Change-Id: Ia007648e23beadcc017e117e59ce3df1b329dcdc --- M includes/specials/SpecialBooksources.php 1 file changed, 3 insertions(+), 1 deletion(-) git pull ssh://gerrit.wikimedia.org:29418/mediawiki/core refs/changes/03/316203/1 diff --git a/includes/specials/SpecialBooksources.php b/includes/specials/SpecialBooksources.php index 2fef725..72e0b88 100644 --- a/includes/specials/SpecialBooksources.php +++ b/includes/specials/SpecialBooksources.php @@ -133,7 +133,9 @@ ], ]; - HTMLForm::factory( 'ooui', $formDescriptor, $this->getContext() ) + $context = new DerivativeContext( $this->getContext() ); + $context->setTitle( $this->getPageTitle() ); + HTMLForm::factory( 'ooui', $formDescriptor, $context ) ->setWrapperLegendMsg( 'booksources-search-legend' ) ->setSubmitTextMsg( 'booksources-search' ) ->setMethod( 'get' ) -- To view, visit https://gerrit.wikimedia.org/r/316203 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: newchange Gerrit-Change-Id: Ia007648e23beadcc017e117e59ce3df1b329dcdc Gerrit-PatchSet: 1 Gerrit-Project: mediawiki/core Gerrit-Branch: master Gerrit-Owner: Legoktm___ MediaWiki-commits mailing list MediaWiki-commits@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits