Project

General

Profile

Enhancement(機能追加・改善) #4064

OpenPNE が http://example.com/sns/ のようにサブディレクトリ以下に設置されている場合に、通知センターからアクセスするURLが正しくない問題に対する opLikePlugin の独自修正を取り除く

Added by kaoru n over 2 years ago. Updated over 2 years ago.

Status:
Won't fix(対応せず)
Priority:
Normal(通常)
Assignee:
Start date:
2016-12-02
Due date:
% Done:

0%


Description

Overview (概要)

#4055 にて報告されている「OpenPNE が http://example.com/sns/ のようにサブディレクトリ以下に設置されている場合に、通知センターからアクセスする日記やコミュニティのURLが正しくない」問題の修正のため、
https://github.com/tejimaya/opLikePlugin/commit/9f99d3619934961c18323a17bc3f66f90b48e5f3
にて、opLikePlugin にて独自に修正が行われている。

#3365, #4053 にて、_layout.php に JavaScriptコードから取得できる baseUrl が追加され、
#4055 にて、通知センターが baseUrl を参照してURLを生成するようになるため、
https://github.com/tejimaya/opLikePlugin/commit/9f99d3619934961c18323a17bc3f66f90b48e5f3
のコミットは不要となる。

Spec (仕様)

https://github.com/tejimaya/opLikePlugin/commit/9f99d3619934961c18323a17bc3f66f90b48e5f3
の修正分を取り除く


Related issues

Related to OpenPNE 3 - Bug(バグ) #4055: OpenPNE が http://example.com/sns/ のようにサブディレクトリ以下に設置されている場合に、通知センターからアクセスする日記やコミュニティのURLが正しくない Pending Testing(テスト待ち) 2016-11-21
Related to OpenPNE 3 - Enhancement(機能追加・改善) #3365: OpenPNEが設置されているURLをJavaScriptコードから取得できるようにする Won't fix(対応せず) 2013-06-17
Related to opSkinThemePlugin - Enhancement(機能追加・改善) #4053: OpenPNEが設置されているURLをJavaScriptコードから取得できるようにする Pending Review(レビュー待ち) 2016-11-17

History

#1 Updated by kaoru n over 2 years ago

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

#2 Updated by kaoru n over 2 years ago

#3 Updated by kaoru n over 2 years ago

#4 Updated by kaoru n over 2 years ago

  • Status changed from New(新規) to Pending Review(レビュー待ち)
  • % Done changed from 0 to 50

https://github.com/tejimaya/opLikePlugin/pull/7
にてプルリクエストしました
※ この修正適用前にいいね!通知されたデータに関しては修正されません。

#5 Updated by kaoru n over 2 years ago

https://github.com/tejimaya/opLikePlugin/pull/7
に、 #4064 修正前に投稿されたデータのURLから $baseUrl として設定された文字列を取り除く migration を追加しました。

#6 Updated by Youichi Kimura over 2 years ago

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

レビューOKです

#7 Updated by kaoru n over 2 years ago

  • Status changed from Pending Testing(テスト待ち) to Rejected(差し戻し)
  • % Done changed from 70 to 50

#4055-13 のように修正するため、差し戻しとします

#8 Updated by kaoru n over 2 years ago

  • Status changed from Rejected(差し戻し) to Won't fix(対応せず)
  • % Done changed from 50 to 0

#4055-13 により、プラグイン側で独自にサブディレクトリを考慮している場合はそのままでよいということになったため、このチケットは対応しないこととします。

Also available in: Atom PDF