Skip to content

[WIP] v7.0.0リリース(PHP7対応)#7

Closed
ghost wants to merge 21 commits into
masterfrom
corresponding-php7
Closed

[WIP] v7.0.0リリース(PHP7対応)#7
ghost wants to merge 21 commits into
masterfrom
corresponding-php7

Conversation

@ghost

@ghost ghost commented Jul 17, 2016

Copy link
Copy Markdown

概要

HAIK v7.0.0リリースに向けた修正です。
主に対応するのは以下の内容です。

  • PHP7への対応
  • QHM表記を削除

TODO

配布用Zipファイル

ご自身のサイトでbeta版を試したい場合は、こちらのZipファイルを解凍し、qhm-7.0.0.beta(もしくはqhm-7.0.0.beta-update)というフォルダの中身をFTPソフトでアップロードしてお使いください。

既存のQHMをバージョンアップする場合は、beta版のファイルをアップロードする前に、必ず現在のサイトのフルバックアップを作成してください。

新規にPHP7対応版を設置する場合

Zipファイルをダウンロード(qhm-7.0.0.beta.4.zip)

既存のQHMをバージョンアップする場合

Zipファイルをダウンロード(qhm-7.0.0.beta.4-update.zip)

@ghost ghost changed the title Update source code for PHP7 [WIP] PHP7対応 Jul 17, 2016
@ghost ghost changed the title [WIP] PHP7対応 [WIP] v6.1.0リリース(PHP7対応) Jul 17, 2016
@ghost ghost changed the title [WIP] v6.1.0リリース(PHP7対応) [WIP] v7.0.0リリース(PHP7対応) Sep 15, 2016
@umihoshi

umihoshi commented Sep 26, 2016

Copy link
Copy Markdown

初めましてうみほしです。
haikをずっと使っています。
PHP7対応版が出ていることを
haikフォーラム
http://haikforum.qhmtips.com/pforum/pforum.php
で知り、さっそくテスト用サイトで試してみました。
結果はフォーラムで報告していますが、まとめると、
(1)編集画面に入れない。
 修正版と入れ替え解決。attach.inc.phpの不具合。
(2)問い合わせフォームのエラー。
 修正版と入れ替え解決。qform.inc.phpの不具合。
(3)「人気の○件」でページ除外が機能しない。
 フォーラムの方の助言で修正解決。popular.inc.phpの不具合。

 今のところベータ版で見つかったのは以上の3件です。
 テスト環境はエックスサーバーのPHP7です。haik6.0.4からの上書きアップデートです。普段は閉鎖しているテスト用サイトです。デザインはflatとfabricで試しました。

もしよろしければフォーラムの方も閲覧いただければ幸いです。
haikの新版リリースを楽しみしています。

@umihoshi

Copy link
Copy Markdown

フォーラムで報告のあった修正コメントをここにも上げておきます。

(1)キラーページが動かない。
 KILLERPAGE2の不具合ではないか。
lib/convert_html.php で使っている関数の splitがPHP7で廃止に
なったのが原因ではないか。
(参考)
http://php.net/manual/ja/function.split.php

lib/convert_html.php で split を使っている2箇所、
splitを explode に書き換えると動くのではというコメントです。

(2)popular.inc.phpの不具合。
PHP7でeregが使えなくなったので、preg_matchに関数を置き換えていますが、その場合に検索文字列を半角の「/」で囲む必要があるそうです。
ベータ版のpopular.inc.php
55行目の
if (($except != '' && preg_match($except, $page)) ||

if (($except != '' && preg_match('/'.$except.'/', $page)) ||
とすれば良い。
 私が書き直して使ったところ正常になりました。

 以上です。

@ghost

ghost commented Sep 28, 2016

Copy link
Copy Markdown
Author

@umihoshi さん

ご報告ありがとうございます。
また、haikフォーラムを立ち上げてくださった方がいらっしゃるのですね。
この度は大変ご迷惑をおかけいたしました。

有志の方々によるフォーラム立ち上げは、本当に心強いです。


ご報告の問題点をまとめると、
以下の2点の不具合があるということでよろしいでしょうか?

  1. 「人気の○件」表示ができない
  2. KILLERPAGE2が動かない

また、それぞれについてフォーラムで議論され、修正方法が提示されている状況ということですね。
フォーラムでの議論と、 @umihoshi さんのコメントを参考に、修正をさせていただきますね。

アップデートを行うたびに、
できるだけhaikフォーラムの方に私からもコメントをさせていただこうと思います。
今後ともよろしくお願いいたします。

@umihoshi

Copy link
Copy Markdown

つながりができてうれしく思います。
今のところ上記の2つのプラグインはFiles changedのviewにあるソースでも不具合が出ます。
フォーラムのいしまるさんの指摘ではちょっと書き直せば良さそうです。

@umihoshi

umihoshi commented Oct 1, 2016

Copy link
Copy Markdown

PHP7対応外の要望も忘れないように書いておきます。旧フォーラムでも要望が出ていました。
Qブログのアーカイブ表示が現在「月ごと」にしかできないので、何年分ものリストが長大に表示されて大変不便です。
#qblog_archives
の仕様に「年ごとの表示」とか「○○年以前」とまとめて表示できる機能を追加してほしいです。

@ghost

ghost commented Oct 4, 2016

Copy link
Copy Markdown
Author

@umihoshi Qブログのアーカイブ表示についての要望は、Issueに上げておきました。
#12 ←こちら

@ghost

ghost commented Oct 4, 2016

Copy link
Copy Markdown
Author

ダウンロードできるzipファイルを最新の状態に更新しました。
#11 #9 #8 への対応済みのデータとなります。

Keishi Tanaka added 3 commits October 5, 2016 05:40
@ghost

ghost commented Oct 17, 2016

Copy link
Copy Markdown
Author

編集メニュー内の「open-qhm.net」の表記を削除し、代わりにスマホプレビューのリンクを表示させるようにする変更を反映させました。

@umihoshi

umihoshi commented Oct 17, 2016

Copy link
Copy Markdown

7.0.0betaのバグ報告です。
ブログのメニュー編集で、編集後のプレビューを行うと、ブログのメニューが表示されずに、トップページのメニューが表示されます。ブログメニューの更新は正常にできます。ブログメニューのプレビューができない障害です。

@umihoshi

umihoshi commented Oct 20, 2016

Copy link
Copy Markdown

バグかfacebookの仕様かは不明です。
haik上のフェイスブックいいねボタンが、httpsでは表示できなくなります。httpでは表示できます。
グーグル+とTwitterはそのような症状は出ていません。

@ghost

ghost commented Oct 23, 2016

Copy link
Copy Markdown
Author

@umihoshi ご報告ありがとうございます。バグとしてリストアップさせていただきます。

@umihoshi

umihoshi commented Nov 2, 2016

Copy link
Copy Markdown

既存サイトに7.0.0betaをそのまま適応した方のエラー報告です。

編集のボタンを押すと
「www.・・・・・・・ ページは機能していません
www.・・・・・・・・では現在このリクエストを処理できません。」

となります。編集できないエラーです。
 このバグはPHP5.6に戻すと出なくなるので、PHP7への対応ができていないことが原因でしょう。
正式リリースでは解消されていることを期待します。

@umihoshi

umihoshi commented Nov 13, 2016

Copy link
Copy Markdown

Google ChromeとWindows10Pro環境です。7.0.0beta.

編集にログインしてから,ブログのページに移動すると,ログイン状態が解除されます。再度ログインすると編集可能になります。
 その後他のページに移動しても同様の現象が出ました。フロントページから他ページへ移動するとログインが解消されます。これは最初の立ち上げの一回のみの現象のようですがよく分かりません。何度も遭遇しています。

@umihoshi

Copy link
Copy Markdown

7betaのバグ報告です。
編集メニューのツールにあるFWD3が機能していません。掲示板の他の方からも同様の報告がありました。

@umihoshi

umihoshi commented Nov 22, 2016

Copy link
Copy Markdown

haikのバグ報告です。
Googleの仕様変更(APIキー導入)にHAIKが対応していないので、gmapが動作しません。googleの仕様変更以後のドメインで発生する障害です。HAIKの対応が必要です。
Forumでの調査結果は、
「htmlで貼り付けてOKの人とNGが出る人との環境の差は、GoogleのURLの問題というのがわかりました。
HAIKに最初から埋め込まれているマップJavaの指定URLが問題で、htmlでjavaを埋め込むと、URLどうしが干渉してしまう事があり、エラーが出ていました。
干渉の有無はHPの構成やらなんやらに左右されるみたいです。」
ということです。

@umihoshi

umihoshi commented Nov 23, 2016

Copy link
Copy Markdown

7beta,6.0.4共通の問題点です。
サイト印刷を行うとリンクの内容(アドレスが)がぞろぞろと印刷されてしまい、画面で見た通りに印刷できません。
CSSの改造である程度対処できましたが、デフォルトで印刷対応してほしいです。

@umihoshi

umihoshi commented Nov 28, 2016

Copy link
Copy Markdown

haik_seed(7beta)での印刷追加実験結果です。
Qブログの表紙の印刷で、グリッド表示にするとサムネイル画像が文字になってしまい、印刷されません。
リスト表示だとブログサムネイル画像が正常に印刷できます。
もう一点は、タグクラウドの文字の大小が無視され、同じ大きさの文字で印刷されてしまいます。
以上2点も画面表示通りに印刷できないバグとして報告しておきます。

@ghost

ghost commented Nov 28, 2016

Copy link
Copy Markdown
Author

@umihoshi さん

ご連絡が遅くなりました。
いつもご報告ありがとうございます。

Githubには不具合や要望を報告する仕組みとして Issues という機能があります。
Issuesには、掲示板のような感覚で、不具合・要望1件につき1スレッド作って投稿していただけると嬉しいです。


※ なお、現在のこのページは、Pull Requestといって、修正したプログラムについてプログラマー同士でコードレビューやフィードバックを行う目的のページです。

@ghost ghost locked and limited conversation to collaborators Nov 28, 2016
@ghost ghost added the v7 label Nov 28, 2016
@ghost

ghost commented Dec 6, 2016

Copy link
Copy Markdown
Author

#22 , #25 で報告された脆弱性にも対応しました。
v7.0.0.beta.1としてリリースしました。

@ghost

ghost commented Dec 22, 2016

Copy link
Copy Markdown
Author

#40 の修正を反映しました。
v7.0.0.beta.2 としてリリースしました。

Keishi Tanaka and others added 3 commits January 13, 2017 16:42
* Remove dead hosts in default ping hosts

* Fix cannot update qblog_enable_ping to true via setting page
@ghost ghost closed this Apr 1, 2017
@ghost ghost deleted the corresponding-php7 branch April 1, 2017 12:18
This pull request was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants