X-Git-Url: http://git.cyclocoop.org/?a=blobdiff_plain;f=includes%2FWikiMap.php;h=3305f9f7d3ad9bf095ff97686ae14aeffd5fda56;hb=51945dbca3594;hp=8bb37b5c882a7a599d63ab658cdaa4a5bfc80a88;hpb=236488d398046838059f758b0915341648b64c7b;p=lhc%2Fweb%2Fwiklou.git diff --git a/includes/WikiMap.php b/includes/WikiMap.php index 8bb37b5c88..3305f9f7d3 100644 --- a/includes/WikiMap.php +++ b/includes/WikiMap.php @@ -104,7 +104,7 @@ class WikiMap { $path .= '?' . $urlParts['query']; } - $canonicalServer = isset( $urlParts['scheme'] ) ? $urlParts['scheme'] : 'http'; + $canonicalServer = $urlParts['scheme'] ?? 'http'; $canonicalServer .= '://' . $urlParts['host']; return new WikiReference( $canonicalServer, $path ); @@ -131,7 +131,7 @@ class WikiMap { * * @param string $wikiID Wiki'd id (generally database name) * @param string $user User name (must be normalised before calling this function!) - * @param string $text Link's text; optional, default to "User:$user" + * @param string|null $text Link's text; optional, default to "User:$user" * @return string HTML link or false if the wiki was not found */ public static function foreignUserLink( $wikiID, $user, $text = null ) { @@ -143,7 +143,7 @@ class WikiMap { * * @param string $wikiID Wiki'd id (generally database name) * @param string $page Page name (must be normalised before calling this function!) - * @param string $text Link's text; optional, default to $page + * @param string|null $text Link's text; optional, default to $page * @return string|false HTML link or false if the wiki was not found */ public static function makeForeignLink( $wikiID, $page, $text = null ) { @@ -196,7 +196,8 @@ class WikiMap { $infoMap = []; // Make sure at least the current wiki is set, for simple configurations. // This also makes it the first in the map, which is useful for common cases. - $infoMap[wfWikiID()] = [ + $wikiId = self::getWikiIdFromDomain( self::getCurrentWikiDomain() ); + $infoMap[$wikiId] = [ 'url' => $wgCanonicalServer, 'parts' => wfParseUrl( $wgCanonicalServer ) ]; @@ -245,17 +246,68 @@ class WikiMap { * Get the wiki ID of a database domain * * This is like DatabaseDomain::getId() without encoding (for legacy reasons) + * and without the schema if it merely set to the generic value "mediawiki" * * @param string|DatabaseDomain $domain * @return string */ public static function getWikiIdFromDomain( $domain ) { - if ( !( $domain instanceof DatabaseDomain ) ) { - $domain = DatabaseDomain::newFromId( $domain ); + $domain = DatabaseDomain::newFromId( $domain ); + + if ( !in_array( $domain->getSchema(), [ null, 'mediawiki' ], true ) ) { + // Include the schema if it is set and is not the default placeholder. + // This means a site admin may have specifically taylored the schemas. + // Domain IDs might use the form --, meaning that + // the schema portion must be accounted for to disambiguate wikis. + return "{$domain->getDatabase()}-{$domain->getSchema()}-{$domain->getTablePrefix()}"; } + // Note that if this wiki ID is passed a a domain ID to LoadBalancer, then it can + // handle the schema by assuming the generic "mediawiki" schema if needed. return strlen( $domain->getTablePrefix() ) ? "{$domain->getDatabase()}-{$domain->getTablePrefix()}" - : $domain->getDatabase(); + : (string)$domain->getDatabase(); + } + + /** + * @return DatabaseDomain Database domain of the current wiki + * @since 1.33 + */ + public static function getCurrentWikiDomain() { + global $wgDBname, $wgDBmwschema, $wgDBprefix; + // Avoid invoking LBFactory to avoid any chance of recursion + return new DatabaseDomain( $wgDBname, $wgDBmwschema, (string)$wgDBprefix ); + } + + /** + * @param DatabaseDomain|string $domain + * @return bool Whether $domain has the same DB/prefix as the current wiki + * @since 1.33 + */ + public static function isCurrentWikiDomain( $domain ) { + $domain = DatabaseDomain::newFromId( $domain ); + $curDomain = self::getCurrentWikiDomain(); + + if ( !in_array( $curDomain->getSchema(), [ null, 'mediawiki' ], true ) ) { + // Include the schema if it is set and is not the default placeholder. + // This means a site admin may have specifically taylored the schemas. + // Domain IDs might use the form --, meaning that + // the schema portion must be accounted for to disambiguate wikis. + return $curDomain->equals( $domain ); + } + + return ( + $curDomain->getDatabase() === $domain->getDatabase() && + $curDomain->getTablePrefix() === $domain->getTablePrefix() + ); + } + + /** + * @param string $wikiId + * @return bool Whether $wikiId has the same DB/prefix as the current wiki + * @since 1.33 + */ + public static function isCurrentWikiId( $wikiId ) { + return ( self::getWikiIdFromDomain( self::getCurrentWikiDomain() ) === $wikiId ); } }