Daniel Werner has uploaded a new change for review.
https://gerrit.wikimedia.org/r/92423
Change subject: jQuery.valueview minor options optimization
......................................................................
jQuery.valueview minor options optimization
Change-Id: I20aff1b24e39101c803be8968385d83c9d40a17e
---
M ValueView/resources/jquery.valueview/valueview.valueview.js
1 file changed, 8 insertions(+), 2 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/DataValues
refs/changes/23/92423/1
diff --git a/ValueView/resources/jquery.valueview/valueview.valueview.js
b/ValueView/resources/jquery.valueview/valueview.valueview.js
index 8ac1ec7..1d6bd91 100644
--- a/ValueView/resources/jquery.valueview/valueview.valueview.js
+++ b/ValueView/resources/jquery.valueview/valueview.valueview.js
@@ -180,12 +180,18 @@
* @see jQuery.widget._setOption
*/
_setOption: function( key, value ) {
+ switch( key ) {
+ case 'autoStartEditing':
+ // doesn't make sense to change this after
initialization
+ throw new Error( 'Can not change
jQuery.valueview option "' + key
+ + '" after widget initialization' );
+ }
+
PARENT.prototype._setOption.call( this, key, value );
switch( key ) {
- case 'autoStartEditing':
- break; // doesn't make sense to change this
after initialization
case 'expertProvider':
+ case 'on': // TODO: make this work properly and test
this._updateExpert();
break;
case 'value':
--
To view, visit https://gerrit.wikimedia.org/r/92423
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I20aff1b24e39101c803be8968385d83c9d40a17e
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/DataValues
Gerrit-Branch: master
Gerrit-Owner: Daniel Werner <[email protected]>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
