操作
Enhancement(機能追加・改善) #723
未完了Change name of OpenPNE.yml.sample to OpenPNE.yml.default (OpenPNE.yml.sample を OpenPNE.yml.default へ変えて欲しい)
開始日:
2010-02-22
期日:
進捗率:
0%
予定工数:
説明
Source¶
http://sns.openpne.jp/?m=pc&a=page_fh_diary&target_c_diary_id=23657 より転載
OpenPNE.yml.sample をcpではなくmv してしまったせいでSNSが設置できなかったという失敗が少なくとも2件あったみたい。 ※うち一件は自分のミス。 どうもsampleという名前だと、サンプル用であって本番では使わないよという印象を受けるので、名前を変えた方がいいんじゃないかと思う。 .sample => .default .base など そもそも、コピーしなくてもすむように OpenPNE.ymlを設置しておいてあげればいいんじゃないかと思うんだけど。 ProjectConfiguration.phpも。 これをしないということは、何か問題(たとえばセキュリティなどで)があるんだろうけど、少しでも設置の手間が少なくなるようにしたほうが、OpenPNEの普及には役立つんじゃないかな。
Shogo Kawahara さんが14年以上前に更新
- 題名 を OpenPNE.yml.sample => OpenPNE.yml.default がいいんじゃない? から OpenPNE.yml.sample を OpenPNE.yml.default へ変えて欲しい に変更
Shogo Kawahara さんがほぼ14年前に更新
- 題名 を OpenPNE.yml.sample を OpenPNE.yml.default へ変えて欲しい から Change name of OpenPNE.yml.sample to OpenPNE.yml.default (OpenPNE.yml.sample を OpenPNE.yml.default へ変えて欲しい) に変更
Shogo Kawahara さんがほぼ14年前に更新
OpenPNE.yml.sample という名称を変更するのはどうかと思います(根本的な解決ではないため)が、
少なくとも config/OpenPNE.yml.sample が無くても動作するように変更したいと考えました。
OpenPNE.yml.default を lib/config/config下におき、デフォルトでそれを読み、
config/OpenPNE.yml があった場合は、config/OpenPNE.yml とマージしたものを
設定とするのが最適かと思います。
この修正のデメリットは、 lib/config/config/OpenPNE.yml.default と config/OpenPNE.yml.sample
を常に同期する必要があるということでしょう。(開発においてミスが発生しやすくなるかもしれない)
Shogo Kawahara さんがほぼ14年前に更新
また、データ取得時に lib/config/opProjectConfiguration.class.php では
array_merge を行ったときに、YAMLファイルが NULL であったときの対策が
取られていない。
情報元: http://twitter.com/#!/miniturbo/status/9215274964226048
Thanks!!!
操作