[grisbi-devel] [PATCH 3/7] Reuse existing GtkListStore in payee editing wizard

[Jean-Luc Duflot]

Bonjour Remi,

Excellente idée, celle de soumettre le code à corrections ! Je ne suis 
pas programmeur moi-même, j'ai juste quelques notions de C. Mais je peux 
surtout aider pour les textes en français et anglais.

Donc une remarque sur la ligne :

label = g_object_get_data ( G_OBJECT (assistant), "nbre_tiers_total" );

tout à la fin : si c'est vraiment un label, et donc affiché, on aurait: 
nbre_total_tiers plutôt que nbre_tiers_total

Et ça serait encore mieux si c'était «nombre total de tiers», ça dépend 
aussi du contexte.

Jean-luc



Le 30/01/2013 08:42, Rémi Cardona a écrit :
> A store is already created and hooked in gsb_assistant_payees_page_3().
> This patch simply reuses the existing store after emptying it.
> ---
>   src/tiers_onglet.c | 9 +++------
>   1 file changed, 3 insertions(+), 6 deletions(-)
>
> diff --git a/src/tiers_onglet.c b/src/tiers_onglet.c
> index b4aff83..df40bc0 100644
> --- a/src/tiers_onglet.c
> +++ b/src/tiers_onglet.c
> @@ -1332,8 +1332,9 @@ static gboolean gsb_assistant_payees_enter_page_3 ( GtkWidget *assistant )
>       gtk_label_set_text ( GTK_LABEL (new_payee_label), new_tiers );
>
>       payee_list_tmp = gsb_data_payee_get_payees_list ();
> -    list_store = gtk_list_store_new ( N_COLUMNS, G_TYPE_BOOLEAN,
> -                        G_TYPE_UINT, G_TYPE_STRING );
> +    treeview = g_object_get_data ( G_OBJECT (assistant), "treeview" );
> +    list_store = GTK_LIST_STORE ( gtk_tree_view_get_model ( treeview ) );
> +    gtk_list_store_clear ( list_store );
>
>       while ( payee_list_tmp )
>       {
> @@ -1362,10 +1363,6 @@ static gboolean gsb_assistant_payees_enter_page_3 ( GtkWidget *assistant )
>
>       if ( i>  0 )
>       {
> -        treeview = g_object_get_data ( G_OBJECT (assistant), "treeview" );
> -        if ( treeview )
> -            gtk_tree_view_set_model (GTK_TREE_VIEW (treeview),
> -                            GTK_TREE_MODEL (list_store));
>           label = g_object_get_data ( G_OBJECT (assistant), "nbre_tiers_total" );
>           gtk_label_set_text ( GTK_LABEL (label ),
>                           g_strdup_printf ("%d", i ) );