Skip to content
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

Fix issue #533 #534

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Fix issue #533 #534

wants to merge 2 commits into from

Conversation

ke20
Copy link

@ke20 ke20 commented Sep 18, 2014

The given variable data is the parameters.ini content file.

In the file capifony / lib / symfony2 / database.rb there is

    data = capture("#{try_sudo} cat #{current_path}/#{app_config_path}/#{app_config_file}")
    config = load_database_config data, symfony_env_prod

Kevin Auvinet added 2 commits September 18, 2014 16:51
The given variable `data`  is the `parameters.ini` content file.

In the file `capifony / lib / symfony2 / database.rb` there is 

```ruby
    data = capture("#{try_sudo} cat #{current_path}/#{app_config_path}/#{app_config_file}")
    config = load_database_config data, symfony_env_prod
```
Replace the hard coded path of the backup directory by the variable `#{backup_path}`
@willdurand
Copy link
Collaborator

The given variable data is the parameters.ini content file.

Could you please rephrase this sentence?

@ke20
Copy link
Author

ke20 commented Apr 27, 2015

Sorry, my english is far to be perfect and I'm not a rubyist ...

The method IniFile.new seems expect as argument a filename for an existing INI file.
But, it's not a filename which is given but the content of the INI file.

So, the symbol :content is missing.
In my case, without the :content there is a bug.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants