Project

General

Profile

Bug(バグ) #3026

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

Added by Youichi Kimura about 7 years ago. Updated about 2 years ago.

Status:
Pending Review(レビュー待ち)
Priority:
Normal(通常)
Target version:
Start date:
2012-04-23
Due date:
% Done:

50%

3.6 で発生するか:
Unknown (未調査)
3.8 で発生するか:
Yes (はい)

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 - Backport(バックポート) #3028: PHP5.2.x環境でjson_encodeの呼び出し時にWarningが発生する Rejected(差し戻し) 2012-04-23
Copied to OpenPNE 3 - Backport(バックポート) #3645: PHP5.2.x環境でjson_encodeの呼び出し時にWarningが発生する New(新規) 2012-04-23

Associated revisions

Revision 140cbe5b (diff)
Added by Youichi Kimura about 7 years ago

fix compatibility with PHP 5.2.x (fixes #3026)

Revision 6b344ef2 (diff)
Added by Youichi Kimura about 7 years ago

fix compatibility with PHP 5.2.x (fixes #3026)

History

#1 Updated by Youichi Kimura about 7 years ago

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

更新履歴 140cbe5b536bdecf70608a2b699599d580ac057b で適用されました。

#2 Updated by Yuma Sakata about 7 years ago

  • Target version set to OpenPNE 3.9.0-old

#3 Updated by Rimpei Ogawa about 7 years ago

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

opJsonApiActions::renderJSON() でも json_encode() の呼び出しに第2引数が指定されています。

lib/action/opJsonApiActions.class.php
42-  public function renderJSON(array $data)
43-  {
44:    $json = json_encode($data, $this->doEscape);
45-    $this->getResponse()->setContentType('application/json');
46-

#4 Updated by Rimpei Ogawa about 7 years ago

修正部分を見直しましたが、JSON_PRETTY_PRINT の指定はデバッグ時以外は不要ではないでしょうか?
対象とする $jsonData も見たところ単純なものですし、デバッグ用途だとしてもそれほど必要性を感じませんでした。

#5 Updated by Shouta Kashiwagi about 7 years ago

  • Target version changed from OpenPNE 3.9.0-old to OpenPNE 3.8.x

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

#6 Updated by Youichi Kimura about 7 years ago

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

更新履歴 6b344ef263672077953a80e97690a20543e68be2 で適用されました。

#7 Updated by Kousuke Ebihara almost 7 years ago

  • 3.8 で発生するか set to Yes (はい)

#8 Updated by Yuma Sakata almost 7 years ago

  • Target version changed from OpenPNE 3.8.x to OpenPNE 3.9.0-old

#9 Updated by Shinichi Urabe about 5 years ago

  • Target version changed from OpenPNE 3.9.0-old to OpenPNE 3.8.12

#10 Updated by 誠二 天重 about 5 years ago

#11 Updated by 誠二 天重 about 5 years ago

  • Target version changed from OpenPNE 3.8.12 to OpenPNE 3.9.0-old

#13 Updated by isao sano about 2 years ago

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

対象バージョン変更により修正内容の確認が必要であるため差し戻します。

#14 Updated by isao sano about 2 years ago

  • Target version changed from OpenPNE 3.9.0-old to OpenPNE 3.9.0

#15 Updated by Shinichi Urabe about 2 years ago

  • Status changed from Rejected(差し戻し) to Pending Review(レビュー待ち)
  • Assignee changed from Youichi Kimura to Shinichi Urabe

Also available in: Atom PDF