ディレクトリの位置をハードコードしているファイルが多数ある†
- 元タイトル: pukiwiki.ini.php: 参照する/されるディレクトリのdefineが足りない (image/など)
- 元タイトル: ./image/ の pukiwiki.ini.phpでの定義
- ページ: BugTrack
- 投稿者: upk
- 優先順位: 重要
- 状態: 完了
- カテゴリー: その他
- 投稿日: 2003-12-23 (火) 14:36:33
- バージョン:
メッセージ†
pukiwiki.ini.php で ./image/ に対する定義が無いため、自作プラグインなどで、ハードコーディングされているものが目立つため、
define('IMAGE_DIR','./image/');
を定義し統一した方が良いと思います。
- imageだけの問題ではない様なので、まとめて片付けます。ただし、face/ については役割付けが重複している image/ 以下に集約(ディレクトリごと移動)すべきと考えますので除外させて下さい。 -- henoheno
- 追加しました。SUB_DIR は func.php などのサブルーチン (ライブラリ) の場所、IMAGE_DIR は外部参照させたい画像の場所としてまずは定義しました。(名称を直すのは簡単なので、別途コメントを下さい) 提案ありがとうございました。 -- henoheno
- ということで、このページはハードコーディングされているディレクトリ情報を駆逐するページになりました :) -- henoheno
- あと目に見えている作業対象は IMAGE_DIR です。スキンにおける SKIN_DIR はどうしたものか XD -- henoheno
- IMAGE_DIR は終了(開発日記/2004-06-22) 。もれがあったらお知らせ下さい。 -- henoheno
- init.php 320行付近、rules.ini.php を読んでいる部分が漏れてます。 -- 三浦克介
- コメントありがとうございます。rules.phpですが、ユーザーごとの(あるいは、PukiWiki一本ごとの)カスタマイズを意図している部分であるため、SUB_DIRよりはDATA_HOME(未実装)にすべきかと思いますので、DATA_HOMEの実装の際に一緒に直させてください。 -- henoheno
- henohenoさんがかなり整理してくださったので、PukiWiki/WikiFarmに載せたWikiFarmのたたき台を楽に作ることができました。このたたき台では、本体のプログラムをSUB_DIRではなく、CORE_DIRとしてます。 -- 三浦克介
DATA_HOMEの追加†
- 派生: BugTrack/656 SKIN_FILEの定義(DATA_HOMEとSKIN_DIRの2つを用いるのは冗長では?)
imageディレクトリのハードコード (IMAGE_DIRと置換)†
- default.ini.php(Ver1.10)でのフェースマークの定義が、IMAGE_DIRを使わずにディレクトリ位置がハードコードされているようです。(ステータスが完了となっていましたが同じトピックでしたので、追記させていただきます。) -- jjyun
- お疲れさまです。上記 IMAGE_DIRの件、早速の対応ありがとうございます。これとは別件で、skinディレクトリの pukiwiki.skin.ja.php などで、SKIN_DIRが呼ばれず、skin/default.ja.cssなどとハードコーディングされている部分を見付けましたので御連絡します。 -- jjyun
- amazon.inc.phpでimage/のハードコードがありました。 -- Ratbeta
--- amazon.inc.php 2004-07-31 12:09:19.000000000 +0900
+++ amazon.inc.php 2004-09-20 14:31:08.915956800 +0900
@@ -59,7 +59,7 @@
define('AMAZON_EXPIRE_TIT',356);
/////////////////////////////////////////////////
// 画像なしの場合の画像
-define('NO_IMAGE','./image/noimage.jpg');
+define('NO_IMAGE',IMAGE_DIR.'noimage.jpg');
/////////////////////////////////////////////////
// amazon ショップ
define('AMAZON_SHOP','http://www.amazon.co.jp/exec/obidos/ASIN/');
- 上記 amazonプラグインについて、今までnoimage.jpg 自体が存在していない理由が不明であったため手をつけずにいましたが、BugTrack/742 にて新規に同梱したことをふまえ、IMAGE_DIR を見る様に修正しました。 -- henoheo