Un-clutter roster_window.py

[steve-e]

I know there is still the modelfilter patch pending, nevertheless we should think about cleaning up the roster sources.

Roster_window.py is 5578 lines long and declares 169 public methods (only about 40% of those are documented)

On a quick glance I spotted the following:

• Music Track notifications should be moved somewhere else
  • enable_syncing_status_msg_from_current_music_track()
  • enable_syncing_status_msg_from_lastfm()
  • _music_track_changed
  • some stuff in init
• change_awn_icon_status should be moved to statusicon.py
• A lot of Icon related stuff could be moved to gtkgui_helpers
  • get_appropriate_state_images
  • load_iconset or make_foo_state_images
  • ...
• on_status_combobox_changed and change_status kill DRY Principal
  • Don't repeat yourself: both fire dialog for invisibility in groupchat

[jim++]

(In [9399]) Clarify how roster.quit_on_next_offline works with comments. Use only one function when user wants to quit. Bad dup code ! Maybe see #3643.

[steve-e]

Do we want that for 0.12 or would it be easier to wait for the plugin system and let MusicTrack? and AWN become a plugin.

[Jim++]

Avoid duplicate icons and move icon code outside of roster_window (not sure better place yet):

Assigned to me (after modelfilter is finished).

[Jim++]

For icon code, see also #2378

[jim++]

(In [9501]) Moved methods for opening new messages controls from roster.py to src/gajim.py. This is not roster related. See #3643. Sorted src/gajim.py functions, using steve-e style :p

[Jim++]

http://trac.gajim.org/attachment/wiki/UmlClassDiagrams/roster-links.png may help for this ticket.