Load multiple configuration files

[awalls]

I'm trying to load multiple configuration files as below. It appears this is possible, and I can pass the Loader class an array of filepaths, but this is not working?

$Loader = (new \josegonzalez\Dotenv\Loader([ __DIR__ . '/../.env', __DIR__ . '/../aws.env', ])) ->parse() ->toEnv($overwrite) ->toServer($overwrite) ->putenv($overwrite);

[josegonzalez]

You say its not working, but what exactly isn't working? Can you give more details on the contents of the files, the values of overwrite, what you are expecting to happen, what is actually happening, and any errors?

[awalls]

When you pass the Loader constructor an array of file names, which gets passed to the function parse() in Loader.php. This loops through $this->filepaths() to parse each file and load its content. However, on line 215 of Loader.php the loop exists on the first iteration as $contents has been set. Therefore, any subsequent files in the loop are not parsed.

[josegonzalez]

Yeah looks like a bug, or maybe I just meant to add the ability to load multiple with fallbacks? In any case, would you be interested in creating a patch to fix this to ensure we parse all of them? I think maybe just concatenating the contents into a single file with two newlines in between might make most sense?

[awalls]

Only you know what your intended design was 😄

Yes, it would make sense just to concatenate the files, if they exist. I'll see if I can get a "fix" in the coming days.

[josegonzalez]

Last worked on this code like 7 years ago so I don't have any idea what the intended design was 😂 .