プロジェクト

全般

プロフィール

Backport(バックポート) #3178

三項演算子の二項目を省略した記述はPHP5.2環境で動作しない

Yuma Sakata11年以上前に追加. 11年以上前に更新.

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

100%


説明

概要

三項演算子の二項目を省略した記述はPHP5.2環境で動作しない

具体的には下記コード部分がこれに該当する.

lib/util/opActivityQueryBuilder.class.php

 72   public function includeFriends($target_member_id = null)
 73   {
 74     $this->include['friend'] = $target_member_id ?: $this->viewerId;
 75     return $this;
 76   }

apps/api/lib/helper/opJsonApiHelper.php

134     'name' => $community->getName(),
'category' => (string)$community->getCommunityCategory() ?: null,
\136     'community_url' => $communityUrl,

原因

三項演算子の二項目を省略した形で記述できるのは PHP 5.3 以降であるため、 PHP 5.2 環境では利用することができない.

参考URL: http://php.net/manual/ja/language.operators.comparison.php

修正内容

diff --git a/apps/api/lib/helper/opJsonApiHelper.php b/apps/api/lib/helper/opJsonApiHelper.php
index e5fd560..c460778 100644
--- a/apps/api/lib/helper/opJsonApiHelper.php
+++ b/apps/api/lib/helper/opJsonApiHelper.php
@@ -139,7 +139,7 @@ function op_api_community($community)
   return array(
     'id' => $community->getId(),
     'name' => $community->getName(),
-    'category' => (string)$community->getCommunityCategory() ?: null,
+    'category' => $community->getCommunityCategory() ? $community->getCommunityCategory()->getName() : null,
     'community_url' => $communityUrl,
     'community_image_url' => $communityImage,
     'joining' => $communityMember ? !$communityMember->getIsPre() : false,
diff --git a/lib/util/opActivityQueryBuilder.class.php b/lib/util/opActivityQueryBuilder.class.php
index 6841e90..661dec5 100644
--- a/lib/util/opActivityQueryBuilder.class.php
+++ b/lib/util/opActivityQueryBuilder.class.php
@@ -71,7 +71,7 @@ class opActivityQueryBuilder

   public function includeFriends($target_member_id = null)
   {
-    $this->include['friend'] = $target_member_id ?: $this->viewerId;
+    $this->include['friend'] = $target_member_id ? $target_member_id : $this->viewerId;
     return $this;
   }

元本文

三項演算子の省略形がPHP5.2で動作しないので、動作するように変更する。
厳密にはバグではないが、これだけの理由でPHP5.3縛りにしてしまうのはもったいない。安定版中に対応する。
すなわち、OpenPNE3.8の対象バージョンはPHP5.2以上である、ということを定義する。

例 {OpenPNE3.8.0}/apps/api/lib/helper
opJsonApiHelper.php
パラメータ省略は、5.3から可能
135行目
修正前
'category' => (string)$community->getCommunityCategory() ?: null,
修正後
'category' => (string)$community->getCommunityCategory() ? $community->getCommunityCategory() : null,


関連するチケット

関連している OpenPNE 3 - Bug(バグ) #3121: 三項演算子の二項目を省略した記述はPHP5.2環境で動作しない Won't fix(対応せず) 2012-07-20

関係しているリビジョン

リビジョン c8a4f449 (差分)
Youichi Kimura11年以上前に追加

fix compatibility with PHP 5.2.x (fixes #3178, BP from #3121)

リビジョン 8e28d934 (差分)
Youichi Kimura11年以上前に追加

fix coding style issue (refs #3178, BP from #3121)

(cherry picked from commit 28c087935d5eb411ded3b902333c30608ef2a845)

履歴

#1 Youichi Kimura11年以上前に更新

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

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

#2 Yuma Sakata11年以上前に更新

  • 説明 を更新 (diff)

#3 Yuma Sakata11年以上前に更新

  • 説明 を更新 (diff)

#4 Yuma Sakata11年以上前に更新

  • 説明 を更新 (diff)

#5 Yuma Sakata11年以上前に更新

・json_encodeのPRETTY_PRINT

については #3028 にて既に修正されているため、

・三項演算子の省略形

のみを当チケットでの修正対象とします。

#6 Yuma Sakata11年以上前に更新

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

テスト実施しましたが、修正が必要な点がありましたので確認お願いします。

コミュニティカテゴリ未設定の場合、API値の確認

  • 試験手順
    1. カテゴリを指定しないで、コミュニティを作成する
    2. SNSトップページで、ページのソースを表示する
    3. 手順2 のソースよりAPIキーを取得する
    (例:var openpne = {"apiKey":"ここにAPIキーが表示されます")
    4. 次のURLのAPIキーの部分に手順3で取得したAPIキーを入力する
    (例:http://sns.example.com/api.php/community/search.json?apiKey=<APIキー>)
    5. API の category の値を確認する
  • 試験結果
    "category":"" と表示される
  • 修正方針
    "category":null と表示されるように修正お願いします。

#7 Yuya Watanabe11年以上前に更新

  • 題名PHP5.2環境で動作するようにする から 三項演算子の二項目を省略した記述はPHP5.2環境で動作しない に変更
  • 説明 を更新 (diff)

#8 Yuya Watanabe11年以上前に更新

差し戻し内容

コーディング規約違反. http://www.openpne.jp/coding-standards-ja/

関数およびメソッド:return 文の直前には可読性向上のために空行を入れるべきです
変数:変数名に含めることができるのは英数字のみです。アンダースコアを使用してはいけません。

lib/util/opActivityQueryBuilder.class.php

 72   public function includeFriends($target_member_id = null)
 73   {
 74     $this->include['friend'] = $target_member_id ? $target_member_id : $this->viewerId;
 75     return $this;
 76   }

#9 Youichi Kimura11年以上前に更新

  • ステータスRejected(差し戻し) から Accepted(着手) に変更
  • 進捗率50 から 0 に変更

#10 Youichi Kimura11年以上前に更新

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

https://redmine.openpne.jp/issues/3121#note-11 と同様にコーディング規約違反の箇所を修正しました。

#11 Yuya Watanabe11年以上前に更新

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

#12 Yuma Sakata11年以上前に更新

  • ステータスPending Testing(テスト待ち) から Fixed(完了) に変更
  • 進捗率70 から 100 に変更

テストOKです。
note-6 指摘は本チケットで扱う問題でないので、別チケットで対応します。

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