プロジェクト

全般

プロフィール

Bug(バグ) #3484

opMemberProfileSearchForm 内で定義されていない $values 変数を用いている

Yuya Watanabe10年以上前に追加. 約4年前に更新.

ステータス:
New(新規)
優先度:
Normal(通常)
担当者:
-
対象バージョン:
開始日:
2013-11-14
期日:
進捗率:

0%

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

説明

概要

下記メソッド内で定義されていない $values 変数を用いているため正常に動作しない可能性がある.
影響範囲などについては調査できていない

OpenPNE-3.8.8 コード

lib/form/searchForm/opMemberProfileSearchForm.class.php
 89   protected function addNameColumnQuery(Doctrine_Query $query, $value)
 90   {
 91     if (!empty($value))
 92     {
 93       if (is_array($value))
 94       {
 95         foreach ($value as $v)
 96         {
 97           $query->andWhereLike('name', $v);
 98         }
 99       }
100       else
101       {
102         if (!empty($value))
103         {
104           $query->andWhereLike('name', $values);
105         }
106       }
107     }
108   }

直接影響はしないが !empty($value) の条件が二回行われている.
また is_array() ではなく array でキャストしてしまえば条件分岐はほぼ無くなるのではないかと思う.

履歴

#1 kaoru n約4年前に更新

  • 対象バージョンOpenPNE 3.10.x にセット

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