Ticket #21811: trac-21811-image-edit-view-01.patch

src/wp-admin/js/image-edit.js

@@ -5 +5 @@
         iasapi : {},
         hold : {},
         postid : '',
+        _view : {},
 
         intval : function(f) {
                 return f | 0;
@@ -287 +288 @@
                 });
         },
 
-        open : function(postid, nonce) {
+        open : function( postid, nonce, view ) {
+                _view = view;
+
                 var data, elem = $('#image-editor-' + postid), head = $('#media-head-' + postid),
                         btn = $('#imgedit-open-btn-' + postid), spin = btn.siblings('.spinner');
 
@@ -397 +400 @@
 
                 this.iasapi = {};
                 this.hold = {};
-                $('#image-editor-' + postid).fadeOut('fast', function() {
-                        $('#media-head-' + postid).fadeIn('fast');
-                        $(this).empty();
-                });
+
+                // If we've loaded the editor in the context of a Media Modal, then switch to the previous view,
+                // whatever that might have been.
+                if ( _view && 'object' == typeof _view ){
+                        _view.controller.setState( _view.controller.lastState() );
+                }
+
+                // In case we are not accessing the image editor in the context of a View, close the editor the old-skool way
+                else {
+                        $('#image-editor-' + postid).fadeOut('fast', function() {
+                                $('#media-head-' + postid).fadeIn('fast');
+                                $(this).empty();
+                        });
+                }
+
+
         },
 
         notsaved : function(postid) {

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

@@ -1062 +1062 @@
         });
 
         /**
+         * wp.media.controller.EditImage
+         *
+         * @constructor
+         * @augments wp.media.controller.State
+         * @augments Backbone.Model
+         */
+        media.controller.EditImage = media.controller.State.extend({
+                defaults: {
+                        id: 'edit-image',
+                        url: '',
+                        menu: false,
+                        content: 'edit-image',
+                        syncSelection: true
+                },
+
+                activate: function() {
+                        this.syncSelection();
+                },
+
+                syncSelection: function() {
+                        var selection = this.get('selection'),
+                                manager = this.frame._selection;
+
+                        if ( ! this.get('syncSelection') || ! manager || ! selection ) {
+                                return;
+                        }
+
+                        // If the selection supports multiple items, validate the stored
+                        // attachments based on the new selection's conditions. Record
+                        // the attachments that are not included; we'll maintain a
+                        // reference to those. Other attachments are considered in flux.
+                        if ( selection.multiple ) {
+                                selection.reset( [], { silent: true });
+                                selection.validateAll( manager.attachments );
+                                manager.difference = _.difference( manager.attachments.models, selection.models );
+                        }
+
+                        // Sync the selection's single item with the master.
+                        selection.single( manager.single );
+                }
+        });
+
+        /**
          * wp.media.controller.Embed
          *
          * @constructor
@@ -1740 +1783 @@
                                         menu:    'gallery'
                                 }),
 
-                                new media.controller.GalleryAdd()
+                                new media.controller.GalleryAdd(),
+
+                                new media.controller.EditImage( { selection: options.selection } )
                         ]);
 
 
@@ -1768 +1813 @@
 
                                 content: {
                                         'embed':          'embedContent',
+                                        'edit-image':     'editImageContent',
                                         'edit-selection': 'editSelectionContent'
                                 },
 
@@ -1866 +1912 @@
                         this.content.set( view );
                 },
 
+                editImageContent: function() {
+                        var selection = this.state().get('selection'),
+                                view = new media.view.EditImage( { model: selection.single() } ).render();
+
+                        this.content.set( view );
+
+                        // after bringing in the frame, load the actual editor via an ajax call
+                        view.loadEditor();
+                },
+
                 // Toolbars
 
                 /**
@@ -2037 +2093 @@
                         media.view.MediaFrame.Select.prototype.bindHandlers.apply( this, arguments );
                         this.on( 'menu:create:image-details', this.createMenu, this );
                         this.on( 'content:render:image-details', this.renderImageDetailsContent, this );
+                        this.on( 'content:render:edit-image', this.editImageContent, this );
                         this.on( 'menu:render:image-details', this.renderMenu, this );
                         this.on( 'toolbar:render:image-details', this.renderImageDetailsToolbar, this );
                         // override the select toolbar
@@ -2060 +2117 @@
                                         toolbar: 'replace',
                                         priority:  80,
                                         displaySettings: true
-                                })
+                                }),
+                                new media.controller.EditImage( { image: this.image } )
                         ]);
                 },
 
@@ -2100 +2158 @@
 
                 },
 
+                editImageContent: function() {
+                        var attachment = this.state().get('image').attachment,
+                                view;
+
+                        if ( ! attachment ) {
+                                return;
+                        }
+
+                        view = new media.view.EditImage({
+                                controller: this,
+                                model: attachment
+                        }).render();
+
+                        this.content.set( view );
+
+                        // after bringing in the frame, load the actual editor via an ajax call
+                        view.loadEditor();
+
+                },
+
                 renderImageDetailsToolbar: function() {
                         this.toolbar.set( new media.view.Toolbar({
                                 controller: this,
@@ -4895 +4973 @@
                         }
                 },
 
-                editAttachment: function() {
-                        this.$el.addClass('needs-refresh');
+                editAttachment: function( event ) {
+                        event.preventDefault();
+                        this.controller.setState( 'edit-image' );
                 },
                 /**
                  * @param {Object} event
@@ -4906 +4985 @@
                         event.preventDefault();
                         this.model.fetch();
                 }
+
         });
 
         /**
@@ -5180 +5260 @@
                 className: 'image-details',
                 template:  media.template('image-details'),
 
+                events: {
+                        'click .edit-attachment': 'editAttachment'
+                },
+
                 initialize: function() {
                         // used in AttachmentDisplay.prototype.updateLinkTo
                         this.options.attachment = this.model.attachment;
@@ -5219 +5303 @@
                 resetFocus: function() {
                         this.$( '.caption textarea' ).focus();
                         this.$( '.embed-image-settings' ).scrollTop( 0 );
+                },
+
+                editAttachment: function( event ) {
+                        event.preventDefault();
+                        this.controller.setState( 'edit-image' );
                 }
         });
+
+
+        media.view.EditImage = media.View.extend({
+                className: 'image-editor',
+                template: media.template('image-editor'),
+
+                initialize: function() {
+
+                        this.editor = window.imageEdit;
+                        media.View.prototype.initialize.apply( this, arguments );
+                },
+
+                prepare: function() {
+                        return this.model.toJSON();
+                },
+
+                render: function() {
+                        media.View.prototype.render.apply( this, arguments );
+                        return this;
+                },
+
+                loadEditor: function() {
+                        this.editor.open( this.model.get('id'), this.model.get('nonces').edit, this );
+                }
+
+        });
+
 }(jQuery));

src/wp-includes/media-template.php

@@ -508 +508 @@
                                 <div class="thumbnail">
                                         <img src="{{ data.model.url }}" draggable="false" />
                                 </div>
+                                <# if ( data.attachment ) { #>
+                                        <input type="button" class="edit-attachment button" value="<?php esc_attr_e( 'Edit Image' ); ?>" />
+                                <# } #>
 
                                 <div class="setting url">
                                         <?php // might want to make the url editable if it isn't an attachment ?>
@@ -597 +600 @@
                         </div>
                 </div>
         </script>
+
+        <script type="text/html" id="tmpl-image-editor">
+                <div id="media-head-{{{ data.id }}}"></div>
+                <div id="image-editor-{{{ data.id }}}"></div>
+        </script>
         <?php
 
         /**

src/wp-includes/media.php

@@ -1869 +1869 @@
                 'nonces'      => array(
                         'update' => false,
                         'delete' => false,
+                        'edit'   => false
                 ),
                 'editLink'   => false,
         );
 
         if ( current_user_can( 'edit_post', $attachment->ID ) ) {
                 $response['nonces']['update'] = wp_create_nonce( 'update-post_' . $attachment->ID );
+                $response['nonces']['edit'] = wp_create_nonce( 'image_editor-' . $attachment->ID );
                 $response['editLink'] = get_edit_post_link( $attachment->ID, 'raw' );
         }
 

src/wp-includes/script-loader.php

@@ -385 +385 @@
         // To enqueue media-views or media-editor, call wp_enqueue_media().
         // Both rely on numerous settings, styles, and templates to operate correctly.
         $scripts->add( 'media-views',  "/wp-includes/js/media-views$suffix.js",  array( 'utils', 'media-models', 'wp-plupload', 'jquery-ui-sortable' ), false, 1 );
-        $scripts->add( 'media-editor', "/wp-includes/js/media-editor$suffix.js", array( 'shortcode', 'media-views' ), false, 1 );
+        $scripts->add( 'media-editor', "/wp-includes/js/media-editor$suffix.js", array( 'shortcode', 'media-views', 'image-edit' ), false, 1 );
         $scripts->add( 'mce-view', "/wp-includes/js/mce-view$suffix.js", array( 'shortcode', 'media-models' ), false, 1 );
 
         if ( is_admin() ) {