WordPress.org
Get WordPress

Make WordPress Core

Ticket #19815: 19815.9.2.diff

File 19815.9.2.diff, 7.4 KB (added by garyc40, 13 years ago)

Removed some redundant variables that I added.

  • wp-admin/js/theme.dev.js

    var ThemeViewer; 
    4545                        init: init
    4646                };
    4747
    48         return api;
     48        return api;
    4949        }
    5050})(jQuery);
    5151
    … … jQuery( document ).ready( function($) { 
    5454        theme_viewer.init();
    5555});
    5656
    57 var wpThemes;
     57var ThemeScroller;
    5858
    5959(function($){
    60         var inputs = {}, Query;
    61 
    62         wpThemes = {
    63                 timeToTriggerQuery: 150,
    64                 minQueryAJAXDuration: 200,
     60        ThemeScroller = {
     61                // Inputs
     62                nonce: '',
     63                search: '',
     64                tab: '',
     65                type: '',
     66                nextPage: 2,
     67                features: {},
     68
     69                // Preferences
     70                scrollThrottleDelay: 500,
     71                failedRetryDelay: 4000,
    6572                outListBottomThreshold: 200,
     73
     74                // Flags
    6675                noMoreResults: false,
    67                
     76                scrolling: false,
     77                querying: false,
     78
    6879                init : function() {
     80                        var self = this,
     81                                startPage,
     82                                queryArray = {},
     83                                queryString = window.location.search;
     84
     85                        // We're using infinite scrolling, so hide all pagination.
    6986                        $('.pagination-links').hide();
    7087
    71                         inputs.nonce = $('#_ajax_fetch_list_nonce').val();
    72        
    73                         // Parse Query
    74                         inputs.queryString = window.location.search;                   
    75                         inputs.queryArray = wpThemes.parseQuery( inputs.queryString.substring( 1 ) );
    76 
    77                         // Handle Inputs from Query
    78                         inputs.search = inputs.queryArray['s'];
    79                         inputs.features = inputs.queryArray['features'];
    80                         inputs.startPage = parseInt( inputs.queryArray['paged'] );     
    81                         inputs.tab = inputs.queryArray['tab'];
    82                         inputs.type = inputs.queryArray['type'];
    83 
    84                         if ( isNaN( inputs.startPage ) )
    85                                 inputs.startPage = 2;
    86                         else
    87                                 inputs.startPage++;
    88 
    89                         // Cache jQuery objects
    90                         inputs.outList = $('#availablethemes');
    91                         inputs.waiting = $('div.tablenav.bottom').children( 'img.ajax-loading' );
    92                         inputs.window = $(window);
    93 
    94                         // Generate Query
    95                         wpThemes.query = new Query();
    96 
    97                         // Start Polling
    98                         inputs.window.scroll( function(){ wpThemes.maybeLoad(); } );
    99                 },
    100                 delayedCallback : function( func, delay ) {
    101                         var timeoutTriggered, funcTriggered, funcArgs, funcContext;
    102 
    103                         if ( ! delay )
    104                                 return func;
    105 
    106                         setTimeout( function() {
    107                                 if ( funcTriggered )
    108                                         return func.apply( funcContext, funcArgs );
    109                                 // Otherwise, wait.
    110                                 timeoutTriggered = true;
    111                         }, delay);
    112 
    113                         return function() {
    114                                 if ( timeoutTriggered )
    115                                         return func.apply( this, arguments );
    116                                 // Otherwise, wait.
    117                                 funcArgs = arguments;
    118                                 funcContext = this;
    119                                 funcTriggered = true;
    120                         };
     88                        // Parse GET query string
     89                        queryArray = this.parseQuery( queryString.substring( 1 ) );
     90
     91                        // Handle inputs
     92                        this.nonce = $('#_ajax_fetch_list_nonce').val();
     93                        this.search = queryArray['s'];
     94                        this.features = queryArray['features'];
     95                        this.tab = queryArray['tab'];
     96                        this.type = queryArray['type'];
     97
     98                        startPage = parseInt( queryArray['paged'] );
     99                        if ( !isNaN( startPage ) )
     100                                this.nextPage++;
     101
     102                        // Cache jQuery selectors
     103                        this.$outList = $('#availablethemes');
     104                        this.$spinner = $('div.tablenav.bottom').children( 'img.ajax-loading' );
     105                        this.$window = $(window);
     106                        this.$document = $(document);
     107
     108                        setInterval(this.poll, this.scrollThrottleDelay);
    121109                },
    122                 ajax: function( callback ) {
    123                         var self = this,
    124                                 response = wpThemes.delayedCallback( function( results, params ) {
    125                                         self.process( results, params );
    126                                         if ( callback )
    127                                                 callback( results, params );
    128                                 }, wpThemes.minQueryAJAXDuration );
     110                poll : function() {
     111                        var t = ThemeScroller,
     112                                bottom = t.$document.scrollTop() + t.$window.innerHeight();
     113
     114                        if ( t.noMoreResults ||
     115                                t.querying ||
     116                                ( bottom < t.$outList.height() - t.outListBottomThreshold ) )
     117                                return;
    129118
    130                         this.query.ajax( response );
     119                        t.maybeLoad();
    131120                },
    132                 process: function( results, params ) {
    133                         // If no Results, for now, mark as no Matches, and bail.
    134                         // Alternately: inputs.outList.append(wpThemesL10n.noMatchesFound);
     121                process: function( results ) {
    135122                        if ( ( results === undefined ) ||
    136                                  ( results.rows.indexOf( "no-items" ) != -1 ) ) {
     123                                ( results.rows.indexOf( "no-items" ) != -1 ) ) {
    137124                                this.noMoreResults = true;
    138125                        } else {
    139                                 inputs.outList.append( results.rows );
     126                                this.$outList.append( results.rows );
    140127                        }
    141128                },
    142129                maybeLoad: function() {
    143                         var self = this,
    144                                 el = $(document),
    145                                 bottom = el.scrollTop() + inputs.window.innerHeight();
    146                                
    147                         if ( this.noMoreResults ||
    148                                  !this.query.ready() ||
    149                                  ( bottom < inputs.outList.height() - wpThemes.outListBottomThreshold ) )
    150                                 return;
    151 
    152                         setTimeout( function() {
    153                                 var newTop = el.scrollTop(),
    154                                         newBottom = newTop + inputs.window.innerHeight();
     130                        var self = this;
     131                        this.querying = true;
    155132
    156                                 if ( !self.query.ready() ||
    157                                          ( newBottom < inputs.outList.height() - wpThemes.outListBottomThreshold ) )
    158                                         return;
     133                        var query = {
     134                                action: 'fetch-list',
     135                                tab: this.tab,
     136                                paged: this.nextPage,
     137                                s: this.search,
     138                                type: this.type,
     139                                _ajax_fetch_list_nonce: this.nonce,
     140                                'features[]': this.features,
     141                                'list_args': list_args
     142                        };
    159143
    160                                 inputs.waiting.css( 'visibility', 'visible' ); // Show Spinner
    161                                 self.ajax( function() { inputs.waiting.css( 'visibility', 'hidden' ) } ); // Hide Spinner
    162                                
    163                         }, wpThemes.timeToTriggerQuery );
     144                        this.$spinner.css( 'visibility', 'visible' );
     145                        $.getJSON( ajaxurl, query )
     146                                .done( function( response ) {
     147                                        self.nextPage++;
     148                                        self.process( response );
     149                                        self.$spinner.css( 'visibility', 'hidden' );
     150                                        self.querying = false;
     151                                })
     152                                .fail( function() {
     153                                        self.$spinner.css( 'visibility', 'hidden' );
     154                                        self.querying = false;
     155                                        setTimeout( function() { self.maybeLoad(); }, self.failedRetryDelay)
     156                                });
    164157                },
    165158                parseQuery: function( query ) {
    166159                        var Params = {};
    167                         if ( ! query ) {return Params;}// return empty object
     160                        if ( !query ) { return Params; }
    168161                        var Pairs = query.split(/[;&]/);
    169162                        for ( var i = 0; i < Pairs.length; i++ ) {
    170163                                var KeyVal = Pairs[i].split('=');
    171                                 if ( ! KeyVal || KeyVal.length != 2 ) {continue;}
     164                                if ( ! KeyVal || KeyVal.length != 2 ) { continue; }
    172165                                var key = unescape( KeyVal[0] );
    173166                                var val = unescape( KeyVal[1] );
    174167                                val = val.replace(/\+/g, ' ');
    175168                                key = key.replace(/\[.*\]$/g, '');
    176        
     169
    177170                                if ( Params[key] === undefined ) {
    178171                                        Params[key] = val;
    179172                                } else {
    180173                                        var oldVal = Params[key];
    181                                         if ( ! jQuery.isArray( Params[key] ) )
     174                                        if ( !jQuery.isArray( Params[key] ) )
    182175                                                Params[key] = new Array( oldVal, val );
    183176                                        else
    184177                                                Params[key].push( val );
    … … var wpThemes; 
    188181                }
    189182        }
    190183
    191         Query = function() {
    192                 this.failedRequest = false;
    193                 this.querying = false;
    194                 this.page = inputs.startPage;
    195         }
    196        
    197         $.extend( Query.prototype, {
    198                 ready: function() {
    199                         return !( this.querying || this.failedRequest );
    200                 },
    201                 ajax: function( callback ) {
    202                         var self = this,
    203                         query = {
    204                                 action: 'fetch-list',
    205                                 tab: inputs.tab,
    206                                 paged: this.page,
    207                                 s: inputs.search,
    208                                 type: inputs.type,
    209                                 _ajax_fetch_list_nonce: inputs.nonce,
    210                                 'features[]': inputs.features,
    211                                 'list_args': list_args
    212                         };
    213 
    214                         this.querying = true;
    215                         $.get( ajaxurl, query, function(r) {
    216                                 self.page++;
    217                                 self.querying = false;
    218                                 self.failedRequest = !r;
    219                                 callback( r, query );
    220                         }, "json" );
    221                 }
    222         });
    223 
    224         $(document).ready( wpThemes.init );
     184        $(document).ready( function($) { ThemeScroller.init(); });
    225185
    226186})(jQuery);