Ticket #834 (assigned バグ指摘) — at Version 15
パスに関わるパラメータ名が不適切
Reported by: | Seasoft | Owned by: | Seasoft |
---|---|---|---|
Priority: | 中 | Milestone: | EC-CUBE2.11.0 |
Component: | その他 | Version: | 2.5-dev |
Keywords: | Cc: | ||
修正済み: |
Description (last modified by Seasoft) (diff)
問題点
- 処理によっては、URL かパスかを意識する必要があり、実装者の意図しない実装に繋がる状況である。
- 命名規則が不明確なため、新しいパラメータを決める際に迷い・揺らぎが生じやすい。
解決策
下記の通りに命名規則を割り当てる方向で検討。
- *_URL: URL
- 先頭は「スキーム名:」
- *_URLPATH: URL における url-path 相当 (絶対パス)
- 先頭は「/」
- *_HTML_PATH: URL 上の EC-CUBE の /html/ からの相対パス
- 先頭に「/」を含まない。(/html/ と同一の場合、空文字)
- 命名に改善余地あり (参考: concrete5=DIR_REL)
- *_REALFILE: (サーバの)ファイルシステム上のファイルの絶対パス
- 先頭は「/」
- *_REALDIR: (サーバの)ファイルシステム上のディレクトリの絶対パス
- 先頭は「/」。末尾は「/」。
- *_DIR: 上記に該当しない断片的なディレクトリ情報。
- 先頭に「/」を含まない。
- *_FILE: 上記に該当しない断片的なファイル情報。
- *_PATH: 上記に該当しないもの。上記の複数に該当するもの。
課題
- 上記太字部分
- パラメータ名 URL_PATH が妥当なのか怪しい。BASE_URLPATH (使用頻度が高いのに長い?) などへの変更を検討。
Change History
comment:1 Changed 13 years ago by nanasess
- Milestone changed from EC-CUBE2.5.0beta to EC-CUBE2.6.0(仮)
comment:2 Changed 13 years ago by Seasoft
- Owner changed from somebody to Seasoft
- Status changed from new to assigned
- Milestone changed from EC-CUBE2.6.0(仮) to EC-CUBE2.5.0beta
comment:9 follow-up: ↓ 10 Changed 13 years ago by nanasess
- Description modified (diff)
初回インストール時, インストーラで 定数 URL_PATH が未定義となるようです
comment:10 in reply to: ↑ 9 Changed 13 years ago by Seasoft
nanasess への返信
初回インストール時, インストーラで 定数 URL_PATH が未定義となるようです
もしかすると、CSS 乱れにも影響がありそうですか?
そうでしたら、本チケット以前からの不具合だと思います。
comment:14 Changed 13 years ago by Seasoft
- Summary changed from パラメータの定数名に「URL」を含むにもかかわらず、パスのみのものがある to パスに関わるパラメータ名が不適切
Note: See
TracTickets for help on using
tickets.