Project

General

Profile

Bug(バグ) #2257

モバイルでコミュニティの管理権を渡そうとしたときに「管理権を渡す」リンクが残ったまま

Added by Yuya Watanabe over 8 years ago. Updated about 2 years ago.

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

100%

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

Description

概要

コミュニティの管理権をほかのメンバに渡す処理を行おうとした場合、メンバ管理画面においてPC版だとほかのメンバに申請を行ったあとだと「管理権交代依頼中」と表示されているが、モバイル版の場合「管理権を渡す」リンクが表示されたままとなっている。(添付画像参照)
このリンクをクリックするとエラーページが表示される。

ここはPC版と同様に「管理権交代依頼中」と表示させ、リンクを表示しないようにするべきだと思われる。

確認バージョン

OpenPNE3.6beta12, master

確認環境

Windows7 Firefox5 FiremobileSimlurator P903i

修正内容

管理者であり管理者交代依頼中であるときに「管理者交代依頼中」と表示されるような処理の状態だったものを、管理者ではなく管理者交代依頼中であるときに「管理者交代依頼中」と表示されるように修正。

キャプチャ.PNG View (7.71 KB) Yuya Watanabe, 2011-07-04 18:49


Subtasks

Backport(バックポート) #4237: モバイルでコミュニティの管理権を渡そうとしたときに「管理権を渡す」リンクが残ったままNew(新規)

Associated revisions

Revision 30bc42ab (diff)
Added by Yuya Watanabe over 8 years ago

fixed to show admin confirm status (fixes #2257)

History

#1 Updated by Yuya Watanabe over 8 years ago

  • Status changed from New(新規) to Accepted(着手)
  • Assignee set to Yuya Watanabe

該当部分では、管理者でなければ「管理権を渡す」リンクを表示するようにする実装となっている。
admin_confirmはadminではないため最初の条件部分でtrueになってしまう。この部分を修正すればいいと思われる。

     if (!$communityMember->hasPosition('admin'))
     {
       $operation[] = link_to(__("Take over this %community%'s administrator to this member"),  'community/changeAdminRequest?id='.$community->getId().'&member_id='.$member->getId());
     }
     elseif ($communityMember->hasPosition('admin_confirm'))
     {
       $operation[] = __("You are taking over this %community%'s administrator to this member now.");
     }

#2 Updated by wa ta over 8 years ago

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

更新履歴 30bc42ab1940763a0bf8cca0094d401ab817a65b で適用されました。

#3 Updated by Minoru Takai over 8 years ago

  • Target version set to OpenPNE 3.7.0
  • 3.6 で発生するか set to Yes

#4 Updated by Minoru Takai over 8 years ago

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

レビューしました。

背景

この問題は #22843ae2a74 での修正に誤りがあったために生じています(当該部分の元実装)。

なお、これは携帯版での問題ですが、 #228 時点でPC版は以下のようになっているようです(ここではインデントをつけています)。

  • #228 時点の PC 版記述
    <?php if (!$communityMember->hasPosition('admin')) : ?>
      <?php if ($communityMember->hasPosition('admin_confirm')): ?>
        <?php echo __("You are taking over this %community%'s administrator to this member now.") ?>
      <?php else: ?>
        <?php echo link_to(__("Take over this %community%'s administrator to this member"), 'community/changeAdminRequest?id='.$community->getId().'&member_id='.$member->getId()) ?>
      <?php endif; ?>
    <?php else: ?>
     &nbsp;
    <?php endif; ?>
    

修正内容について

コミュニティメンバーに対して、

admin (何も表示しない)
admin_confirm 管理者交代依頼中
other 管理権を渡す

修正後は、上記の分岐を正しく表現できているので問題ないと判断します(分岐の書き方は #228 時点のPC版と同様になっています)。また、コーディング規約に反するような記述やミスも無いようなので問題ありません。

#5 Updated by isao sano about 8 years ago

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

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

#6 Updated by Chiharu Nakajima about 2 years ago

3.6未対応
3.8発生せず

Also available in: Atom PDF