Project

General

Profile

Backport(バックポート) #3729

スマートフォンからコミュニティ画像を変更すると画像だけのコミュニティが作成される

Added by isao sano about 5 years ago. Updated over 4 years ago.

Status:
Fixed(完了)
Priority:
High(高め)
Assignee:
Target version:
Start date:
2014-04-09
Due date:
% Done:

100%


Description

概要

スマートフォンからコミュニティ画像を変更すると、参加者、コミュニティ名、説明文のないコミュニティが作成される。
このコミュニティは管理者が存在しないため削除を行うことは出来ない。

なお、スマートフォン版のコミュニティトップページには「コミュニティ編集」の導線はないため、直接URLを指定して編集を行った。
そもそもスマートフォンで「コミュニティ編集」画面に遷移できる事自体に問題があるのかもしれない。

再現環境

  • OpenPNE 3.8.10
  • OpenPNE 3.8.11

opCommunityTopicPlugin の有無は関連しない

再現手順

1. スマートフォンより任意のコミュニティAの管理者でログイン
2. コミュニティAのコミュニティ編集ページのURL(/community/edit?id=XX)を指定する
3. 写真を選択し、編集ボタンを押下
4. コミュニティAの画像は変更されない
5. 画像だけのコミュニティが作成されている

画像だけコミュニティ.png View (75.1 KB) isao sano, 2014-04-09 13:52

スクリーンショット 2014-05-26 16.11.58.png View (620 KB) Akihiro KOBAYASHI, 2014-05-26 16:27


Related issues

Copied from OpenPNE 3 - Bug(バグ) #3588: スマートフォンからコミュニティ画像を変更すると画像だけのコミュニティが作成される Won't fix(対応せず) 2014-04-09

Associated revisions

Revision 32588ac6 (diff)
Added by isao sano over 4 years ago

fix not to create empty Community( except image) if a Community created by smartphone(refs #3729, BP from #3588)

Revision 866ad999 (diff)
Added by isao sano over 4 years ago

Modified so as not to create a communityFileForm class if do not use file upload(refs #3729, BP from #3588)

Revision 6cd6fc25 (diff)
Added by isao sano over 4 years ago

It aligned the indentation (refs #3729, BP from #3588)

Revision 8792c5bf (diff)
Added by isao sano over 4 years ago

Modified to branch on whether non- mobile or mobile (refs #3729, BP from #3588)

Revision b06f4edf (diff)
Added by isao sano over 4 years ago

Processing of communityConfigForm has been changed to the common ones (refs #3729, BP from #3588)

Revision 3f69572b (diff)
Added by isao sano over 4 years ago

Change to get the Parameters from the Form (refs #3729, BP from #3588)

Revision d8591063 (diff)
Added by isao sano over 4 years ago

Fixed a getFiles() to get the parameters from the Form (refs #3729, BP from #3588)

Revision eac6a052
Added by Shinichi Urabe over 4 years ago

Merge pull request #253 from isaosano/t-3729

fix not to create empty Community( except image) if a Community created by smartphone(refs #3729, BP from #3588)

History

#1 Updated by isao sano about 5 years ago

  • Copied from Bug(バグ) #3588: スマートフォンからコミュニティ画像を変更すると画像だけのコミュニティが作成される added

#2 Updated by isao sano about 5 years ago

  • Target version set to OpenPNE 3.8.14

#3 Updated by isao sano about 5 years ago

  • Status changed from New(新規) to Accepted(着手)

#4 Updated by isao sano about 5 years ago

  • Status changed from Accepted(着手) to Pending Review(レビュー待ち)
  • % Done changed from 0 to 50

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

#5 Updated by Shinichi Urabe almost 5 years ago

  • Assignee set to isao sano

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

#6 Updated by isao sano almost 5 years ago

  • Target version changed from OpenPNE 3.8.14 to OpenPNE 3.8.x

#7 Updated by Chiharu Nakajima almost 5 years ago

  • Target version changed from OpenPNE 3.8.x to OpenPNE 3.8.15

#8 Updated by Rimpei Ogawa over 4 years ago

  • Status changed from Pending Review(レビュー待ち) to Rejected(差し戻し)

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

#9 Updated by Chiharu Nakajima over 4 years ago

  • Target version changed from OpenPNE 3.8.15 to OpenPNE 3.8.x

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

#10 Updated by kaoru n over 4 years ago

  • Target version changed from OpenPNE 3.8.x to OpenPNE 3.8.16

#11 Updated by isao sano over 4 years ago

  • Status changed from Rejected(差し戻し) to Accepted(着手)
  • % Done changed from 50 to 0

#12 Updated by isao sano over 4 years ago

  • Status changed from Accepted(着手) to Pending Review(レビュー待ち)
  • % Done changed from 0 to 50

プルリクエストしました。
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());
        }
    }

レビューお願いします。

#13 Updated by Shinichi Urabe over 4 years ago

  • Status changed from Pending Review(レビュー待ち) to Rejected(差し戻し)

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) で分岐すれば良いだけかと思います

#14 Updated by isao sano over 4 years ago

  • Status changed from Rejected(差し戻し) to Accepted(着手)
  • % Done changed from 50 to 0

#15 Updated by isao sano over 4 years ago

  • Status changed from Accepted(着手) to Pending Review(レビュー待ち)
  • % Done changed from 0 to 50

#16 Updated by isao sano over 4 years ago

  • Status changed from Pending Review(レビュー待ち) to Accepted(着手)
  • % Done changed from 50 to 0

#17 Updated by isao sano over 4 years ago

  • Status changed from Accepted(着手) to Pending Review(レビュー待ち)
  • % Done changed from 0 to 50

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

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

#18 Updated by Shinichi Urabe over 4 years ago

  • Status changed from Pending Review(レビュー待ち) to Rejected(差し戻し)

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

#19 Updated by isao sano over 4 years ago

  • Status changed from Rejected(差し戻し) to Accepted(着手)
  • % Done changed from 50 to 0

#20 Updated by isao sano over 4 years ago

  • Status changed from Accepted(着手) to Pending Review(レビュー待ち)
  • % Done changed from 0 to 50

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

#21 Updated by Shinichi Urabe over 4 years ago

  • Status changed from Pending Review(レビュー待ち) to Pending Testing(テスト待ち)
  • % Done changed from 50 to 70

#22 Updated by isao sano over 4 years ago

  • Status changed from Pending Testing(テスト待ち) to Pending Merge(マージ待ち)
  • % Done changed from 70 to 80

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

#23 Updated by Shinichi Urabe over 4 years ago

  • Status changed from Pending Merge(マージ待ち) to Fixed(完了)
  • % Done changed from 80 to 100

Also available in: Atom PDF