fix: move initial image data storing to a reasonable place

Fixed fengyuanchen/jquery-cropper#19
fengyuanchen · Jun 21, 2020 · ed89b3e · ed89b3e
1 parent 50547ba
commit ed89b3e
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 1 deletion.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -2,6 +2,7 @@
 
 ## next
 
+- Store the initial image data in the right place for avoiding [side-effect](https://github.com/fengyuanchen/jquery-cropper/issues/19).
 - Fix the issue when the value of the `minContainerWidth/Height` options is `0` (#683).
 
 ## 1.5.7 (May 23, 2020)

diff --git a/src/js/cropper.js b/src/js/cropper.js
@@ -249,6 +249,7 @@ class Cropper {
         naturalHeight,
         aspectRatio: naturalWidth / naturalHeight,
       });
+      this.initialImageData = assign({}, this.imageData);
       this.sizing = false;
       this.sized = true;
       this.build();

diff --git a/src/js/render.js b/src/js/render.js
@@ -105,7 +105,6 @@ export default {
     this.canvasData = canvasData;
     this.limited = (viewMode === 1 || viewMode === 2);
     this.limitCanvas(true, true);
-    this.initialImageData = assign({}, imageData);
     this.initialCanvasData = assign({}, canvasData);
   },