Pre Merge pull request !462 from 聪小陈/N/A

CHANGELOG.md

@@ -1,6 +1,3 @@
-## 3.2.2-rc.1
-- Support EXIF metadata carried by images as display orientation
-
 ## 3.2.2-rc.0
 - Add ImageKnifeComponent to destroy network request interruption
 - Code refactoring during the download of image resources stage

entry/src/main/ets/pages/Index.ets

@@ -224,11 +224,6 @@ struct Index {
 
           });
         })
-        Button($r('app.string.test_exif')).margin({ top: 10 }).onClick(() => {
-          router.push({
-            uri: 'pages/TestImageExif',
-          });
-        })
       }
     } .width('100%')
     .height('100%')

entry/src/main/ets/pages/TestImageExif.ets

@@ -1,92 +0,0 @@
-/*
- * Copyright (C) 2025 Huawei Device Co., Ltd.
- * Licensed under the Apache License, Version 2.0 (the 'License');
- * you may not use this file except in compliance with the License.
- * You may obtain a copy of the License at
- *
- *     http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an 'AS IS' BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
- */
-import { ImageKnifeComponent } from '@ohos/libraryimageknife';
-import fs from '@ohos.file.fs';
-
-@Entry
-@Component
-struct LocalImage {
-  scroller: Scroller = new Scroller;
-
-  build() {
-    Scroll(this.scroller) {
-      Column() {
-        Column() {
-          Text($r('app.string.base_image'))
-            .fontSize(20)
-            .fontWeight(FontWeight.Bold)
-          Row() {
-            Image($rawfile('rotate/rotate.jpg')).width(100).height(100).margin({ right: 10 })
-            ImageKnifeComponent({
-              imageKnifeOption: {
-                loadSrc: $rawfile('rotate/rotate.jpg'),
-                objectFit: ImageFit.Contain
-              }
-            }).width(100).height(100)
-          }
-        }
-        .margin({ bottom: 20 })
-
-        Column() {
-          Text($r('app.string.rotate_mirror'))
-            .fontSize(20)
-            .fontWeight(FontWeight.Bold)
-          Row() {
-            Image($rawfile('rotate/rotate_mirror.jpg')).width(100).height(100).margin({ right: 10 })
-            ImageKnifeComponent({
-              imageKnifeOption: {
-                loadSrc: $rawfile('rotate/rotate_mirror.jpg'),
-                objectFit: ImageFit.Contain
-              }
-            }).width(100).height(100)
-          }
-        }.margin({ bottom: 20 })
-
-        Column() {
-          Text($r('app.string.rotate_rotate90'))
-            .fontSize(20)
-            .fontWeight(FontWeight.Bold)
-          Row() {
-            Image($rawfile('rotate/rotate_rotate90.jpg')).width(100).height(100).margin({ right: 10 })
-            ImageKnifeComponent({
-              imageKnifeOption: {
-                loadSrc: $rawfile('rotate/rotate_rotate90.jpg'),
-                objectFit: ImageFit.Contain
-              }
-            }).width(100).height(100)
-          }
-        }.margin({ bottom: 20 })
-
-        Column() {
-          Text($r('app.string.rotate_mirror_rotate270'))
-            .fontSize(20)
-            .fontWeight(FontWeight.Bold)
-          Row() {
-            Image($rawfile('rotate/rotate_mirror_rotate270.jpg')).width(100).height(100).margin({ right: 10 })
-            ImageKnifeComponent({
-              imageKnifeOption: {
-                loadSrc: $rawfile('rotate/rotate_mirror_rotate270.jpg'),
-                objectFit: ImageFit.Contain
-              }
-            }).width(100).height(100)
-          }
-        }
-      }
-      .width('100%')
-    }
-    .height('100%')
-  }
-}
-

entry/src/main/resources/base/element/string.json

@@ -687,26 +687,6 @@
     {
       "name": "Error_Message",
       "value": "error message"
-    },
-    {
-      "name": "test_exif",
-      "value": "Test display orientation base on the EXIF metadata "
-    },
-    {
-      "name": "base_image",
-      "value": "The image don't carry rotation information"
-    },
-    {
-      "name": "rotate_mirror",
-      "value": "Mirror horizontal"
-    },
-    {
-    "name": "rotate_rotate90",
-    "value": "Rotate 90°"
-    },
-    {
-      "name": "rotate_mirror_rotate270",
-      "value": "Mirror horizontal and rotate 270°"
     }
   ]
 }

entry/src/main/resources/base/profile/main_pages.json

@@ -42,7 +42,6 @@
     "pages/SingleImageCallBack",
     "pages/MultipleImageCallBack",
     "pages/LocalImage",
-    "pages/ErrorMessageDownload",
+    "pages/ErrorMessageDownload"
-    "pages/TestImageExif"
   ]
 }

entry/src/main/resources/zh_CN/element/string.json

@@ -679,26 +679,6 @@
     {
       "name": "Error_Message",
       "value": "错误信息"
-    },
-    {
-      "name": "base_image",
-      "value": "图片不携带旋转信息"
-    },
-    {
-      "name": "test_exif",
-      "value": "测试图片携带的EXIF元数据作为显示方向"
-    },
-    {
-      "name": "rotate_mirror",
-      "value": "水平翻转"
-    },
-    {
-      "name": "rotate_rotate90",
-      "value": "顺时针90°"
-    },
-    {
-      "name": "rotate_mirror_rotate270",
-      "value": "水平翻转后再顺时针270°"
     }
   ]
 }

library/oh-package.json5

@@ -14,7 +14,7 @@
   "main": "index.ets",
   "repository": "https://gitee.com/openharmony-tpc/ImageKnife",
   "type": "module",
-  "version": "3.2.2-rc.1",
+  "version": "3.2.2-rc.0",
   "dependencies": {
     "@ohos/gpu_transform": "^1.0.2"
   },

library/src/main/ets/ImageKnifeLoader.ets

@@ -16,7 +16,6 @@ import {
   CacheStrategy,
   DecodeImageInfo,
   ErrorInfo,
-  FlipRotate,
   ImageKnifeData,
   ImageKnifeRequestSource,
   ImageKnifeRequestWithSource, RequestJobRequest,
@@ -165,14 +164,6 @@ export class ImageKnifeLoader {
     }
     timeInfo.decodeStartTime = Date.now();
 
-    // 获取旋转信息
-    let exif: string | undefined = undefined;
-    await imageSource.getImageProperty(image.PropertyKey.ORIENTATION).then((res)=>{
-      exif = res;
-    }).catch((error: BusinessError)=>{
-      LogUtil.info("The normal image don't have rotation information, " + error.message);
-    })
-
     await imageSource.createPixelMap(decodingOptions)
       .then((pixelmap: PixelMap) => {
         timeInfo.decodeEndTime = Date.now();
@@ -195,17 +186,6 @@ export class ImageKnifeLoader {
     } catch (e) {
       LogUtil.error('PixelMap setTransferDetached failed:' + JSON.stringify(e))
     }
-    // 设置翻转和旋转角度
-    if(exif && exif !== 'Top-left'){
-      let result = ImageKnifeLoader.getOrientation(exif);
-      if(result.horizontal || result.vertical) {
-        resPixelmap?.flipSync(result.horizontal, result.vertical);
-      }
-      if(result.rotate > 0) {
-        resPixelmap?.rotateSync(result.rotate);
-      }
-      LogUtil.log('The normal image set flip , horizontal=' + result.horizontal + ', vertical=' +result.vertical + ', rotate=' + result.rotate);
-    }
 
     //获取各个pixelMap的大小
     if (resPixelmap !== undefined) {
@@ -273,15 +253,6 @@ export class ImageKnifeLoader {
       return
     }
     timeInfo.decodeStartTime = Date.now();
-
-    // 获取旋转信息
-    let exif: string | undefined = undefined;
-    await imageSource.getImageProperty(image.PropertyKey.ORIENTATION).then((res)=>{
-      exif = res;
-    }).catch((error: BusinessError)=>{
-      LogUtil.info("Svg image don't have rotation information, " + error.message);
-    })
-
     await imageSource.createPixelMap(opts)
       .then((pixelmap: PixelMap) => {
         timeInfo.decodeEndTime = Date.now();
@@ -289,17 +260,6 @@ export class ImageKnifeLoader {
         imageSource.release()
         try {
           resPixelmap.setTransferDetached(true)
-          // 设置翻转和旋转角度
-          if(exif && exif !== 'Top-left'){
-            let result = ImageKnifeLoader.getOrientation(exif);
-            if(result.horizontal || result.vertical) {
-              resPixelmap?.flipSync(result.horizontal, result.vertical);
-            }
-            if(result.rotate > 0) {
-              resPixelmap?.rotateSync(result.rotate);
-            }
-            LogUtil.log('Svg image set flip , horizontal=' + result.horizontal + ', vertical=' +result.vertical + ', rotate=' + result.rotate);
-          }
         } catch (e) {
           LogUtil.error('PixelMap setTransferDetached failed:' + JSON.stringify(e))
         }
@@ -529,39 +489,4 @@ export class ImageKnifeLoader {
       }
     }
   }
-
-  static getOrientation(orientation: string | undefined){
-    let horizontal: boolean = false;
-    let vertical: boolean = false;
-    let rotate: number= 0;
-    switch (orientation){
-      case 'Top-left': break
-      case 'Top-right':
-        horizontal = true;
-        break;
-      case 'Bottom-left':
-        vertical = true;
-        break
-      case 'Bottom-right':
-        rotate = 180;
-        break;
-      case 'Left-top':
-        horizontal = true;
-        rotate = 270;
-        break
-      case 'Right-top':
-        rotate = 90;
-        break;
-      case 'Left-bottom':
-        rotate = 270;
-        break
-      case 'Right-bottom':
-        horizontal = true;
-        rotate = 90;
-        break;
-    }
-
-    let data: FlipRotate = { horizontal: horizontal, vertical: vertical, rotate:rotate };
-    return data
-  }
 }

library/src/main/ets/model/ImageKnifeData.ets

@@ -158,9 +158,3 @@ export interface RequestJobRequest {
   readTimeout?: number
 }
 
-export interface FlipRotate{
-  horizontal: boolean,
-  vertical: boolean,
-  rotate: number,
-}
-