Unable to nest variables

[CMDann]

Unable to nest variables

[gmcgibbon]

This has been fixed. We changed DB_DATABASE="BUILD_${DRONE_BUILD_NUMBER}" to DB_DATABASE="BUILD_{$DRONE_BUILD_NUMBER}".

Turns out ${...} is used for phpdotenv variables while {$...} is used for environment variables. You might want to clarify this in the documentation.

[vlucas]

phpdotenv follows the correct bash/shell syntax, which is ${...} for variable expansion and string interpolation. I just tried it in my shell:

$> export DRONE_BUILD_NUMBER=42
$> export DB_DATABASE="BUILD_${DRONE_BUILD_NUMBER}"
$> echo $DB_DATABASE                                                                                            ]
$> BUILD_42

You can also test it with:

source .env && echo $DB_DATABASE

[CMDann]

👍

[VanTanev]

phpdotenv does not properly inline env variables in laravel 5.1

$ printenv | grep MYSQL_USER
MYSQL_USER=root

$ cat .env | grep DB_USERNAME
DB_USERNAME="${MYSQL_USER}"

$ php artisan migrate --force
SQLSTATE[HY000] [1045] Access denied for user '${MYSQL_USER}'@'localhost' (using password: YES)

$ source .env && echo $DB_USERNAME
root

Switching to DB_USERNAME="{$MYSQL_USER}" inside .env fixes the issue.

I don't know if this behavior has been changed since laravel 5.1, but it would definitely be good to mention it somewhere visible. Or, since the {$VAR} version seems to always work, maybe use that example in the documentation?

[GrahamCampbell]

Laravel 5.1 uses an old version of this package.

[GrahamCampbell]

https://github.com/vlucas/phpdotenv/tree/1.1

[VanTanev]

But since Laravel's docs do not specify inline semantics, and people will look to this documentation for guidance, we're left with a very confusing error. I can try and open a PR to fix the Laravel docs, but probably not the only one who would look to your docs first.

Do you think mentioning the old syntax in the main README would be useful? Something along the lines of:

"For old versions of dotenv, use {$VAR} instead of ${VAR}"

[GrahamCampbell]

I'd say that I'd expect people to look at the readme for the correct version?

[VanTanev]

Unfortunately, it's not that easy to know what the correct version is when it's vendored like this. Also, 5.1 is the LTS release of Laravel, so it's expected to stick around for a good while still (2 years active dev, 3 years security fixes, starting July 19, 2016).

The docs (https://laravel.com/docs/5.1#environment-configuration) also simply link to the latest version of DotEnv (instead of to ~1.0) adding to the confusion.