Ticket #598 (closed バグ指摘: 修正済)

Opened 11 years ago

Last modified 10 years ago

SITE_URLの設定と異なるURLでのアクセス時にuser_data/以下のページが表示されない

Reported by: shutta Owned by: shutta
Priority: Milestone: EC-CUBE2.4.4
Component: フロント Version: 2.4.2
Keywords: Cc:
修正済み:

Description

 http://xoops.ec-cube.net/modules/newbb/viewtopic.php?topic_id=3256&forum=10 のように、

SITE_URLを  http://example.com/ としていた場合に、

 http://example.com/user_data/test.php

は正常に表示されるが、

 http://www.example.com/user_data/test.php

だとコンテンツ部分が表示されない。

本来ならSITE_URLに記述されているURLでアクセスすべきですが、
回避できるんであればと思い登録しました。

Change History

comment:1 Changed 11 years ago by shutta

  • Owner changed from somebody to shutta
  • Status changed from new to assigned

comment:2 Changed 11 years ago by shutta

  • Status changed from assigned to closed
  • Resolution set to 修正済

仕様の違い(dtb_pagelayoutのurlの値が、comu-ver2は、SITE_URLからの相対パス、正式版は絶対URL)により、

comu-ver2 r18549, r18551

version2_4-dev r18550

で、それぞれ対応

comment:3 Changed 11 years ago by kajiwara

  • Status changed from closed to reopened
  • Resolution 修正済 deleted
  • Milestone set to EC-CUBE2.4.3

修正ありがとうございます。
こちら、マイルストーンを2.4.3とし、version2_4の方に取込が完了した時点で、最終的に修正済みにさせていただきたいと思います。

よろしくお願いします。

comment:4 follow-up: ↓ 5 Changed 11 years ago by kajiwara

  • Priority changed from to
  • Milestone changed from EC-CUBE2.4.3 to EC-CUBE2.4.4

意外と確認すべき事項が多く、若干の時間を要すると思われるため、次期バージョンにマイルストーンを変更させていただきたいと思います。
(r18550の対応では不十分かもしれません。コミュニティ版では正常に動作しますので、そちらの確認をしつつ、対応することになると思います。)

また、EC-CUBE全体の優先順位から考えまして、優先度を下げさせていただきたいと思います。
大変申し訳ございません。

comment:5 in reply to: ↑ 4 Changed 11 years ago by shutta

有用度の低い修正ですので、優先順位は最初から低に設定しておけばよかったですね。すみません。

関連して、dtb_pagelayout の urlカラムの仕様ですが、user_data以下のページを登録する際に、

正式版

絶対URLで登録される。


コミュニティ版

SITE_URL部分を削った相対パスで登録される。
seasoftさんが、r17416, r17487, r17602 あたりで仕様変更されている。


と、仕様が異なっているのですが、どちらかに統一した方が良いような気がします。

どちらの版もuser_data以外のurlは相対パスになっているので、コミュニティ版のように全てのurlが相対パスになっている方が混乱が無いと思いますが如何でしょうか?

同じ内容の修正をしようと思っても、それぞれに実装と検証をしないといけないのは効率が悪いので、可能であれば、どちらかに統一されていればと思いますのでご検討頂けますでしょうか。

comment:6 Changed 11 years ago by shutta

とりあえず、2_4-devにも、 r17416, r17487, r17602 を取り込んで、コミュニティ版と同じ仕様になるようにしておきました。(r18606)

comment:7 Changed 10 years ago by nanasess

  • Status changed from reopened to closed
  • Resolution set to 修正済

r18703 でマージしました

Note: See TracTickets for help on using tickets.