From 1059e35da2570c27eb5325ec50e40aba001c248c Mon Sep 17 00:00:00 2001
From: Julia Radzhabova <Julia.Radzhabova@onlyoffice.com>
Date: Mon, 8 Apr 2019 11:04:34 +0300
Subject: [PATCH] [DE] Disable crop button when can't crop image

---
 .../main/app/view/ImageSettings.js            | 39 +++++++++++--------
 1 file changed, 22 insertions(+), 17 deletions(-)

diff --git a/apps/documenteditor/main/app/view/ImageSettings.js b/apps/documenteditor/main/app/view/ImageSettings.js
index d81a40e52..07bd5652a 100644
--- a/apps/documenteditor/main/app/view/ImageSettings.js
+++ b/apps/documenteditor/main/app/view/ImageSettings.js
@@ -264,23 +264,6 @@ define([
             $(this.el).on('click', '#image-advanced-link', _.bind(this.openAdvancedSettings, this));
         },
 
-        onCrop: function(btn, e) {
-            btn.pressed ? this.api.asc_startEditCrop() : this.api.asc_endEditCrop();
-            this.fireEvent('editcomplete', this);
-        },
-
-        onCropMenu: function(menu, item) {
-            if (item.value == 1) {
-                this.api.asc_cropFill();
-            } else if (item.value == 2) {
-                this.api.asc_cropFit();
-            } else {
-                item.checked ? this.api.asc_startEditCrop() : this.api.asc_endEditCrop();
-            }
-
-            this.fireEvent('editcomplete', this);
-        },
-
         _changeCropState: function(state) {
             this.btnCrop.toggle(state, true);
             this.btnCrop.menu.items[0].setChecked(state, true);
@@ -500,6 +483,26 @@ define([
             this.fireEvent('editcomplete', this);
         },
 
+        onCrop: function(btn, e) {
+            if (this.api) {
+                btn.pressed ? this.api.asc_startEditCrop() : this.api.asc_endEditCrop();
+            }
+            this.fireEvent('editcomplete', this);
+        },
+
+        onCropMenu: function(menu, item) {
+            if (this.api) {
+                if (item.value == 1) {
+                    this.api.asc_cropFill();
+                } else if (item.value == 2) {
+                    this.api.asc_cropFit();
+                } else {
+                    item.checked ? this.api.asc_startEditCrop() : this.api.asc_endEditCrop();
+                }
+            }
+            this.fireEvent('editcomplete', this);
+        },
+
         openAdvancedSettings: function(e) {
             if (this.linkAdvanced.hasClass('disabled')) return;
 
@@ -554,6 +557,8 @@ define([
                 });
                 this.linkAdvanced.toggleClass('disabled', disable);
             }
+
+            this.btnCrop.setDisabled(disable || !this.api.asc_canEditCrop());
         },
 
         textSize:       'Size',