Skip to:
現状、 get_image_type および check_upload 経由で get_image_info が呼ばれるが、同じファイルの情報を二度も呼ぶのは無駄だし、場合によっては非常に時間がかかるので、 get_image_info で取得した情報を使い回すなりして get_image_info を呼びすぎないようにする。
同時に、ImageInfo は内部で余分なものを呼びすぎるので、必要がないなら ExtractInfo に書き換えて、必要な情報だけ GetValue するようにした方がなおよさそう。
@Reiko Sato ありがとうございます。パッチのPRをマージしました。パッチ提供は でするのでこちらは閉じます。
@Kenichi Ishigaki 8.0.2/8.0.3共に問題ないように見えます。
あ、ファイルのリンクが違っていました。 https://github.com/movabletype/movabletype-patches/raw/5b9a340f7056331fcec0192fc36c77e9583e6dae/package/8.0.3/MT-8.0.3-support-336.zip ですね。
@Taku Amano ありがとうございます。diff はこちらで追加したつもりでコミット漏れしていました(すみません)。個別に追加でよいとおもいます。
@Reiko Sato 前のコメントに書いた 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 向けのパッチの確認もおねがいします。確認内容は同じです。
@Kenichi Ishigaki ありがとうごいます。
diff/support-336.patch がなかったので追加して(共通の場所に移動したほうがいいか、微妙なところだと思うのですが、そんなに頻繁にあるものでもないのでここはよいかなと思い 8.0.3 の下に追加してしまいました)パッチの作成まで行いました。
どうでしょうか!
現状、 get_image_type および check_upload 経由で get_image_info が呼ばれるが、同じファイルの情報を二度も呼ぶのは無駄だし、場合によっては非常に時間がかかるので、 get_image_info で取得した情報を使い回すなりして get_image_info を呼びすぎないようにする。
同時に、ImageInfo は内部で余分なものを呼びすぎるので、必要がないなら ExtractInfo に書き換えて、必要な情報だけ GetValue するようにした方がなおよさそう。