Backport（バックポート） #4058: OpenPNE が http://example.com/sns/ のようにサブディレクトリ以下に設置されている場合に、通知センターからアクセスする日記やコミュニティのURLが正しくない - OpenPNE 3 - OpenPNE Issue Tracking System

Backport（バックポート） #4058

Bug（バグ） #4055: OpenPNE が http://example.com/sns/ のようにサブディレクトリ以下に設置されている場合に、通知センターからアクセスする日記やコミュニティのURLが正しくない

OpenPNE が http://example.com/sns/ のようにサブディレクトリ以下に設置されている場合に、通知センターからアクセスする日記やコミュニティのURLが正しくない

Added by isao sano almost 8 years ago. Updated about 4 years ago.

Status:

Fixed（完了）

Priority:

Normal（通常）

Assignee:

Youichi Kimura

Target version:

OpenPNE 3.8.36

Start date:

2016-11-21

Due date:

% Done:

100%

Description

詳細は #4055 を参照

Related issues

Associated revisions

Revision b337ffc6 (diff)
Added by Youichi Kimura over 4 years ago

allow internal uri (the 'url_for()' format) for notification url (fixes #4058, BP from #4055)

(cherry picked from commit b6a97087aefa7ef4ffef2e6bc1001c9d88816348)

Revision 9bda3736 (diff)
Added by Youichi Kimura over 4 years ago

make case-insensitive to notification URI scheme (refs #4058, BP from #4055)

Revision 9c066cd0
Added by kaoru n about 4 years ago

Merge pull request #571 from nishizoe/t-4058

allow internal uri (the 'url_for()' format) for notification url (fixes #4058, BP from #4055)

History

#1 Updated by isao sano almost 8 years ago

Related to Bug（バグ） #4055: OpenPNE が http://example.com/sns/ のようにサブディレクトリ以下に設置されている場合に、通知センターからアクセスする日記やコミュニティのURLが正しくない added

#2 Updated by isao sano almost 8 years ago

Status changed from New（新規） to Accepted（着手）

#3 Updated by isao sano almost 8 years ago

Status changed from Accepted（着手） to Pending Review（レビュー待ち）
% Done changed from 0 to 50

https://github.com/openpne/OpenPNE3/pull/394
にてプルリクエストしました。

#5 Updated by kaoru n almost 8 years ago

Target version changed from OpenPNE 3.8.22 to OpenPNE 3.8.x

#6 Updated by kaoru n almost 8 years ago

Target version changed from OpenPNE 3.8.x to OpenPNE 3.8.22

#7 Updated by Shinichi Urabe almost 8 years ago

Status changed from Pending Review（レビュー待ち） to Rejected（差し戻し）

親チケットと同様の理由で差し戻し

#8 Updated by kaoru n almost 8 years ago

https://github.com/openpne/OpenPNE3/blob/stable-3.8.x/apps/pc_frontend/templates/_layout.php#L22
のように、openpne.baseUrl は末尾に '/' が一つ付与されるため、data-location-url の先頭の '/' を取り除いて文字列を連結するようにプルリクエストを更新しました。
https://github.com/openpne/OpenPNE3/pull/394

#9 Updated by kaoru n almost 8 years ago

Status changed from Rejected（差し戻し） to Pending Review（レビュー待ち）
Assignee changed from isao sano to kaoru n

#10 Updated by Shinichi Urabe almost 8 years ago

Status changed from Pending Review（レビュー待ち） to Rejected（差し戻し）

#4055 と同様の理由

#11 Updated by kaoru n almost 8 years ago

Status changed from Rejected（差し戻し） to Pending Review（レビュー待ち）

変換文字列をダブルクォートではなくシングルクオートで囲むように修正しました
https://github.com/openpne/OpenPNE3/pull/394

#12 Updated by Shinichi Urabe almost 8 years ago

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

#13 Updated by Chiharu Nakajima almost 8 years ago

Status changed from Pending Testing（テスト待ち） to Rejected（差し戻し）
% Done changed from 70 to 50

以下のように、コミュニティトピックへのリンクとメッセージへのリンクのURLが正しくありません（日記への遷移はOKです）。

http://example.com/sns/ がベースURLの場合
自分の日記にコメントが付いた時の通知『１件の日記に新着コメントがあります』→ http://example.com/sns/diary/1 となる（OK）
自分が作ったトピックにコメントが付いた時の通知『１件のトピックに新着コメントがあります』→ http://example.com/sns/sns/communityTopic/1 となる（NG）
自分にメッセージが届いた時『新着メッセージが１件あります！』→ http://example.com/sns/sns/message/read/1 となる（NG）

動作確認を行ったプラグインのバージョン
opDiaryPlugin(ver1.5.1.1)
opCommunityTopicPlugin(ver1.1.2.1)
opMessagePlugin(ver2.0.0)