Package: redmine
Severity: normal

Dear Maintainer,

When trying to change the path used for file storage, I've updated my
configuration file to change attachments_storage_path according to my
needs. Turns out it didn't work.

After digging, I found that Debian patch 2002_FHS_through_env_vars.patch
is broken.
The conditional change is wrong. It's interpreted as:
(Redmine::Configuration['attachments_storage_path'] || ENV['RAILS_VAR']) ? ....

The change should have been like:
Redmine::Configuration['attachments_storage_path'] || (ENV['RAILS_VAR']) ? ....

By adding the missing parentheses, redmine now takes
attachments_storage_path into account.

I've not checked if other parts of the patch suffer from similar bug.

I believe that this bug is limited to jessie/stable as this change doesn't seem
to exist in newer versions.

Thanks,
Arnaud Patard

Reply via email to