Changeset 33338

Timestamp:

07/20/2015 11:55:46 PM (11 years ago)

Author:

iseulde

Message:

TinyMCE: views: make sure gallery captions update

Part props dmsnell.
Fixes #32779.

File:

• trunk/src/wp-includes/js/mce-view.js (modified) (10 diffs)

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

@@ -146 +146 @@
                  * Create a view instance.
                  *
-                 * @param {String} type    The view type.
-                 * @param {String} text    The textual representation of the view.
-                 * @param {Object} options Options.
+                 * @param {String}  type    The view type.
+                 * @param {String}  text    The textual representation of the view.
+                 * @param {Object}  options Options.
+                 * @param {Boolean} force   Recreate the instance. Optional.
                  *
                  * @return {wp.mce.View} The view instance.
                  */
-                createInstance: function( type, text, options ) {
+                createInstance: function( type, text, options, force ) {
                         var View = this.get( type ),
                                 encodedText,
@@ -158 +159 @@
 
                         text = tinymce.DOM.decode( text );
-                        instance = this.getInstance( text );
-
-                        if ( instance ) {
-                                return instance;
+
+                        if ( ! force ) {
+                                instance = this.getInstance( text );
+
+                                if ( instance ) {
+                                        return instance;
+                                }
                         }
 
@@ -217 +221 @@
                  * @param {tinymce.Editor} editor The TinyMCE editor instance the view node is in.
                  * @param {HTMLElement}    node   The view node to update.
-                 */
-                update: function( text, editor, node ) {
+                 * @param {Boolean}        force  Recreate the instance. Optional.
+                 */
+                update: function( text, editor, node, force ) {
                         var instance = this.getInstance( node );
 
                         if ( instance ) {
-                                instance.update( text, editor, node );
+                                instance.update( text, editor, node, force );
                         }
                 },
@@ -236 +241 @@
 
                         if ( instance && instance.edit ) {
-                                instance.edit( instance.text, function( text ) {
-                                        instance.update( text, editor, node );
+                                instance.edit( instance.text, function( text, force ) {
+                                        instance.update( text, editor, node, force );
                                 } );
                         }
@@ -303 +308 @@
                  * Renders all view nodes tied to this view instance that are not yet rendered.
                  *
-                 * @param {String} content The content to render. Optional.
-                 * @param {Boolean} force Rerender all view nodes tied to this view instance.
+                 * @param {String}  content The content to render. Optional.
+                 * @param {Boolean} force   Rerender all view nodes tied to this view instance. Optional.
                  */
                 render: function( content, force ) {
@@ -648 +653 @@
                  *
                  * @param {String} message  The error message to set.
-                 * @param {String} dashicon A dashicon ID (optional). {@link https://developer.wordpress.org/resource/dashicons/}
+                 * @param {String} dashicon A dashicon ID. Optional. {@link https://developer.wordpress.org/resource/dashicons/}
                  */
                 setError: function( message, dashicon ) {
@@ -686 +691 @@
                  * @param {tinymce.Editor} editor The TinyMCE editor instance the view node is in.
                  * @param {HTMLElement}    node   The view node to update.
-                 */
-                update: function( text, editor, node ) {
+                 * @param {Boolean}        force  Recreate the instance. Optional.
+                 */
+                update: function( text, editor, node, force ) {
                         _.find( views, function( view, type ) {
                                 var match = view.prototype.match( text );
@@ -694 +700 @@
                                         $( node ).data( 'rendered', false );
                                         editor.dom.setAttrib( node, 'data-wpview-text', encodeURIComponent( text ) );
-                                        wp.mce.views.createInstance( type, text, match.options ).render();
+                                        wp.mce.views.createInstance( type, text, match.options, force ).render();
                                         editor.focus();
 
@@ -730 +736 @@
 
                 edit: function( text, update ) {
-                        var media = wp.media[ this.type ],
+                        var type = this.type,
+                                media = wp.media[ type ],
                                 frame = media.edit( text );
 
@@ -737 +744 @@
                         _.each( this.state, function( state ) {
                                 frame.state( state ).on( 'update', function( selection ) {
-                                        update( media.shortcode( selection ).string() );
+                                        update( media.shortcode( selection ).string(), type === 'gallery' );
                                 } );
                         } );