WordPress.org

Make WordPress Core

Changeset 31613


Ignore:
Timestamp:
03/04/2015 09:00:52 PM (5 years ago)
Author:
azaozz
Message:

PressThis: remove the extra .length tests for strings from press-this.js.
Props stephdau, see #31373.

File:
1 edited

Legend:

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

    r31609 r31613  
    103103            var link = '';
    104104
    105             if ( data._links ) {
    106                 if ( data._links.canonical && data._links.canonical.length ) {
    107                     link = data._links.canonical;
    108                 }
    109             }
    110 
    111             if ( ! link.length && data.u ) {
     105            if ( data._links && data._links.canonical ) {
     106                link = data._links.canonical;
     107            }
     108
     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                }
     
    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                }
     
    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                }
     
    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                }
     
    223221            if ( embeds.length ) {
    224222                $.each( embeds, function ( i, src ) {
    225                     if ( !src || !src.length ) {
     223                    if ( ! src ) {
    226224                        // Skip: no src value
    227225                        return;
     
    258256                    src = checkUrl( src );
    259257
    260                     if ( ! src || ! src.length ) {
     258                    if ( ! src ) {
    261259                        // Skip: no src value
    262260                        return;
     
    370368
    371369            if ( 'img' === type ) {
    372                 if ( ! link || ! link.length ) {
     370                if ( ! link ) {
    373371                    link = src;
    374372                }
     
    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 );
     
    522520         */
    523521        function renderSuggestedContent() {
    524             if ( ! suggestedContentStr || ! suggestedContentStr.length ) {
     522            if ( ! suggestedContentStr ) {
    525523                return;
    526524            }
     
    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 );
     
    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>'
     
    594592            }
    595593
    596             if ( interestingImages && interestingImages.length ) {
     594            if ( interestingImages ) {
    597595                $.each( interestingImages, function ( i, src ) {
    598596                    src = checkUrl( src );
     
    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                }
     
    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
Note: See TracChangeset for help on using the changeset viewer.