Unable to get all items of a package config

[billmn]

I've created a package with it's own config file ... I can retrieve an item of this configuration with :

Config::get('package::config_key')

but I cannot retrieve all items of this package using :

Config::get('package')

After this I think that can be useful add this tip to the docs

[bencorlett]

Hi @billmn

This is true, maybe you should put a pull request onto http://www.github.com/laravel/docs :)

Good point though as this can throw people off initially.

[jasonlewis]

You still have to provide the filename, even if that file is the config.php file. This should work: Config::get('package::config') or if you have a different file, Config::get('package::filename')

Remember you can have multiple config files so Laravel doesn't know which file to fetch the keys out of. Unless you want to load all config items from all files with Config::get('package')?

[billmn]

No ... it doesn't work, I've tried :

Config::package('billmn/twig', 'twig');
var_dump(Config::get('twig::config'));

with this I can get options for the specified key ('view_extensions') :

Config::package('billmn/twig', 'twig');
var_dump(Config::get('twig::view_extensions'));

Remember you can have multiple config files so Laravel doesn't know which file to fetch the keys out of. Unless you want to load all config items from all files with Config::get('package')?

I would need to get all options for a specified config file ... I don't need to have them all

[jasonlewis]

You're setting up the config wrong for the package. The second parameter should be the path to the config directory, the third parameter is an optional namespace if you prefer to specify one.

Config::package('billmn/twig', __DIR__.'/../../config', 'twig');

Laravel will automatically use twig though so in this case you don't really need to provide the 3rd parameter.

[bencorlett]

Oh Jason that's not right as of the last 3 days. It's now reversed cheek out the ServiceProvider

Sent from my iPhone
Please excuse my brevity

On 15/01/2013, at 9:25 PM, Jason Lewis notifications@github.com wrote:

You're setting up the config wrong for the package. The second parameter should be the path to the config directory, the third parameter is an optional namespace if you prefer to specify one.

Config::package('billmn/twig', DIR.'/../../config', 'twig');
Laravel will automatically use twig though so in this case you don't really need to provide the 3rd parameter.

—
Reply to this email directly or view it on GitHub.

[jasonlewis]

@bencorlett I'm talking about config registration separately, not the package registration as per the service provider. See here: https://github.com/laravel/framework/blob/master/src/Illuminate/Config/Repository.php#L218

[bencorlett]

Oh, sorry my bad misread there

Sent from my iPhone
Please excuse my brevity

On 15/01/2013, at 9:33 PM, Jason Lewis notifications@github.com wrote:

@bencorlett I'm talking about config registration separately, not the package registration as per the service provider. See here: https://github.com/laravel/framework/blob/master/src/Illuminate/Config/Repository.php#L218

—
Reply to this email directly or view it on GitHub.

[billmn]

@jasonlewis Right ... now it works fine.

Thanks for the support

[billmn]

@jasonlewis What about add this on the docs? is missing now

[jasonlewis]

@billmn No worries at all.

Which part needs clarifying in the docs? Actually getting all keys from an entire file?

[billmn]

Exactly ... in particular in this section :
http://four.laravel.com/docs/packages#package-configuration

unfortunately my english sucks :)

[jasonlewis]

No worries at all. I've noted it down and will clarify it when I get the chance. Thanks for the feedback @billmn. :)