Noise

column selection resets after restarting noise

Bug #1117964 reported by Corey Kincaid
14
This bug affects 3 people
Affects Status Importance Assigned to Milestone
Noise
Fix Released
High
Corentin Noël
Noise 0.2.1

Bug Description

After restarting noise the columns are reset to their default value.
For example I uncheck the album and genre columns because I do not use that information, but after restarting noise the columns are reset and I must go and uncheck them again.

Revision history for this message
Victor Martinez (victored) wrote :

This has been broken for a long time (since that bug about noise_playlist_get_tree_view_setup).

Everytime noise starts up, the followng can be seen:

[_LOG_LEVEL_DEBUG 23:31:02.980620] TreeViewSetup.vala:57: Creating a new TreeViewSetup for NOISE_VIEW_WRAPPER_HINT_MUSIC
[_LOG_LEVEL_DEBUG 23:31:03.265326] TreeViewSetup.vala:57: Creating a new TreeViewSetup for NOISE_VIEW_WRAPPER_HINT_READ_ONLY_PLAYLIST
[_LOG_LEVEL_DEBUG 23:31:03.500957] TreeViewSetup.vala:57: Creating a new TreeViewSetup for NOISE_VIEW_WRAPPER_HINT_READ_ONLY_PLAYLIST
[_LOG_LEVEL_DEBUG 23:31:03.689860] TreeViewSetup.vala:57: Creating a new TreeViewSetup for NOISE_VIEW_WRAPPER_HINT_SMART_PLAYLIST
[_LOG_LEVEL_DEBUG 23:31:03.873887] TreeViewSetup.vala:57: Creating a new TreeViewSetup for NOISE_VIEW_WRAPPER_HINT_SMART_PLAYLIST
[_LOG_LEVEL_DEBUG 23:31:04.058392] TreeViewSetup.vala:57: Creating a new TreeViewSetup for NOISE_VIEW_WRAPPER_HINT_SMART_PLAYLIST
[_LOG_LEVEL_DEBUG 23:31:04.287502] TreeViewSetup.vala:57: Creating a new TreeViewSetup for NOISE_VIEW_WRAPPER_HINT_SMART_PLAYLIST
[_LOG_LEVEL_DEBUG 23:31:04.550596] TreeViewSetup.vala:57: Creating a new TreeViewSetup for NOISE_VIEW_WRAPPER_HINT_SMART_PLAYLIST
[_LOG_LEVEL_DEBUG 23:31:04.762457] TreeViewSetup.vala:57: Creating a new TreeViewSetup for NOISE_VIEW_WRAPPER_HINT_SMART_PLAYLIST

That should only happen during first run. Afterwards, noise should save the user's current setup using TreeViewSetup.columns_to_string() and load those setups using TreeViewSetup.import_columns(). The whole point of TreeViewSetup is serializing what columns are visible.

If you need me to have a look into this, please let me know :)

Changed in noise:
status: New → Triaged
importance: Undecided → High
Cody Garver (codygarver)
Changed in noise:
milestone: none → luna-beta3
Corentin Noël (tintou)
Changed in noise:
assignee: nobody → Corentin Noël (tintou)
status: Triaged → Fix Committed
Revision history for this message
Jason (nikeh34d) wrote :

This was fixed with last update for me, :D

Cody Garver (codygarver)
Changed in noise:
status: Fix Committed → Fix Released
Cody Garver (codygarver)
Changed in noise:
milestone: luna-rc1 → noise-0.3
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.