Merge "RCFilters: Actually check whether defaults are empty and update state"
authorjenkins-bot <jenkins-bot@gerrit.wikimedia.org>
Mon, 18 Sep 2017 20:42:42 +0000 (20:42 +0000)
committerGerrit Code Review <gerrit@wikimedia.org>
Mon, 18 Sep 2017 20:42:42 +0000 (20:42 +0000)
1  2 
resources/src/mediawiki.rcfilters/ui/mw.rcfilters.ui.FilterTagMultiselectWidget.js

@@@ -70,7 -70,7 +70,7 @@@
                                ]
                        },
                        input: {
 -                              icon: 'menu',
 +                              icon: 'add',
                                placeholder: mw.msg( 'rcfilters-search-placeholder' )
                        }
                }, config ) );
@@@ -99,7 -99,8 +99,8 @@@
                        } );
                        this.queriesModel.connect( this, {
                                itemUpdate: 'onSavedQueriesItemUpdate',
-                               initialize: 'onSavedQueriesInitialize'
+                               initialize: 'onSavedQueriesInitialize',
+                               'default': 'reevaluateResetRestoreState'
                        } );
                }
  
                        this.controller.trackFilterGroupings( 'filtermenu' );
                }
  
 -              this.input.setIcon( isVisible ? 'search' : 'menu' );
 +              this.input.setIcon( isVisible ? 'search' : 'add' );
        };
  
        /**
         * Reevaluate the restore state for the widget between setting to defaults and clearing all filters
         */
        mw.rcfilters.ui.FilterTagMultiselectWidget.prototype.reevaluateResetRestoreState = function () {
-               var defaultsAreEmpty = this.model.areDefaultFiltersEmpty(),
+               var defaultsAreEmpty = this.controller.areDefaultsEmpty(),
                        currFiltersAreEmpty = this.model.areCurrentFiltersEmpty(),
                        hideResetButton = currFiltersAreEmpty && defaultsAreEmpty;