puppet-php icon indicating copy to clipboard operation
puppet-php copied to clipboard

php.ini variables doesn't work using dwim method

Open wojtek-alef opened this issue 11 years ago • 5 comments

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'
}

wojtek-alef avatar May 14 '14 14:05 wojtek-alef

Can you check this one out @igalic ? :)

jippi avatar May 14 '14 19:05 jippi

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

igalic avatar May 16 '14 13:05 igalic

@wojtek-alef bump?

jippi avatar May 23 '14 18:05 jippi

looks like it's working when changing include

wojtek-alef avatar May 27 '14 08:05 wojtek-alef

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

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

igalic avatar May 27 '14 08:05 igalic