jenkins-bot has submitted this change and it was merged.

Change subject: Add a "SkinMinervaLicenseFooter" hook
......................................................................


Add a "SkinMinervaLicenseFooter" hook

Needed to overwrite the whole license text with a Wikidata
specific one and not just a part of it.

Bug: T112088
Change-Id: I2f3cebc6dc59ebd19869056d6450e44964a2b53c
---
M includes/skins/SkinMinerva.php
1 file changed, 10 insertions(+), 7 deletions(-)

Approvals:
  Florianschmidtwelzow: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/includes/skins/SkinMinerva.php b/includes/skins/SkinMinerva.php
index dd0ed4c..5605d04 100644
--- a/includes/skins/SkinMinerva.php
+++ b/includes/skins/SkinMinerva.php
@@ -1227,13 +1227,16 @@
 </ul>
 HTML;
 
-               // Generate the licensing text displayed in the footer of each 
page.
-               // See Skin::getCopyright for desktop equivalent.
-               $license = self::getLicense( 'footer' );
-               if ( isset( $license['link'] ) && $license['link'] ) {
-                       $licenseText = $this->msg( 'mobile-frontend-copyright' 
)->rawParams( $license['link'] )->text();
-               } else {
-                       $licenseText = '';
+               // @TODO: Consolidate all of this with the core handling for 
license
+               // footers (implement license handling using the 
"SkinCopyrightFooter" hook).
+               $licenseText = '';
+               if ( Hooks::run( 'SkinMinervaLicenseFooter', array( 
&$licenseText ) ) ) {
+                       // Generate the licensing text displayed in the footer 
of each page.
+                       // See Skin::getCopyright for desktop equivalent.
+                       $license = self::getLicense( 'footer' );
+                       if ( isset( $license['link'] ) && $license['link'] ) {
+                               $licenseText = $this->msg( 
'mobile-frontend-copyright' )->rawParams( $license['link'] )->text();
+                       }
                }
 
                // Enable extensions to add links to footer in Mobile view, too 
- bug 66350

-- 
To view, visit https://gerrit.wikimedia.org/r/242339
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I2f3cebc6dc59ebd19869056d6450e44964a2b53c
Gerrit-PatchSet: 3
Gerrit-Project: mediawiki/extensions/MobileFrontend
Gerrit-Branch: master
Gerrit-Owner: Hoo man <[email protected]>
Gerrit-Reviewer: Aude <[email protected]>
Gerrit-Reviewer: Daniel Kinzler <[email protected]>
Gerrit-Reviewer: Florianschmidtwelzow <[email protected]>
Gerrit-Reviewer: Hoo man <[email protected]>
Gerrit-Reviewer: Jdlrobson <[email protected]>
Gerrit-Reviewer: jenkins-bot <>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to