Henning Snater has uploaded a new change for review.

  https://gerrit.wikimedia.org/r/65271


Change subject: Checking for valid qualifiers in claimview
......................................................................

Checking for valid qualifiers in claimview

(bug 48776)
Change If4e1196551835776100382416430ae289f82c825 caused claimview's save button 
to be enabled
with empty qualifiers being present.

Change-Id: Iaa8636e8878abb379681be3e3ca44174ecc2fc44
---
M lib/resources/jquery.wikibase/jquery.wikibase.claimview.js
1 file changed, 4 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/Wikibase 
refs/changes/71/65271/1

diff --git a/lib/resources/jquery.wikibase/jquery.wikibase.claimview.js 
b/lib/resources/jquery.wikibase/jquery.wikibase.claimview.js
index 6344718..3ba48ca 100644
--- a/lib/resources/jquery.wikibase/jquery.wikibase.claimview.js
+++ b/lib/resources/jquery.wikibase/jquery.wikibase.claimview.js
@@ -184,6 +184,10 @@
         * @return {boolean}
         */
        isValid: function() {
+               if( this._qualifiers && !this._qualifiers.isValid() ) {
+                       return false;
+               }
+
                try {
                        this._instantiateClaim( null );
                } catch( e ) {

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Iaa8636e8878abb379681be3e3ca44174ecc2fc44
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/Wikibase
Gerrit-Branch: master
Gerrit-Owner: Henning Snater <henning.sna...@wikimedia.de>

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

Reply via email to