Project

General

Profile

Actions

Defect #8847

closed

I18n YAML files not parsable with psych yaml library

Added by Etienne Massip almost 13 years ago. Updated over 11 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Toshi MARUYAMA
Category:
Ruby support
Target version:
-
Start date:
2011-07-19
Due date:
% Done:

0%

Estimated time:
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)
end

Patch 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).


Files


Related issues

Related to Redmine - Feature #8209: I18n YAML files not parsable with psych yaml libraryClosedToshi MARUYAMA2011-04-22

Actions
Related to Redmine - Feature #4050: Ruby 1.9 supportClosed2009-10-18

Actions
Related to Redmine - Patch #35026: Remove rake task check_parsing_by_psychClosedGo MAEDA

Actions
Actions #1

Updated by Toshi MARUYAMA almost 13 years ago

I have known this problem.
https://www.chiliproject.org/issues/503

Should we change all i18n yaml?

Actions #2

Updated by Etienne Massip almost 13 years ago

What do you mean by "all i18n yaml" ?

Actions #3

Updated by Etienne Massip almost 13 years ago

All the changes included in the patch are necessary, if this is your question ?

Actions #4

Updated by Toshi MARUYAMA almost 13 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 ?

Actions #5

Updated by Etienne Massip almost 13 years ago

  • File deleted (locales_YAML_parsing_errors_fix.patch)
Actions #6

Updated by Etienne Massip almost 13 years ago

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.

Actions #7

Updated by Etienne Massip almost 13 years ago

I don't know why they missed the other files...

Actions #8

Updated by Etienne Massip over 11 years ago

  • Status changed from New to Closed
  • Target version deleted (Candidate for next minor release)
  • Resolution set to Fixed
Actions #9

Updated by Go MAEDA about 3 years ago

  • Related to Patch #35026: Remove rake task check_parsing_by_psych added
Actions

Also available in: Atom PDF