OpenPNE 3

該当部分では、管理者でなければ「管理権を渡す」リンクを表示するようにする実装となっている。
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.");
     }

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

レビューしました。

背景¶

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

なお、これは携帯版での問題ですが、 #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; ?>
  

修正内容について¶

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

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

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

3.6未対応
3.8発生せず