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

Opened 8 years ago

Closed 8 years ago

#4084 closed defect (fixed)

携帯日記コメント投稿時権限エラーメッセージ違い

Reported by: imamura623 Owned by: kudo
Priority: minor Milestone: OpenPNE2.14.2
Component: 指定しない Version: 2.12.x & 2.14.x
Keywords: OpenPNE2.12.15 Cc:

Description (last modified by imamura623)

■現象

携帯版・日記へのコメント送信時、文章入力中に日記の公開レベルが変更されていると、日記閲覧権限エラーの「この日記にはアクセスできません」ではなく「申し訳ございませんがこのメンバーのページにはアクセスできません」と、アクセスブロック時のメッセージが表示されてしまう。

■発生バージョン

  • Ver2.14beta4
  • Ver.2.12.12

■再現手順

  • 1.携帯にてコメント投稿時に公開範囲を変更する
  • 2.「この日記にはアクセスできません」ではなく「このページにはアクセスできません」と表示される

■環境

  • Firefox3のFireMobileSimulatorで確認

■修正内容

  • エラーページのリダイレクト先をアクセスブロックエラーページからアクセス権限エラーページに変更

■関連情報

http://sns.openpne.jp/?m=pc&a=page_fh_diary&target_c_diary_id=20859より転載

携帯版・日記へのコメント送信時、
文章入力中に日記の公開レベルが変更されていると、
日記閲覧権限エラーの「この日記にはアクセスできません」ではなく
「申し訳ございませんがこのメンバーのページにはアクセスできません」と、
アクセスブロック時のメッセージが表示されてしまう。

原因箇所
webapp\modules\ktai\do\fh_diary_insert_c_diary_comment.php(36)

公開レベルエラー時に
openpne_redirect('ktai', 'page_h_access_block');
とアクセスブロック通知ページへと飛ばしていますが、
ここはPC版と同様、
openpne_redirect('pc', 'page_h_err_diary_access');
ではないかと思います。

当方OpenPNE v2.13.8ですが、2.14beta2も同じ処理でした。

どうぞ対応よろしくお願いいたします。

※
こちらも参加者の方から問合せがあり個別対応しました。
実際にはアクセス拒否されていないのに「アクセス拒否されたかも…」と参加者間のいらぬ誤解を生んでいました…。

Change History (17)

comment:1 Changed 8 years ago by imamura623

Keywords: 再現待ち added

comment:2 Changed 8 years ago by imamura623

Description: modified (diff)
Keywords: 再現待ち removed
Milestone: OpenPNE2.14RC1
Version: 2.14.x2.12.x & 2.14.x

再現テスト実施しました。

comment:3 Changed 8 years ago by kiwa

Milestone: OpenPNE2.14RC1

comment:4 Changed 8 years ago by imamura623

Keywords: OpenPNE2.12.15 added
Milestone: OpenPNE2.14.2

comment:5 Changed 8 years ago by kudo

Owner: changed from nobody to kudo
Status: newassigned

comment:6 Changed 8 years ago by kudo

Keywords: 確認中 added

r12841,r12842にてtrunk

r12843にて2.12

r12844にて2.14を修正しました。確認お願いします。

comment:7 Changed 8 years ago by imamura623

動作テストしました。問題ありません。

comment:8 Changed 8 years ago by ogawa

Keywords: 差し戻し added; 確認中 removed

携帯版では、$WORD_DIARY ではなく $WORD_DIARY_HALF を使うようにしてください。

ktai_page_fh_diary で同様の権限チェック箇所があるので同じ処理を行うように合わせてください。今回の変更でページ作成済みなので、ktai_page_fh_diary を openpne_redirect() に合わせる変更で OK だと思います。

comment:9 in reply to:  8 Changed 8 years ago by imamura623

Keywords: 確認中 added; 差し戻し removed

Replying to ogawa:

携帯版では、$WORD_DIARY ではなく $WORD_DIARY_HALF を使うようにしてください。

ktai_page_fh_diary で同様の権限チェック箇所があるので同じ処理を行うように合わせてください。今回の変更でページ作成済みなので、ktai_page_fh_diary を openpne_redirect() に合わせる変更で OK だと思います。

修正いたしました。

  • r12996でstable-2.14.xにコミット
  • r12997でtrunkにコミット
  • r12998でstable-2.12.xにコミット

comment:10 Changed 8 years ago by ogawa

Keywords: 差し戻し added; 確認中 removed

trunk のみ、webapp/modules/ktai/page/fh_diary.php の変更行の末尾に不要な空白が追加されています。

comment:11 Changed 8 years ago by imamura623

Keywords: 確認中 added; 差し戻し removed

r12999でwebapp/modules/ktai/page/fh_diary.phpに含まれていた不要な空白を削除しました。

comment:12 Changed 8 years ago by imamura623

Description: modified (diff)

comment:13 Changed 8 years ago by ogawa

Keywords: 差し戻し added; 確認中 removed

description の「修正内容」にある、

  • PC,ktaiともに修正

は間違いではないでしょうか。

comment:14 Changed 8 years ago by imamura623

Description: modified (diff)
Keywords: 確認中 added; 差し戻し removed

失礼しました。携帯のみの対応です。

comment:15 Changed 8 years ago by ogawa

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

確認しました。ktai_page_fh_diary も合わせて再度テストしてください。

comment:16 Changed 8 years ago by imamura623

Keywords: テスト待ち removed

ktai_page_fh_diary,ktai_do_fh_diary_insert_c_diary_commentともに確認しました。問題ありません。

修正完了としチケットクローズします。

comment:17 Changed 8 years ago by imamura623

Resolution: fixed
Status: assignedclosed
Note: See TracTickets for help on using tickets.