[Sugar-devel] [PATCH Finance] expenses with no category specified are displayed as "Unknown" fix to SL#3097
Kalpa Welivitigoda
callkalpa at gmail.com
Sat Mar 17 16:16:32 EDT 2012
On Sun, Mar 18, 2012 at 1:06 AM, Gonzalo Odiard <gonzalo at laptop.org> wrote:
> Thanks Kalpa,
> What do you think about replacing:
>
> - catbox = gtk.Label(c)
> + # If there is no category, display as Unknown
> + if c is '':
> + catbox = gtk.Label(_('Unknown'))
> + else:
> + catbox = gtk.Label(c)
>
> by:
>
> + # If there is no category, display as Unknown
> + description = c
> + if c is '':
> + description = _('Unknown')
> catbox = gtk.Label(description)
>
> Looks simpler, right?
>
and much better.
I've attached the new patch.
> Gonzalo
>
> On Sat, Mar 17, 2012 at 4:21 PM, <callkalpa at gmail.com> wrote:
>>
>> From: Kalpa Pathum Welivitigoda <callkalpa at gmail.com>
>>
>> ---
>> budgetscreen.py | 6 +++++-
>> chartscreen.py | 7 +++++--
>> 2 files changed, 10 insertions(+), 3 deletions(-)
>>
>> diff --git a/budgetscreen.py b/budgetscreen.py
>> index 0e2cd98..bcbceb5 100644
>> --- a/budgetscreen.py
>> +++ b/budgetscreen.py
>> @@ -96,7 +96,11 @@ class BudgetScreen(gtk.VBox):
>>
>> # Build categories.
>> for c in self.sorted_categories:
>> - catbox = gtk.Label(c)
>> + # If there is no category, display as Unknown
>> + if c is '':
>> + catbox = gtk.Label(_('Unknown'))
>> + else:
>> + catbox = gtk.Label(c)
>> catbox.set_padding(10, 0)
>>
>> color = finance.get_category_color_str(c)
>> diff --git a/chartscreen.py b/chartscreen.py
>> index 129be9b..b6d1977 100644
>> --- a/chartscreen.py
>> +++ b/chartscreen.py
>> @@ -89,8 +89,11 @@ class ChartScreen(gtk.HBox):
>> for c in self.sorted_categories:
>> hbox = gtk.HBox()
>>
>> - catlabel = gtk.Label()
>> - catlabel.set_markup(c)
>> + # If there is no category, display as Unknown
>> + if c is '':
>> + catlabel = gtk.Label(_('Unknown'))
>> + else:
>> + catlabel = gtk.Label(c)
>> catgroup.add_widget(catlabel)
>>
>> color = finance.get_category_color_str(c)
>> --
>> 1.7.7.6
>>
>> _______________________________________________
>> Sugar-devel mailing list
>> Sugar-devel at lists.sugarlabs.org
>> http://lists.sugarlabs.org/listinfo/sugar-devel
>
>
--
Best Regards,
Kalpa Pathum Welivitigoda
http://about.me/callkalpa
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0002-expenses-with-no-category-specified-are-displayed-as.patch
Type: text/x-patch
Size: 1596 bytes
Desc: not available
URL: <http://lists.sugarlabs.org/archive/sugar-devel/attachments/20120318/ec77af00/attachment.bin>
More information about the Sugar-devel
mailing list