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

Opened 9 years ago

Last modified 9 years ago

パスに関わるパラメータ名が不適切

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

wiki/EC-CUBE標準規約 へ展開しました。以降の最新情報は、そちらを参照してください。

Change History

comment:1 Changed 9 years ago by nanasess

  • Milestone changed from EC-CUBE2.5.0beta to EC-CUBE2.6.0(仮)

comment:2 Changed 9 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 9 years ago by Seasoft

  • Description modified (diff)

comment:4 Changed 9 years ago by Seasoft

  • Description modified (diff)

comment:5 Changed 9 years ago by Seasoft

  • Description modified (diff)

comment:6 Changed 9 years ago by Seasoft

  • Description modified (diff)

comment:7 Changed 9 years ago by Seasoft

  • Description modified (diff)

comment:8 Changed 9 years ago by Seasoft

  • Description modified (diff)

comment:9 follow-up: ↓ 10 Changed 9 years ago by nanasess

  • Description modified (diff)

初回インストール時, インストーラで 定数 URL_PATH が未定義となるようです

comment:10 in reply to: ↑ 9 Changed 9 years ago by Seasoft

nanasess への返信

初回インストール時, インストーラで 定数 URL_PATH が未定義となるようです

もしかすると、CSS 乱れにも影響がありそうですか?
そうでしたら、本チケット以前からの不具合だと思います。

comment:11 Changed 9 years ago by Seasoft

  • Description modified (diff)

comment:12 Changed 9 years ago by Seasoft

  • Description modified (diff)

comment:13 Changed 9 years ago by Seasoft

  • Description modified (diff)

comment:14 Changed 9 years ago by Seasoft

  • Summary changed from パラメータの定数名に「URL」を含むにもかかわらず、パスのみのものがある to パスに関わるパラメータ名が不適切

comment:15 Changed 9 years ago by Seasoft

  • Description modified (diff)

comment:16 Changed 9 years ago by Seasoft

  • Description modified (diff)

comment:17 Changed 9 years ago by Seasoft

  • Description modified (diff)
  • Milestone changed from EC-CUBE2.5.0beta to EC-CUBE2.5.0

comment:18 Changed 9 years ago by Seasoft

  • Status changed from assigned to closed
  • Resolution set to 修正済
  • Description modified (diff)
Note: See TracTickets for help on using tickets.