[PATCH conductor 2/4] properly quote strings
Jan Provaznik
jprovazn at redhat.com
Thu May 3 12:39:14 UTC 2012
On 05/03/2012 02:04 PM, Mo Morsi wrote:
> On 05/03/2012 11:47 AM, Jan Provaznik wrote:
>> On 05/02/2012 03:48 PM, Mo Morsi wrote:
>>> ---
>>> src/config/locales/de.yml | 12 ++++++------
>>> src/config/locales/en.yml | 6 +++---
>>> src/config/locales/fr.yml | 4 ++--
>>> src/config/locales/it.yml | 4 ++--
>>> src/config/locales/pt-BR.yml | 2 +-
>>> 5 files changed, 14 insertions(+), 14 deletions(-)
>>>
>>> diff --git a/src/config/locales/de.yml b/src/config/locales/de.yml
>>> index 902480f..2be3fcb 100644
>>> --- a/src/config/locales/de.yml
>>> +++ b/src/config/locales/de.yml
>>> @@ -260,7 +260,7 @@ de:
>>> no_catalog_selected: "Kein Katalog ausgewählt"
>>> new_from_image:
>>> catalog_entry_name: "%{name}"
>>> - description: "Markup- und Image-Referenzen in \%{name}\ wurden
>>> erfolgreich überprüft. Sie können das Image nun speichern und eine
>>> Deployable erstellen."
>>> + description: "Markup- und Image-Referenzen in \"%{name}\"
>>> wurden erfolgreich überprüft. Sie können das Image nun speichern und
>>> eine Deployable erstellen."
>>
>> Now when switching to 'de' on F16/ruby 1.8, I'm getting this error:
>> missing interpolation argument in "Markup- und Image-Referenzen in
>> \"%{name}\" wurden erfolgreich überprüft. Sie können das Image nun
>> speichern und eine Deployable erstellen." ({:environment=>"default"}
>> given)
>>
>
> How are you producing this error? I enabled the 'de' localization in
> config/application.rb, and started the rails server. The application
> came up fine and I was able to login.
>
I'm getting this error when enabling German language in browser and
clicking on "New Image" on "Environments" tab. Now I tried w/o your
patch with same result -> it's not related to your patch. CCing Jozef.
> In any case the problem is if we don't change the localizations in this
> way, we get yaml parsing errors when running rake tasks on F17.
>
> See mtaylor's feedback to my original patchset [1]. As soon as this
> patch is applied the errors that were were getting when running rake
> db:migrate go away.
>
> Just talked to Jozef and he mentioned that we can remove the backslashes
> alltogether which should fix this issue. Just sent an updated patch
> doing just this.
>
> Thanks for testing it out,
> -Mo
>
> [1] https://fedorahosted.org/pipermail/aeolus-devel/2012-April/010051.html
>
More information about the aeolus-devel
mailing list