プロジェクト

全般

プロフィール

Bug(バグ) #3026

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

Youichi Kimura約12年前に追加. 3年以上前に更新.

ステータス:
Fixed(完了)
優先度:
Normal(通常)
担当者:
対象バージョン:
開始日:
2012-04-23
期日:
進捗率:

100%

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

説明

Overview (現象)

Causes (原因)

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

参考: PHP: json_encode - Manual

Way to fix (修正内容)

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


子チケット

Backport(バックポート) #4394: PHP5.2.x環境でjson_encodeの呼び出し時にWarningが発生するFixed(完了)isao sano


関連するチケット

関連している OpenPNE 3 - Backport(バックポート) #3028: PHP5.2.x環境でjson_encodeの呼び出し時にWarningが発生する Fixed(完了) 2012-04-23
コピー先 OpenPNE 3 - Backport(バックポート) #3645: PHP5.2.x環境でjson_encodeの呼び出し時にWarningが発生する Invalid(無効) 2012-04-23

関係しているリビジョン

リビジョン 140cbe5b (差分)
Youichi Kimura約12年前に追加

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

リビジョン 6b344ef2 (差分)
Youichi Kimuraほぼ12年前に追加

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

リビジョン 93716497 (差分)
Youichi Kimuraほぼ7年前に追加

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

リビジョン 0304406a
kaoru n3年以上前に追加

Merge pull request #475 from ShinichiU/t-3026

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

履歴

#1 Youichi Kimura約12年前に更新

  • ステータスNew(新規) から Pending Review(レビュー待ち) に変更
  • 進捗率0 から 50 に変更

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

#2 Yuma Sakata約12年前に更新

  • 対象バージョンOpenPNE 3.9.0-old にセット

#3 Rimpei Ogawaほぼ12年前に更新

  • ステータスPending Review(レビュー待ち) から 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 Rimpei Ogawaほぼ12年前に更新

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

#5 Shouta Kashiwagiほぼ12年前に更新

  • 対象バージョンOpenPNE 3.9.0-old から OpenPNE 3.8.x に変更

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

#6 Youichi Kimuraほぼ12年前に更新

  • ステータスRejected(差し戻し) から Pending Review(レビュー待ち) に変更

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

#7 Kousuke Ebihara11年以上前に更新

  • 3.8 で発生するかYes (はい) にセット

#8 Yuma Sakata11年以上前に更新

  • 対象バージョンOpenPNE 3.8.x から OpenPNE 3.9.0-old に変更

#9 Shinichi Urabeほぼ10年前に更新

  • 対象バージョンOpenPNE 3.9.0-old から OpenPNE 3.8.12 に変更

#10 誠二 天重ほぼ10年前に更新

#11 誠二 天重ほぼ10年前に更新

  • 対象バージョンOpenPNE 3.8.12 から OpenPNE 3.9.0-old に変更

#13 isao sano約7年前に更新

  • ステータスPending Review(レビュー待ち) から Rejected(差し戻し) に変更

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

#14 isao sano約7年前に更新

  • 対象バージョンOpenPNE 3.9.0-old から OpenPNE 3.9.0 に変更

#15 Shinichi Urabeほぼ7年前に更新

  • ステータスRejected(差し戻し) から Pending Review(レビュー待ち) に変更
  • 担当者Youichi Kimura から Shinichi Urabe に変更

#16 kaoru n約4年前に更新

  • 対象バージョンOpenPNE 3.9.0 から OpenPNE 3.10.x に変更

#17 kaoru n3年以上前に更新

  • 対象バージョンOpenPNE 3.10.x から OpenPNE 3.11.x に変更

#18 Rimpei Ogawa3年以上前に更新

  • ステータスPending Review(レビュー待ち) から Pending Testing(テスト待ち) に変更
  • 進捗率50 から 70 に変更

#19 isao sano3年以上前に更新

  • ステータスPending Testing(テスト待ち) から Pending Merge(マージ待ち) に変更
  • 進捗率70 から 80 に変更

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

#20 kaoru n3年以上前に更新

  • ステータスPending Merge(マージ待ち) から Fixed(完了) に変更
  • 進捗率80 から 100 に変更

マージしました

他の形式にエクスポート: Atom PDF