Merge "mw.widgets.UsersMultiselectWidget: Compat with mw.special.preferences.confirmC...
authorjenkins-bot <jenkins-bot@gerrit.wikimedia.org>
Tue, 17 Apr 2018 15:55:08 +0000 (15:55 +0000)
committerGerrit Code Review <gerrit@wikimedia.org>
Tue, 17 Apr 2018 15:55:08 +0000 (15:55 +0000)
resources/src/mediawiki.widgets/mw.widgets.UsersMultiselectWidget.js

index d250ad8..f0ad056 100644 (file)
                this.limit = config.limit;
 
                if ( 'name' in config ) {
-                       // If used inside HTML form, then create hidden input, which will store
-                       // the results.
-                       this.hiddenInput = $( '<input>' )
-                               .attr( 'type', 'hidden' )
+                       // Use this instead of <input type="hidden">, because hidden inputs do not have separate
+                       // 'value' and 'defaultValue' properties. The script on Special:Preferences
+                       // (mw.special.preferences.confirmClose) checks this property to see if a field was changed.
+                       this.hiddenInput = $( '<textarea>' )
+                               .addClass( 'oo-ui-element-hidden' )
                                .attr( 'name', config.name )
                                .appendTo( this.$element );
-
                        // Update with preset values
                        this.updateHiddenInput();
+                       // Set the default value (it might be different from just being empty)
+                       this.hiddenInput.prop( 'defaultValue', this.getSelectedUsernames().join( '\n' ) );
                }
 
                this.menu = this.getMenu();
        mw.widgets.UsersMultiselectWidget.prototype.updateHiddenInput = function () {
                if ( 'hiddenInput' in this ) {
                        this.hiddenInput.val( this.getSelectedUsernames().join( '\n' ) );
-                       // Hidden inputs do not trigger onChange.
-                       // @see https://developer.mozilla.org/en-US/docs/Web/HTML/Element/input/hidden
+                       // Trigger a 'change' event as if a user edited the text
+                       // (it is not triggered when changing the value from JS code).
                        this.hiddenInput.trigger( 'change' );
                }
        };