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

Change subject: Per legal, add "license prompt"
......................................................................


Per legal, add "license prompt"

Add extra license prefix text in front of the
license name:

   Data available under Creative Commons Zero

where the license name is a URL link

Bug: T152553
Change-Id: If5c149aeac55c820772e89512e9f3ee37d8292be
---
M i18n/en.json
M i18n/qqq.json
M includes/JCDataContent.php
3 files changed, 15 insertions(+), 9 deletions(-)

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



diff --git a/i18n/en.json b/i18n/en.json
index 7f73a52..e4e452f 100644
--- a/i18n/en.json
+++ b/i18n/en.json
@@ -39,8 +39,9 @@
        "jsonconfig-type-name-number": "number",
        "jsonconfig-type-abbr-boolean": "(B)",
        "jsonconfig-type-name-boolean": "boolean",
+       "jsonconfig-license": "Data available under $1",
        "jsonconfig-license-or-later": "$1, or (at your option) any later 
version",
-       "jsonconfig-license-CC0-1.0": "Creative Commons Zero v1.0 Universal",
+       "jsonconfig-license-CC0-1.0": "Creative Commons Zero",
        "jsonconfig-license-url-CC0-1.0": 
"https://creativecommons.org/publicdomain/zero/1.0/";,
        "right-jsonconfig-flush": "Manipulate JsonConfig via API",
        "apihelp-jsonconfig-description": "Allows direct access to JsonConfig 
subsystem.",
diff --git a/i18n/qqq.json b/i18n/qqq.json
index a40a222..c6aea8e 100644
--- a/i18n/qqq.json
+++ b/i18n/qqq.json
@@ -47,7 +47,8 @@
        "jsonconfig-type-name-number": "Description of the type of data 
contained in this column. Shown for columns that contain 
numbers\n{{Identical|Number}}",
        "jsonconfig-type-abbr-boolean": "A very short indicator of the type of 
data contained in this column. Shown for columns that contain booleans (true or 
false)",
        "jsonconfig-type-name-boolean": "Description of the type of data 
contained in this column. Shown for columns that contain booleans (true or 
false)\n{{Identical|Boolean}}",
-       "jsonconfig-license-or-later": "In case the data can be used with any 
future versions of a license, this string will be used instead of 
'jsonconfig-license-*' message.\n\nParameters:\n* $1 - license name, as defined 
in the 'jsonconfig-license-*' messages",
+       "jsonconfig-license": "Shows license text prefix before giving the 
specific license.\n\nParameters:\n* $1 - * 
{{msg-mw|jsonconfig-license-or-later}} or {{msg-mw|jsonconfig-license-CC0-1.0}} 
or other similar license messages",
+       "jsonconfig-license-or-later": "In case the data can be used with any 
future versions of a license, this string will be used instead of 
'jsonconfig-license-*' message.\n\nParameters:\n* $1 - license name, as defined 
in the {{msg-mw|jsonconfig-license-CC0-1.0}} and similar messages",
        "jsonconfig-license-CC0-1.0": "Name of the license",
        "jsonconfig-license-url-CC0-1.0": "URL to the license description, 
localized if available",
        "right-jsonconfig-flush": "{{doc-right|jsonconfig-flush}}",
diff --git a/includes/JCDataContent.php b/includes/JCDataContent.php
index 0188d4f..b71fb02 100644
--- a/includes/JCDataContent.php
+++ b/includes/JCDataContent.php
@@ -72,10 +72,11 @@
                $license = $this->getLicenseObject();
                if ( $license ) {
                        $text = $license['text']->inLanguage( $lang )->plain();
-                       if ( $license['laterVersion'] ) {
-                               $text = wfMessage( 
'jsonconfig-license-or-later', $text )
-                                       ->inLanguage( $lang )->plain();
-                       }
+                       // Legal suggested we don't add this just yet
+//                     if ( $license['laterVersion'] ) {
+//                             $text = wfMessage( 
'jsonconfig-license-or-later', $text )
+//                                     ->inLanguage( $lang )->plain();
+//                     }
                        $result->license = (object)[
                                'code' => $license['code'],
                                'text' => $text,
@@ -112,9 +113,12 @@
                                'href' => $license['url']->plain()
                        ], $license['text']->plain() );
 
-                       if ( $license['laterVersion'] ) {
-                               $text = wfMessage( 
'jsonconfig-license-or-later', $text )->plain();
-                       }
+                       // Legal suggested we don't add this just yet
+//                     if ( $license['laterVersion'] ) {
+//                             $text = wfMessage( 
'jsonconfig-license-or-later', $text )->plain();
+//                     }
+
+                       $text = wfMessage( 'jsonconfig-license' )->rawParams( 
$text )->parse();
 
                        $html = Html::rawElement( 'p', [ 'class' => 
'mw-jsonconfig-license' ], $text );
                } else {

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

Gerrit-MessageType: merged
Gerrit-Change-Id: If5c149aeac55c820772e89512e9f3ee37d8292be
Gerrit-PatchSet: 4
Gerrit-Project: mediawiki/extensions/JsonConfig
Gerrit-Branch: master
Gerrit-Owner: Yurik <yu...@wikimedia.org>
Gerrit-Reviewer: MaxSem <maxsem.w...@gmail.com>
Gerrit-Reviewer: Siebrand <siebr...@kitano.nl>
Gerrit-Reviewer: Yurik <yu...@wikimedia.org>
Gerrit-Reviewer: jenkins-bot <>

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to