WordPress.org
Get WordPress

Make WordPress Core

Changeset 28748


Ignore:
Timestamp:
06/12/2014 02:48:17 AM (12 years ago)
Author:
azaozz
Message:

wpView:

  • Don't wrap single-line URLs in [embed]. Use them directly in generating a view.
  • If the embedding HTML contains a script, "sandbox" it in an iframe to prevent it from changing the editor DOM.
  • Automatically add toolbar and overlay when needed.
  • Try to embed single-line URLs only if they are pasted in an empty paragraph.

Props avryl, see #28195

Location:
trunk/src/wp-includes
Files:
3 edited

Legend:

Unmodified
Added
Removed

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

    r28689 r28748  
    99window.wp = window.wp || {};
    1010
    11 (function($){
     11( function( $ ) {
     12        'use strict';
     13
    1214        var views = {},
    1315                instances = {},
    … …  
    2830        wp.mce.View = function( options ) {
    2931                options = options || {};
     32                this.type = options.type;
    3033                _.extend( this, _.pick( options, viewOptions ) );
    3134                this.initialize.apply( this, arguments );
    … …  
    3639                getHtml: function() {},
    3740                render: function() {
    38                         var html = this.getHtml();
    39                         // Search all tinymce editor instances and update the placeholders
     41                        this.setContent(
     42                                '<div class="toolbar">' +
     43                                        ( _.isFunction( views[ this.type ].edit ) ? '<div class="dashicons dashicons-edit edit"></div>' : '' ) +
     44                                        '<div class="dashicons dashicons-no-alt remove"></div>' +
     45                                '</div>' +
     46                                '<div class="wpview-content">' +
     47                                        this.getHtml() +
     48                                '</div>' +
     49                                ( this.overlay ? '<div class="wpview-overlay"></div>' : '' ) +
     50                                // The <ins> is used to mark the end of the wrapper div (has to be the last child node).
     51                                // Needed when comparing the content as string for preventing extra undo levels.
     52                                '<ins data-wpview-end="1"></ins>',
     53                                function( self, editor, node ) {
     54                                        $( self ).trigger( 'ready', [ editor, node ] );
     55                                }
     56                        );
     57                },
     58                unbind: function() {},
     59                setContent: function( html, callback, replace ) {
    4060                        _.each( tinymce.editors, function( editor ) {
    4161                                var self = this;
    4262                                if ( editor.plugins.wpview ) {
    43                                         $( editor.getDoc() ).find( '[data-wpview-text="' + this.encodedText + '"]' ).each( function ( i, element ) {
    44                                                 var node = $( element );
    45                                                 // The <ins> is used to mark the end of the wrapper div. Needed when comparing
    46                                                 // the content as string for preventing extra undo levels.
    47                                                 node.html( html ).append( '<ins data-wpview-end="1"></ins>' );
    48                                                 $( self ).trigger( 'ready', element );
    49                                         });
     63                                        $( editor.getBody() )
     64                                        .find( '[data-wpview-text="' + this.encodedText + '"]' )
     65                                        .each( function ( i, element ) {
     66                                                var contentWrap = $( element ).children( '.wpview-content' ),
     67                                                        wrap = element;
     68
     69                                                if ( contentWrap.length ) {
     70                                                        element = contentWrap = contentWrap[0];
     71                                                }
     72
     73                                                if ( _.isString( html ) ) {
     74                                                        if ( replace ) {
     75                                                                element = editor.dom.replace( editor.dom.createFragment( html ), wrap );
     76                                                        } else {
     77                                                                editor.dom.setHTML( element, html );
     78                                                        }
     79                                                } else {
     80                                                        if ( replace ) {
     81                                                                element = editor.dom.replace( html, wrap );
     82                                                        } else {
     83                                                                element.appendChild( html );
     84                                                        }
     85                                                }
     86
     87                                                if ( _.isFunction( callback ) ) {
     88                                                        callback( self, editor, $( element ).children( '.wpview-content' )[0] );
     89                                                }
     90                                        } );
    5091                                }
    5192                        }, this );
    5293                },
    53                 unbind: function() {}
     94                setError: function( message, dashicon ) {
     95                        this.setContent(
     96                                '<div class="wpview-error">' +
     97                                        '<div class="dashicons dashicons-' + ( dashicon ? dashicon : 'no' ) + '"></div>' +
     98                                        '<p>' + message + '</p>' +
     99                                '</div>'
     100                        );
     101                }
    54102        } );
    55103
    … …  
    210258                        if ( ! wp.mce.views.getInstance( encodedText ) ) {
    211259                                viewOptions = options;
     260                                viewOptions.type = viewType;
    212261                                viewOptions.encodedText = encodedText;
    213262                                instance = new view.View( viewOptions );
    … …  
    245294                                result = view.toView( text );
    246295                                viewOptions = result.options;
     296                                viewOptions.type = view.type;
    247297                                viewOptions.encodedText = encodedText;
    248298                                instance = new view.View( viewOptions );
    … …  
    332382
    333383                                return this.template( options );
    334 
    335384                        }
    336385                },
    … …  
    362411
    363412                View: _.extend( {}, wp.media.mixin, {
     413                        overlay: true,
     414
    364415                        initialize: function( options ) {
    365416                                this.players = [];
    … …  
    376427                         * @global MediaElementPlayer
    377428                         *
    378                          * @param {Event} e
     429                         * @param {Event} event
     430                         * @param {Object} editor
    379431                         * @param {HTMLElement} node
    380432                         */
    381                         setPlayer: function(e, node) {
    382                                 // if the ready event fires on an empty node
    383                                 if ( ! node ) {
    384                                         return;
    385                                 }
    386 
     433                        setPlayer: function( event, editor, node ) {
    387434                                var self = this,
    388                                         media,
    389                                         firefox = this.ua.is( 'ff' ),
    390                                         className = '.wp-' +  this.shortcode.tag + '-shortcode';
    391 
    392                                 media = $( node ).find( className );
     435                                        media;
     436
     437                                media = $( node ).find( '.wp-' +  this.shortcode.tag + '-shortcode' );
    393438
    394439                                if ( ! this.isCompatible( media ) ) {
    395440                                        media.closest( '.wpview-wrap' ).addClass( 'wont-play' );
    396                                         if ( ! media.parent().hasClass( 'wpview-wrap' ) ) {
    397                                                 media.parent().replaceWith( media );
    398                                         }
    399441                                        media.replaceWith( '<p>' + media.find( 'source' ).eq(0).prop( 'src' ) + '</p>' );
    400442                                        return;
    401443                                } else {
    402444                                        media.closest( '.wpview-wrap' ).removeClass( 'wont-play' );
    403                                         if ( firefox ) {
     445                                        if ( this.ua.is( 'ff' ) ) {
    404446                                                media.prop( 'preload', 'metadata' );
    405447                                        } else {
    … …  
    509551                View: _.extend( {}, wp.media.mixin, {
    510552                        template:  media.template( 'editor-playlist' ),
     553                        overlay: true,
    511554
    512555                        initialize: function( options ) {
    … …  
    532575                        },
    533576
    534                         setPlaylist: function( event, element ) {
     577                        setPlaylist: function( event, editor, element ) {
    535578                                if ( ! this.data.tracks ) {
    536579                                        return;
    … …  
    635678         * TinyMCE handler for the embed shortcode
    636679         */
     680        wp.mce.embedView = _.extend( {}, wp.media.mixin, {
     681                overlay: true,
     682                initialize: function( options ) {
     683                        this.players = [];
     684                        this.content = options.content;
     685                        this.fetching = false;
     686                        this.parsed = false;
     687                        this.original = options.url || options.shortcode.string();
     688
     689                        if ( options.url ) {
     690                                this.shortcode = '[embed]' + options.url + '[/embed]';
     691                        } else {
     692                                this.shortcode = options.shortcode.string();
     693                        }
     694
     695                        _.bindAll( this, 'setHtml', 'setNode', 'fetch' );
     696                        $( this ).on( 'ready', this.setNode );
     697                },
     698                unbind: function() {
     699                        var self = this;
     700                        _.each( this.players, function ( player ) {
     701                                player.pause();
     702                                self.removePlayer( player );
     703                        } );
     704                        this.players = [];
     705                },
     706                setNode: function () {
     707                        if ( this.parsed ) {
     708                                this.setHtml( this.parsed );
     709                                this.parseMediaShortcodes();
     710                        } else if ( ! this.fetching ) {
     711                                this.fetch();
     712                        }
     713                },
     714                fetch: function () {
     715                        var self = this;
     716
     717                        this.fetching = true;
     718
     719                        wp.ajax.send( 'parse-embed', {
     720                                data: {
     721                                        post_ID: $( '#post_ID' ).val(),
     722                                        content: this.shortcode
     723                                }
     724                        } )
     725                        .done( function( content ) {
     726                                self.fetching = false;
     727
     728                                if ( content.substring( 0, ( '<a href' ).length ) === '<a href' ) {
     729                                        if ( self.type === 'embed' ) {
     730                                                self.setError( self.original + ' failed to embed.', 'admin-media' );
     731                                        } else {
     732                                                self.setContent( self.original, null, true );
     733                                        }
     734                                } else {
     735                                        self.parsed = content;
     736                                        self.setHtml( content );
     737                                }
     738                        } )
     739                        .fail( function() {
     740                                self.fetching = false;
     741                                self.setError( self.original + ' failed to embed due to a server error.', 'admin-media' );
     742                        } );
     743                },
     744                /* jshint scripturl: true */
     745                setHtml: function ( content ) {
     746                        var MutationObserver = window.MutationObserver || window.WebKitMutationObserver || window.MozMutationObserver,
     747                                iframe, iframeDoc, i, resize,
     748                                dom = tinymce.DOM;
     749
     750                        if ( content.indexOf( '<script' ) !== -1 ) {
     751                                iframe = dom.create( 'iframe', {
     752                                        src: tinymce.Env.ie ? 'javascript:""' : '',
     753                                        frameBorder: '0',
     754                                        allowTransparency: 'true',
     755                                        style: {
     756                                                width: '100%',
     757                                                display: 'block'
     758                                        }
     759                                } );
     760
     761                                this.setContent( iframe );
     762                                iframeDoc = iframe.contentWindow.document;
     763
     764                                iframeDoc.open();
     765                                iframeDoc.write(
     766                                        '<!DOCTYPE html>' +
     767                                        '<html>' +
     768                                                '<head>' +
     769                                                        '<meta http-equiv="Content-Type" content="text/html; charset=UTF-8" />' +
     770                                                '</head>' +
     771                                                '<body>' +
     772                                                        content +
     773                                                '</body>' +
     774                                        '</html>'
     775                                );
     776                                iframeDoc.close();
     777
     778                                resize = function() {
     779                                        $( iframe ).height( $( iframeDoc ).height() );
     780                                };
     781
     782                                if ( MutationObserver ) {
     783                                        new MutationObserver( _.debounce( function() {
     784                                                resize();
     785                                        }, 100 ) )
     786                                        .observe( iframeDoc.body, {
     787                                                attributes: true,
     788                                                childList: true,
     789                                                subtree: true
     790                                        } );
     791                                } else {
     792                                        for ( i = 1; i < 6; i++ ) {
     793                                                setTimeout( resize, i * 700 );
     794                                        }
     795                                }
     796                        } else {
     797                                this.setContent( content );
     798                        }
     799
     800                        this.parseMediaShortcodes();
     801                },
     802                parseMediaShortcodes: function () {
     803                        var self = this;
     804                        $( '.wp-audio-shortcode, .wp-video-shortcode', this.node ).each( function ( i, element ) {
     805                                self.players.push( new MediaElementPlayer( element, self.mejsSettings ) );
     806                        } );
     807                },
     808                getHtml: function() {
     809                        return '';
     810                }
     811        } );
     812
    637813        wp.mce.views.register( 'embed', {
    638                 View: _.extend( {}, wp.media.mixin, {
    639                         template: media.template( 'editor-embed' ),
    640                         initialize: function( options ) {
    641                                 this.players = [];
    642                                 this.content = options.content;
    643                                 this.parsed = false;
    644                                 this.shortcode = options.shortcode;
    645                                 _.bindAll( this, 'setHtml', 'setNode', 'fetch' );
    646                                 $( this ).on( 'ready', this.setNode );
    647                         },
    648                         unbind: function() {
    649                                 var self = this;
    650                                 _.each( this.players, function ( player ) {
    651                                         player.pause();
    652                                         self.removePlayer( player );
    653                                 } );
    654                                 this.players = [];
    655                         },
    656                         setNode: function ( e, node ) {
    657                                 this.node = node;
    658                                 if ( this.parsed ) {
    659                                         this.parseMediaShortcodes();
    660                                 } else {
    661                                         this.fetch();
    662                                 }
    663                         },
    664                         fetch: function () {
    665                                 wp.ajax.send( 'parse-embed', {
    666                                         data: {
    667                                                 post_ID: $( '#post_ID' ).val(),
    668                                                 content: this.shortcode.string()
    669                                         }
    670                                 } ).done( this.setHtml );
    671                         },
    672                         setHtml: function ( content ) {
    673                                 this.parsed = content;
    674                                 $( this.node ).html( this.getHtml() );
    675                                 this.parseMediaShortcodes();
    676                         },
    677                         parseMediaShortcodes: function () {
    678                                 var self = this;
    679                                 $( '.wp-audio-shortcode, .wp-video-shortcode', this.node ).each( function ( i, element ) {
    680                                         self.players.push( new MediaElementPlayer( element, self.mejsSettings ) );
    681                                 } );
    682                         },
    683                         getHtml: function() {
    684                                 if ( ! this.parsed ) {
    685                                         return '';
    686                                 }
    687                                 return this.template({ content: this.parsed });
    688                         }
    689                 } ),
    690                 edit: function() {}
     814                View: wp.mce.embedView
    691815        } );
    692816
     817        wp.mce.views.register( 'embedURL', {
     818                toView: function( content ) {
     819                        var re = /(?:^|<p>)(https?:\/\/[^\s"]+?)(?:<\/p>\s*|$)/gi,
     820                                match = re.exec( tinymce.trim( content ) );
     821
     822                        if ( ! match ) {
     823                                return;
     824                        }
     825
     826                        return {
     827                                index: match.index,
     828                                content: match[0],
     829                                options: {
     830                                        url: match[1]
     831                                }
     832                        };
     833                },
     834                View: wp.mce.embedView
     835        } );
     836
    693837}(jQuery));

  • trunk/src/wp-includes/js/tinymce/plugins/wpview/plugin.js

    r28687 r28748  
    159159        // view wrappers.
    160160        editor.on( 'BeforeSetContent', function( event ) {
     161                var node;
     162
    161163                if ( ! event.content ) {
    162164                        return;
    … …  
    167169                }
    168170
     171                node = editor.selection.getNode();
     172
     173                // When a url is pasted, only try to embed it when pasted in an empty paragrapgh.
     174                if ( event.content.match( /^\s*(https?:\/\/[^\s"]+)\s*$/i ) &&
     175                        ( node.nodeName !== 'P' || node.parentNode !== editor.getBody() || ! editor.dom.isEmpty( node ) ) ) {
     176                        return;
     177                }
     178
    169179                event.content = wp.mce.views.toViews( event.content );
    170180        });
    171 
    172         editor.on( 'PastePreProcess', function( event ) {
    173                 if ( event.content.match( /^\s*(https?:\/\/[^\s"]+)\s*$/im ) ) {
    174                         event.content = '[embed]' + event.content + '[/embed]';
    175                 }
    176         } );
    177181
    178182        // When the editor's content has been updated and the DOM has been

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

    r28683 r28748  
    995995
    996996        <script type="text/html" id="tmpl-editor-gallery">
    997                 <div class="toolbar">
    998                         <div class="dashicons dashicons-edit edit"></div><div class="dashicons dashicons-no-alt remove"></div>
    999                 </div>
    1000997                <# if ( data.attachments ) { #>
    1001998                        <div class="gallery gallery-columns-{{ data.columns }}">
    … …  
    10281025
    10291026        <script type="text/html" id="tmpl-editor-audio">
    1030                 <div class="toolbar">
    1031                         <div class="dashicons dashicons-edit edit"></div>
    1032                         <div class="dashicons dashicons-no-alt remove"></div>
    1033                 </div>
    10341027                <?php wp_underscore_audio_template() ?>
    1035                 <div class="wpview-overlay"></div>
    10361028        </script>
    10371029
    10381030        <script type="text/html" id="tmpl-editor-video">
    1039                 <div class="toolbar">
    1040                         <div class="dashicons dashicons-edit edit"></div>
    1041                         <div class="dashicons dashicons-no-alt remove"></div>
    1042                 </div>
    10431031                <?php wp_underscore_video_template() ?>
    1044                 <div class="wpview-overlay"></div>
    10451032        </script>
    10461033
    … …  
    10481035
    10491036        <script type="text/html" id="tmpl-editor-playlist">
    1050                 <div class="toolbar">
    1051                         <div class="dashicons dashicons-edit edit"></div>
    1052                         <div class="dashicons dashicons-no-alt remove"></div>
    1053                 </div>
    10541037                <# if ( data.tracks ) { #>
    10551038                        <div class="wp-playlist wp-{{ data.type }}-playlist wp-playlist-{{ data.style }}">
    … …  
    10631046                                <div class="wp-playlist-prev"></div>
    10641047                        </div>
    1065                         <div class="wpview-overlay"></div>
    10661048                <# } else { #>
    10671049                        <div class="wpview-error">
    … …  
    10741056                <img class="crop-image" src="{{ data.url }}">
    10751057                <div class="upload-errors"></div>
    1076         </script>
    1077 
    1078         <script type="text/html" id="tmpl-editor-embed">
    1079                 <div class="toolbar">
    1080                         <div class="dashicons dashicons-no-alt remove"></div>
    1081                 </div>
    1082                 {{{ data.content }}}
    1083                 <div class="wpview-overlay"></div>
    10841058        </script>
    10851059
Note: See TracChangeset for help on using the changeset viewer.