diff options
author | Robert Speicher <rspeicher@gmail.com> | 2017-08-15 15:08:56 -0400 |
---|---|---|
committer | Robert Speicher <rspeicher@gmail.com> | 2017-08-15 15:08:56 -0400 |
commit | f5cb3ac14da14a527a9a259d8a1bfb372868f352 (patch) | |
tree | b7d1e4976b671ae655be18ed8aee1ba250c9d230 /locale/de | |
parent | a2918d56899fbbe2fb2c4cfe8c0d6c2dffd32253 (diff) | |
download | gitlab-ce-rs-redis-config-paths.tar.gz |
Don't depend on `Rails` for Redis configuration file pathsrs-redis-config-paths
The `Rails` object was not always available in all tasks that require
Redis access, such as `mail_room`, so the constant pointing to the
configuration path was never defined, but we still attempted to access
it in `config_file_name`, resulting in a `NameError` exception.
Further, there was no benefit to defining these paths in a constant to
begin with -- they're only accessed in one place, and it was within the
class where they were being defined. We can just provide them at
run-time instead.
Further _still_, we were calling `File.expand_path` on the absolute path
returned by `Rails.root.join`, which was rather pointless.
Diffstat (limited to 'locale/de')
0 files changed, 0 insertions, 0 deletions