Project

General

Profile

Actions
Copy link

Bug（バグ） #3026

closed

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

Added by Youichi Kimura over 12 years ago. Updated about 4 years ago.

Status:
Fixed（完了）
Priority:
Normal（通常）
Assignee:
Shinichi Urabe
Target version:
OpenPNE 3.11.x
Start date:
2012-04-23
Due date:
% Done:

100%

Estimated time:
(Total: 0:00 h)
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引数を指定しないように修正する。

Subtasks 1 (0 open — 1 closed)

Backport（バックポート） #4394: PHP5.2.x環境でjson_encodeの呼び出し時にWarningが発生するFixed（完了）isao sano2012-04-23

Actions

Related issues 2 (0 open — 2 closed)

Related to OpenPNE 3 - Backport（バックポート） #3028: PHP5.2.x環境でjson_encodeの呼び出し時にWarningが発生するFixed（完了）Youichi Kimura2012-04-23

Actions
Copied to OpenPNE 3 - Backport（バックポート） #3645: PHP5.2.x環境でjson_encodeの呼び出し時にWarningが発生するInvalid（無効）2012-04-23

Actions
Actions
Copy link
 #1

Updated by Youichi Kimura over 12 years ago

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

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

Actions
Copy link
 #2

Updated by Yuma Sakata over 12 years ago

  • Target version set to OpenPNE 3.9.0-old
Actions
Copy link
 #3

Updated by Rimpei Ogawa over 12 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-
Actions
Copy link
 #4

Updated by Rimpei Ogawa over 12 years ago

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

Actions
Copy link
 #5

Updated by Shouta Kashiwagi over 12 years ago

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

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

Actions
Copy link
 #6

Updated by Youichi Kimura over 12 years ago

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

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

Actions
Copy link
 #7

Updated by Kousuke Ebihara over 12 years ago

  • 3.8 で発生するか set to Yes (はい)
Actions
Copy link
 #8

Updated by Yuma Sakata over 12 years ago

  • Target version changed from OpenPNE 3.8.x to OpenPNE 3.9.0-old
Actions
Copy link
 #9

Updated by Shinichi Urabe over 10 years ago

  • Target version changed from OpenPNE 3.9.0-old to OpenPNE 3.8.12
Actions
Copy link
 #10

Updated by 誠二 天重 over 10 years ago

Actions
Copy link
 #11

Updated by 誠二 天重 over 10 years ago

  • Target version changed from OpenPNE 3.8.12 to OpenPNE 3.9.0-old
Actions
Copy link
 #13

Updated by isao sano almost 8 years ago

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

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

Actions
Copy link
 #14

Updated by isao sano almost 8 years ago

  • Target version changed from OpenPNE 3.9.0-old to OpenPNE 3.9.0
Actions
Copy link
 #15

Updated by Shinichi Urabe over 7 years ago

  • Status changed from Rejected（差し戻し） to Pending Review（レビュー待ち）
  • Assignee changed from Youichi Kimura to Shinichi Urabe
Actions
Copy link
 #16

Updated by kaoru n almost 5 years ago

  • Target version changed from OpenPNE 3.9.0 to OpenPNE 3.10.x
Actions
Copy link
 #17

Updated by kaoru n over 4 years ago

  • Target version changed from OpenPNE 3.10.x to OpenPNE 3.11.x
Actions
Copy link
 #18

Updated by Rimpei Ogawa about 4 years ago

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

Updated by isao sano about 4 years ago

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

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

Actions
Copy link
 #20

Updated by kaoru n about 4 years ago

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

マージしました

Actions
Copy link

Also available in: Atom PDF