Project

General

Profile

Backport(バックポート) #3363

日記やアルバムなどに投稿された画像のサムネイル表示が全て正方形に切り抜かれる問題

Added by Chiharu Nakajima almost 6 years ago. Updated almost 6 years ago.

Status:
Fixed(完了)
Priority:
Normal(通常)
Target version:
Start date:
2013-06-06
Due date:
% Done:

100%


Description

現象

画像付きの日記を日記の表示画面 (/diary/:id) などでサムネイル表示した際に、投稿された画像の縦横比に関わらず 長辺が切り落とされた正方形 で表示されてしまう。
この挙動は OpenPNE3.8 から追加されたメンバー画像の正方形化 (#2890) が他方にも影響したバグであると推測して調査を行う。

調査時に注意すべき点

このバグに関する調査の注意点として、 #3357 の影響によりサムネイル画像のキャッシュ (web/cache/ 以下に生成される) は ./symfony cc コマンドでは削除されないため、挙動の変化を確認する際には手動でこのキャッシュを削除する必要がある。

原因

#2890 の変更が sfImageHandlerPlugin を介する全ての画像に影響していることによるもの。

修正内容

  • 正方形化した画像とそうでない画像でURLを区別しキャッシュの混在を避ける
    • /cache/img/png/w120_h120/hogehoge.png (通常の画像)
    • /cache/img/png/w120_h120_sq/hogehoge.png (正方形化する画像)
  • ヘルパー関数 sf_image_path に square オプションを追加し、テンプレートにおいて画像を正方形化した状態で表示するか否かを選択できる状態にする
    • このオプションは、メンバー画像 (nameが m_ から始まる) とコミュニティ画像 (nameが c_ から始まる) についてはデフォルトで true とし、それ以外はデフォルトで false とする。

Related issues

Related to OpenPNE 3 - Bug(バグ) #3356: 日記やアルバムなどに投稿された画像のサムネイル表示が全て正方形に切り抜かれる問題 Won't fix(対応せず) 2013-06-06

Associated revisions

Revision 25c8164c (diff)
Added by Youichi Kimura almost 6 years ago

[sfImageHandlerPlugin] add `square' option for sf_image_path (fixes #3356)

(cherry picked from commit f0be644f44ef7d51fb8d310f5789557dc49e60c8)

History

#1 Updated by Youichi Kimura almost 6 years ago

  • Status changed from New(新規) to Accepted(着手)

#2 Updated by Youichi Kimura almost 6 years ago

  • Status changed from Accepted(着手) to Pending Review(レビュー待ち)
  • % Done changed from 0 to 50

25c8164c7eff503ab70358a9435efae41cc17262 にて stable-3.8.x ブランチに取り込みました。
(コミットメッセージの修正ミスで一時 #3356 に紐付いてしまいました)

#3 Updated by Kousuke Ebihara almost 6 years ago

  • Status changed from Pending Review(レビュー待ち) to Pending Testing(テスト待ち)
  • % Done changed from 50 to 70

#4 Updated by Chiharu Nakajima almost 6 years ago

  • Status changed from Pending Testing(テスト待ち) to Fixed(完了)
  • % Done changed from 70 to 100

試験実施完了しました。

Also available in: Atom PDF