jenkins-bot has submitted this change and it was merged. Change subject: Introduce _hasMoreLanguages method ......................................................................
Introduce _hasMoreLanguages method Checks length of languages before calculating actual additional languages. Change-Id: I2e5573ead7b97b4b6e743fd8ee220b22d0439e84 --- M view/resources/jquery/wikibase/jquery.wikibase.entitytermsforlanguagelistview.js 1 file changed, 24 insertions(+), 2 deletions(-) Approvals: Bene: Looks good to me, approved jenkins-bot: Verified diff --git a/view/resources/jquery/wikibase/jquery.wikibase.entitytermsforlanguagelistview.js b/view/resources/jquery/wikibase/jquery.wikibase.entitytermsforlanguagelistview.js index 09d429a..b8ad1aa 100644 --- a/view/resources/jquery/wikibase/jquery.wikibase.entitytermsforlanguagelistview.js +++ b/view/resources/jquery/wikibase/jquery.wikibase.entitytermsforlanguagelistview.js @@ -224,7 +224,7 @@ * @private */ _createEntitytermsforlanguagelistviewMore: function() { - if ( $.isEmptyObject( this._getAdditionalLanguages() ) ) { + if ( !this._hasMoreLanguages() ) { return; } @@ -241,7 +241,29 @@ }, /** - * Click handler for more languages button + * Checks whether there are more languages to display + * + * @private + */ + _hasMoreLanguages: function() { + var fingerprint = this.options.value, + minLength = this.options.userLanguages.length; + + if ( fingerprint.getLabels().length > minLength + || fingerprint.getDescriptions().length > minLength + || fingerprint.getAliases().length > minLength ) { + return true; + } + + if ( !$.isEmptyObject( this._getAdditionalLanguages() ) ) { + return true; + } + + return false; + }, + + /** + * Click handler for more languages button. * * @private */ -- To view, visit https://gerrit.wikimedia.org/r/267043 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2e5573ead7b97b4b6e743fd8ee220b22d0439e84 Gerrit-PatchSet: 4 Gerrit-Project: mediawiki/extensions/Wikibase Gerrit-Branch: master Gerrit-Owner: Jonas Kress (WMDE) <jonas.kr...@wikimedia.de> Gerrit-Reviewer: Adrian Lang <adrian.he...@wikimedia.de> Gerrit-Reviewer: Bene <benestar.wikime...@gmail.com> Gerrit-Reviewer: Thiemo Mättig (WMDE) <thiemo.maet...@wikimedia.de> Gerrit-Reviewer: jenkins-bot <> _______________________________________________ MediaWiki-commits mailing list MediaWiki-commits@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits