Ticket #834 (assigned バグ指摘) — at Version 11

Opened 13 years ago

Last modified 13 years ago

パラメータの定数名に「URL」を含むにもかかわらず、パスのみのものがある

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
    • 先頭は「(スキーム名):」
  • *_URL_PATH: URL における url-path 相当 (絶対パス)
    • 先頭は「/」
    • *_URLPATH へ変更を検討中
  • *_HTML_PATH: URL 上の EC-CUBE の /html/ からの相対パス
    • 先頭に「/」を含まない。(/html/ と同一の場合、空文字)
    • 命名に改善余地あり
  • *_REALFILE: (サーバの)ファイルシステム上のファイルの絶対パス
    • 先頭は「/」
  • *_REALDIR: (サーバの)ファイルシステム上のディレクトリの絶対パス
    • 先頭は「/」。末尾は「/」。
  • *_DIR: 上記に該当しない断片的なディレクトリ情報。
    • 先頭に「/」を含まない。
  • *_FILE: 上記に該当しない断片的なファイル情報。
  • *_PATH: 上記に該当しないもの。上記の複数に該当するもの。

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:3 Changed 13 years ago by Seasoft

  • Description modified (diff)

comment:4 Changed 13 years ago by Seasoft

  • Description modified (diff)

comment:5 Changed 13 years ago by Seasoft

  • Description modified (diff)

comment:6 Changed 13 years ago by Seasoft

  • Description modified (diff)

comment:7 Changed 13 years ago by Seasoft

  • Description modified (diff)

comment:8 Changed 13 years ago by Seasoft

  • Description modified (diff)

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:11 Changed 13 years ago by Seasoft

  • Description modified (diff)
Note: See TracTickets for help on using tickets.