Project

General

Profile

Bug(バグ) #2481

携帯版で副管理者がコミュニティを削除するボタンが表示されてしまっている

Added by Yuya Watanabe over 7 years ago. Updated over 3 years ago.

Status:
Fixed(完了)
Priority:
Normal(通常)
Assignee:
Target version:
Start date:
2011-10-11
Due date:
% Done:

100%

3.6 で発生するか:
Yes (はい)
3.8 で発生するか:
Unknown (未調査)

Description

概要

携帯版で副管理者がコミュニティを削除するボタンが表示されている.
そのボタンを押すと「このページにはアクセスできません。」と表示されるため,実際には削除できない.
PC版では表示されないため,携帯版もPC版と同様に削除ボタンを表示しないようにする必要がある.

再現方法

前提:メンバAがコミュニティAの副管理者である

  1. コミュニティトップ画面(community/○○)のページにアクセスする
  2. 「コミュニティの編集」リンクをクリック
  3. コミュニティ編集画面(community/edit/○○)の表示に成功する
    • ページ最下部あたりに「コミュニティを削除する」の項目が表示されている

NOTE: ○○はコミュニティID

原因

下記部分において削除可能かどうかの確認が行われていない.

/apps/mobile_frontend/modules/community/templates/editSuccess.php 24行目

 22 )) ?>
 23 
 24 <?php if (!$communityForm->isNew()): ?>
 25 <?php
 26   op_include_parts('buttonBox', 'deleteForm', array(
 27     'title' => __('Delete this %community%'),
 28     'body' => __('delete this %community%.if you delete this %community% please to report in advance for all this %community% members.'),
 29     'button' => __('Delete'),
 30     'method' => 'get',
 31     'url' => url_for('@community_delete?id=' . $community->getId()),
 32   ));
 33 ?>
 34 <hr color="<?php echo $op_color['core_color_11'] ?>">
 35 <?php echo link_to(__('Community Top'), '@community_home?id='.$community->getId()) ?>
 36 <?php endif; ?>

修正方針

削除項目を表示する際に削除できるかどうかを確認する.
具体的には下記のPC版のようにすることがいいと考えられる.

apps/pc_frontend/modules/community/templates/editSuccess.php 19行目

 18                                                                                                                    
 19 if (!$communityForm->isNew() && $isDeleteCommunity)                                                                
 20 {                                                                                                                  
 21   op_include_parts('buttonBox', 'deleteForm', array(                                                               
 22     'title' => __('Delete this %community%'),                                                                      
 23     'body' => __('delete this %community%.if you delete this %community% please to report in advance for all this %community% members.'),
 24     'button' => __('Delete'),                                                                                      
 25     'method' => 'get',
 26     'url' => url_for('@community_delete?id=' . $community->getId()),                                               
 27   ));
 28 }
 29 

実装案

diff --git a/apps/mobile_frontend/modules/community/templates/editSuccess.php b/apps/mobile_frontend/modules/community/templates/editSuccess.php
index f568b4c..6859e9d 100644
--- a/apps/mobile_frontend/modules/community/templates/editSuccess.php
+++ b/apps/mobile_frontend/modules/community/templates/editSuccess.php
@@ -21,7 +21,7 @@ else
   'align' => 'center',
 )) ?>

-<?php if (!$communityForm->isNew()): ?>
+<?php if (!$communityForm->isNew() && $isDeleteCommunity): ?>
 <?php
   op_include_parts('buttonBox', 'deleteForm', array(
     'title' => __('Delete this %community%'),

Related issues

Related to OpenPNE 3 - Backport(バックポート) #2656: 携帯版で副管理者がコミュニティを削除するボタンが表示されてしまっている Fixed(完了) 2011-12-02
Related to OpenPNE 3 - Backport(バックポート) #2693: 携帯版で副管理者がコミュニティを削除するボタンが表示されてしまっている Fixed(完了) 2011-10-11
Related to OpenPNE 3 - Backport(バックポート) #3098: 携帯版で副管理者がコミュニティを削除するボタンが表示されてしまっている Fixed(完了) 2011-10-11

Associated revisions

Revision 8e62d7e6 (diff)
Added by Yuya Watanabe over 7 years ago

(fixes #2481) fixed not to display delete buttom in mobile page

History

#1 Updated by wa ta over 7 years ago

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

更新履歴 8e62d7e68ad2bdfdd397d85fabec15239b809e14 で適用されました。

#2 Updated by Kousuke Ebihara over 7 years ago

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

#3 Updated by isao sano over 7 years ago

  • Status changed from Pending Testing(テスト待ち) to Fixed(完了)
  • % Done changed from 70 to 100

テスト完了致しました。
問題ありませんので、Fixedに致します。

#4 Updated by Yuya Watanabe over 7 years ago

  • 3.6 で発生するか set to Unknown (未調査)
  • 3.4 で発生するか set to Yes (はい)

#5 Updated by Yuya Watanabe over 7 years ago

  • 3.6 で発生するか changed from Unknown (未調査) to Yes (はい)

#6 Updated by kaoru n over 3 years ago

  • 3.8 で発生するか set to Unknown (未調査)

Also available in: Atom PDF