Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

epubmaker: params["foo"] should be same result as params["epubmaker"]["foo"] #515

Closed
takahashim opened this issue Apr 4, 2016 · 4 comments
Milestone

Comments

@takahashim
Copy link
Collaborator

YAMLで @config["epubmaker"]["foo"] で指定した値をepubmakerの@producer.params["foo"]で参照できるようにしたいです。

期待される挙動・優先度

  • a) @config["foo"]が設定されていればそれを使い、設定されてなければ@config[maker]["foo"]を使う
  • b) @config[maker]["foo"]が設定されていればそれを使い、設定されてなければ@config["foo"]を使う

挙動としてはa)の方が素直だけど、b)で上書きできるのは便利そうです。
ちなみにepubmakerじゃなくても同様。

@kmuto
Copy link
Owner

kmuto commented Apr 5, 2016

本当はparamsじゃなくてconfigにしてうまく統一できればよかった とこっそり思っていますが、
用途としてはどっちのほうがありえそうでしょう(いまいちケースがわからず)。

@takahashim
Copy link
Collaborator Author

お、そうなんですね。ではReVIEW::Configureと統一する方向で考えてみます。
用途として分かりやすいところでは、EPUBでは拡張子を*.xhtmlにしたいけどWEBでは*.htmlにしたい、とかですね…。

@kmuto
Copy link
Owner

kmuto commented Apr 13, 2016

#540 マージを受けてほかに手を入れるところはあるかな。

@takahashim
Copy link
Collaborator Author

たぶんこれで大丈夫そうです。閉じますね。

@kdmsnr kdmsnr added this to the 2.0.0 milestone Apr 24, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants