WordPress.org
Get WordPress

Make WordPress Core

Changeset 31613


Ignore:
Timestamp:
03/04/2015 09:00:52 PM (12 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.