ファイルアップロード時にファイルのメタ情報を取得する回数を減らす

Description

現状、 get_image_type および check_upload 経由で get_image_info が呼ばれるが、同じファイルの情報を二度も呼ぶのは無駄だし、場合によっては非常に時間がかかるので、 get_image_info で取得した情報を使い回すなりして get_image_info を呼びすぎないようにする。

同時に、ImageInfo は内部で余分なものを呼びすぎるので、必要がないなら ExtractInfo に書き換えて、必要な情報だけ GetValue するようにした方がなおよさそう。

Checklists

None

Attachments

1
  • 13 Mar 2024, 01:38 AM

Activity

Show:

Kenichi IshigakiApril 16, 2024 at 3:35 AM

ありがとうございます。パッチのPRをマージしました。パッチ提供は でするのでこちらは閉じます。

Reiko SatoApril 16, 2024 at 3:09 AM


8.0.2/8.0.3共に問題ないように見えます。

Kenichi IshigakiApril 15, 2024 at 3:28 AM

Kenichi IshigakiApril 15, 2024 at 3:27 AM

ありがとうございます。diff はこちらで追加したつもりでコミット漏れしていました(すみません)。個別に追加でよいとおもいます。

前のコメントに書いた MT8.0.2 向けのパッチに加えて、 https://github.com/movabletype/movabletype-patches/blob/5b9a340f7056331fcec0192fc36c77e9583e6dae/package/8.0.3/MT-8.0.3-support-336.zip の MT8.0.3 向けのパッチの確認もおねがいします。確認内容は同じです。

Taku AmanoApril 15, 2024 at 2:14 AM

ありがとうごいます。

diff/support-336.patch がなかったので追加して(共通の場所に移動したほうがいいか、微妙なところだと思うのですが、そんなに頻繁にあるものでもないのでここはよいかなと思い 8.0.3 の下に追加してしまいました)パッチの作成まで行いました。

どうでしょうか!

Implemented

Details

Assignee

Reporter

Fix versions

Priority

Smart Checklist

Created February 14, 2024 at 9:20 AM
Updated April 16, 2024 at 3:35 AM
Resolved April 16, 2024 at 3:35 AM