プロジェクト

全般

プロフィール

Enhancement(機能追加・改善) #2890

デフォルトで生成されるキャッシュ画像を正方形に切り取る

Shouta Kashiwagi約12年前に追加. ほぼ12年前に更新.

ステータス:
Fixed(完了)
優先度:
High(高め)
対象バージョン:
開始日:
2012-03-22
期日:
進捗率:

100%


説明

説明

180x180 や 36x36 などのサイズで生成される画像を正方形に切り取り画像生成を行うようにする


関連するチケット

関連している OpenPNE 3 - Bug(バグ) #2797: スマートフォンUIの顔写真サイズを適正に Invalid(無効) 2012-02-13
関連している OpenPNE 3 - Bug(バグ) #3195: ImageMagickによる画像生成に失敗する Won't fix(対応せず) 2012-09-11
関連している OpenPNE 3 - Bug(バグ) #3356: 日記やアルバムなどに投稿された画像のサムネイル表示が全て正方形に切り抜かれる問題 Won't fix(対応せず) 2013-06-06

関係しているリビジョン

リビジョン 4ff9ae53 (差分)
Shouta Kashiwagiほぼ12年前に追加

fixed to generate image for square size. (fixes #2890)

リビジョン 207d349e (差分)
Shouta Kashiwagiほぼ12年前に追加

fixed to generate image for square size. (fixes #2890)

リビジョン 54a5a24d (差分)
Shouta Kashiwagiほぼ12年前に追加

fixed valiable type in sfImageHandlerPlugin (fixes #2890)

リビジョン b842f268 (差分)
Shouta Kashiwagiほぼ12年前に追加

fixed valiable type in sfImageHandlerPlugin (fixes #2890)

履歴

#1 Shouta Kashiwagi約12年前に更新

  • 優先度Normal(通常) から High(高め) に変更

#2 Shouta Kashiwagi約12年前に更新

  • 対象バージョン252 から OpenPNE 3.8beta1 に変更

#3 Shouta Kashiwagiほぼ12年前に更新

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

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

#4 Shouta Kashiwagiほぼ12年前に更新

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

#5 Yuya Watanabeほぼ12年前に更新

  • ステータスPending Review(レビュー待ち) から Rejected(差し戻し) に変更

PHP であるため差し戻す理由として適切かどうか判断に迷いますが,下記コードの ceil() が返す値の型が double になることが気になります.getImageWidth() や getImageHeight() で返す値は int 型であり,crop() で渡す値も int 型であるとされているため,キャストしておいたほうが無難かもしれません.

 59     $srcWidth = $this->transform->getImageWidth();
 60     $srcHeight = $this->transform->getImageHeight();
 61     if ($srcWidth > $srcHeight)
 62     {
 63       $srcX = ceil(($srcWidth - $srcHeight) / 2);
 64       $srcY = 0;
 65       $srcW = $srcHeight;
 66       $srcH = $srcHeight;
 67     }
 68     else
 69     {
 70       $srcX = 0;
 71       $srcY = ceil(($srcHeight - $srcWidth) / 2);
 72       $srcW = $srcWidth;
 73       $srcH = $srcWidth;
 74     }
...
 84       $this->transform->crop($srcW, $srcH, $srcX, $srcY);

#6 Shouta Kashiwagiほぼ12年前に更新

  • ステータスRejected(差し戻し) から Pending Review(レビュー待ち) に変更

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

#7 Shouta Kashiwagiほぼ12年前に更新

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

#8 Yuya Watanabeほぼ12年前に更新

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

#9 Yuma Sakataほぼ12年前に更新

  • ステータスPending Testing(テスト待ち) から Fixed(完了) に変更
  • 進捗率70 から 100 に変更

テストOKです。

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