Project

General

Profile

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

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

Added by Shouta Kashiwagi over 7 years ago. Updated over 7 years ago.

Status:
Fixed(完了)
Priority:
High(高め)
Target version:
Start date:
2012-03-22
Due date:
% Done:

100%


Description

説明

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


Related issues

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

Associated revisions

Revision 4ff9ae53 (diff)
Added by Shouta Kashiwagi over 7 years ago

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

Revision 207d349e (diff)
Added by Shouta Kashiwagi over 7 years ago

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

Revision 54a5a24d (diff)
Added by Shouta Kashiwagi over 7 years ago

fixed valiable type in sfImageHandlerPlugin (fixes #2890)

Revision b842f268 (diff)
Added by Shouta Kashiwagi over 7 years ago

fixed valiable type in sfImageHandlerPlugin (fixes #2890)

History

#1 Updated by Shouta Kashiwagi over 7 years ago

  • Priority changed from Normal(通常) to High(高め)

#2 Updated by Shouta Kashiwagi over 7 years ago

  • Target version changed from 252 to OpenPNE 3.8beta1

#3 Updated by Shouta Kashiwagi over 7 years ago

  • Status changed from New(新規) to Pending Review(レビュー待ち)
  • % Done changed from 0 to 50

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

#4 Updated by Shouta Kashiwagi over 7 years ago

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

#5 Updated by Yuya Watanabe over 7 years ago

  • Status changed from Pending Review(レビュー待ち) to 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 Updated by Shouta Kashiwagi over 7 years ago

  • Status changed from Rejected(差し戻し) to Pending Review(レビュー待ち)

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

#7 Updated by Shouta Kashiwagi over 7 years ago

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

#8 Updated by Yuya Watanabe over 7 years ago

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

#9 Updated by Yuma Sakata over 7 years ago

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

テストOKです。

Also available in: Atom PDF