OpenPNE 3

• 環境
  • PHPUnit 3.5.14
  • SeleniumRC 2.7.0

<?php

require_once 'PHPUnit/Extensions/SeleniumTestCase.php';

class t2426Test extends PHPUnit_Extensions_SeleniumTestCase
{
    public static $browsers = array(
        array(
            'browser' => '*firefox',
            'host'    => 'localhost',
            'port'    => 4444,
            'timeout' => 30000,
        ),
    );

    protected function setUp()
    {
        $url = 'http://localhost';
        $this->setBrowserUrl($url);
    }

    public function testT2426()
    {
        // 管理ページで管理者でログイン
        $this->open("/pc_backend.php/");
        $this->type("admin_user_username", "admin");
        $this->type("admin_user_password", "password");
        $this->click("//input[@value=\"ログイン\"]");
        $this->waitForPageToLoad();

        // プロフィール項目登録で「性別」の項目を「Webへの公開の許可」を「禁止」にし、
        // 「公開範囲変更の可否」を「メンバー選択」にする。
        $this->open("/pc_backend.php/profile/edit/id/1");
        $this->click("profile_is_public_web_0");
        $this->click("profile_is_edit_public_flag_1");
        $this->click("//input[@value=\"変更\"]");
        $this->waitForPageToLoad();

        // SNSログイン
        $this->open();
        $this->type("authMailAddress_mail_address", "sns@example.com");
        $this->type("authMailAddress_password", "password");
        $this->click("//input[@value=\"ログイン\"]");
        $this->waitForPageToLoad();

        // 「プロフィール編集画面」で「性別」の項目の公開範囲設定で「Web全体に公開」の項目が存在していると失敗
        $this->open("member/edit/profile");
        $isExist = $this->isElementPresent("//select[@id=\"profile_op_preset_sex_public_flag\"]/option[@value=\"4\"]");
        $this->assertFalse($isExist, "#2426 『Web への公開の許可を「表示しない」に設定しても「Web全体に公開」の選択肢が表示される』");

    }

}
?>

原因¶

「Web全体に公開」等の項目を含むかどうかを規定している部分は下記部分である．

lib/widget/opWidgetFormProfile.class.php 68行目

 65     {
 66       return $input;
 67     }
 68     $publicFlagWidget = new sfWidgetFormSelect(array('choices' => Doctrine::getTable('Profile')->getPublicFlags()));
 69 
 70     return strtr($this->getOption('template'), array('%input%' => $input, '%public_flag%' =>$publicFlagWidget->render($name.'[public_flag]', $value['public_flag'], $attributes, $errors)));
 71   }
 72 }

Doctrine::getTable('Profile')->getPublicFlags()が返すのは「Web全体に公開」「全体に公開」「マイフレンドまで公開」「公開しない」のリストをそのまま返すだけであり，「Web全体に公開」を項目に含むかどうかの確認は行われていない．また同様に，バリデータについても下記の部分ように，同じメソッドを用いている．

lib/validator/opValidatorProfile.class.php 46行目

 32   /**
 33    * @see sfValidatorBase
 34    */
 35   protected function doClean($value)
 36   {
 37     $clean = array();
 38     $clean['value'] = $this->getOption('validator')->clean($value['value']);
 39 
 40     if ($this->getOption('is_edit_public_flag'))
 41     {
 42       if (!isset($value['public_flag']))
 43       {
 44         throw new sfValidatorError($this, 'invalid');
 45       }
 46       $validator = new sfValidatorChoice(array('choices' => array_keys(Doctrine::getTable('Profile')->getPublicFlags())));
 47       $clean['public_flag'] = $validator->clean($value['public_flag']);
 48     }
 49 
 50     return $clean;
 51   }

また，プロフィール確認画面においても公開範囲に関しては考慮されているが「Web への公開の許可」の項目を考慮するコードではないと思われる．

apps/pc_frontend/modules/member/templates/_profileListBox.php　53行目 59行目

 52 
 53   if ($member->getId() == $sf_user->getMemberId() && $profile->getPublicFlag() == ProfileTable::PUBLIC_FLAG_FRIEND)
 54   {
 55     $profileValue .= ' ('.__('Only Open to %my_friend%', array(
 56       '%my_friend%' => $op_term['my_friend']->titleize()->pluralize(),
 57     )).')';
 58   }
 59   elseif ($member->getId() == $sf_user->getMemberId() && $profile->getPublicFlag() == ProfileTable::PUBLIC_FLAG_WEB)
 60   {
 61     $profileValue .= ' ('.__('All Users on the Web').')';
 62   }
 63   $list[$caption] = $profileValue;
 64 }

修正案¶

本チケットの問題を解決するには各プロフィール項目について「Web全体に公開」の設定を反映した公開範囲一覧を得る必要がある．

今回の場合，公開範囲一覧で表示しない設定が存在するというのは各プロフィール項目に固有であると考えられる．そのため，プロフィール項目に固有な情報を保持するlib/model/doctrine/Profile.class.phpから公開範囲一覧を取得できることが望ましい．

また，ウィジェットおよびバリデータ内では，それ自身が表示および検証するプロフィール項目を得る手段が必要となる．ウィジェットの場合は以下の方法でプロフィール項目名を取得することができることを確認した．

1. opWidgetFormProfileクラスのrender()メソッドの第一引数である$nameから正規表現等を用いて識別名を取得し，ProfileTableからProfileを取得する．
2. opWidgetFormProfileクラスのrender()メソッド内において$this->parent->getFields()の連想配列の$thisに相当する値と対応するキーが識別名となるため，その識別名を用いてProfileTableからProfileを取得する．
3. opWidgetFormProfileクラスを呼び出す側でProfileクラスを渡してrender()メソッド内で$this->profileとして呼び出す．

しかし，バリデータ内でウィジェットと同様の方法でProfileを得る方法を検討したが1と2では得ることが出来なかった．
そのためウィジェットおよびバリデータ内でプロフィール情報を取得する部分の修正案は3の方法を検討する．

実装案¶

diff --git a/lib/form/doctrine/MemberProfileForm.class.php b/lib/form/doctrine/MemberProfileForm.class.php
index ad45aa6..09dd0e4 100644
--- a/lib/form/doctrine/MemberProfileForm.class.php
+++ b/lib/form/doctrine/MemberProfileForm.class.php
@@ -163,6 +163,9 @@ class MemberProfileForm extends BaseForm
       $this->widgetSchema[$profile->getName()] = new opWidgetFormProfile($widgetOptions);
       $this->validatorSchema[$profile->getName()] = new opValidatorProfile($validatorOptions);

+      $this->widgetSchema[$profile->getName()]->profile = $profile;
+      $this->validatorSchema[$profile->getName()]->profile = $profile;
+
       $this->widgetSchema->setHelp($profile->getName(), $profileWithI18n['info']);
       if ($profile->isPreset())
       {
diff --git a/lib/model/doctrine/Profile.class.php b/lib/model/doctrine/Profile.class.php
index aa43dad..4d66565 100644
--- a/lib/model/doctrine/Profile.class.php
+++ b/lib/model/doctrine/Profile.class.php
@@ -133,4 +133,15 @@ class Profile extends BaseProfile
       ->orderBy('sort_order')
       ->execute();
   }
+
+  public function getPublicFlags($isI18n = true)
+  {
+    $publicFlags = Doctrine::getTable('profile')->getPublicFlags($isI18n);
+    if (!$this['is_public_web'])
+    {
+      unset($publicFlags[ProfileTable::PUBLIC_FLAG_WEB]);
+    }
+
+    return $publicFlags;
+  }
 }
diff --git a/lib/validator/opValidatorProfile.class.php b/lib/validator/opValidatorProfile.class.php
index 9a4dcbf..dd47cd4 100644
--- a/lib/validator/opValidatorProfile.class.php
+++ b/lib/validator/opValidatorProfile.class.php
@@ -43,7 +43,7 @@ class opValidatorProfile extends sfValidatorBase
       {
         throw new sfValidatorError($this, 'invalid');
       }
-      $validator = new sfValidatorChoice(array('choices' => array_keys(Doctrine::getTable('Profile')->getPublicFlags())));
+      $validator = new sfValidatorChoice(array('choices' => array_keys($this->profile->getPublicFlags())));
       $clean['public_flag'] = $validator->clean($value['public_flag']);
     }

diff --git a/lib/widget/opWidgetFormProfile.class.php b/lib/widget/opWidgetFormProfile.class.php
index 7126a91..eab3f2e 100644
--- a/lib/widget/opWidgetFormProfile.class.php
+++ b/lib/widget/opWidgetFormProfile.class.php
@@ -65,7 +65,8 @@ class opWidgetFormProfile extends sfWidgetForm
     {
       return $input;
     }
-    $publicFlagWidget = new sfWidgetFormSelect(array('choices' => Doctrine::getTable('Profile')->getPublicFlags()));
+    $publicFlagWidget = new sfWidgetFormSelect(array('choices' => $this->profile->getPublicFlags()));

     return strtr($this->getOption('template'), array('%input%' => $input, '%public_flag%' =>$publicFlagWidget->render($name.'[public_flag]', $value['public_flag'], $attributes, $errors)));
   }

また、この手順で未ログイン時にアクセスした際にプロフィール画像が表示されない方が良いのではないでしょうか？

このチケットで検討されたことのない事項である以上、なぜそう思うかの理由を書いてください。

あと、「この手順で」とおっしゃいますが、では、どの手順であればプロフィール画像を表示してよいことになるのですか？

Kousuke Ebihara は書きました:

また、この手順で未ログイン時にアクセスした際にプロフィール画像が表示されない方が良いのではないでしょうか？

このチケットで検討されたことのない事項である以上、なぜそう思うかの理由を書いてください。

あと、「この手順で」とおっしゃいますが、では、どの手順であればプロフィール画像を表示してよいことになるのですか？

この件について、管理画面のSNS設定ページで「メンバーのプロフィールページ公開範囲設定」が「 メンバーの設定を許可しない（Web全体に公開）」になっていたためで全く問題ないです。
これは不要な指摘でした。お騒がせしました。

プロフィール画面で「Web全体に公開」の設定を反映するように修正を行いました．