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

Add parameter wp_additional_inline_config #38

Open
atxulo opened this issue Apr 11, 2014 · 2 comments
Open

Add parameter wp_additional_inline_config #38

atxulo opened this issue Apr 11, 2014 · 2 comments

Comments

@atxulo
Copy link

atxulo commented Apr 11, 2014

With automatic parameter lookup you don't need to create a puppet recipe to install wordpress using this module any longer, because if you add the required parameters to hiera, the agent will launch the class with the values loaded from hiera.

But in this scenario, you cannot use the wp_additional_config parameter as you don't have a module with the template, so if you need to add additional config to wordpress, you have to write your recipe.

In this situation, a new parameter called wp_additional_inline_config would be helpful, so you could avoid writting the recipe, loading the additional config into hiera.

Related: #24

@nicolasbrechet
Copy link

Hi,

I've added this parameter, could you test if that's what you were thinking of ?

@atxulo
Copy link
Author

atxulo commented Nov 10, 2014

It seems to fit my needs (WP_PROXY_USERNAME, WP_PROXY_PASSWORD, DISALLOW_FILE_MODS,WP_AUTO_UPDATE_CORE...) 👍

I can't think of a setting that doesn't match define('<%=key%>', <%=value%>) and it looks that it should work with all these settings (http://codex.wordpress.org/Editing_wp-config.php) so i'd like to see this change merged 👍

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

2 participants