X-Git-Url: https://git.cyclocoop.org/%7B%24www_url%7Dadmin/compta/comptes/?a=blobdiff_plain;f=includes%2FTitle.php;h=0687a1558958e85cba569ba4b74bce9937b23247;hb=f4022fe876525b49d557194d084b35bd59e0f336;hp=5dececee8e5fe4cdc9c68070a10ba77c29d586d2;hpb=36395150104588f2afea866c330b683e4329fa48;p=lhc%2Fweb%2Fwiklou.git

diff --git a/includes/Title.php b/includes/Title.php
index 5dececee8e..0687a15589 100644
--- a/includes/Title.php
+++ b/includes/Title.php
@@ -1036,6 +1036,7 @@ class Title implements LinkTarget {
 	 * Can this title have a corresponding talk page?
 	 *
 	 * @see MWNamespace::hasTalkNamespace
+	 * @since 1.30
 	 *
 	 * @return bool True if this title either is a talk page or can have a talk page associated.
 	 */
@@ -1324,7 +1325,7 @@ class Title implements LinkTarget {
 	 *
 	 * @since 1.30
 	 *
-	 * @return Title The object for the talk page,
+	 * @return Title|null The object for the talk page,
 	 *         or null if no associated talk page can exist, according to canHaveTalkPage().
 	 */
 	public function getTalkPageIfDefined() {
@@ -1896,6 +1897,8 @@ class Title implements LinkTarget {
 	 * protocol-relative, the URL will be expanded to http://
 	 *
 	 * @see self::getLocalURL for the arguments.
+	 * @param string $query
+	 * @param string|bool $query2
 	 * @return string The URL
 	 */
 	public function getInternalURL( $query = '', $query2 = false ) {
@@ -2667,24 +2670,33 @@ class Title implements LinkTarget {
 
 		if ( $this->mTitleProtection === null ) {
 			$dbr = wfGetDB( DB_REPLICA );
+			$commentStore = new CommentStore( 'pt_reason' );
+			$commentQuery = $commentStore->getJoin();
 			$res = $dbr->select(
-				'protected_titles',
+				[ 'protected_titles' ] + $commentQuery['tables'],
 				[
 					'user' => 'pt_user',
-					'reason' => 'pt_reason',
 					'expiry' => 'pt_expiry',
 					'permission' => 'pt_create_perm'
-				],
+				] + $commentQuery['fields'],
 				[ 'pt_namespace' => $this->getNamespace(), 'pt_title' => $this->getDBkey() ],
-				__METHOD__
+				__METHOD__,
+				[],
+				$commentQuery['joins']
 			);
 
 			// fetchRow returns false if there are no rows.
 			$row = $dbr->fetchRow( $res );
 			if ( $row ) {
-				$row['expiry'] = $dbr->decodeExpiry( $row['expiry'] );
+				$this->mTitleProtection = [
+					'user' => $row['user'],
+					'expiry' => $dbr->decodeExpiry( $row['expiry'] ),
+					'permission' => $row['permission'],
+					'reason' => $commentStore->getComment( $row )->text,
+				];
+			} else {
+				$this->mTitleProtection = false;
 			}
-			$this->mTitleProtection = $row;
 		}
 		return $this->mTitleProtection;
 	}
@@ -3693,7 +3705,7 @@ class Title implements LinkTarget {
 	 * Returns true if ok, or a getUserPermissionsErrors()-like array otherwise
 	 *
 	 * @deprecated since 1.25, use MovePage's methods instead
-	 * @param Title $nt The new title
+	 * @param Title &$nt The new title
 	 * @param bool $auth Whether to check user permissions (uses $wgUser)
 	 * @param string $reason Is the log summary of the move, used for spam checking
 	 * @return array|bool True on success, getUserPermissionsErrors()-like array on failure
@@ -3745,7 +3757,7 @@ class Title implements LinkTarget {
 	 * Move a title to a new location
 	 *
 	 * @deprecated since 1.25, use the MovePage class instead
-	 * @param Title $nt The new title
+	 * @param Title &$nt The new title
 	 * @param bool $auth Indicates whether $wgUser's permissions
 	 *  should be checked
 	 * @param string $reason The reason for the move