プロジェクト

全般

プロフィール

Bug(バグ) #3356

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

Youichi Kimuraほぼ11年前に追加. ほぼ7年前に更新.

ステータス:
Won't fix(対応せず)
優先度:
Normal(通常)
担当者:
対象バージョン:
開始日:
2013-06-06
期日:
進捗率:

0%

3.6 で発生するか:
No (いいえ)
3.8 で発生するか:
Yes (はい)

説明

現象

画像付きの日記を日記の表示画面 (/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 とする。

関連するチケット

関連している OpenPNE 3 - Enhancement(機能追加・改善) #2890: デフォルトで生成されるキャッシュ画像を正方形に切り取る Fixed(完了) 2012-03-22
関連している OpenPNE 3 - Backport(バックポート) #3363: 日記やアルバムなどに投稿された画像のサムネイル表示が全て正方形に切り抜かれる問題 Fixed(完了) 2013-06-06

関係しているリビジョン

リビジョン f0be644f (差分)
Youichi Kimuraほぼ11年前に追加

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

履歴

#1 Youichi Kimuraほぼ11年前に更新

  • 説明 を更新 (diff)

#2 Youichi Kimuraほぼ11年前に更新

  • 説明 を更新 (diff)

手元の環境で bbebcb5a 54a5a24d 4ff9ae53 の各コミットをrevertして動作確認したところ、日記において以前と同じ (OpenPNE3.6と同等な) 挙動をすることを確認しました。

#3 Youichi Kimuraほぼ11年前に更新

手嶋さん と確認の上、次の通り対応する予定です。

  • OpenPNE3.8 から追加されたメンバー画像の正方形化を維持する
  • 日記やアルバム等など他のプラグインに影響を与えないよう、正方形化しない従来のサムネイルを選択できるようにする
  • 正方形化を行ったサムネイルと行わなかったサムネイルを、キャッシュURLによって区別するようにする
    • (例) /cache/img/png/w120_h120/hogehoge.png と /cache/img/png/w120_h120/sq_hogehoge.png

#4 Youichi Kimuraほぼ11年前に更新

note-3 の正方形化を有効にした場合のURLについて、下記のように変更する

(変更前) /cache/img/png/w120_h120/sq_hogehoge.png
(変更後) /cache/img/png/w120_h120_sq/hogehoge.png

理由は hogehoge の部分に当たる file テーブルの name にはアンダーバーを含んだ名前が存在する可能性があるため。

#5 Youichi Kimuraほぼ11年前に更新

  • 説明 を更新 (diff)

#6 Youichi Kimuraほぼ11年前に更新

  • ステータスNew(新規) から Pending Review(レビュー待ち) に変更
  • 進捗率0 から 50 に変更

更新履歴 f0be644f44ef7d51fb8d310f5789557dc49e60c8 で適用されました。

#7 Chiharu Nakajimaほぼ11年前に更新

  • 3.6 で発生するかUnknown (未調査) から No (いいえ) に変更
  • 3.8 で発生するかUnknown (未調査) から Yes (はい) に変更

#8 Youichi Kimura10年以上前に更新

更新履歴 25c8164c7eff503ab70358a9435efae41cc17262 で適用されました。

#9 Youichi Kimura10年以上前に更新

上記 25c8164c7eff503ab70358a9435efae41cc17262 は stable-3.8.x ブランチに取り込んだものです。
コミットメッセージの修正ミスで一時このチケットに紐付いてしまいましたが、正しくは #3363 に対するコミットです。

#10 Kousuke Ebihara10年以上前に更新

  • ステータスPending Review(レビュー待ち) から Pending Testing(テスト待ち) に変更
  • 進捗率50 から 70 に変更

#12 isao sanoほぼ7年前に更新

  • ステータスPending Testing(テスト待ち) から Won't fix(対応せず) に変更
  • 進捗率70 から 0 に変更

OpenPNE 3.8.7 にて対応済みであったため、対応せずとします。

他の形式にエクスポート: Atom PDF