Ticket #21785: 21785-doCrop-move.3.diff

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

@@ -322 +322 @@
                         }, data ) ).done( function( resp, status, xhr ) {
                                 model.set( model.parse( resp, xhr ), options );
                         });
+                },
+                crop: function( cropDetails ) {
+                        var model = this;
+
+                        return wp.ajax.post( 'custom-header-crop', {
+                                nonce: this.get('nonces').edit,
+                                id: this.get('id'),
+                                cropDetails: cropDetails
+                        } ).done( function( resp, status, xhr ) {
+                                model.set( model.parse( resp, xhr ) );
+                        });
                 }
         }, {
                 /**

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

@@ -1299 +1299 @@
                 activate: function() {
                         this.frame.on( 'content:create:crop', this.createCropContent, this );
                         this.frame.on( 'close', this.removeCropper, this );
-                        this.set('selection', new Backbone.Collection(this.frame._selection.single));
+                        this.set('selection', new media.model.Selection(this.frame._selection.single));
                 },
 
                 deactivate: function() {
@@ -1308 +1308 @@
 
                 createCropContent: function() {
                         this.cropperView = new wp.media.view.Cropper({controller: this,
-                                        attachment: this.get('selection').first() });
+                                        attachment: this.get('selection').single() });
                         this.cropperView.on('image-loaded', this.createCropToolbar, this);
                         this.frame.content.set(this.cropperView);
 
@@ -1335 +1335 @@
 
                                                 click: function() {
                                                         var self = this,
-                                                                selection = this.controller.state().get('selection').first();
-
-                                                        selection.set({cropDetails: this.controller.state().imgSelect.getSelection()});
+                                                                cropDetails = this.controller.state().imgSelect.getSelection(),
+                                                                selection = this.controller.state().get('selection').single();
 
                                                         this.$el.text(l10n.cropping);
                                                         this.$el.attr('disabled', true);
-                                                        this.controller.state().doCrop( selection ).done( function( croppedImage ) {
+                                                        selection.crop(cropDetails).done( function( croppedImage ) {
                                                                 self.controller.trigger('cropped', croppedImage );
                                                                 self.controller.close();
                                                         }).fail( function() {
@@ -1360 +1359 @@
                                                 priority:   70,
                                                 requires:   { library: false, selection: false },
                                                 click:      function() {
-                                                        var selection = this.controller.state().get('selection').first();
+                                                        var selection = this.controller.state().get('selection').single();
                                                         this.controller.state().cropperView.remove();
                                                         this.controller.trigger('skippedcrop', selection);
                                                         this.controller.close();
@@ -1370 +1369 @@
                         }
 
                         this.frame.toolbar.set( new wp.media.view.Toolbar(toolbarOptions) );
-                },
-
-                doCrop: function( attachment ) {
-                        return wp.ajax.post( 'custom-header-crop', {
-                                nonce: attachment.get('nonces').edit,
-                                id: attachment.get('id'),
-                                cropDetails: attachment.get('cropDetails')
-                        } );
                 }
         });