Project

General

Profile

Backport(バックポート) #3677

リサイズ後の画像サイズが荒い

Added by 誠二 天重 about 6 years ago. Updated about 6 years ago.

Status:
Fixed(完了)
Priority:
Normal(通常)
Assignee:
Target version:
Start date:
2012-05-24
Due date:
% Done:

100%


Description

Overview (現象)

Bug(バグ) #1931: 透過PNG画像が透過されずに背景が黒くなってしまう - OpenPNE 3 - OpenPNE Issue Tracking System
https://redmine.openpne.jp/issues/1931

の対策後、画像をリサイズした際に画質が荒くなるようになった。

Causes (原因)

#1931imagecopyresampled() の代わりに imagecopyresized() をコールするように変更されたことが原因だが、この変更は truecolor 画像に対してはおこなう必要がなかった。

Way to fix (修正内容)

#1931 でおこなわれた変更をより好ましい形に改善する形で、以下のような対処を実施した。

1. truecolor 時のみ PEAR::Image_Transform のオプションの scaleMethod を指定する (= imagecopyresampled() の代わりに imagecopyresized() をコールする)
2. save() 前の crop() はこれまでにリサイズをおこなわなかったときにのみ実行する

詳しくは https://redmine.openpne.jp/issues/3062#note-5 の記載内容を参考のこと

スクリーンショット.png View (42.2 KB) isao sano, 2014-09-10 17:32


Related issues

Copied from OpenPNE 3 - Bug(バグ) #3062: リサイズ後の画像サイズが荒い Won't fix(対応せず) 2012-05-24

Associated revisions

Revision 1e334378 (diff)
Added by Kousuke Ebihara about 6 years ago

changed to set scaleMethod option only for non-truecolor image (e.g. well-used gif images) (refs #3677, BP from #3062)

Conflicts:
lib/plugins/sfImageHandlerPlugin/lib/image/generator/sfImageGeneratorImageTransform.php

Revision 0b6c92db (diff)
Added by Kousuke Ebihara about 6 years ago

changed not to call crop() method twice (refs #3677, BP from #3062)

History

#1 Updated by 誠二 天重 about 6 years ago

#2 Updated by 誠二 天重 about 6 years ago

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

#3 Updated by 誠二 天重 about 6 years ago

  • Target version changed from OpenPNE 3.8.x to OpenPNE 3.8.13

#4 Updated by Shinichi Urabe about 6 years ago

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

#5 Updated by isao sano about 6 years ago

単体試験完了しました。

スクリーンショット添付しておきます。
左が修正後、右が修正前

#6 Updated by Shinichi Urabe about 6 years ago

  • Status changed from Pending Merge(マージ待ち) to Fixed(完了)
  • % Done changed from 80 to 100

Also available in: Atom PDF