Skip to content
This repository has been archived by the owner on Oct 11, 2018. It is now read-only.

php.ini variables doesn't work using dwim method #67

Open
wojtek-alef opened this issue May 14, 2014 · 5 comments
Open

php.ini variables doesn't work using dwim method #67

wojtek-alef opened this issue May 14, 2014 · 5 comments

Comments

@wojtek-alef
Copy link
Contributor

php::config losses default $inifile value when calling from php::fpm::config

looks like

include ::php

cause $inifile gets empty after line 44 of manifests/config.pp.

it's working fine when file is defined ie.

php::fpm::config { 'display_errors':
  file => '/etc/php5/fpm/php.ini',
  setting => 'display_errors',
  value => 'On'
}
@jippi
Copy link
Owner

jippi commented May 14, 2014

Can you check this one out @igalic ? :)

@igalic
Copy link
Collaborator

igalic commented May 16, 2014

Strange, I thought I had tested for exactly that, from what I gather from my test: https://github.com/jippi/puppet-php/blob/master/spec/defines/fpm_config_spec.rb#L4-L6 you should probably only include php::fpm

@jippi
Copy link
Owner

jippi commented May 23, 2014

@wojtek-alef bump?

@wojtek-alef
Copy link
Contributor Author

looks like it's working when changing include

@igalic
Copy link
Collaborator

igalic commented May 27, 2014

This, right now, is due to the cascade of inheritances, I believe.

I'm not sure how to properly fix it, though

cegeka-jenkins pushed a commit to cegeka/puppet-php that referenced this issue Mar 21, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants