Defect #32500
Spent time report csv shows translation missing text if custom fields are involved
| Status: | Closed | Start date: | ||
|---|---|---|---|---|
| Priority: | Normal | Due date: | ||
| Assignee: |  Go MAEDA | % Done: | 0% | |
| Category: | Time tracking | |||
| Target version: | 4.0.6 | |||
| Resolution: | Fixed | Affected version: | 4.0.5 |
Description
- Create a spent time's custom field (list format) with two values. In this example it was named Perfil, with values Perfil1 and Perfil2
- Log some time, with different Perfil values.
- Go to Spent Time > Report
- Group by custom field Perfil
The report on the page looks fine, but if you export it using the CSV link it shows a translation missing: en.Perfil message at first column, first row:
If you choose another language the message will change accordingly: translation missing: es.Perfil, translation missing: gl.Perfil, etc...
My environment:
Environment: Redmine version 4.0.5.stable Ruby version 2.5.7-p206 (2019-10-01) [x86_64-linux] Rails version 5.2.3 Environment production Database adapter Mysql2 Mailer queue ActiveJob::QueueAdapters::AsyncAdapter Mailer delivery smtp SCM: Subversion 1.12.2 Cvs 1.12.13 Git 2.19.1 Filesystem Redmine plugins: no plugin installed
error.translation-missing.timelog.csv.png (37.8 KB)
fix-32500.patch 
(1.53 KB)
Associated revisions
Fix that spent time report CSV shows translation missing text if custom fields are involved (#32500).
Patch by Yuichi HARADA.
History
Updated by #2
Updated by Yuichi HARADA 11 months ago
- File fix-32500.patch added
Changed method l to l_or_humanize . I attached a patch.
diff --git a/app/helpers/timelog_helper.rb b/app/helpers/timelog_helper.rb
index 019b441a0..2c3e500a5 100644
--- a/app/helpers/timelog_helper.rb
+++ b/app/helpers/timelog_helper.rb
@@ -79,7 +79,7 @@ module TimelogHelper
def report_to_csv(report)
Redmine::Export::CSV.generate(:encoding => params[:encoding]) do |csv|
# Column headers
- headers = report.criteria.collect {|criteria| l(report.available_criteria[criteria][:label]) }
+ headers = report.criteria.collect {|criteria| l_or_humanize(report.available_criteria[criteria][:label]) }
headers += report.periods
headers << l(:label_total_time)
csv << headers
#4
Updated by Guillermo ML 11 months ago
Go MAEDA wrote:
I have confirmed that the patch fixes the issue.
The patch also worked for us. Now it shows Perfil instead of translation missing: en.Perfil. CSV output is the same as html.
Many thanks to Yuichi HARADA and Go MAEDA.