Bug(バグ) #3121
三項演算子の二項目を省略した記述はPHP5.2環境で動作しない
0%
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
Associated revisions
fix compatibility with PHP 5.2.x (fixes #3121)
fix coding style issue (refs #3121)
History
#1
Updated by Yuma Sakata over 10 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 10 years ago
- Target version changed from OpenPNE 3.8.2 to OpenPNE 3.9.0-old
#3
Updated by Youichi Kimura over 10 years ago
- Description updated (diff)
#4
Updated by Youichi Kimura over 10 years ago
- Description updated (diff)
#5
Updated by Youichi Kimura over 10 years ago
- 3.8 で発生するか changed from Unknown (未調査) to Yes (はい)
#6
Updated by Youichi Kimura over 10 years ago
- Assignee set to Youichi Kimura
#7
Updated by Youichi Kimura over 10 years ago
- Status changed from New(新規) to Pending Review(レビュー待ち)
- % Done changed from 0 to 50
更新履歴 cfcc95b7a960e0c64bad54c494b9e398981d55a1 で適用されました。
#8
Updated by Yuya Watanabe over 10 years ago
- Subject changed from PHP5.2環境で動作するようにする to 三項演算子の二項目を省略した記述はPHP5.2環境で動作しない
- Description updated (diff)
#9
Updated by Yuya Watanabe over 10 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 10 years ago
- Status changed from Rejected(差し戻し) to Accepted(着手)
- % Done changed from 50 to 0
#11
Updated by Youichi Kimura over 10 years ago
- Status changed from Accepted(着手) to Pending Review(レビュー待ち)
- % Done changed from 0 to 50
28c087935d5eb411ded3b902333c30608ef2a845 にて、note-9 で指摘されているコーディング規約違反に該当する箇所を修正しました。
#12
Updated by Yuya Watanabe over 10 years ago
- Status changed from Pending Review(レビュー待ち) to Pending Testing(テスト待ち)
- % Done changed from 50 to 70
#13
Updated by Chiharu Nakajima about 8 years ago
- Priority changed from Normal(通常) to Low(低め)
このチケットの内容は下記の項目に該当する不具合であるため、一旦優先度を下げます。
・一部の特殊なサーバー環境のみで発生する不具合
#14
Updated by isao sano almost 6 years ago
- Status changed from Pending Testing(テスト待ち) to Won't fix(対応せず)
- % Done changed from 70 to 0
OpenPNE 3.8.2 にて対応済みであったため、対応せずとします。