Backport(バックポート) #3330
完了
opJsonApiActions 内で Content-Type の設定が2度呼び出されている
Chiharu Nakajima さんが11年以上前に追加.
11年以上前に更新.
説明
Overview (現象)¶
opJsonApiActions クラス内で $this->getResponse()->setContentType('application/json')
が execute() メソッドと renderJSON() メソッドで 2 度呼び出されている。また、これによって各 API のアクション内で Content-Type の設定を上書きすることが出来ない。
(現象補足)
apiを使用しているプラグインでは、使用しているjavascriptライブラリとの都合上、content-typeが「text/html」である必要があるものがあります。
(デフォルトでライブラリの中でstringをjsonオブジェクトに変換します。)
そのため、actionsのexecuteXxx()内でcontent-typeの書き換えを行っています。
その変更をopJsonApiActionsのrenderJSON()で「application/json」に再度置き換えられるために不具合発生となります。
Way to fix (修正内容)¶
opJsonApiActions::renderJSON() メソッド内の setContentType の呼び出しが不要なため除去する。
関連するチケット
1 (0件未完了 — 1件完了)
- ステータス を New(新規) から Accepted(着手) に変更
- 担当者 を kaoru n にセット
- ステータス を Accepted(着手) から Pending Review(レビュー待ち) に変更
- 進捗率 を 0 から 50 に変更
原因について¶
https://redmine.openpne.jp/issues/3198#note-6
のとおりですが、少しだけ補足します。
apiを使用しているプラグインでは、使用しているjavascriptライブラリとの都合上、content-typeが「text/html」である必要があるものがあります。
(デフォルトでライブラリの中でstringをjsonオブジェクトに変換します。)
そのため、actionsのexecuteXxx()内でcontent-typeの書き換えを行っています。
その変更をopJsonApiActionsのrenderJSON()で「application/json」に再度置き換えられるために不具合発生となります。
上記についてチケット本文にも記載を行いました。
- ステータス を Pending Review(レビュー待ち) から Pending Testing(テスト待ち) に変更
- 進捗率 を 50 から 70 に変更
- ステータス を Pending Testing(テスト待ち) から Pending Review(レビュー待ち) に変更
- 進捗率 を 70 から 50 に変更
- ステータス を Pending Review(レビュー待ち) から Pending Testing(テスト待ち) に変更
- 進捗率 を 50 から 70 に変更
- ステータス を Pending Testing(テスト待ち) から Fixed(完了) に変更
- 進捗率 を 70 から 100 に変更
確認しました。問題ありません。
チケットをクローズします。
他の形式にエクスポート: Atom
PDF