OpenPNE 3

症状まとめです

スマートフォンからコミュニティ画像を変更すると、
・参加者、コミュニティ名、説明文のないコミュニティが作成される
・スマホ版ではコミュニティ検索ができなくなる(/community/searchにはアクセスできるものの、ずっと検索結果表示待機状態のまま)
※PC版では検索が可能
・一度スマホからコミュニティの画像を変更する処理を行うと、それ以降同じ処理を行おうとしても「現在サーバーが混み合っているか〜〜〜」と表示される。さらに、参加者、コミュニティ名、説明文のないコミュニティは作成されない

という結果になりました。これらの情報を元に修正を行います。

プルリクエストしました
https://github.com/openpne/OpenPNE3/pull/138

元々、スマホではコミュニティの編集ができないことから、スマホでのアクセスをいかなる形を用いても制限することでデータベースが破壊されることを防ぎます。

lib/action/opCommunityAction.class.php内
public function executeEdit(opWebRequest $request) {
$this->forward404If($this->id && !$this->isEditCommunity);

ここでスマホでもアクセスできないようにという条件を追加することで対応しました

具体的には
public function executeEdit(opWebRequest $request) {
if ($request->isSmartphone()) {
$this->forward404();
}
$this->forward404If($this->id && !$this->isEditCommunity);

という風にif文を追加しました

メモ：関連しそうなチケット
http://trac.openpne.jp/ticket/4230
この修正が入っているコミットは 319c599e25d822d36183f4fff9c57aa160e037c5 で以下のような修正が入っている。

diff --git a/apps/pc_frontend/modules/community/actions/actions.class.php b/apps/pc_frontend/modules/community/actions/actions.class.php
index 92dd419..cca9328 100644
--- a/apps/pc_frontend/modules/community/actions/actions.class.php
+++ b/apps/pc_frontend/modules/community/actions/actions.class.php
@@ -24,13 +24,13 @@ class communityActions extends sfOpenPNECommunityAction
   */
   public function executeEdit($request)
   {
+    $this->enableImage = true;
     $result = parent::executeEdit($request);

     if ($this->community->isNew()) {
       sfConfig::set('sf_nav_type', 'default');
     }

-    $this->communityFileForm   = new CommunityFileForm(array(), array('community' => $this->community));

     return $result;
   }
diff --git a/lib/action/sfOpenPNECommunityAction.class.php b/lib/action/sfOpenPNECommunityAction.class.php
index a51a6f4..0b303c7 100644
--- a/lib/action/sfOpenPNECommunityAction.class.php
+++ b/lib/action/sfOpenPNECommunityAction.class.php
@@ -74,6 +74,9 @@ abstract class sfOpenPNECommunityAction extends sfActions

     $this->communityForm       = new CommunityForm($this->community);
     $this->communityConfigForm = new CommunityConfigForm(array(), array('community' => $this->community));
+    $this->communityFileForm = isset($this->enableImage) && $this->enableImage ?
+      new CommunityFileForm(array(), array('community' => $this->community)) :
+      new CommunityFileForm();

     if ($request->isMethod('post'))
     {

検証¶

該当する箇所を以下のようにtry-catchで囲ってみたが、エラー発生せず。

    if ($request->isMethod('post'))
    {
      $conn = opDoctrineQuery::getMasterConnectionDirect();

      try
      {
        $params = $request->getParameter('community');
        $params['id'] = $this->id;
        $this->communityForm->bind($params);
        $this->communityConfigForm->bind($request->getParameter('community_config'));
        $this->communityFileForm->bind($request->getParameter('community_file'), $request->getFiles('community_file'));
        if ($this->communityForm->isValid() && $this->communityConfigForm->isValid() && $this->communityFileForm->isValid())
        {
          $this->communityForm->save($conn);
          $this->communityConfigForm->save($conn);
          $this->communityFileForm->save($conn);

          $this->redirect('@community_home?id='.$this->community->getId());
        }
      }
      catch (Exception $e)
      {
        $conn->rollback();
        $this->forward404();
      }
    }

現象¶

• スマートフォンから画像を設定してコミュニティ作成を行うと、画像が設定されていない正常なコミュニティと、画像が設定されているだけで他の情報を持たないコミュニティの二つのコミュニティができる。

上記の検証から、CommunityForm, CommunityConfigForm, CommunityFileForm の全てがバリデーションが正しいと判定されているが故に、それぞれのフォームがsaveメソッドを実行していることになる。
問題としては、CommunityConfigFormはCommunityFormと紐付いてデータが保存されるが、CommunityFileFormが紐付いていない状態で保存されるため、新たにデータが空の状態のコミュニティが作成される、ということである。

原因¶

一度の投稿で二つのコミュニティ（片方は画像以外の情報を持たない）が登録されてしまうのは、CommunityFileFormにコミュニティオブジェクトが設定されていなくてもエラーとならないことが根本的な原因である。
CommunityFileForm のconfigureは以下になっている。

  public function configure()
  {
    $this->setCommunity($this->getOption('community'));

75    $this->communityConfigForm = new CommunityConfigForm(array(), array('community' => $this->community));
76    $this->communityFileForm = isset($this->enableImage) && $this->enableImage ?
77      new CommunityFileForm(array(), array('community' => $this->community)) :
78      new CommunityFileForm();

55  public function executeEdit(opWebRequest $request)
56  {
57    $this->forwardIf($request->isSmartphone(), 'community', 'smtEdit');
58
59    $this->enableImage = true;
60    $result = parent::executeEdit($request);
...
75  public function executeSmtEdit(opWebRequest $request)
76  {
77    $result = parent::executeEdit($request);

修正方針¶

もっとも簡単な解決としては、スマホ版のeditアクションでも enableImage に true をセットすれば解決する。
だが、その場合 lib/action/opCommunityAction.class.php の78行目を通ることはないのだから、 enableImage の値で分岐する理由がそもそもない。
もともと、enableImage という変数名から、コミュニティ作成時に画像を設定できるかどうかの設定値を保持するような想定だったのかもしれない。
現状でその実装がなされていないことと、すでに当初想定されていた仕様が不明であるため、enableImage の値による分岐自体を削除するというのが妥当な修正方針であると判断する。

#note-18 の指摘にともない以下のように修正してみたのですが、mobileでコミュニティ編集画面を開き「確定」ボタンを押下しても画面が遷移しません。なにが原因でしょうか。

    if ($request->isMethod('post'))
    {
      $params = $request->getParameter('community');
      $params['id'] = $this->id;
      $this->communityForm->bind($params);
      $this->communityConfigForm->bind($request->getParameter('community_config'));
      if($this->communityFileForm)
      {
        $this->communityFileForm->bind($request->getParameter('community_file'), $request->getFiles('community_file'));
        if ($this->communityForm->isValid() && $this->communityConfigForm->isValid() && (!$this->communityFileForm || $this->communityFileForm->isValid()))
        {
          $this->communityForm->save();
          $this->communityConfigForm->save();
          if ($this->communityFileForm)
          {
          $this->communityFileForm->save();
          }

          $this->redirect('@community_home?id='.$this->community->getId());
        }
      }
    }