resize_image.php の見直し

[nanasess]

ここ近年、回線速度も向上し、サーバーに負荷をかけてまで、毎回縮小画像を生成する必要は薄れてきたと思われます。 少々サイズの大きな画像でも、1度表示してしまえば、ブラウザのキャッシュに乗るので、トラフィックの影響も少ないのではないでしょうか。

この際、何かとトラブルの多い resize_image.php を廃止するのはいかがでしょう。

以下のような CSS を使えば、縦横比を維持したままリサイズできますし。

.image-resize {
    max-width: 64px;
    max-height: 64px;
}

参考  http://kudakurage.hatenadiary.com/entry/20090629/1246245252

-- 追記 CSSだとIE7で問題がありそう。 Feed生成のプラグイン等で、利用するなど、利用価値はあり、2.13での廃止は時期早々と思われます。 いちお、既存の処理も利用可能な形にはしておくが、商品ID等を渡してファイルへのパスをDBから取得する方法を標準としたい。

[poego]

r23005 で作成してみました。
resize_image.phpを呼び出すたびに、DBへのアクセスを行う事がやはり気になったので、フロントのおすすめ商品のテンプレートだけひとまず修正
IE7で問題がないのであれば、resize_image.php 使用箇所をすべてCSSの
max-width: 80px;max-height: 80;
のような形にするほうが良いと思われます。
手元にIE7がないので未検証。

利用箇所は管理画面7箇所、defaultテンプレート7箇所、sphoneテンプレート11箇所

[poego]

表示速度検証（Virtualboxでローカルに作った開発環境）
デフォルトインストール状態でTOPページのお薦め商品画像で検証

• resize _image.php 利用時（パス渡し）
  常に200 1.95s ~1.97s
  
• resize_image.php 利用時（ID渡し）
  常に200 1.74 s~2.08s
  
• CSS利用時
  200 152ms〜204 ms
  304 57 ms〜107ms
  

圧倒的にCSSのほうが早いので、CSSへの置き換えを行なっていきます。 CSSがうまくいかない環境や、resize_imageを使いたい方ように、resize_image自体は残しておきます。

[poego]

r23041 でコミットしました。

[poego]

r23042 で一部間違いを修正

[nanasess]

max-height: 65; となっているようです。

max-height: 65px; が正しい。

[poego]

nanasess への返信

max-height: 65; となっているようです。

max-height: 65px; が正しい。

r23122 で対応しました。

[m_uehara]

・お問い合わせに画像なし商品を登録するとフロントでリンク切れ表示となります。
・購入確認画面で複数配送先を指定している場合、配送先に表示される商品画像に「画像はありません」の画像が表示されております。

ご確認のほどよろしくお願いいたします。

[m_uehara]

r23166 にて、お問い合わせ商品のリンク切れ、購入確認画面の複数配送先の商品画像表示を修正いたしました。