Skip to content

QBlogのPing送信機能を修正#58

Merged
2 commits merged into
corresponding-php7from
hotfix/qblog-setting
Apr 1, 2017
Merged

QBlogのPing送信機能を修正#58
2 commits merged into
corresponding-php7from
hotfix/qblog-setting

Conversation

@ghost

@ghost ghost commented Apr 1, 2017

Copy link
Copy Markdown

Close #47
Close #48

概要

  • PHP5.6 以降で設定画面においてQBlog の Ping 送信を有効化できない問題を修正
  • 一度のリクエスト内にセッションの更新が複数回あったことが原因
    • plugin_qhmsetting_update_ini はセッション連想配列の読み込み/書き込み/削除を行っている関数のため、複数回呼び出すとセッションファイルに書き出すところで不整合が出そう(適当)
  • デフォルトのPing先に死んだホストがいくつかあり、タイムアウトが発生するため、設定やブログ記事の更新に非常に時間がかかることがあった問題を修正
  • 死んでいるホストをリストから削除した
    • 新規に有効にする分にはいいけど既に保存されているものについてはちょっと保留。。
    • 現在のタイムアウトは 2 秒にしているけど、体感ではもうちょっと長いように感じる

変更ファイル

  • plugin/qblog.inc.php

テスト方法

PHP 5.6 以上で確認すること。

  1. QHMにログイン
  2. 設定へ移動
  3. ブログ設定へ移動
  4. 外部連携タブへ移動
  5. Ping送信「送信する」を選び、保存する
  6. 「有効にしました」とメッセージが出るのを確認できれば 成功1
  7. 一度「設定」トップへ戻る
  8. もう一度「ブログ設定」>「外部連携」タブへ移動
  9. Ping送信が「送信する」になっていれば 成功2

死んだPing先リスト

以下を既に保存している場合、削除推奨。

http://api.my.yahoo.co.jp/RPC2
http://ping.rss.drecom.jp
http://rpc.reader.livedoor.com/ping

タスク

  • レビュー
  • パッチバージョンアップ

@ghost ghost merged commit bbafc2e into corresponding-php7 Apr 1, 2017
@ghost ghost deleted the hotfix/qblog-setting branch April 1, 2017 12:08
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant