Update docs/hooks.txt for ShowSearchHitTitle
authorThiemo Mättig <thiemo.maettig@wikimedia.de>
Fri, 20 Jan 2017 09:40:28 +0000 (10:40 +0100)
committerThiemo Mättig <thiemo.maettig@wikimedia.de>
Fri, 20 Jan 2017 09:40:28 +0000 (10:40 +0100)
Reasons:
* $text is vague. It's the label of the link. There is a function named
  getTitleSnippet, which is what I'm reusing for the variable name.
* $page is vague. It's not the page of the search result, but the search
  special page.
* $queryString is not a string but an array of strings.

I also updated https://www.mediawiki.org/wiki/Manual:Hooks/ShowSearchHitTitle
accordingly.

Change-Id: Ia18dbf5ccf7f9fcc8890ad01641d0b7ff3e6eee8

docs/hooks.txt
includes/widget/search/FullSearchResultWidget.php

index c5b112d..803c20d 100644 (file)
@@ -2823,11 +2823,11 @@ $terms: Search terms, for highlighting
 
 'ShowSearchHitTitle': Customise display of search hit title/link.
 &$title: Title to link to
-&$text: Text to use for the link
-$result: The search result
-$terms: The search terms entered
-$page: The SpecialSearch object
-&$query: Query string to be appended to the link
+&$titleSnippet: Label for the link representing the search result. Typically the article title.
+$result: The SearchResult object
+$terms: String of the search terms entered
+$specialSearch: The SpecialSearch object
+&$query: Array of query string parameters for the link representing the search result.
 
 'SidebarBeforeOutput': Allows to edit sidebar just before it is output by skins.
 Warning: This hook is run on each display. You should consider to use
index 69bfa6d..0d0fa12 100644 (file)
@@ -131,16 +131,16 @@ class FullSearchResultWidget implements SearchResultWidget {
 
                // clone to prevent hook from changing the title stored inside $result
                $title = clone $result->getTitle();
-               $queryString = [];
+               $query = [];
 
                Hooks::run( 'ShowSearchHitTitle',
-                       [ &$title, &$snippet, $result, $terms, $this->specialPage, &$queryString ] );
+                       [ &$title, &$snippet, $result, $terms, $this->specialPage, &$query ] );
 
                $link = $this->linkRenderer->makeLink(
                        $title,
                        $snippet,
                        [ 'data-serp-pos' => $position ],
-                       $queryString
+                       $query
                );
 
                return $link;