Mooeypoo has uploaded a new change for review. ( 
https://gerrit.wikimedia.org/r/369538 )

Change subject: RCFilters: Unsticky the 'limit' preference temporarily
......................................................................

RCFilters: Unsticky the 'limit' preference temporarily

Bug: T172156
Bug: T171514
Change-Id: I0bed6de70655a4a8a90ceb2f900ffbe98e643d33
---
M resources/src/mediawiki.rcfilters/mw.rcfilters.Controller.js
1 file changed, 11 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/core 
refs/changes/38/369538/1

diff --git a/resources/src/mediawiki.rcfilters/mw.rcfilters.Controller.js 
b/resources/src/mediawiki.rcfilters/mw.rcfilters.Controller.js
index 4051eda..a573ca4 100644
--- a/resources/src/mediawiki.rcfilters/mw.rcfilters.Controller.js
+++ b/resources/src/mediawiki.rcfilters/mw.rcfilters.Controller.js
@@ -105,7 +105,11 @@
                                        validate: $.isNumeric,
                                        sortFunc: function ( a, b ) { return 
Number( a.name ) - Number( b.name ); },
                                        'default': String( limitDefault ),
-                                       isSticky: true,
+                                       // Temporarily making this not sticky 
until we resolve the proble
+                                       // with the misleading preference. Note 
that if this is to be permanent
+                                       // we should remove all sticky behavior 
methods completely
+                                       // See T172156
+                                       // isSticky: true,
                                        filters: [ 50, 100, 250, 500 ].map( 
function ( num ) {
                                                return 
controller._createFilterDataFromNumber( num, num );
                                        } )
@@ -813,6 +817,10 @@
         * @param {number} newValue New value
         */
        mw.rcfilters.Controller.prototype.updateLimitDefault = function ( 
newValue ) {
+               // HACK: Temporarily remove this from being sticky
+               // See T172156
+
+               /*
                if ( !$.isNumeric( newValue ) ) {
                        return;
                }
@@ -825,6 +833,8 @@
                        // Update the preference for this session
                        mw.user.options.set( 'rcfilters-rclimit', newValue );
                }
+               */
+               return;
        };
 
        /**

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I0bed6de70655a4a8a90ceb2f900ffbe98e643d33
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/core
Gerrit-Branch: master
Gerrit-Owner: Mooeypoo <[email protected]>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to