Project

General

Profile

Backport（バックポート） #3028

PHP5.2.x環境でjson_encodeの呼び出し時にWarningが発生する

Added by Yuma Sakata about 10 years ago. Updated over 1 year ago.

Status:
Fixed（完了）
Priority:
Normal（通常）
Assignee:
Youichi Kimura
Target version:
OpenPNE 3.8.38
Start date:
2012-04-23
Due date:
% Done:

100%

Description

Overview (現象)

Causes (原因)

json_encode の第2引数はPHP5.3.0で追加されたもので、それ以前のバージョンで実行されることを考慮されていなかったことが原因。

参考: PHP: json_encode - Manual

Way to fix (修正内容)

JSON_PRETTY_PRINT が定義されていない場合に json_encode の第2引数を指定しないように修正する。

Related issues

Related to OpenPNE 3 - Bug（バグ） #3026: PHP5.2.x環境でjson_encodeの呼び出し時にWarningが発生する Fixed（完了） 2012-04-23

Associated revisions

Revision 94bb5047 (diff)
Added by Youichi Kimura about 10 years ago

fix compatibility with PHP 5.2.x (fixes #3028, BP from #3026)

History

#1 Updated by Youichi Kimura about 10 years ago

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

更新履歴 94bb50479d37eb9428751883a3b6903ddb8dcbff で適用されました。

#2 Updated by Rimpei Ogawa about 10 years ago

  • Status changed from Pending Review（レビュー待ち） to Rejected（差し戻し）

https://redmine.openpne.jp/issues/3026#note-3
で親チケットを差し戻しています。

#3 Updated by Shouta Kashiwagi about 10 years ago

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

3.8.1以降に対応することになりました

#4 Updated by Yuya Watanabe about 10 years ago

本チケットのステータスは Rejected になっていますが， OpenPNE-3.8.0 リリース時に本チケットのコミットが混入しています．

94bb50479d37eb9428751883a3b6903ddb8dcbff

#6 Updated by kaoru n over 1 year ago

  • Status changed from Rejected（差し戻し） to Pending Review（レビュー待ち）

https://github.com/openpne/OpenPNE3/pull/616
にてプルリクエストしました

#7 Updated by Rimpei Ogawa over 1 year ago

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

#8 Updated by kaoru n over 1 year ago

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

#9 Updated by isao sano over 1 year ago

  • Status changed from Pending Testing（テスト待ち） to Pending Merge（マージ待ち）
  • % Done changed from 70 to 80

試験完了しました。
問題ありません。

#10 Updated by kaoru n over 1 year ago

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

マージしました

Also available in: Atom PDF