WordPress.org

Make WordPress Core

Changeset 26269


Ignore:
Timestamp:
11/19/2013 11:09:01 AM (7 years ago)
Author:
SergeyBiryukov
Message:

Fix JSHint errors in gallery.js.

props dougwollison.
fixes #25999.

File:
1 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/wp-admin/js/gallery.js

    r21592 r26269  
     1/* global unescape, getUserSetting, setUserSetting */
     2
    13jQuery(document).ready(function($) {
    2     var gallerySortable, gallerySortableInit, w, desc = false;
     4    var gallerySortable, gallerySortableInit, sortIt, clearAll, w, desc = false;
    35
    46    gallerySortableInit = function() {
     
    911            distance: 2,
    1012            handle: 'div.filename',
    11             stop: function(e, ui) {
     13            stop: function() {
    1214                // When an update has occurred, adjust the order for each item
    1315                var all = $('#media-items').sortable('toArray'), len = all.length;
     
    1820            }
    1921        } );
    20     }
     22    };
    2123
    2224    sortIt = function() {
     
    2628            $(this).val(order);
    2729        });
    28     }
     30    };
    2931
    3032    clearAll = function(c) {
    3133        c = c || 0;
    32         $('.menu_order_input').each(function(){
    33             if ( this.value == '0' || c ) this.value = '';
     34        $('.menu_order_input').each( function() {
     35            if ( this.value === '0' || c ) {
     36                this.value = '';
     37            }
    3438        });
    35     }
    36 
    37     $('#asc').click(function(){desc = false; sortIt(); return false;});
    38     $('#desc').click(function(){desc = true; sortIt(); return false;});
    39     $('#clear').click(function(){clearAll(1); return false;});
    40     $('#showall').click(function(){
     39    };
     40
     41    $('#asc').click( function() {
     42        desc = false;
     43        sortIt();
     44        return false;
     45    });
     46    $('#desc').click( function() {
     47        desc = true;
     48        sortIt();
     49        return false;
     50    });
     51    $('#clear').click( function() {
     52        clearAll(1);
     53        return false;
     54    });
     55    $('#showall').click( function() {
    4156        $('#sort-buttons span a').toggle();
    4257        $('a.describe-toggle-on').hide();
     
    4560        return false;
    4661    });
    47     $('#hideall').click(function(){
     62    $('#hideall').click( function() {
    4863        $('#sort-buttons span a').toggle();
    4964        $('a.describe-toggle-on').show();
     
    6176
    6277        $('#save-all, #gallery-settings').show();
    63         if ( typeof w.tinyMCE != 'undefined' && w.tinyMCE.activeEditor && ! w.tinyMCE.activeEditor.isHidden() ) {
     78        if ( typeof w.tinyMCE !== 'undefined' && w.tinyMCE.activeEditor && ! w.tinyMCE.activeEditor.isHidden() ) {
    6479            wpgallery.mcemode = true;
    6580            wpgallery.init();
     
    89104        var t = this, li, q, i, it, w = t.getWin();
    90105
    91         if ( ! t.mcemode ) return;
     106        if ( ! t.mcemode ) {
     107            return;
     108        }
    92109
    93110        li = ('' + document.location.search).replace(/^\?/, '').split('&');
     
    98115        }
    99116
    100         if (q.mce_rdomain)
     117        if ( q.mce_rdomain ) {
    101118            document.domain = q.mce_rdomain;
     119        }
    102120
    103121        // Find window & API
     
    115133    setup : function() {
    116134        var t = this, a, ed = t.editor, g, columns, link, order, orderby;
    117         if ( ! t.mcemode ) return;
     135        if ( ! t.mcemode ) {
     136            return;
     137        }
    118138
    119139        t.el = ed.selection.getNode();
    120140
    121         if ( t.el.nodeName != 'IMG' || ! ed.dom.hasClass(t.el, 'wpGallery') ) {
    122             if ( (g = ed.dom.select('img.wpGallery')) && g[0] ) {
     141        if ( t.el.nodeName !== 'IMG' || ! ed.dom.hasClass(t.el, 'wpGallery') ) {
     142            if ( ( g = ed.dom.select('img.wpGallery') ) && g[0] ) {
    123143                t.el = g[0];
    124144            } else {
    125                 if ( getUserSetting('galfile') == '1' ) t.I('linkto-file').checked = "checked";
    126                 if ( getUserSetting('galdesc') == '1' ) t.I('order-desc').checked = "checked";
    127                 if ( getUserSetting('galcols') ) t.I('columns').value = getUserSetting('galcols');
    128                 if ( getUserSetting('galord') ) t.I('orderby').value = getUserSetting('galord');
     145                if ( getUserSetting('galfile') === '1' ) {
     146                    t.I('linkto-file').checked = 'checked';
     147                }
     148                if ( getUserSetting('galdesc') === '1' ) {
     149                    t.I('order-desc').checked = 'checked';
     150                }
     151                if ( getUserSetting('galcols') ) {
     152                    t.I('columns').value = getUserSetting('galcols');
     153                }
     154                if ( getUserSetting('galord') ) {
     155                    t.I('orderby').value = getUserSetting('galord');
     156                }
    129157                jQuery('#insert-gallery').show();
    130158                return;
     
    144172            orderby = a.match(/orderby=['"]([^'"]+)['"]/i);
    145173
    146             if ( link && link[1] ) t.I('linkto-file').checked = "checked";
    147             if ( order && order[1] ) t.I('order-desc').checked = "checked";
    148             if ( columns && columns[1] ) t.I('columns').value = ''+columns[1];
    149             if ( orderby && orderby[1] ) t.I('orderby').value = orderby[1];
     174            if ( link && link[1] ) {
     175                t.I('linkto-file').checked = 'checked';
     176            }
     177            if ( order && order[1] ) {
     178                t.I('order-desc').checked = 'checked';
     179            }
     180            if ( columns && columns[1] ) {
     181                t.I('columns').value = '' + columns[1];
     182            }
     183            if ( orderby && orderby[1] ) {
     184                t.I('orderby').value = orderby[1];
     185            }
    150186        } else {
    151187            jQuery('#insert-gallery').show();
     
    157193
    158194        if ( ! t.mcemode || ! t.is_update ) {
    159             s = '[gallery'+t.getSettings()+']';
     195            s = '[gallery' + t.getSettings() + ']';
    160196            t.getWin().send_to_editor(s);
    161197            return;
    162198        }
    163199
    164         if (t.el.nodeName != 'IMG') return;
    165 
    166         all = ed.dom.decode(ed.dom.getAttrib(t.el, 'title'));
     200        if ( t.el.nodeName !== 'IMG' ) {
     201            return;
     202        }
     203
     204        all = ed.dom.decode( ed.dom.getAttrib( t.el, 'title' ) );
    167205        all = all.replace(/\s*(order|link|columns|orderby)=['"]([^'"]+)['"]/gi, '');
    168206        all += t.getSettings();
     
    185223        }
    186224
    187         if ( I('columns').value != 3 ) {
    188             s += ' columns="'+I('columns').value+'"';
     225        if ( I('columns').value !== 3 ) {
     226            s += ' columns="' + I('columns').value + '"';
    189227            setUserSetting('galcols', I('columns').value);
    190228        }
    191229
    192         if ( I('orderby').value != 'menu_order' ) {
    193             s += ' orderby="'+I('orderby').value+'"';
     230        if ( I('orderby').value !== 'menu_order' ) {
     231            s += ' orderby="' + I('orderby').value + '"';
    194232            setUserSetting('galord', I('orderby').value);
    195233        }
Note: See TracChangeset for help on using the changeset viewer.