[PATCH 1/1] Hob: fixed a little view issue about package selection page

[PATCH 1/1] Hob: fixed a little view issue about package selection page

[Liming An]

In package selection page, treeview cell data callback function not cover the 'else' case, so parent item will render the pervious value.

Signed-off-by: Liming An <limingx.l.an@intel.com>
---
 bitbake/lib/bb/ui/crumbs/hobwidget.py |    3 ++-
 1 files changed, 2 insertions(+), 1 deletions(-)

diff --git a/bitbake/lib/bb/ui/crumbs/hobwidget.py b/bitbake/lib/bb/ui/crumbs/hobwidget.py
index a42cdec..8cbd6aa 100644
--- a/bitbake/lib/bb/ui/crumbs/hobwidget.py
+++ b/bitbake/lib/bb/ui/crumbs/hobwidget.py
@@ -175,7 +175,8 @@ class HobViewTable (gtk.VBox):
         if binb:
             bin = binb.split(', ')
             cell.set_property('text', bin[0])
-
+        else:
+            cell.set_property('text', "")
         return True
 
     def set_model(self, tree_model):
-- 
1.7.5.4

Re: [PATCH 1/1] Hob: fixed a little view issue about package selection page

[Joshua Lock]

On 16/04/12 03:09, Liming An wrote:
> In package selection page, treeview cell data callback function not cover the 'else' case, so parent item will render the pervious value.
>
> Signed-off-by: Liming An<limingx.l.an@intel.com>

Good catch, thanks!

Signed-off-by: Joshua Lock<josh@linux.intel.com>

> ---
>   bitbake/lib/bb/ui/crumbs/hobwidget.py |    3 ++-
>   1 files changed, 2 insertions(+), 1 deletions(-)
>
> diff --git a/bitbake/lib/bb/ui/crumbs/hobwidget.py b/bitbake/lib/bb/ui/crumbs/hobwidget.py
> index a42cdec..8cbd6aa 100644
> --- a/bitbake/lib/bb/ui/crumbs/hobwidget.py
> +++ b/bitbake/lib/bb/ui/crumbs/hobwidget.py
> @@ -175,7 +175,8 @@ class HobViewTable (gtk.VBox):
>           if binb:
>               bin = binb.split(', ')
>               cell.set_property('text', bin[0])
> -
> +        else:
> +            cell.set_property('text', "")
>           return True
>
>       def set_model(self, tree_model):

-- 
Joshua Lock
         Yocto Project
         Intel Open Source Technology Centre