Enhancement(機能追加・改善) #2890
デフォルトで生成されるキャッシュ画像を正方形に切り取る
開始日:
2012-03-22
期日:
進捗率:
100%
関連するチケット
関係しているリビジョン
fixed to generate image for square size. (fixes #2890)
fixed to generate image for square size. (fixes #2890)
fixed valiable type in sfImageHandlerPlugin (fixes #2890)
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です。