-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Config error #6
Comments
@SvanThuijl , hi! Do you use |
My framework version is 5.1.10 and I do use .env to configure everything. I have "solved" this issue by renaming the file to "propel..php" for now. |
Do you use artisan config:cache? If yes, please run |
I did not do anything to the cache configuration so I don't think so. When I try this I already get the errors because the propel config fails. Renaming the file and execute this command does not solve the issue. |
@SvanThuijl , you can use |
-bash-4.1$ php artisan propel:config:convert [ErrorException] -bash-4.1$ |
@SvanThuijl , hmm, i digged config loading bootstrapper and seems like it's rarely your system behavior. You can use workaround. Add in your
and next text to the end of file:
|
Hmm. It will not solve issue (it will replace error happening, but it will be still broken). We need to reread config in ServiceProvider somehow.... @SvanThuijl , let me one more day to think. |
@SvanThuijl , i tried to fix it by a little hack. Try dev version please. You can follow migration guide |
Thanks for the effort. Your migration manual is not correct. The dev-master has no PropelIntegrationServiceProvider but I configured the GeneratorServiceProvider and RuntimeServiceProvider. This doe snot solve the issue. |
@SvanThuijl , did you changed |
Yes I did. I also just noticed the command line tool is not fixed with renaming the file. The config is not loaded at all when I try to run
|
@SvanThuijl , hm, i thought that composer Try to explicitly point commit:
and update. |
Composer doesn't update anything at all after changing this. |
@SvanThuijl , can you provide |
@SvanThuijl , i hope that i totally fixed your issue. Can you install latest |
Got a machine crash and couldn't/didn't work for a while. I'll get back to this asap. |
Problem still persists. Do you have a private contact option? I'll give you access to my install so you can test on my production server. |
@SvanThuijl , feel free to write emails to |
Had the same issue and the fix works perfectly, thus by using
in my composer.json as the fixes are in develop branch. |
Hey Guys,
I have bene trying to figure this out by myself but I can't.
In my local development environment I have this plugin working perfectly but when deploying there is an issue. I get these errors.
[Wed Aug 12 18:44:37.429070 2015] [:error] [pid 30427] [client 190.88.209.114:13186] PHP Warning: array_filter() expects parameter 1 to be array, null given in /home/project/laravel/config/propel.php on line 83, referer: http://www.project.com/
[Wed Aug 12 18:44:37.429163 2015] [:error] [pid 30427] [client 190.88.209.114:13186] PHP Warning: array_map(): Argument #2 should be an array in /home/project/laravel/config/propel.php on line 84, referer: http://www.project.com/
I thinks the issue comes from the order laravel has in loading the config files. By adding the line below to the config files I found out that propel.php is loaded before database.php in the production environment. Locally this is the other way around.
echo 'Loaded: ' . microtime(true) . ': ' . FILE . '
';
Result on production:
Loaded: 1439398148.1305: /home/project/laravel/config/propel.php
Loaded: 1439398148.1313: /home/project/laravel/config/database.php
Result local:
Loaded: 1439398248.01: /Users/SvanThuijl/PhpstormProjects/laravel/config/database.php
Loaded: 1439398248.02: /Users/SvanThuijl/PhpstormProjects/laravel/config/propel.php
Does anybody know how to solve this issue?
The text was updated successfully, but these errors were encountered: