Skip to content

Imagick を利用するサーバーで画像のアップロードが正常にできるよう修正#54

Merged
big2men merged 2 commits into
masterfrom
feature/fix-imagick-uploading
Feb 27, 2017
Merged

Imagick を利用するサーバーで画像のアップロードが正常にできるよう修正#54
big2men merged 2 commits into
masterfrom
feature/fix-imagick-uploading

Conversation

@big2men

@big2men big2men commented Feb 25, 2017

Copy link
Copy Markdown
Member

概要

  • 画像編集を Imagick を使って行うサーバーでテーマ編集から画像のアップロードができない問題に対応
  • 主にロリポップサーバーが対象サーバーとなる
  • Imagick を使わないサーバー(Xserver など)には影響が無い更新

原因と対処法について

Imagick::readImage に渡すパスが相対パスになっていると unable to open image という例外が発生し、処理が中断されてしまっていたようだ。
渡すパスを絶対パスにすることで正常に動作することを確認した。
以前は動いていたことから、 Imagick のバージョンによっては相対パスでもいけていたようだ。(未確認)

@big2men big2men added the bug label Feb 25, 2017
@big2men

big2men commented Feb 25, 2017

Copy link
Copy Markdown
Member Author

セルフマージします。
予告:HAIK TEAM によりマージされます。
セルフマージしました

@ishimaru130

Copy link
Copy Markdown

動作確認の報告です。

ロリポップスタンダードプラン、PHP5.6モジュール版
テーマ「haik_seed」でアイキャッチ画像、ロゴ画像が
アップロード出来ませんでしたが、今回の修正の
plugin/skin_customizer/UploadHandler.php
を上書きアップロードしたところ、アイキャッチ画像、ロゴ画像が
アップロード出来るようになりました。

@big2men big2men force-pushed the feature/fix-imagick-uploading branch from 1e9ab6e to 9e9b0dc Compare February 27, 2017 07:59
@big2men big2men force-pushed the feature/fix-imagick-uploading branch from 9e9b0dc to 884cb91 Compare February 27, 2017 08:00
@big2men big2men merged commit 63c440c into master Feb 27, 2017
@big2men big2men deleted the feature/fix-imagick-uploading branch February 27, 2017 08:00
@big2men

big2men commented Feb 27, 2017

Copy link
Copy Markdown
Member Author

@ishimaru130 ありがとうございます!リリースいたしました。

@umihoshi

Copy link
Copy Markdown

beta4のアップデート版は出ないのですか?

@big2men

big2men commented Feb 28, 2017

Copy link
Copy Markdown
Member Author

@umihoshi もうお気づきかもですが、リンク置いておきますね。
https://github.com/open-qhm/qhm/releases/tag/7.0.0.beta.4

@umihoshi

Copy link
Copy Markdown

これで既存サイトを上書きしてもいいんですか?

@ghost

ghost commented Feb 28, 2017

Copy link
Copy Markdown

@umihoshi
上書き用のデータですね!ちょっとまだ用意できてないので、今日中にはアップできると思います。またこちらで通知しますね。

@ghost

ghost commented Feb 28, 2017

Copy link
Copy Markdown

@umihoshi
遅くなりましたが、上書き用の最新データを用意しました。
#7 ←こちらよりどうぞ

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants