WebsiteURL、 WebsitePathタグに id 属性対応を追加する

Description

https://movabletype.atlassian.net/browse/DOC-176 より。BlogURL は id 対応しているが、 WebsiteURL は id 対応していないので追加する(かどうかをまずは検討する)。

Checklists

None

Activity

Show:

Kenichi IshigakiJuly 31, 2024 at 6:12 AM

こちらでマージしておきました。対応完了ということでクローズします。

Chieko MatsuoJuly 25, 2024 at 11:15 PM

なるほど。Perl と PHP とで id に存在しない ID を入れたときの挙動が異なる件も含め、了解しました。

マージよろしくお願いします。

Kenichi IshigakiJuly 25, 2024 at 9:59 AM

ありがとうございます。これ悩ましいのですが、歴史的に見ると 2009 年に GitHub に移行した時点で PHP 側の WebsitePath は id 属性の対応があり、その当時は指定した id の Website/Blog のサイトパスをそのまま返すようになっていたのですが、2013年の https://movabletype.atlassian.net/browse/MTC-20410 によって、id指定して読み込んだサイトないしコンテキストに保存されているサイトが Blog ならその親である Website に切り替える修正が入って、いまに至っています(ダイナミックで挙動確認していただければ様子がつかめるはずです)。ご提案通りの修正を入れると既存のタグの挙動が変わってしまうので、いささか気持ち悪いながらも本件の対応はそのままとしたいとおもいますが、いかがでしょう?(この挙動が気に入らない方は BlogURL/BlogSitePath 由来の SiteURL / SitePath を使っていただければこの問題を回避できます)

なお、厳密には Perl と PHP とで id に存在しない ID を入れたときの挙動が異なるのですが、影響する範囲を調べてまわる余裕がないのでひとまず見ない振りをしていただければと。

Chieko MatsuoJuly 25, 2024 at 7:58 AM

WebsiteURL、 WebsitePath のテンプレートタグで id 属性を利用できることを確認しました。

ところで、今のMTではBlogは存在しなくなっていて、親サイトも子サイトも「サイト」だと思うのですが、id に子サイトが該当する値を設定した場合には、WebsiteURL、 WebsitePathが表示したのは、設定した子サイトの親サイトのURLおよびPathでした。

子サイトであっても、サイトなので、そのURL/Pathを表示するのが妥当ではないかと思ったのですが、いかがでしょうか?

okanumaJuly 25, 2024 at 6:59 AM

QAをお願いします。
WebsiteURL、 WebsitePath のテンプレートタグで id 属性を利用できるように修正したので、ご確認をお願いします。

Implemented

Details

Assignee

Reporter

Fix versions

Release Notes

Priority

Smart Checklist

Created October 18, 2023 at 6:17 AM
Updated August 16, 2024 at 4:48 AM
Resolved July 31, 2024 at 6:12 AM