Project

General

Profile

Bug(バグ) #3121

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

Added by 守 手嶋 over 7 years ago. Updated over 2 years ago.

Status:
Won't fix(対応せず)
Priority:
Low(低め)
Target version:
Start date:
2012-07-20
Due date:
% Done:

0%

3.6 で発生するか:
No (いいえ)
3.8 で発生するか:
Yes (はい)

Description

概要

三項演算子の二項目を省略した記述は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,


Related issues

Related to OpenPNE 3 - Backport(バックポート) #3178: 三項演算子の二項目を省略した記述はPHP5.2環境で動作しない Fixed(完了) 2012-07-20

Associated revisions

Revision cfcc95b7 (diff)
Added by Youichi Kimura over 7 years ago

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

Revision 28c08793 (diff)
Added by Youichi Kimura over 7 years ago

fix coding style issue (refs #3121)

History

#1 Updated by Yuma Sakata over 7 years ago

  • Target version changed from OpenPNE 3.8.x to OpenPNE 3.8.2
  • 3.8 で発生するか set to Unknown (未調査)

#2 Updated by Yuma Sakata over 7 years ago

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

#3 Updated by Youichi Kimura over 7 years ago

  • Description updated (diff)

#4 Updated by Youichi Kimura over 7 years ago

  • Description updated (diff)

・json_encodeのPRETTY_PRINT

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

・三項演算子の省略形

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

#5 Updated by Youichi Kimura over 7 years ago

  • 3.8 で発生するか changed from Unknown (未調査) to Yes (はい)

#6 Updated by Youichi Kimura over 7 years ago

  • Assignee set to Youichi Kimura

#7 Updated by Youichi Kimura over 7 years ago

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

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

#8 Updated by Yuya Watanabe over 7 years ago

  • Subject changed from PHP5.2環境で動作するようにする to 三項演算子の二項目を省略した記述はPHP5.2環境で動作しない
  • Description updated (diff)

#9 Updated by Yuya Watanabe over 7 years ago

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

差し戻し内容

コーディング規約違反. 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   }

#10 Updated by Youichi Kimura over 7 years ago

  • Status changed from Rejected(差し戻し) to Accepted(着手)
  • % Done changed from 50 to 0

#11 Updated by Youichi Kimura over 7 years ago

  • Status changed from Accepted(着手) to Pending Review(レビュー待ち)
  • % Done changed from 0 to 50

28c087935d5eb411ded3b902333c30608ef2a845 にて、note-9 で指摘されているコーディング規約違反に該当する箇所を修正しました。

#12 Updated by Yuya Watanabe over 7 years ago

  • Status changed from Pending Review(レビュー待ち) to Pending Testing(テスト待ち)
  • % Done changed from 50 to 70

#13 Updated by Chiharu Nakajima almost 5 years ago

  • Priority changed from Normal(通常) to Low(低め)

このチケットの内容は下記の項目に該当する不具合であるため、一旦優先度を下げます。
・一部の特殊なサーバー環境のみで発生する不具合

#14 Updated by isao sano over 2 years ago

  • Status changed from Pending Testing(テスト待ち) to Won't fix(対応せず)
  • % Done changed from 70 to 0

OpenPNE 3.8.2 にて対応済みであったため、対応せずとします。

Also available in: Atom PDF