After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 779565 - Treeview header combos do not work at first load.
Treeview header combos do not work at first load.
Status: RESOLVED OBSOLETE
Product: GnuCash
Classification: Other
Component: Import - CSV
git-master
Other Linux
: Normal normal
: ---
Assigned To: gnucash-import-maint
gnucash-import-maint
Depends on:
Blocks: 778981
 
 
Reported: 2017-03-04 11:04 UTC by Bob
Modified: 2018-06-29 23:55 UTC
See Also:
GNOME target: ---
GNOME version: ---



Description Bob 2017-03-04 11:04:54 UTC
If you have a comma separated file and load it, the column header combo boxes do not work, unable to change. If you select an additional separator or other option, like say the semicolon, they start to work.

I have looked at preview_cbox_factory to check the model has the right number of options and it does, monitored the call back and it does not fire, just can not see what is wrong.
Comment 1 Bob 2017-03-04 11:18:47 UTC
After further looking I can make it work by changing line 1605 in assistant-csv-trans-import.cpp from this...

preview_refresh_table ();
to
g_idle_add ((GSourceFunc)csv_imp_preview_queue_rebuild_table, this);

I also think that the preview_refresh() in the file_confirm_cb may not be required as there is one in preview_settings_load which executes any way and you can no go back to the file page anyway. 

There is another preview_refresh_table in the 'preview_update_encoding' and 'assist_preview_page_prepare' that also execute at first load pass, just wondering if there should be a mechanism to stop all this execution of the same code ?


Bob
Comment 2 John Ralls 2018-06-29 23:55:00 UTC
GnuCash bug tracking has moved to a new Bugzilla host. The new URL for this bug is https://bugs.gnucash.org/show_bug.cgi?id=779565. Please continue processing the bug there and please update any external references or bookmarks.