ここの情報は古いです。ご理解頂いた上でお取り扱いください。

Opened 12 years ago

Closed 12 years ago

Last modified 12 years ago

#1620 closed defect (fixed)

イベントに参加していない状態で「このイベントをマイフレンドに教える」を選択するとエラー

Reported by: ogawa Owned by: ebihara
Priority: major Milestone: OpenPNE2.10.2
Component: その他 Version: 2.10.x & 2.11.x
Keywords: OpenPNE2.8.7 OpenPNE2.11.2 Cc:

Description (last modified by ebihara)

■現象

携帯版イベントページで、イベントに参加していない状態で「このイベントをマイフレンドに教える」を選択すると、下記のエラーページへ遷移してしまう。

  • タイトル:エラー
  • 内容:このページにはアクセスすることが出来ません

また、PC版イベントページでは、イベント管理者だが参加者ではないメンバーが「イベントお知らせメッセージを送る」をクリックすると、権限エラーが表示されてしまう。

■原因

遷移元と遷移先の権限チェックが矛盾している。また、PC版と携帯版で仕様が統一されていない。

■修正内容

権限チェックを「イベントの閲覧権限があるかどうか」のみを見るように統一する。つまり閲覧権限さえあれば、イベントお知らせメッセージを送信できるようにする。

■関連情報

Change History (14)

comment:1 Changed 12 years ago by kiwa

Priority: minorcritical

comment:2 Changed 12 years ago by kiwa

Keywords: 再現待ち removed
Milestone: OpenPNE2.10.2
Priority: criticalmajor
Version: 2.8.x & 2.10.x & 2.11.x2.10.x & 2.11.x

現象が確認されました

■発生バージョン

  • 2.10.1
  • 2.11.1

※2.8系には携帯版に「イベントをマイフレンドに教える」機能がありませんでした

■再現手順

以下の条件のメンバーで携帯からイベントの「イベントをマイフレンドに教える」にアクセスする

  • コミュニティに参加していて、イベントにも参加している
  • コミュニティに参加しているが、イベントには参加していない
  • コミュニティにも参加していないし、イベントにも参加していない

■再現結果

  • コミュニティに参加していて、イベントにも参加している
    • ○ イベント紹介画面が表示される
  • コミュニティに参加しているが、イベントには参加していない
    • × エラーページが表示される
    • 「このページにはアクセスすることができません。」
  • コミュニティにも参加していないし、イベントにも参加していない
    • × エラーページが表示される
    • 「このページにはアクセスすることができません。」
    • × エラーページが表示される

comment:3 Changed 12 years ago by ebihara

Owner: changed from nobody to ebihara
Status: newassigned

やります。

comment:4 Changed 12 years ago by ebihara

Description: modified (diff)

PC版での挙動を調査してみましたが、

  • 「このイベントをマイフレンドに教える」リンク表示:イベント参加者かイベント管理者の場合
  • 「このイベントをマイフレンドに教える」リンク先表示:イベント参加者の場合

という権限チェックになっていました。
つまり、イベント管理者だが参加者ではないメンバーがアクセスした場合、権限エラーが出る形になってしまっています。

ただ、実装を見る限りでは、この機能はイベント参加者だけでなく、イベント閲覧権限を持つすべてのメンバーが使えることを意図しているようです。

というわけで、PC版および携帯版両方の権限チェックを、「閲覧権限を持つメンバー」という形に修正したいと思います。

comment:5 Changed 12 years ago by ebihara

Keywords: 確認中 added

以下のリビジョンで修正しました。ご確認ください。

comment:6 Changed 12 years ago by ogawa

Keywords: テスト待ち added; 確認中 removed

comment:7 Changed 12 years ago by kiwa

Keywords: テスト待ち removed

PC版および携帯版両方の権限チェックを、「閲覧権限を持つメンバー」という形に修正

となると、携帯版に「このイベントをマイフレンドに教える」機能がない2.8系も修正が必要だと思うのですが、2.8系のPC版は修正しないで大丈夫なのでしょうか?

comment:8 Changed 12 years ago by ebihara

確認してみます。

comment:9 Changed 12 years ago by ebihara

Keywords: OpenPNE2.8.7 added

2.8.xでも修正以前の 2.10.x, trunk と同様に、権限エラーが出てしまいました。

PC版の変更のみ、2.8.xに取り込みたいと思います。

comment:10 Changed 12 years ago by ebihara

Keywords: 確認中 added

r4744 で2.8.xのPC版に対して変更を取り込みました。ご確認ください。

comment:11 Changed 12 years ago by ogawa

Keywords: テスト待ち added; 確認中 removed

comment:12 Changed 12 years ago by kiwa

Resolution: fixed
Status: assignedclosed

確認しました。OKです。

comment:13 Changed 12 years ago by kiwa

Keywords: テスト待ち removed

comment:14 Changed 12 years ago by kiwa

Keywords: OpenPNE2.11.2 added
Note: See TracTickets for help on using tickets.