Aude has uploaded a new change for review. https://gerrit.wikimedia.org/r/199270
Change subject: Update Wikidata ...................................................................... Update Wikidata * bug fixes to dispatch changes script * switch order of description and alias in header (back to how it is currently on Wikidata) * Fix css on Special:NewItem and Special:NewProperty * Fix rendering of qualifiers on save * Security fix for MergeItems api module Change-Id: Ied92e1f088c5287409e05bec26a2aef40d44358a --- M composer.lock M extensions/Wikibase/lib/maintenance/dispatchChanges.php M extensions/Wikibase/lib/resources/jquery.wikibase/jquery.wikibase.entitytermsforlanguagelistview.js M extensions/Wikibase/lib/resources/jquery.wikibase/jquery.wikibase.entitytermsforlanguageview.js M extensions/Wikibase/lib/resources/jquery.wikibase/jquery.wikibase.entitytermsview.js M extensions/Wikibase/lib/resources/jquery.wikibase/jquery.wikibase.statementview.js M extensions/Wikibase/repo/includes/View/EntityTermsView.php M extensions/Wikibase/repo/includes/api/MergeItems.php M extensions/Wikibase/repo/resources/templates.php M extensions/Wikibase/repo/resources/wikibase.special/wikibase.special.css M vendor/composer/installed.json 11 files changed, 39 insertions(+), 31 deletions(-) git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/Wikidata refs/changes/70/199270/1 diff --git a/composer.lock b/composer.lock index 1ed7281..1d8c9bb 100644 --- a/composer.lock +++ b/composer.lock @@ -1237,12 +1237,12 @@ "source": { "type": "git", "url": "https://github.com/wikimedia/mediawiki-extensions-Wikibase.git", - "reference": "c5f4cd35a8ff2c7e95157fbb354909964e5a1391" + "reference": "75b3a2284f1780663c54655957db2992d774d7dd" }, "dist": { "type": "zip", - "url": "https://api.github.com/repos/wikimedia/mediawiki-extensions-Wikibase/zipball/c5f4cd35a8ff2c7e95157fbb354909964e5a1391", - "reference": "c5f4cd35a8ff2c7e95157fbb354909964e5a1391", + "url": "https://api.github.com/repos/wikimedia/mediawiki-extensions-Wikibase/zipball/75b3a2284f1780663c54655957db2992d774d7dd", + "reference": "75b3a2284f1780663c54655957db2992d774d7dd", "shasum": "" }, "require": { @@ -1309,7 +1309,7 @@ "wikibaserepo", "wikidata" ], - "time": "2015-03-18 14:23:48" + "time": "2015-03-24 14:53:33" }, { "name": "wikibase/wikimedia-badges", diff --git a/extensions/Wikibase/lib/maintenance/dispatchChanges.php b/extensions/Wikibase/lib/maintenance/dispatchChanges.php index f2131d8..8b99b38 100644 --- a/extensions/Wikibase/lib/maintenance/dispatchChanges.php +++ b/extensions/Wikibase/lib/maintenance/dispatchChanges.php @@ -418,9 +418,9 @@ , __METHOD__, array( - 'ORDER BY chd_seen ASC', + 'ORDER BY' => 'chd_seen ASC', 'FOR UPDATE', - 'LIMIT ' . (int)$this->randomness + 'LIMIT' => (int)$this->randomness ) ); @@ -860,7 +860,7 @@ // The change is relevant if it alters any sitelinks referring to $siteID, // or the item currently links to $siteID. if ( isset( $linkedItems[$itemId] ) - || $this->isRelevantChange( $change, $siteID ) !== null ) { + || $this->isRelevantChange( $change, $siteID ) ) { $filteredChanges[] = $change; $numberOfChangesFound++; diff --git a/extensions/Wikibase/lib/resources/jquery.wikibase/jquery.wikibase.entitytermsforlanguagelistview.js b/extensions/Wikibase/lib/resources/jquery.wikibase/jquery.wikibase.entitytermsforlanguagelistview.js index 1cf2d1d..02f2e69 100644 --- a/extensions/Wikibase/lib/resources/jquery.wikibase/jquery.wikibase.entitytermsforlanguagelistview.js +++ b/extensions/Wikibase/lib/resources/jquery.wikibase/jquery.wikibase.entitytermsforlanguagelistview.js @@ -52,8 +52,8 @@ templateParams: [ mw.msg( 'wikibase-entitytermsforlanguagelistview-language' ), mw.msg( 'wikibase-entitytermsforlanguagelistview-label' ), - mw.msg( 'wikibase-entitytermsforlanguagelistview-aliases' ), mw.msg( 'wikibase-entitytermsforlanguagelistview-description' ), + mw.msg( 'wikibase-entitytermsforlanguagelistview-aliases' ), '' // entitytermsforlanguageview ], templateShortCuts: { diff --git a/extensions/Wikibase/lib/resources/jquery.wikibase/jquery.wikibase.entitytermsforlanguageview.js b/extensions/Wikibase/lib/resources/jquery.wikibase/jquery.wikibase.entitytermsforlanguageview.js index 8f36f77..ffca465 100644 --- a/extensions/Wikibase/lib/resources/jquery.wikibase/jquery.wikibase.entitytermsforlanguageview.js +++ b/extensions/Wikibase/lib/resources/jquery.wikibase/jquery.wikibase.entitytermsforlanguageview.js @@ -67,8 +67,8 @@ ); }, '', // label - '', // aliases '', // description + '', // aliases '' // toolbar placeholder ], templateShortCuts: { diff --git a/extensions/Wikibase/lib/resources/jquery.wikibase/jquery.wikibase.entitytermsview.js b/extensions/Wikibase/lib/resources/jquery.wikibase/jquery.wikibase.entitytermsview.js index a6296c4..a26d60c 100644 --- a/extensions/Wikibase/lib/resources/jquery.wikibase/jquery.wikibase.entitytermsview.js +++ b/extensions/Wikibase/lib/resources/jquery.wikibase/jquery.wikibase.entitytermsview.js @@ -55,10 +55,10 @@ templateParams: [ '', // label class '', // labelview - '', // aliases class - '', // aliasesview '', // description class '', // descriptionview + '', // aliases class + '', // aliasesview '', // entitytermsforlanguagelistview '', // additional entitytermsforlanguagelistview container class(es) '' // toolbar placeholder diff --git a/extensions/Wikibase/lib/resources/jquery.wikibase/jquery.wikibase.statementview.js b/extensions/Wikibase/lib/resources/jquery.wikibase/jquery.wikibase.statementview.js index bcb6c72..72f1435 100644 --- a/extensions/Wikibase/lib/resources/jquery.wikibase/jquery.wikibase.statementview.js +++ b/extensions/Wikibase/lib/resources/jquery.wikibase/jquery.wikibase.statementview.js @@ -478,7 +478,11 @@ : this.option( 'predefined' ).mainSnak || null ); - if( this.isInEditMode() ) { + if( this.isInEditMode() + || this.options.value + && this.options.value.getClaim().getQualifiers().length + && !this.$qualifiers.children().length + ) { this._createQualifiersListview( this.options.value ? this.options.value.getClaim().getQualifiers() diff --git a/extensions/Wikibase/repo/includes/View/EntityTermsView.php b/extensions/Wikibase/repo/includes/View/EntityTermsView.php index d1f06d7..88ee5bb 100644 --- a/extensions/Wikibase/repo/includes/View/EntityTermsView.php +++ b/extensions/Wikibase/repo/includes/View/EntityTermsView.php @@ -83,10 +83,10 @@ return $this->templateFactory->render( 'wikibase-entitytermsview', $labels->hasTermForLanguage( $this->languageCode ) ? '' : 'wb-empty', $this->getHtmlForLabel( $labels, $entityId ), - $aliasGroups->hasGroupForLanguage( $this->languageCode ) ? '' : 'wb-empty', - $this->getHtmlForAliases( $aliasGroups ), $descriptions->hasTermForLanguage( $this->languageCode ) ? '' : 'wb-empty', $this->getDescriptionText( $descriptions ), + $aliasGroups->hasGroupForLanguage( $this->languageCode ) ? '' : 'wb-empty', + $this->getHtmlForAliases( $aliasGroups ), $termBoxHtml, $textInjector->newMarker( 'entityViewPlaceholder-entitytermsview-entitytermsforlanguagelistview-class' @@ -182,8 +182,8 @@ return $this->templateFactory->render( 'wikibase-entitytermsforlanguagelistview', $this->msg( 'wikibase-entitytermsforlanguagelistview-language' ), $this->msg( 'wikibase-entitytermsforlanguagelistview-label' ), - $this->msg( 'wikibase-entitytermsforlanguagelistview-aliases' ), $this->msg( 'wikibase-entitytermsforlanguagelistview-description' ), + $this->msg( 'wikibase-entitytermsforlanguagelistview-aliases' ), $entityTermsForLanguageViewsHtml ); } @@ -226,7 +226,6 @@ '', '' ), - $this->getAliasesView( $aliasGroups, $languageCode ), $this->templateFactory->render( 'wikibase-descriptionview', $hasDescription ? '' : 'wb-empty', htmlspecialchars( $hasDescription @@ -236,6 +235,7 @@ '', '' ), + $this->getAliasesView( $aliasGroups, $languageCode ), '' ); } diff --git a/extensions/Wikibase/repo/includes/api/MergeItems.php b/extensions/Wikibase/repo/includes/api/MergeItems.php index 38ad1e4..adaf84e 100644 --- a/extensions/Wikibase/repo/includes/api/MergeItems.php +++ b/extensions/Wikibase/repo/includes/api/MergeItems.php @@ -176,6 +176,15 @@ } /** + * @see ApiBase::needsToken + * + * @return string + */ + public function needsToken() { + return 'csrf'; + } + + /** * @see ApiBase::getAllowedParams */ protected function getAllowedParams() { @@ -194,7 +203,6 @@ 'summary' => array( ApiBase::PARAM_TYPE => 'string', ), - 'token' => null, 'bot' => false ); } diff --git a/extensions/Wikibase/repo/resources/templates.php b/extensions/Wikibase/repo/resources/templates.php index 55c7835..0c6263c 100644 --- a/extensions/Wikibase/repo/resources/templates.php +++ b/extensions/Wikibase/repo/resources/templates.php @@ -158,8 +158,8 @@ <div class="wikibase-entitytermsview"> <div class="wikibase-entitytermsview-heading"> <h1 class="wikibase-entitytermsview-heading-label $1"><!-- wikibase-entitytermsview-label -->$2</h1> - <div class="wikibase-entitytermsview-heading-aliases $3"><!-- wikibase-entitytermsview-aliases -->$4</div> - <div class="wikibase-entitytermsview-heading-description $5">$6</div> + <div class="wikibase-entitytermsview-heading-description $3">$4</div> + <div class="wikibase-entitytermsview-heading-aliases $5"><!-- wikibase-entitytermsview-aliases -->$6</div> </div> <!-- ? wikibase-toolbar -->$9 <div class="wikibase-entitytermsview-entitytermsforlanguagelistview $8"><!-- wikibase-entitytermsforlanguagelistview -->$7</div> @@ -188,8 +188,8 @@ <tr class="wikibase-entitytermsforlanguagelistview-header-row"> <td class="wikibase-entitytermsforlanguagelistview-cell wikibase-entitytermsforlanguagelistview-language">$1</td> <td class="wikibase-entitytermsforlanguagelistview-cell wikibase-entitytermsforlanguagelistview-label">$2</td> - <td class="wikibase-entitytermsforlanguagelistview-cell wikibase-entitytermsforlanguagelistview-aliases">$3</td> - <td class="wikibase-entitytermsforlanguagelistview-cell wikibase-entitytermsforlanguagelistview-description">$4</td> + <td class="wikibase-entitytermsforlanguagelistview-cell wikibase-entitytermsforlanguagelistview-description">$3</td> + <td class="wikibase-entitytermsforlanguagelistview-cell wikibase-entitytermsforlanguagelistview-aliases">$4</td> </tr> </thead> <tbody class="wikibase-entitytermsforlanguagelistview-listview"><!-- [0,*] wikibase-entitytermsforlanguageview -->$5</tbody> @@ -201,8 +201,8 @@ <$1 class="wikibase-entitytermsforlanguageview wikibase-entitytermsforlanguageview-$3" > <$2 class="wikibase-entitytermsforlanguageview-language"><!-- wikibase-entitytermsforlanguageview-language -->$4</$2> <$2 class="wikibase-entitytermsforlanguageview-label">$5</$2> - <$2 class="wikibase-entitytermsforlanguageview-aliases">$6</$2> - <$2 class="wikibase-entitytermsforlanguageview-description">$7</$2> + <$2 class="wikibase-entitytermsforlanguageview-description">$6</$2> + <$2 class="wikibase-entitytermsforlanguageview-aliases">$7</$2> <!-- ? wikibase-toolbar -->$8 </$1> HTML; diff --git a/extensions/Wikibase/repo/resources/wikibase.special/wikibase.special.css b/extensions/Wikibase/repo/resources/wikibase.special/wikibase.special.css index f5c9e68..cb37733 100644 --- a/extensions/Wikibase/repo/resources/wikibase.special/wikibase.special.css +++ b/extensions/Wikibase/repo/resources/wikibase.special/wikibase.special.css @@ -9,10 +9,6 @@ /********** Create item/property page **********/ -.wb-fieldset { - float: left; -} - .wb-label, .wb-input, .wb-button, .wb-select { display: block; float: left; diff --git a/vendor/composer/installed.json b/vendor/composer/installed.json index 7f71d89..c5640fd 100644 --- a/vendor/composer/installed.json +++ b/vendor/composer/installed.json @@ -1329,12 +1329,12 @@ "source": { "type": "git", "url": "https://github.com/wikimedia/mediawiki-extensions-Wikibase.git", - "reference": "c5f4cd35a8ff2c7e95157fbb354909964e5a1391" + "reference": "75b3a2284f1780663c54655957db2992d774d7dd" }, "dist": { "type": "zip", - "url": "https://api.github.com/repos/wikimedia/mediawiki-extensions-Wikibase/zipball/c5f4cd35a8ff2c7e95157fbb354909964e5a1391", - "reference": "c5f4cd35a8ff2c7e95157fbb354909964e5a1391", + "url": "https://api.github.com/repos/wikimedia/mediawiki-extensions-Wikibase/zipball/75b3a2284f1780663c54655957db2992d774d7dd", + "reference": "75b3a2284f1780663c54655957db2992d774d7dd", "shasum": "" }, "require": { @@ -1362,7 +1362,7 @@ "conflict": { "mediawiki/mediawiki": "<1.23" }, - "time": "2015-03-18 14:23:48", + "time": "2015-03-24 14:53:33", "type": "mediawiki-extension", "installation-source": "dist", "autoload": { -- To view, visit https://gerrit.wikimedia.org/r/199270 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: newchange Gerrit-Change-Id: Ied92e1f088c5287409e05bec26a2aef40d44358a Gerrit-PatchSet: 1 Gerrit-Project: mediawiki/extensions/Wikidata Gerrit-Branch: wmf/1.25wmf22 Gerrit-Owner: Aude <aude.w...@gmail.com> _______________________________________________ MediaWiki-commits mailing list MediaWiki-commits@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits