Defect #8847
I18n YAML files not parsable with psych yaml library
| Status: | Closed | Start date: | 2011-07-19 | |
|---|---|---|---|---|
| Priority: | Normal | Due date: | ||
| Assignee: |  Toshi MARUYAMA | % Done: | 0% | |
| Category: | Ruby support | |||
| Target version: | - | |||
| Resolution: | Fixed | Affected version: |
Description
Follows #8209.
This scriptlet returned new errors :
require 'psych'
parser = Psych::Parser.new
Dir.entries('config/locales').each do |filename|
next if File.directory? filename
puts "parsing #{filename}..."
parser.parse File.open('config/locales/' + filename)
endPatch attached based on your redmine-31 bitbucket mirror; beware, the updates on ja, zh and zh-TW locales are made "blindly" (don't have the according fonts installed).
locales_YAML_parsing_errors_fix.patch 
(5.16 KB)
Related issues
Associated revisions
Ruby 1.9: fix parsing error en.yml with psych yaml library (#8847).
Ruby 1.9: fix parsing error fr.yml with psych yaml library (#8847).
It seems that double quote and escape is need for non ASCII and backslash
on ruby 1.9.2p290 (2011-07-09 revision 32553) [i686-linux].
Contributed by Etienne Massip.
Ruby 1.9: fix parsing error of pt-BR.yml with psych yaml library (#8847).
It seems that double quote and escape is need for non ASCII and backslash
on ruby 1.9.2p290 (2011-07-09 revision 32553) [i686-linux].
Contributed by Etienne Massip.
Ruby 1.9: fix parsing error of hu.yml with psych yaml library (#8847).
Ruby 1.9: fix parsing error of ja.yml with psych yaml library (#8847).
Contributed by Etienne Massip.
Ruby 1.9: fix parsing error of zh.yml with psych yaml library (#8847).
Contributed by Etienne Massip.
Ruby 1.9: fix parsing error of zh.yml with psych yaml library (#8847).
Ruby 1.9: fix parsing error of sl.yml with psych yaml library (#8847)
History
#1
Updated by Toshi MARUYAMA over 8 years ago
I have known this problem.
https://www.chiliproject.org/issues/503
Should we change all i18n yaml?
#2
Updated by Etienne Massip over 8 years ago
What do you mean by "all i18n yaml" ?
#3
Updated by Etienne Massip over 8 years ago
All the changes included in the patch are necessary, if this is your question ?
#4
Updated by Toshi MARUYAMA over 8 years ago
Chiliproject replaced '\' to '\\' in only en.yml and pt-BR.yml.
https://www.chiliproject.org/projects/chiliproject/repository/revisions/f41dac1b6b593890c8a6fde2b72d87c5c51fd321
Should we change all *.yml at source:trunk/config/locales ?
#5
Updated by Etienne Massip over 8 years ago
- File deleted (
locales_YAML_parsing_errors_fix.patch)
#6
Updated by Etienne Massip over 8 years ago
- File locales_YAML_parsing_errors_fix.patch added
Yes, that is : en, fr, ja, pt-BR, zh and zh-TW.
Was a glitch in my previous patch, here's a patch based on trunk.
#7
Updated by Etienne Massip over 8 years ago
I don't know why they missed the other files...
#8
Updated by Etienne Massip over 7 years ago
- Status changed from New to Closed
- Target version deleted (
Candidate for next minor release) - Resolution set to Fixed