プロジェクト

全般

プロフィール

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

完了

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

Shouta Kashiwagi さんがほぼ13年前に追加. 12年以上前に更新.

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

100%

予定工数:

説明

説明

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


関連するチケット 3 (0件未完了3件完了)

関連している OpenPNE 3 - Bug(バグ) #2797: スマートフォンUIの顔写真サイズを適正にInvalid(無効)2012-02-13

操作
関連している OpenPNE 3 - Bug(バグ) #3195: ImageMagickによる画像生成に失敗するWon't fix(対応せず)Youichi Kimura2012-09-11

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

操作

Shouta Kashiwagi さんが12年以上前に更新

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

Shouta Kashiwagi さんが12年以上前に更新

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

Shouta Kashiwagi さんが12年以上前に更新

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

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

Shouta Kashiwagi さんが12年以上前に更新

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

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);

Shouta Kashiwagi さんが12年以上前に更新

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

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

Shouta Kashiwagi さんが12年以上前に更新

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

Yuya Watanabe さんが12年以上前に更新

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

Yuma Sakata さんが12年以上前に更新

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

テストOKです。

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