Project

General

Profile

Enhancement(機能追加・改善) #723

Change name of OpenPNE.yml.sample to OpenPNE.yml.default (OpenPNE.yml.sample を OpenPNE.yml.default へ変えて欲しい)

Added by pnetan   over 9 years ago. Updated over 2 years ago.

Status:
New(新規)
Priority:
Normal(通常)
Assignee:
-
Target version:
Start date:
2010-02-22
Due date:
% Done:

0%


Description

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の普及には役立つんじゃないかな。 

Related issues

Related to OpenPNE 3 - Enhancement(機能追加・改善) #2825: OpenPNE.yml.sample ProjectConfiguration.class.php.sampleは「サンプル」なのに消すと動かなくなるのはバグ Invalid(無効) 2012-02-20

History

#1 Updated by Shogo Kawahara over 9 years ago

  • Subject changed from OpenPNE.yml.sample => OpenPNE.yml.default がいいんじゃない? to OpenPNE.yml.sample を OpenPNE.yml.default へ変えて欲しい

#2 Updated by Shogo Kawahara over 8 years ago

  • Target version set to OpenPNE 3.7.0

#3 Updated by Shogo Kawahara over 8 years ago

  • Subject changed from OpenPNE.yml.sample を OpenPNE.yml.default へ変えて欲しい to Change name of OpenPNE.yml.sample to OpenPNE.yml.default (OpenPNE.yml.sample を OpenPNE.yml.default へ変えて欲しい)

#4 Updated by Shogo Kawahara over 8 years ago

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
を常に同期する必要があるということでしょう。(開発においてミスが発生しやすくなるかもしれない)

#5 Updated by Shogo Kawahara over 8 years ago

また、データ取得時に lib/config/opProjectConfiguration.class.php では
array_merge を行ったときに、YAMLファイルが NULL であったときの対策が
取られていない。

情報元: http://twitter.com/#!/miniturbo/status/9215274964226048
Thanks!!!

#6 Updated by Shouta Kashiwagi over 7 years ago

  • Target version changed from OpenPNE 3.7.0 to 252

#7 Updated by Shouta Kashiwagi over 7 years ago

  • Target version changed from 252 to OpenPNE 3.8.x

#8 Updated by 守 手嶋 almost 7 years ago

  • Target version changed from OpenPNE 3.8.x to OpenPNE 3.9.0-old

安定版では実装できない内容。開発版で検討。

#9 Updated by kaoru n over 2 years ago

  • Target version changed from OpenPNE 3.9.0-old to OpenPNE 3.9.0

Also available in: Atom PDF