WordPress.org
Get WordPress

Make WordPress Core

Ticket #31373: length-less.diff

File length-less.diff, 6.3 KB (added by stephdau, 10 years ago)

Cleans ip the use of unnecessary ( vartest && vartest.length ) in JS in press-this.php, as Ox did in bookmarklet.

  • src/wp-admin/js/press-this.js

     
    102102                function getCanonicalLink() {
    103103                        var link = '';
    104104
    105                         if ( data._links ) {
    106                                 if ( data._links.canonical && data._links.canonical.length ) {
    107                                         link = data._links.canonical;
    108                                 }
     105                        if ( data._links && data._links.canonical ) {
     106                                link = data._links.canonical;
    109107                        }
    110108
    111                         if ( ! link.length && data.u ) {
     109                        if ( ! link && data.u ) {
    112110                                link = data.u;
    113111                        }
    114112
    115                         if ( ! link.length && data._meta ) {
    116                                 if ( data._meta['twitter:url'] && data._meta['twitter:url'].length ) {
     113                        if ( ! link && data._meta ) {
     114                                if ( data._meta['twitter:url'] ) {
    117115                                        link = data._meta['twitter:url'];
    118                                 } else if ( data._meta['og:url'] && data._meta['og:url'].length ) {
     116                                } else if ( data._meta['og:url'] ) {
    119117                                        link = data._meta['og:url'];
    120118                                }
    121119                        }
    … …  
    132130                        var name = '';
    133131
    134132                        if ( data._meta ) {
    135                                 if ( data._meta['og:site_name'] && data._meta['og:site_name'].length ) {
     133                                if ( data._meta['og:site_name'] ) {
    136134                                        name = data._meta['og:site_name'];
    137                                 } else if ( data._meta['application-name'] && data._meta['application-name'].length ) {
     135                                } else if ( data._meta['application-name'] ) {
    138136                                        name = data._meta['application-name'];
    139137                                }
    140138                        }
    … …  
    155153                        }
    156154
    157155                        if ( ! title && data._meta ) {
    158                                 if ( data._meta['twitter:title'] && data._meta['twitter:title'].length ) {
     156                                if ( data._meta['twitter:title'] ) {
    159157                                        title = data._meta['twitter:title'];
    160                                 } else if ( data._meta['og:title'] && data._meta['og:title'].length ) {
     158                                } else if ( data._meta['og:title'] ) {
    161159                                        title = data._meta['og:title'];
    162                                 } else if ( data._meta.title && data._meta.title.length ) {
     160                                } else if ( data._meta.title ) {
    163161                                        title = data._meta.title;
    164162                                }
    165163                        }
    … …  
    185183                                url      = getCanonicalLink(),
    186184                                siteName = getSourceSiteName();
    187185
    188                         if ( data.s && data.s.length ) {
     186                        if ( data.s ) {
    189187                                text = data.s;
    190188                        } else if ( data._meta ) {
    191                                 if ( data._meta['twitter:description'] && data._meta['twitter:description'].length ) {
     189                                if ( data._meta['twitter:description'] ) {
    192190                                        text = data._meta['twitter:description'];
    193                                 } else if ( data._meta['og:description'] && data._meta['og:description'].length ) {
     191                                } else if ( data._meta['og:description'] ) {
    194192                                        text = data._meta['og:description'];
    195                                 } else if ( data._meta.description && data._meta.description.length ) {
     193                                } else if ( data._meta.description ) {
    196194                                        text = data._meta.description;
    197195                                }
    198196                        }
    … …  
    220218                                interestingEmbeds  = [],
    221219                                alreadySelected    = [];
    222220
    223                         if ( embeds.length ) {
     221                        if ( embeds ) {
    224222                                $.each( embeds, function ( i, src ) {
    225                                         if ( !src || !src.length ) {
     223                                        if ( ! src ) {
    226224                                                // Skip: no src value
    227225                                                return;
    228226                                        }
    … …  
    252250                                interestingImgs  = [],
    253251                                alreadySelected  = [];
    254252
    255                         if ( imgs.length ) {
     253                        if ( imgs ) {
    256254                                $.each( imgs, function ( i, src ) {
    257255                                        src = src.replace( /http:\/\/[\d]+\.gravatar\.com\//, 'https://secure.gravatar.com/' );
    258256                                        src = checkUrl( src );
    259257
    260                                         if ( ! src || ! src.length ) {
     258                                        if ( ! src ) {
    261259                                                // Skip: no src value
    262260                                                return;
    263261                                        }
    … …  
    369367                        link = checkUrl( link );
    370368
    371369                        if ( 'img' === type ) {
    372                                 if ( ! link || ! link.length ) {
     370                                if ( ! link ) {
    373371                                        link = src;
    374372                                }
    375373
    … …  
    419417                                                                .append( $( '<input type="checkbox" name="post_category[]" checked>' ).attr( 'value', newCat.term_id ) ) );
    420418
    421419                                                if ( newCat.parent ) {
    422                                                         if ( ! $ul || ! $ul.length ) {
     420                                                        if ( ! $ul ) {
    423421                                                                $parent = $wrap.find( '#category-' + newCat.parent );
    424422                                                                $ul = $parent.find( 'ul.children:first' );
    425423
    426                                                                 if ( ! $ul.length ) {
     424                                                                if ( ! $ul ) {
    427425                                                                        $ul = $( '<ul class="children">' ).appendTo( $parent );
    428426                                                                }
    429427                                                        }
    … …  
    480478                 */
    481479                function renderStartupNotices() {
    482480                        // Render errors sent in the data, if any
    483                         if ( data.errors && data.errors.length ) {
     481                        if ( data.errors ) {
    484482                                $.each( data.errors, function( i, msg ) {
    485483                                        renderError( msg );
    486484                                } );
    … …  
    521519                 * Render the suggested content, if any
    522520                 */
    523521                function renderSuggestedContent() {
    524                         if ( ! suggestedContentStr || ! suggestedContentStr.length ) {
     522                        if ( ! suggestedContentStr ) {
    525523                                return;
    526524                        }
    527525
    … …  
    547545
    548546                        listContainer.empty();
    549547
    550                         if ( ( interestingEmbeds && interestingEmbeds.length ) || ( interestingImages && interestingImages.length ) ) {
     548                        if ( interestingEmbeds || interestingImages ) {
    551549                                listContainer.append( '<h2 class="screen-reader-text">' + __( 'allMediaHeading' ) + '</h2><ul class="wppt-all-media-list"/>' );
    552550                        }
    553551
    554                         if ( interestingEmbeds && interestingEmbeds.length ) {
     552                        if ( interestingEmbeds ) {
    555553                                $.each( interestingEmbeds, function ( i, src ) {
    556554                                        src = checkUrl( src );
    557555
    … …  
    580578                                                'class': cssClass,
    581579                                                'tabindex': '0'
    582580                                        } ).css( {
    583                                                 'background-image': ( displaySrc.length ) ? 'url(' + displaySrc + ')' : null
     581                                                'background-image': ( displaySrc ) ? 'url(' + displaySrc + ')' : null
    584582                                        } ).html(
    585583                                                '<span class="screen-reader-text">' + __( 'suggestedEmbedAlt' ).replace( '%d', i + 1 ) + '</span>'
    586584                                        ).on( 'click keypress', function ( e ) {
    … …  
    593591                                } );
    594592                        }
    595593
    596                         if ( interestingImages && interestingImages.length ) {
     594                        if ( interestingImages ) {
    597595                                $.each( interestingImages, function ( i, src ) {
    598596                                        src = checkUrl( src );
    599597
    … …  
    748746                        } );
    749747
    750748                        $selector.on( 'blur', function() {
    751                                 var textLength = $( this ).text().length;
    752 
    753                                 if ( ! textLength ) {
     749                                if ( ! $( this ).text() ) {
    754750                                        $placeholder.removeClass('is-hidden');
    755751                                }
    756752                        } );
    … …  
    805801                        // Needs more work, doesn't detect when the other JS changes the value of #tax-input-post_tag
    806802                        $( '#tax-input-post_tag' ).on( 'change', function() {
    807803                                var val =  $( this ).val();
    808                                 $( '#post-option-tags' ).text( ( val.length ) ? val.replace( /,([^\s])/g, ', $1' ) : '' );
     804                                $( '#post-option-tags' ).text( ( val ) ? val.replace( /,([^\s])/g, ', $1' ) : '' );
    809805                        } );
    810806
    811807                        $( window ).on( 'beforeunload.press-this', function() {