WordPress.org
Get WordPress

Make WordPress Core

Changeset 32258


Ignore:
Timestamp:
04/21/2015 10:40:23 PM (9 years ago)
Author:
azaozz
Message:

Revert editing of video embed parameters in the media modal, [31620] and [31626] for now. Plan on revisiting in 4.3.
Props iseulde. Fixes #31139, fixes #32006.

Location:
trunk/src
Files:
7 edited

Legend:

Unmodified
Added
Removed

  • trunk/src/wp-admin/includes/ajax-actions.php

    r32079 r32258  
    27152715
    27162716    $shortcode = wp_unslash( $_POST['shortcode'] );
    2717 
    2718     preg_match( '/' . get_shortcode_regex() . '/s', $shortcode, $matches );
    2719     $atts = shortcode_parse_atts( $matches[3] );
    2720     if ( ! empty( $matches[5] ) ) {
    2721         $url = $matches[5];
    2722     } elseif ( ! empty( $atts['src'] ) ) {
    2723         $url = $atts['src'];
    2724     }
     2717    $url = str_replace( '[embed]', '', str_replace( '[/embed]', '', $shortcode ) );
    27252718
    27262719    $parsed = false;
    … …  
    27292722    $wp_embed->return_false_on_fail = true;
    27302723
    2731     if ( is_ssl() && 0 === strpos( $url, 'http://' ) ) {
     2724    if ( is_ssl() && preg_match( '%^\\[embed[^\\]]*\\]http://%i', $shortcode ) ) {
    27322725        // Admin is ssl and the user pasted non-ssl URL.
    27332726        // Check if the provider supports ssl embeds and use that for the preview.
    … …  
    27822775
    27832776    wp_send_json_success( array(
    2784         'body' => $parsed,
    2785         'attr' => $wp_embed->last_attr
     2777        'body' => $parsed
    27862778    ) );
    27872779}

  • trunk/src/wp-includes/class-wp-embed.php

    r31620 r32258  
    1212    public $usecache = true;
    1313    public $linkifunknown = true;
    14     public $last_attr = array();
    15     public $last_url = '';
    1614
    1715    /**
    … …  
    137135        }
    138136
    139         $this->last_url = $url;
    140 
    141         if ( empty( $url ) ) {
    142             $this->last_attr = $attr;
     137
     138        if ( empty( $url ) )
    143139            return '';
    144         }
    145140
    146141        $rawattr = $attr;
    147142        $attr = wp_parse_args( $attr, wp_embed_defaults( $url ) );
    148 
    149         $this->last_attr = $attr;
    150143
    151144        // kses converts & into & and we need to undo this

  • trunk/src/wp-includes/css/media-views.css

    r32125 r32258  
    19821982}
    19831983
    1984 .embed-link-settings .setting {
    1985     display: none;
    1986 }
    1987 
    1988 .embed-link-dimensions:after {
    1989     content: '';
    1990     display: block;
    1991     clear: both;
    1992 }
    1993 
    1994 .embed-link-dimensions .width,
    1995 .embed-link-dimensions .height {
    1996     float: left;
    1997     width: 125px;
    1998     clear: none;
    1999     margin-right: 10px;
    2000 }
    2001 
    2002 .embed-link-dimensions input {
    2003     width: auto;
    2004     max-width: 110px;
    2005 }
    2006 
    20071984.image-details .embed-media-settings .setting {
    20081985    float: none;
    … …  
    20512028    display: block;
    20522029    width: 200px;
    2053     max-width: 100%;
    20542030    font-size: 13px;
    20552031    line-height: 24px;

  • trunk/src/wp-includes/js/mce-view.js

    r32255 r32258  
    819819            var media = wp.media.embed,
    820820                frame = media.edit( text, this.url ),
    821                 self = this,
    822                 events = 'change:url change:width change:height';
     821                self = this;
    823822
    824823            this.pausePlayers();
    825824
    826             frame.state( 'embed' ).props.on( events, function( model, url ) {
     825            frame.state( 'embed' ).props.on( 'change:url', function( model, url ) {
    827826                if ( url && model.get( 'url' ) ) {
    828827                    frame.state( 'embed' ).metadata = model.toJSON();
    … …  
    833832                var data = frame.state( 'embed' ).metadata;
    834833
    835                 if ( self.url && ! data.width ) {
     834                if ( self.url ) {
    836835                    update( data.url );
    837836                } else {

  • trunk/src/wp-includes/js/media-views.js

    r32228 r32258  
    45364536        this.spinner = $('<span class="spinner" />');
    45374537        this.$el.append( this.spinner[0] );
    4538         this.listenTo( this.model, 'change:url change:width change:height', this.updateoEmbed );
     4538        this.listenTo( this.model, 'change:url', this.updateoEmbed );
    45394539    },
    45404540
    … …  
    45554555
    45564556    fetch: function() {
    4557         var embed;
    4558 
    45594557        // check if they haven't typed in 500 ms
    45604558        if ( $('#embed-url-field').val() !== this.model.get('url') ) {
    … …  
    45624560        }
    45634561
    4564         embed = new wp.shortcode({
    4565             tag: 'embed',
    4566             attrs: _.pick( this.model.attributes, [ 'width', 'height', 'src' ] ),
    4567             content: this.model.get('url')
    4568         });
    4569 
    45704562        wp.ajax.send( 'parse-embed', {
    45714563            data : {
    45724564                post_ID: wp.media.view.settings.post.id,
    4573                 shortcode: embed.string()
     4565                shortcode: '[embed]' + this.model.get('url') + '[/embed]'
    45744566            }
    45754567        } )
    … …  
    45794571
    45804572    renderFail: function () {
    4581         this.$( '.setting' ).hide().filter( '.link-text' ).show();
     4573        this.$( '.link-text' ).show();
    45824574    },
    45834575
    45844576    renderoEmbed: function( response ) {
    4585         var html = ( response && response.body ) || '',
    4586             attr = {},
    4587             opts = { silent: true };
    4588 
    4589         this.$( '.setting' ).hide()
    4590             .filter( '.link-text' )[ html ? 'hide' : 'show' ]();
    4591 
    4592         if ( response && response.attr ) {
    4593             attr = response.attr;
    4594 
    4595             _.each( [ 'width', 'height' ], function ( key ) {
    4596                 var $el = this.$( '.setting.' + key ),
    4597                     value = attr[ key ];
    4598 
    4599                 if ( value ) {
    4600                     this.model.set( key, value, opts );
    4601                     $el.show().find( 'input' ).val( value );
    4602                 } else {
    4603                     this.model.unset( key, opts );
    4604                     $el.hide().find( 'input' ).val( '' );
    4605                 }
    4606             }, this );
     4577        var html = ( response && response.body ) || '';
     4578
     4579        if ( html ) {
     4580            this.$('.embed-container').show().find('.embed-preview').html( html );
    46074581        } else {
    4608             this.model.unset( 'height', opts );
    4609             this.model.unset( 'width', opts );
    4610         }
    4611 
    4612         this.$('.embed-container').show().find('.embed-preview').html( html );
     4582            this.renderFail();
     4583        }
    46134584    }
    46144585});

  • trunk/src/wp-includes/js/media/views/embed/link.js

    r32101 r32258  
    2020        this.spinner = $('<span class="spinner" />');
    2121        this.$el.append( this.spinner[0] );
    22         this.listenTo( this.model, 'change:url change:width change:height', this.updateoEmbed );
     22        this.listenTo( this.model, 'change:url', this.updateoEmbed );
    2323    },
    2424
    … …  
    3939
    4040    fetch: function() {
    41         var embed;
    42 
    4341        // check if they haven't typed in 500 ms
    4442        if ( $('#embed-url-field').val() !== this.model.get('url') ) {
    … …  
    4644        }
    4745
    48         embed = new wp.shortcode({
    49             tag: 'embed',
    50             attrs: _.pick( this.model.attributes, [ 'width', 'height', 'src' ] ),
    51             content: this.model.get('url')
    52         });
    53 
    5446        wp.ajax.send( 'parse-embed', {
    5547            data : {
    5648                post_ID: wp.media.view.settings.post.id,
    57                 shortcode: embed.string()
     49                shortcode: '[embed]' + this.model.get('url') + '[/embed]'
    5850            }
    5951        } )
    … …  
    6355
    6456    renderFail: function () {
    65         this.$( '.setting' ).hide().filter( '.link-text' ).show();
     57        this.$( '.link-text' ).show();
    6658    },
    6759
    6860    renderoEmbed: function( response ) {
    69         var html = ( response && response.body ) || '',
    70             attr = {},
    71             opts = { silent: true };
     61        var html = ( response && response.body ) || '';
    7262
    73         this.$( '.setting' ).hide()
    74             .filter( '.link-text' )[ html ? 'hide' : 'show' ]();
    75 
    76         if ( response && response.attr ) {
    77             attr = response.attr;
    78 
    79             _.each( [ 'width', 'height' ], function ( key ) {
    80                 var $el = this.$( '.setting.' + key ),
    81                     value = attr[ key ];
    82 
    83                 if ( value ) {
    84                     this.model.set( key, value, opts );
    85                     $el.show().find( 'input' ).val( value );
    86                 } else {
    87                     this.model.unset( key, opts );
    88                     $el.hide().find( 'input' ).val( '' );
    89                 }
    90             }, this );
     63        if ( html ) {
     64            this.$('.embed-container').show().find('.embed-preview').html( html );
    9165        } else {
    92             this.model.unset( 'height', opts );
    93             this.model.unset( 'width', opts );
     66            this.renderFail();
    9467        }
    95 
    96         this.$('.embed-container').show().find('.embed-preview').html( html );
    9768    }
    9869});

  • trunk/src/wp-includes/media-template.php

    r32055 r32258  
    819819        <div class="embed-container" style="display: none;">
    820820            <div class="embed-preview"></div>
    821         </div>
    822         <div class="embed-link-dimensions">
    823             <label class="setting width">
    824                 <span><?php _e( 'Maximum Width' ); ?></span>
    825                 <input type="text" class="alignment" data-setting="width" />
    826             </label>
    827             <label class="setting height">
    828                 <span><?php _e( 'Maximum Height' ); ?></span>
    829                 <input type="text" class="alignment" data-setting="height" />
    830             </label>
    831821        </div>
    832822    </script>
Note: See TracChangeset for help on using the changeset viewer.