Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ExifInferfaceの不具合改修版 #36

Merged
merged 4 commits into from
Dec 1, 2023
Merged

Conversation

JangSungChul
Copy link
Collaborator

@JangSungChul JangSungChul commented Nov 29, 2023

issue

#30

  • AndroidでXMPデータが2回作成される問題

変更内容

  • Android版で利用するExifInterfaceの不具合対応
    • EXIF_TAGSにIFD_TIFF_TAGSが2回宣言されている値を1回のみに修正
  • 対象のファイルをデプロイ後にソース元のpacakgeに移動するHookを作成

確認したこと

  • 写真データにXMP情報が1回のみ入っている

スクリーンショット

スクリーンショット 2023-11-29 13 34 08

補足事項

Pluginのみ更新

npx cordova plugin rm cordova-plugin-blackboard-camera && npx cordova plugin add https://github.com/ncdcdev/cordova-plugin-blackboard-camera#fix/exif_xmp_android && npm run build && npx cordova prepare android

PR時のセルフチェック

  • 関数や変数の命名は一目で分かるものになってますか?
  • コメントは適切な量で、誰が見ても分かるコメントになってますか?
  • PR内容に関するテストは書きましたか?
  • PR内容に付随する各種ドキュメントは一緒に修正しましたか?
  • Issue の完了の定義は満たせていますか?

@karuru6225 karuru6225 merged commit cf3843f into master Dec 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants