プロジェクト

全般

プロフィール

Backport(バックポート) #4058

完了

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

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

isao sano さんが約8年前に追加. 4年以上前に更新.

ステータス:
Fixed(完了)
優先度:
Normal(通常)
担当者:
対象バージョン:
開始日:
2016-11-21
期日:
進捗率:

100%

予定工数:

説明

詳細は #4055 を参照


関連するチケット 1 (0件未完了1件完了)

コピー先 OpenPNE 3 - Backport(バックポート) #4379: OpenPNE が http://example.com/sns/ のようにサブディレクトリ以下に設置されている場合に、通知センターからアクセスする日記やコミュニティのURLが正しくないFixed(完了)isao sano2016-11-21

操作

isao sano さんが約8年前に更新

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

isao sano さんが約8年前に更新

  • ステータスNew(新規) から Accepted(着手) に変更

isao sano さんが約8年前に更新

  • ステータスAccepted(着手) から Pending Review(レビュー待ち) に変更
  • 進捗率0 から 50 に変更

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

kaoru n さんが約8年前に更新

  • 対象バージョンOpenPNE 3.8.22 から OpenPNE 3.8.x に変更

kaoru n さんが約8年前に更新

  • 対象バージョンOpenPNE 3.8.x から OpenPNE 3.8.22 に変更

Shinichi Urabe さんが約8年前に更新

  • ステータスPending Review(レビュー待ち) から Rejected(差し戻し) に変更

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

kaoru n さんが約8年前に更新

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

kaoru n さんが約8年前に更新

  • ステータスRejected(差し戻し) から Pending Review(レビュー待ち) に変更
  • 担当者isao sano から kaoru n に変更

Shinichi Urabe さんが約8年前に更新

  • ステータスPending Review(レビュー待ち) から Rejected(差し戻し) に変更

#4055 と同様の理由

kaoru n さんが約8年前に更新

  • ステータスRejected(差し戻し) から Pending Review(レビュー待ち) に変更

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

Shinichi Urabe さんが約8年前に更新

  • ステータスPending Review(レビュー待ち) から Pending Testing(テスト待ち) に変更
  • 進捗率50 から 70 に変更

Chiharu Nakajima さんが約8年前に更新

  • ステータスPending Testing(テスト待ち) から Rejected(差し戻し) に変更
  • 進捗率70 から 50 に変更

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

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

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

kaoru n さんが約8年前に更新

  • 対象バージョンOpenPNE 3.8.22 から OpenPNE 3.8.x に変更

kaoru n さんが約8年前に更新

opCommunityTopicPlugin(ver1.1.2.1), opMessagePlugin(ver2.0.0) については、それぞれプラグイン内でサブディレクトリに考慮しているため、通知センター内でサブディレクトリを付与することで二重になってしまう。

opCommunityTopicPlugin(ver1.1.2.1)
https://github.com/tejimaya/opCommunityTopicPlugin/blob/dev-1.1.x/lib/util/opCommunityTopicPluginUtil.class.php#L7
opMessagePlugin(ver2.0.0)
https://github.com/tejimaya/opMessagePlugin/blob/master/lib/util/opMessagePluginUtil.class.php#L22

プラグインへの影響を考慮すると、リリースするかどうかを検討する必要があるので、リリースを保留にします。

kaoru n さんが約8年前に更新

#4055-13 のように修正することとします

Youichi Kimura さんがほぼ8年前に更新

  • ステータスRejected(差し戻し) から Pending Review(レビュー待ち) に変更
  • 担当者kaoru n から Youichi Kimura に変更

#4055-17 をバックポートする Pull Request を作成しました
https://github.com/openpne/OpenPNE3/pull/448

Shinichi Urabe さんがほぼ8年前に更新

  • ステータスPending Review(レビュー待ち) から Rejected(差し戻し) に変更

#4055-19 と同様の理由で差し戻します

kaoru n さんが約7年前に更新

  • 親チケット#4055 にセット

kaoru n さんがほぼ5年前に更新

kaoru n さんがほぼ5年前に更新

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

https://github.com/openpne/OpenPNE3/pull/448
についてはクローズしています

kaoru n さんがほぼ5年前に更新

  • ステータスRejected(差し戻し) から Pending Review(レビュー待ち) に変更

isao sano さんがほぼ5年前に更新

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

Rimpei Ogawa さんが4年以上前に更新

  • ステータスPending Review(レビュー待ち) から Pending Testing(テスト待ち) に変更
  • 進捗率50 から 70 に変更

kaoru n さんが4年以上前に更新

  • 対象バージョンOpenPNE 3.8.x から OpenPNE 3.8.36 に変更

isao sano さんが4年以上前に更新

  • ステータスPending Testing(テスト待ち) から Pending Merge(マージ待ち) に変更
  • 進捗率70 から 80 に変更

確認しました。問題ありません。

kaoru n さんが4年以上前に更新

  • ステータスPending Merge(マージ待ち) から Fixed(完了) に変更
  • 進捗率80 から 100 に変更

マージしました

他の形式にエクスポート: Atom PDF