OpenPNE 3

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

担当を設定ください(差し戻し対応が誰か不明になるため)

親チケットを差し戻しました。
https://redmine.openpne.jp/issues/3588#note-15

今回の対応から外します。

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

https://github.com/isaosano/OpenPNE3/commit/7b5ce17f48faaa4c4ccaa706ceda853665b4f50a
の lib/action/opCommunityAction.class.php の修正に関してですが、コードをなるべく書き足さない方法で以下も考えられましたが、可読性を優先させました。

$this->communityForm       = new CommunityForm($this->community);
    $this->communityConfigForm = new CommunityConfigForm(array(), array('community' => $this->community));
    if (!$this->unusedImage){$this->communityFileForm = new CommunityFileForm(array(), array('community' => $this->community));}

    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());
        }
    }

レビューお願いします。

master にまずプルリクエストして、フィードバックをレビュー待ちにしてください
if の書き方がコーディング規約に違反しているので、見直してください。
コーディング規約にはないですが不要な空白文字があります

[OpenPNE3]$ git diff 9e513ac320adc55cf5efeeb7a36cc9d3de6440be.. --check
lib/action/opCommunityAction.class.php:95: trailing whitespace.
+    

opCommunityAction::$unusedImage はPC、スマホからの処理の場合、未定義の変数です。dev モードだと Notice がでているはずです
モバイル専用のアクションを追加する必要もないので !sfConfig::get('app_is_mobile', false) で分岐すれば良いだけかと思います

https://github.com/openpne/OpenPNE3/pull/253
にてプルリクエストしました。

note-12 で行ったプルリクエストは誤りなため取り下げました。

一個見落としていました。細かいですし、動作に問題ないですが http://redmine.openpne.jp/issues/3588#note-27 の差し戻しの理由で、本チケットも差し戻します

note-18 に関して修正しました。
https://github.com/openpne/OpenPNE3/pull/253

単体テスト完了しました。
マージお願いします。