comparation output

Signed-off-by: Juan Pablo Garcia Ripa <sarabadu@gmail.com>
jimp-dev · Jul 26, 2023 · 18e33f3 · 18e33f3
1 parent 620d271
commit 18e33f3
Show file tree

Hide file tree

Showing 16 changed files with 32 additions and 10 deletions.
diff --git a/packages/core/src/utils/image-bitmap.js b/packages/core/src/utils/image-bitmap.js
@@ -3,13 +3,16 @@ import FileType from "file-type";
 import { throwError } from "@jimp/utils";
 import EXIFParser from "exif-parser";
 // modify-exif
+import modifyEXIF from "modify-exif";
 
 import * as constants from "../constants";
 import * as MIME from "./mime";
 import promisify from "./promisify";
 
 const EXIF_TAGS = {
   ORIENTATION: 0x0112, // decimal: 274
+  IMAGE_WIDTH: 0x0100, // decimal: 256
+  IMAGE_HEIGHT: 0x0101, // decimal: 257
 };
 
 async function getMIMEFromBuffer(buffer, path) {
@@ -132,15 +135,18 @@ function transformBitmap(img, width, height, transformation) {
     }
   }
 
+  img.bitmap.data = modifyEXIF(data, (_data) => {
+    console.log(_data);
+    _data["0th"][274] = 1;
+    _data["0th"][256] = width;
+    _data["0th"][257] = height;
+    return _data;
+  });
+
   img.bitmap.width = width;
   img.bitmap.height = height;
 
-  // img.bitmap.data = modifyEXIF(data, (_data) => {
-  //   _data["0th"][274] = 1;
-  //   return _data;
-  // });
-
-  img.bitmap.data = data;
+  // img.bitmap.data = data;
 
   img._exif = EXIFParser.create(data).parse();
 }

diff --git a/packages/jimp/test/Landscape-output-2-PR.jpg b/packages/jimp/test/Landscape-output-2-PR.jpg
diff --git a/packages/jimp/test/Landscape-output-3-PR.jpg b/packages/jimp/test/Landscape-output-3-PR.jpg
diff --git a/packages/jimp/test/Landscape-output-4-PR.jpg b/packages/jimp/test/Landscape-output-4-PR.jpg
diff --git a/packages/jimp/test/Landscape-output-5-PR.jpg b/packages/jimp/test/Landscape-output-5-PR.jpg
diff --git a/packages/jimp/test/Landscape-output-6-PR.jpg b/packages/jimp/test/Landscape-output-6-PR.jpg
diff --git a/packages/jimp/test/Landscape-output-7-PR.jpg b/packages/jimp/test/Landscape-output-7-PR.jpg
diff --git a/packages/jimp/test/Landscape-output-8-PR.jpg b/packages/jimp/test/Landscape-output-8-PR.jpg
diff --git a/packages/jimp/test/Portrait-output-2-PR.jpg b/packages/jimp/test/Portrait-output-2-PR.jpg
diff --git a/packages/jimp/test/Portrait-output-3-PR.jpg b/packages/jimp/test/Portrait-output-3-PR.jpg
diff --git a/packages/jimp/test/Portrait-output-4-PR.jpg b/packages/jimp/test/Portrait-output-4-PR.jpg
diff --git a/packages/jimp/test/Portrait-output-5-PR.jpg b/packages/jimp/test/Portrait-output-5-PR.jpg
diff --git a/packages/jimp/test/Portrait-output-6-PR.jpg b/packages/jimp/test/Portrait-output-6-PR.jpg
diff --git a/packages/jimp/test/Portrait-output-7-PR.jpg b/packages/jimp/test/Portrait-output-7-PR.jpg
diff --git a/packages/jimp/test/Portrait-output-8-PR.jpg b/packages/jimp/test/Portrait-output-8-PR.jpg
diff --git a/packages/jimp/test/exif-rotation.test.js b/packages/jimp/test/exif-rotation.test.js
@@ -15,8 +15,16 @@ describe("EXIF orientation", () => {
       orientedImg.write(
         getTestDir(__dirname) + `/Landscape-output-${orientation}-PR.jpg`
       );
-      expect(orientedImg.getWidth()).toBe(regularImg.getWidth());
-      expect(orientedImg.getHeight()).toBe(regularImg.getHeight());
+
+      if (orientation > 4) {
+        // 5, 6, 7, 8 dimmensions are swapped
+        expect(orientedImg.getWidth()).toBe(regularImg.getHeight());
+        expect(orientedImg.getHeight()).toBe(regularImg.getWidth());
+      } else {
+        expect(orientedImg.getWidth()).toBe(regularImg.getWidth());
+        expect(orientedImg.getHeight()).toBe(regularImg.getHeight());
+      }
+
       expect(Jimp.distance(regularImg, orientedImg)).toBeLessThan(0.07);
 
       expect(getExifOrientation(orientedImg)).toBe(
@@ -33,8 +41,16 @@ describe("EXIF orientation", () => {
       orientedImg.write(
         getTestDir(__dirname) + `/Portrait-output-${orientation}-PR.jpg`
       );
-      expect(orientedImg.getWidth()).toBe(regularImg.getWidth());
-      expect(orientedImg.getHeight()).toBe(regularImg.getHeight());
+
+      if (orientation > 4) {
+        // 5, 6, 7, 8 dimmensions are swapped
+        expect(orientedImg.getWidth()).toBe(regularImg.getHeight());
+        expect(orientedImg.getHeight()).toBe(regularImg.getWidth());
+      } else {
+        expect(orientedImg.getWidth()).toBe(regularImg.getWidth());
+        expect(orientedImg.getHeight()).toBe(regularImg.getHeight());
+      }
+
       expect(Jimp.distance(regularImg, orientedImg)).toBeLessThan(0.07);
 
       expect(getExifOrientation(orientedImg)).toBe(