Re: [Sugar-devel] 0.90 Exception: Activities list view and Journal view crashes in Arabic

2010-09-29 Thread Simon Schampijer
On 09/24/2010 04:22 PM, Tomeu Vizoso wrote:
 On Fri, Sep 24, 2010 at 16:10, Simon Schampijersi...@schampijer.de  wrote:
 On 09/24/2010 10:06 AM, Tomeu Vizoso wrote:

 On Wed, Sep 22, 2010 at 18:06, Simon Schampijersi...@schampijer.de
   wrote:

 Hi,

 I am asking for an exception for [1] 0.90. Originally discovered on 0.84
 this is also present on 0.90. The patch [2] has been reviewed by Sascha
 and Tomeu already and it is low risk. Otherwise Arabic will be unusable
 as this effects the Journal and the Activities list view.

 I think there should be a big FIXME comment with a reference to a
 ticket in trac because it's a hack and because we really need to find
 an appropriate fix in the near future.

 Also, we don't have plurals in any other place?

 Regards,

 Tomeu

 http://bugs.sugarlabs.org/attachment/ticket/2354/0001-If-the-string-does-not-contain-a-conversion-specifie.patch

 The latest patch does add the comment.

 Sorry, I was referring to a ticket about correctly translating strings
 with plurals. If you want to use the same ticket, at least it should
 be clear that it is about a proper solution and not just about the
 crash. It also helps if it starts with FIXME.

 I will check if there are other plurals at other places - none I am aware of
 for now.

 I have only found the activity updater.

 Regards,

 Tomeu

Ok, addressed the raised issues and committed.

Thanks for taking the time,
Simon
___
Sugar-devel mailing list
Sugar-devel@lists.sugarlabs.org
http://lists.sugarlabs.org/listinfo/sugar-devel


Re: [Sugar-devel] 0.90 Exception: Activities list view and Journal view crashes in Arabic

2010-09-24 Thread Tomeu Vizoso
On Wed, Sep 22, 2010 at 18:06, Simon Schampijer si...@schampijer.de wrote:
 Hi,

 I am asking for an exception for [1] 0.90. Originally discovered on 0.84
 this is also present on 0.90. The patch [2] has been reviewed by Sascha
 and Tomeu already and it is low risk. Otherwise Arabic will be unusable
 as this effects the Journal and the Activities list view.

I think there should be a big FIXME comment with a reference to a
ticket in trac because it's a hack and because we really need to find
an appropriate fix in the near future.

Also, we don't have plurals in any other place?

Regards,

Tomeu

 Regards,
    Simon

 [1] http://bugs.sugarlabs.org/ticket/2354
 [2] Patch: http://bugs.sugarlabs.org/ticket/2354#comment:3
 ___
 Sugar-devel mailing list
 Sugar-devel@lists.sugarlabs.org
 http://lists.sugarlabs.org/listinfo/sugar-devel

___
Sugar-devel mailing list
Sugar-devel@lists.sugarlabs.org
http://lists.sugarlabs.org/listinfo/sugar-devel


Re: [Sugar-devel] 0.90 Exception: Activities list view and Journal view crashes in Arabic

2010-09-24 Thread Simon Schampijer
On 09/24/2010 10:06 AM, Tomeu Vizoso wrote:
 On Wed, Sep 22, 2010 at 18:06, Simon Schampijersi...@schampijer.de  wrote:
 Hi,

 I am asking for an exception for [1] 0.90. Originally discovered on 0.84
 this is also present on 0.90. The patch [2] has been reviewed by Sascha
 and Tomeu already and it is low risk. Otherwise Arabic will be unusable
 as this effects the Journal and the Activities list view.

 I think there should be a big FIXME comment with a reference to a
 ticket in trac because it's a hack and because we really need to find
 an appropriate fix in the near future.

 Also, we don't have plurals in any other place?

 Regards,

 Tomeu

http://bugs.sugarlabs.org/attachment/ticket/2354/0001-If-the-string-does-not-contain-a-conversion-specifie.patch

The latest patch does add the comment.

I will check if there are other plurals at other places - none I am 
aware of for now.

Regards,
Simon
___
Sugar-devel mailing list
Sugar-devel@lists.sugarlabs.org
http://lists.sugarlabs.org/listinfo/sugar-devel


Re: [Sugar-devel] 0.90 Exception: Activities list view and Journal view crashes in Arabic

2010-09-24 Thread Tomeu Vizoso
On Fri, Sep 24, 2010 at 16:10, Simon Schampijer si...@schampijer.de wrote:
 On 09/24/2010 10:06 AM, Tomeu Vizoso wrote:

 On Wed, Sep 22, 2010 at 18:06, Simon Schampijersi...@schampijer.de
  wrote:

 Hi,

 I am asking for an exception for [1] 0.90. Originally discovered on 0.84
 this is also present on 0.90. The patch [2] has been reviewed by Sascha
 and Tomeu already and it is low risk. Otherwise Arabic will be unusable
 as this effects the Journal and the Activities list view.

 I think there should be a big FIXME comment with a reference to a
 ticket in trac because it's a hack and because we really need to find
 an appropriate fix in the near future.

 Also, we don't have plurals in any other place?

 Regards,

 Tomeu

 http://bugs.sugarlabs.org/attachment/ticket/2354/0001-If-the-string-does-not-contain-a-conversion-specifie.patch

 The latest patch does add the comment.

Sorry, I was referring to a ticket about correctly translating strings
with plurals. If you want to use the same ticket, at least it should
be clear that it is about a proper solution and not just about the
crash. It also helps if it starts with FIXME.

 I will check if there are other plurals at other places - none I am aware of
 for now.

I have only found the activity updater.

Regards,

Tomeu

 Regards,
   Simon

___
Sugar-devel mailing list
Sugar-devel@lists.sugarlabs.org
http://lists.sugarlabs.org/listinfo/sugar-devel


[Sugar-devel] 0.90 Exception: Activities list view and Journal view crashes in Arabic

2010-09-22 Thread Simon Schampijer
Hi,

I am asking for an exception for [1] 0.90. Originally discovered on 0.84 
this is also present on 0.90. The patch [2] has been reviewed by Sascha 
and Tomeu already and it is low risk. Otherwise Arabic will be unusable 
as this effects the Journal and the Activities list view.

Regards,
Simon

[1] http://bugs.sugarlabs.org/ticket/2354
[2] Patch: http://bugs.sugarlabs.org/ticket/2354#comment:3
___
Sugar-devel mailing list
Sugar-devel@lists.sugarlabs.org
http://lists.sugarlabs.org/listinfo/sugar-devel