Project

General

Profile

Actions

Defect #28109

closed

Incorrect interpolation in Swedish locale

Added by Felix Schäfer almost 7 years ago. Updated almost 7 years ago.

Status:
Closed
Priority:
Normal
Assignee:
-
Category:
Translations
Target version:
Start date:
Due date:
% Done:

0%

Estimated time:
Resolution:
Fixed
Affected version:

Description

There is an incorrect interpolation in the Swedish locale file in source:/trunk/config/locales/sv.yml@17186#L137


Files

28109.patch (812 Bytes) 28109.patch Felix Schäfer, 2018-01-31 11:10

Related issues

Related to Redmine - Patch #18587: Swedish translation (updated)Closed

Actions
Actions #1

Updated by Felix Schäfer almost 7 years ago

  • File 28109.patch added

The attached patch fixes the problem on trunk/master.

Actions #2

Updated by Felix Schäfer almost 7 years ago

Felix Schäfer wrote:

The attached patch fixes the problem on trunk/master.

It turns out this was only half the problem, in addition to the % being at the wrong place for the interpolation, there is also an issue with the variable used for the interpolation being the wrong one. This new patch fixes both issues.

Thank you Jan for catching the second issue!

Actions #3

Updated by Marius BĂLTEANU almost 7 years ago

  • File deleted (28109.patch)
Actions #4

Updated by Go MAEDA almost 7 years ago

  • Related to Patch #18587: Swedish translation (updated) added
Actions #5

Updated by Go MAEDA almost 7 years ago

  • Target version set to 3.3.7

Confirmed the broken message. Setting target version to 3.3.7.

Actions #6

Updated by Toshi MARUYAMA almost 7 years ago

  • Subject changed from Incorrect interpolation in swedisch locale to Incorrect interpolation in Swedish locale
Actions #7

Updated by Toshi MARUYAMA almost 7 years ago

  • Status changed from New to Closed
  • Resolution set to Fixed

Committed in trunk, 3.4-stable, and 3.3-stable, thanks.

locale:check_interpolation cannot catch nested keys, so I could not catch this mistake.
source:tags/3.4.4/lib/tasks/locales.rake#L58

Actions

Also available in: Atom PDF