Backport（バックポート） #3729
完了
スマートフォンからコミュニティ画像を変更すると画像だけのコミュニティが作成される
isao sano さんが約10年前に追加.
9年以上前に更新.
説明
スマートフォンからコミュニティ画像を変更すると、参加者、コミュニティ名、説明文のないコミュニティが作成される。
このコミュニティは管理者が存在しないため削除を行うことは出来ない。
なお、スマートフォン版のコミュニティトップページには「コミュニティ編集」の導線はないため、直接URLを指定して編集を行った。
そもそもスマートフォンで「コミュニティ編集」画面に遷移できる事自体に問題があるのかもしれない。
再現環境¶
- OpenPNE 3.8.10
- OpenPNE 3.8.11
opCommunityTopicPlugin の有無は関連しない
再現手順¶
1. スマートフォンより任意のコミュニティAの管理者でログイン
2. コミュニティAのコミュニティ編集ページのURL(/community/edit?id=XX)を指定する
3. 写真を選択し、編集ボタンを押下
4. コミュニティAの画像は変更されない
5. 画像だけのコミュニティが作成されている
ファイル
関連するチケット
1 (0件未完了 — 1件完了)
- 対象バージョン を OpenPNE 3.8.14 にセット
- ステータス を New（新規） から Accepted（着手） に変更
- ステータス を Accepted（着手） から Pending Review（レビュー待ち） に変更
- 進捗率 を 0 から 50 に変更
担当を設定ください(差し戻し対応が誰か不明になるため)
- 対象バージョン を OpenPNE 3.8.14 から OpenPNE 3.8.x に変更
- 対象バージョン を OpenPNE 3.8.x から OpenPNE 3.8.15 に変更
- ステータス を Pending Review（レビュー待ち） から Rejected（差し戻し） に変更
- 対象バージョン を OpenPNE 3.8.15 から OpenPNE 3.8.x に変更
- 対象バージョン を OpenPNE 3.8.x から OpenPNE 3.8.16 に変更
- ステータス を Rejected（差し戻し） から Accepted（着手） に変更
- 進捗率 を 50 から 0 に変更
- ステータス を Accepted（着手） から Pending Review（レビュー待ち） に変更
- 進捗率 を 0 から 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());
}
}
レビューお願いします。
- ステータス を Pending Review（レビュー待ち） から 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) で分岐すれば良いだけかと思います
- ステータス を Rejected（差し戻し） から Accepted（着手） に変更
- 進捗率 を 50 から 0 に変更
- ステータス を Accepted（着手） から Pending Review（レビュー待ち） に変更
- 進捗率 を 0 から 50 に変更
- ステータス を Pending Review（レビュー待ち） から Accepted（着手） に変更
- 進捗率 を 50 から 0 に変更
- ステータス を Accepted（着手） から Pending Review（レビュー待ち） に変更
- 進捗率 を 0 から 50 に変更
- ステータス を Pending Review（レビュー待ち） から Rejected（差し戻し） に変更
- ステータス を Rejected（差し戻し） から Accepted（着手） に変更
- 進捗率 を 50 から 0 に変更
- ステータス を Accepted（着手） から Pending Review（レビュー待ち） に変更
- 進捗率 を 0 から 50 に変更
- ステータス を Pending Review（レビュー待ち） から Pending Testing（テスト待ち） に変更
- 進捗率 を 50 から 70 に変更
- ステータス を Pending Testing（テスト待ち） から Pending Merge（マージ待ち） に変更
- 進捗率 を 70 から 80 に変更
- ステータス を Pending Merge（マージ待ち） から Fixed（完了） に変更
- 進捗率 を 80 から 100 に変更
他の形式にエクスポート: Atom
PDF
