comparation output

Signed-off-by: Juan Pablo Garcia Ripa <[email protected]>

Author: Sarabadu

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();
 }

packages/jimp/test/Landscape-output-2-PR.jpg

@@ -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(