diff --git a/deluge/ui/gtkui/addtorrentdialog.py b/deluge/ui/gtkui/addtorrentdialog.py index 59c0d58f1..045514061 100644 --- a/deluge/ui/gtkui/addtorrentdialog.py +++ b/deluge/ui/gtkui/addtorrentdialog.py @@ -53,18 +53,7 @@ class AddTorrentDialog(component.Component): self.dialog.connect('delete-event', self._on_delete_event) - self.builder.connect_signals({ - 'on_button_file_clicked': self._on_button_file_clicked, - 'on_button_url_clicked': self._on_button_url_clicked, - 'on_button_hash_clicked': self._on_button_hash_clicked, - 'on_button_remove_clicked': self._on_button_remove_clicked, - 'on_button_trackers_clicked': self._on_button_trackers_clicked, - 'on_button_cancel_clicked': self._on_button_cancel_clicked, - 'on_button_add_clicked': self._on_button_add_clicked, - 'on_button_apply_clicked': self._on_button_apply_clicked, - 'on_button_revert_clicked': self._on_button_revert_clicked, - 'on_chk_move_completed_toggled': self._on_chk_move_completed_toggled - }) + self.builder.connect_signals(self) # download?, path, filesize, sequence number, inconsistent? self.files_treestore = gtk.TreeStore( @@ -517,8 +506,8 @@ class AddTorrentDialog(component.Component): _iter = self.files_treestore.iter_next(_iter) return this_level_toggle - def _on_button_file_clicked(self, widget): - log.debug('_on_button_file_clicked') + def on_button_file_clicked(self, widget): + log.debug('on_button_file_clicked') # Setup the filechooserdialog chooser = gtk.FileChooserDialog( _('Choose a .torrent file'), @@ -561,8 +550,8 @@ class AddTorrentDialog(component.Component): chooser.destroy() self.add_from_files(result) - def _on_button_url_clicked(self, widget): - log.debug('_on_button_url_clicked') + def on_button_url_clicked(self, widget): + log.debug('on_button_url_clicked') dialog = self.builder.get_object('url_dialog') entry = self.builder.get_object('entry_url') @@ -647,8 +636,8 @@ class AddTorrentDialog(component.Component): os.close(tmp_fd) d.addCallbacks(on_download_success, on_download_fail) - def _on_button_hash_clicked(self, widget): - log.debug('_on_button_hash_clicked') + def on_button_hash_clicked(self, widget): + log.debug('on_button_hash_clicked') dialog = self.builder.get_object('dialog_infohash') entry = self.builder.get_object('entry_hash') textview = self.builder.get_object('text_trackers') @@ -683,8 +672,8 @@ class AddTorrentDialog(component.Component): textview.get_buffer().set_text('') dialog.hide() - def _on_button_remove_clicked(self, widget): - log.debug('_on_button_remove_clicked') + def on_button_remove_clicked(self, widget): + log.debug('on_button_remove_clicked') (model, row) = self.listview_torrents.get_selection().get_selected() if row is None: return @@ -695,15 +684,15 @@ class AddTorrentDialog(component.Component): del self.files[torrent_id] del self.infos[torrent_id] - def _on_button_trackers_clicked(self, widget): - log.debug('_on_button_trackers_clicked') + def on_button_trackers_clicked(self, widget): + log.debug('on_button_trackers_clicked') - def _on_button_cancel_clicked(self, widget): - log.debug('_on_button_cancel_clicked') + def on_button_cancel_clicked(self, widget): + log.debug('on_button_cancel_clicked') self.hide() - def _on_button_add_clicked(self, widget): - log.debug('_on_button_add_clicked') + def on_button_add_clicked(self, widget): + log.debug('on_button_add_clicked') self.add_torrents() self.hide() @@ -745,8 +734,8 @@ class AddTorrentDialog(component.Component): log.info('Successfully added %d torrents.', len(torrents_to_add)) client.core.add_torrent_files(torrents_to_add).addCallback(on_torrents_added) - def _on_button_apply_clicked(self, widget): - log.debug('_on_button_apply_clicked') + def on_button_apply_clicked(self, widget): + log.debug('on_button_apply_clicked') (model, row) = self.listview_torrents.get_selection().get_selected() if row is None: return @@ -764,8 +753,8 @@ class AddTorrentDialog(component.Component): self.options[torrent_id].update(options) row = model.iter_next(row) - def _on_button_revert_clicked(self, widget): - log.debug('_on_button_revert_clicked') + def on_button_revert_clicked(self, widget): + log.debug('on_button_revert_clicked') (model, row) = self.listview_torrents.get_selection().get_selected() if row is None: return @@ -773,7 +762,7 @@ class AddTorrentDialog(component.Component): del self.options[model.get_value(row, 0)] self.set_default_options() - def _on_chk_move_completed_toggled(self, widget): + def on_chk_move_completed_toggled(self, widget): value = widget.get_active() self.move_completed_path_chooser.set_sensitive(value) diff --git a/deluge/ui/gtkui/createtorrentdialog.py b/deluge/ui/gtkui/createtorrentdialog.py index 62f645b4a..502027b61 100644 --- a/deluge/ui/gtkui/createtorrentdialog.py +++ b/deluge/ui/gtkui/createtorrentdialog.py @@ -57,17 +57,7 @@ class CreateTorrentDialog(object): self.dialog = self.builder.get_object('create_torrent_dialog') self.dialog.set_transient_for(component.get('MainWindow').window) - self.builder.connect_signals({ - 'on_button_file_clicked': self._on_button_file_clicked, - 'on_button_folder_clicked': self._on_button_folder_clicked, - 'on_button_remote_path_clicked': self._on_button_remote_path_clicked, - 'on_button_cancel_clicked': self._on_button_cancel_clicked, - 'on_button_save_clicked': self._on_button_save_clicked, - 'on_button_up_clicked': self._on_button_up_clicked, - 'on_button_add_clicked': self._on_button_add_clicked, - 'on_button_remove_clicked': self._on_button_remove_clicked, - 'on_button_down_clicked': self._on_button_down_clicked - }) + self.builder.connect_signals(self) # path, icon, size self.files_treestore = gtk.TreeStore(str, str, TYPE_UINT64) @@ -132,8 +122,8 @@ class CreateTorrentDialog(object): self.builder.get_object('combo_piece_size').set_active(index) break - def _on_button_file_clicked(self, widget): - log.debug('_on_button_file_clicked') + def on_button_file_clicked(self, widget): + log.debug('on_button_file_clicked') # Setup the filechooserdialog chooser = gtk.FileChooserDialog(_('Choose a file'), self.dialog, @@ -161,8 +151,8 @@ class CreateTorrentDialog(object): self.adjust_piece_size() chooser.destroy() - def _on_button_folder_clicked(self, widget): - log.debug('_on_button_folder_clicked') + def on_button_folder_clicked(self, widget): + log.debug('on_button_folder_clicked') # Setup the filechooserdialog chooser = gtk.FileChooserDialog(_('Choose a folder'), self.dialog, @@ -189,8 +179,8 @@ class CreateTorrentDialog(object): self.adjust_piece_size() chooser.destroy() - def _on_button_remote_path_clicked(self, widget): - log.debug('_on_button_remote_path_clicked') + def on_button_remote_path_clicked(self, widget): + log.debug('on_button_remote_path_clicked') dialog = self.builder.get_object('remote_path_dialog') entry = self.builder.get_object('entry_path') dialog.set_transient_for(self.dialog) @@ -212,12 +202,12 @@ class CreateTorrentDialog(object): dialog.hide() - def _on_button_cancel_clicked(self, widget): - log.debug('_on_button_cancel_clicked') + def on_button_cancel_clicked(self, widget): + log.debug('on_button_cancel_clicked') self.dialog.destroy() - def _on_button_save_clicked(self, widget): - log.debug('_on_button_save_clicked') + def on_button_save_clicked(self, widget): + log.debug('on_button_save_clicked') if len(self.files_treestore) == 0: return @@ -393,8 +383,8 @@ class CreateTorrentDialog(object): # crash the UI while updating it. idle_add(update_pbar_with_gobject, percent) - def _on_button_up_clicked(self, widget): - log.debug('_on_button_up_clicked') + def on_button_up_clicked(self, widget): + log.debug('on_button_up_clicked') row = self.builder.get_object('tracker_treeview').get_selection().get_selected()[1] if row is None: return @@ -403,15 +393,15 @@ class CreateTorrentDialog(object): else: self.trackers_liststore[row][0] -= 1 - def _on_button_down_clicked(self, widget): - log.debug('_on_button_down_clicked') + def on_button_down_clicked(self, widget): + log.debug('on_button_down_clicked') row = self.builder.get_object('tracker_treeview').get_selection().get_selected()[1] if row is None: return self.trackers_liststore[row][0] += 1 - def _on_button_add_clicked(self, widget): - log.debug('_on_button_add_clicked') + def on_button_add_clicked(self, widget): + log.debug('on_button_add_clicked') builder = gtk.Builder() builder.add_from_file(resource_filename( 'deluge.ui.gtkui', os.path.join('glade', 'edit_trackers.add.ui') @@ -441,8 +431,8 @@ class CreateTorrentDialog(object): dialog.destroy() - def _on_button_remove_clicked(self, widget): - log.debug('_on_button_remove_clicked') + def on_button_remove_clicked(self, widget): + log.debug('on_button_remove_clicked') row = self.builder.get_object('tracker_treeview').get_selection().get_selected()[1] if row is None: return diff --git a/deluge/ui/gtkui/edittrackersdialog.py b/deluge/ui/gtkui/edittrackersdialog.py index 56fdad8f2..5a2842d0d 100644 --- a/deluge/ui/gtkui/edittrackersdialog.py +++ b/deluge/ui/gtkui/edittrackersdialog.py @@ -109,18 +109,7 @@ class EditTrackersDialog(object): self.dialog.set_transient_for(parent) # Connect the signals - self.builder.connect_signals({ - 'on_button_up_clicked': self.on_button_up_clicked, - 'on_button_add_clicked': self.on_button_add_clicked, - 'on_button_edit_clicked': self.on_button_edit_clicked, - 'on_button_edit_cancel_clicked': self.on_button_edit_cancel_clicked, - 'on_button_edit_ok_clicked': self.on_button_edit_ok_clicked, - 'on_button_remove_clicked': self.on_button_remove_clicked, - 'on_button_down_clicked': self.on_button_down_clicked, - 'on_button_add_ok_clicked': self.on_button_add_ok_clicked, - 'on_button_add_cancel_clicked': self.on_button_add_cancel_clicked, - 'on_edit_trackers_dialog_configure_event': self.on_edit_trackers_dialog_configure_event - }) + self.builder.connect_signals(self) # Create a liststore for tier, url self.liststore = gtk.ListStore(int, str) diff --git a/deluge/ui/gtkui/files_tab.py b/deluge/ui/gtkui/files_tab.py index b2ccc7c31..322d05ff9 100644 --- a/deluge/ui/gtkui/files_tab.py +++ b/deluge/ui/gtkui/files_tab.py @@ -181,15 +181,7 @@ class FilesTab(Tab): self.listview.connect('drag_data_get', self._on_drag_data_get_data) self.listview.connect('drag_data_received', self._on_drag_data_received_data) - component.get('MainWindow').connect_signals({ - 'on_menuitem_open_file_activate': self._on_menuitem_open_file_activate, - 'on_menuitem_show_file_activate': self._on_menuitem_show_file_activate, - 'on_menuitem_ignore_activate': self._on_menuitem_ignore_activate, - 'on_menuitem_low_activate': self._on_menuitem_low_activate, - 'on_menuitem_normal_activate': self._on_menuitem_normal_activate, - 'on_menuitem_high_activate': self._on_menuitem_high_activate, - 'on_menuitem_expand_all_activate': self._on_menuitem_expand_all_activate - }) + component.get('MainWindow').connect_signals(self) # Connect to various events from the daemon client.register_event_handler('TorrentFileRenamedEvent', self._on_torrentfilerenamed_event) @@ -289,7 +281,7 @@ class FilesTab(Tab): self.torrent_id = None def _on_row_activated(self, tree, path, view_column): - self._on_menuitem_open_file_activate(None) + self.on_menuitem_open_file_activate(None) def get_file_path(self, row, path=''): if not row: @@ -498,12 +490,12 @@ class FilesTab(Tab): self.listview.set_cursor(path, column, True) return True - def _on_menuitem_open_file_activate(self, menuitem): + def on_menuitem_open_file_activate(self, menuitem): if client.is_localhost: component.get('SessionProxy').get_torrent_status( self.torrent_id, ['download_location']).addCallback(self._on_open_file) - def _on_menuitem_show_file_activate(self, menuitem): + def on_menuitem_show_file_activate(self, menuitem): if client.is_localhost: component.get('SessionProxy').get_torrent_status( self.torrent_id, ['download_location']).addCallback(self._on_show_file) @@ -525,23 +517,23 @@ class FilesTab(Tab): log.debug('priorities: %s', priorities) client.core.set_torrent_options([self.torrent_id], {'file_priorities': priorities}) - def _on_menuitem_ignore_activate(self, menuitem): + def on_menuitem_ignore_activate(self, menuitem): self._set_file_priorities_on_user_change( self.get_selected_files(), FILE_PRIORITY['Ignore']) - def _on_menuitem_low_activate(self, menuitem): + def on_menuitem_low_activate(self, menuitem): self._set_file_priorities_on_user_change( self.get_selected_files(), FILE_PRIORITY['Low']) - def _on_menuitem_normal_activate(self, menuitem): + def on_menuitem_normal_activate(self, menuitem): self._set_file_priorities_on_user_change( self.get_selected_files(), FILE_PRIORITY['Normal']) - def _on_menuitem_high_activate(self, menuitem): + def on_menuitem_high_activate(self, menuitem): self._set_file_priorities_on_user_change( self.get_selected_files(), FILE_PRIORITY['High']) - def _on_menuitem_expand_all_activate(self, menuitem): + def on_menuitem_expand_all_activate(self, menuitem): self.listview.expand_all() def _on_filename_edited(self, renderer, path, new_text): diff --git a/deluge/ui/gtkui/filtertreeview.py b/deluge/ui/gtkui/filtertreeview.py index 5ac6af7fb..eda7295f5 100644 --- a/deluge/ui/gtkui/filtertreeview.py +++ b/deluge/ui/gtkui/filtertreeview.py @@ -109,11 +109,7 @@ class FilterTreeView(component.Component): builder = gtk.Builder() builder.add_from_file(resource_filename('deluge.ui.gtkui', os.path.join('glade', 'filtertree_menu.ui'))) self.menu = builder.get_object('filtertree_menu') - builder.connect_signals({ - 'select_all': self.on_select_all, - 'pause_all': self.on_pause_all, - 'resume_all': self.on_resume_all - }) + builder.connect_signals(self) self.default_menu_items = self.menu.get_children() diff --git a/deluge/ui/gtkui/glade/filtertree_menu.ui b/deluge/ui/gtkui/glade/filtertree_menu.ui index 53b422f88..d85518dee 100644 --- a/deluge/ui/gtkui/glade/filtertree_menu.ui +++ b/deluge/ui/gtkui/glade/filtertree_menu.ui @@ -31,7 +31,7 @@ True image22 False - + @@ -43,7 +43,7 @@ True menu-item-image22 False - + @@ -56,7 +56,7 @@ True menu-item-image23 False - + diff --git a/deluge/ui/gtkui/menubar.py b/deluge/ui/gtkui/menubar.py index a02094aa2..1bf0cbfe7 100644 --- a/deluge/ui/gtkui/menubar.py +++ b/deluge/ui/gtkui/menubar.py @@ -109,48 +109,10 @@ class MenuBar(component.Component): self.main_builder.get_object('sidebar_show_owners').set_active(self.config['sidebar_show_owners']) # Connect main window Signals # - self.mainwindow.connect_signals({ - # File Menu - 'on_menuitem_addtorrent_activate': self.on_menuitem_addtorrent_activate, - 'on_menuitem_createtorrent_activate': self.on_menuitem_createtorrent_activate, - 'on_menuitem_quitdaemon_activate': self.on_menuitem_quitdaemon_activate, - 'on_menuitem_quit_activate': self.on_menuitem_quit_activate, - - # Edit Menu - 'on_menuitem_preferences_activate': self.on_menuitem_preferences_activate, - 'on_menuitem_connectionmanager_activate': self.on_menuitem_connectionmanager_activate, - - # View Menu - 'on_menuitem_toolbar_toggled': self.on_menuitem_toolbar_toggled, - 'on_menuitem_sidebar_toggled': self.on_menuitem_sidebar_toggled, - 'on_menuitem_statusbar_toggled': self.on_menuitem_statusbar_toggled, - - # Help Menu - 'on_menuitem_homepage_activate': self.on_menuitem_homepage_activate, - 'on_menuitem_faq_activate': self.on_menuitem_faq_activate, - 'on_menuitem_community_activate': self.on_menuitem_community_activate, - 'on_menuitem_about_activate': self.on_menuitem_about_activate, - 'on_menuitem_sidebar_zero_toggled': self.on_menuitem_sidebar_zero_toggled, - 'on_menuitem_sidebar_trackers_toggled': self.on_menuitem_sidebar_trackers_toggled, - 'on_menuitem_sidebar_owners_toggled': self.on_menuitem_sidebar_owners_toggled - }) + self.mainwindow.connect_signals(self) # Connect menubar signals - self.builder.connect_signals({ - # Torrent Menu - 'on_menuitem_pause_activate': self.on_menuitem_pause_activate, - 'on_menuitem_resume_activate': self.on_menuitem_resume_activate, - 'on_menuitem_updatetracker_activate': self.on_menuitem_updatetracker_activate, - 'on_menuitem_edittrackers_activate': self.on_menuitem_edittrackers_activate, - 'on_menuitem_remove_activate': self.on_menuitem_remove_activate, - 'on_menuitem_recheck_activate': self.on_menuitem_recheck_activate, - 'on_menuitem_open_folder_activate': self.on_menuitem_open_folder_activate, - 'on_menuitem_move_activate': self.on_menuitem_move_activate, - 'on_menuitem_queue_top_activate': self.on_menuitem_queue_top_activate, - 'on_menuitem_queue_up_activate': self.on_menuitem_queue_up_activate, - 'on_menuitem_queue_down_activate': self.on_menuitem_queue_down_activate, - 'on_menuitem_queue_bottom_activate': self.on_menuitem_queue_bottom_activate - }) + self.builder.connect_signals(self) self.change_sensitivity = [ 'menuitem_addtorrent' diff --git a/deluge/ui/gtkui/options_tab.py b/deluge/ui/gtkui/options_tab.py index 1015f55cf..1f323d7fd 100644 --- a/deluge/ui/gtkui/options_tab.py +++ b/deluge/ui/gtkui/options_tab.py @@ -52,14 +52,7 @@ class OptionsTab(Tab): self.prev_torrent_id = None self.prev_status = None - component.get('MainWindow').connect_signals({ - 'on_button_apply_clicked': self._on_button_apply_clicked, - 'on_chk_move_completed_toggled': self._on_chk_move_completed_toggled, - 'on_chk_stop_at_ratio_toggled': self._on_chk_stop_at_ratio_toggled, - 'on_chk_toggled': self._on_chk_toggled, - 'on_spin_value_changed': self._on_spin_value_changed, - 'on_move_completed_file_set': self._on_move_completed_file_set - }) + component.get('MainWindow').connect_signals(self) self.spin_max_download.connect('key-press-event', self._on_key_press_event) self.spin_max_upload.connect('key-press-event', self._on_key_press_event) @@ -163,7 +156,7 @@ class OptionsTab(Tab): self.prev_status = status - def _on_button_apply_clicked(self, button): + def on_button_apply_clicked(self, button): options = {} if self.spin_max_download.get_value() != self.prev_status['max_download_speed']: options['max_download_speed'] = self.spin_max_download.get_value() @@ -194,13 +187,13 @@ class OptionsTab(Tab): client.core.set_torrent_options([self.prev_torrent_id], options) self.button_apply.set_sensitive(False) - def _on_chk_move_completed_toggled(self, widget): + def on_chk_move_completed_toggled(self, widget): value = self.chk_move_completed.get_active() self.move_completed_path_chooser.set_sensitive(value) if not self.button_apply.is_sensitive(): self.button_apply.set_sensitive(True) - def _on_chk_stop_at_ratio_toggled(self, widget): + def on_chk_stop_at_ratio_toggled(self, widget): value = widget.get_active() self.spin_stop_ratio.set_sensitive(value) @@ -209,11 +202,11 @@ class OptionsTab(Tab): if not self.button_apply.is_sensitive(): self.button_apply.set_sensitive(True) - def _on_chk_toggled(self, widget): + def on_chk_toggled(self, widget): if not self.button_apply.is_sensitive(): self.button_apply.set_sensitive(True) - def _on_spin_value_changed(self, widget): + def on_spin_value_changed(self, widget): if not self.button_apply.is_sensitive(): self.button_apply.set_sensitive(True) @@ -223,7 +216,7 @@ class OptionsTab(Tab): if not self.button_apply.is_sensitive(): self.button_apply.set_sensitive(True) - def _on_move_completed_file_set(self, widget): + def on_move_completed_file_set(self, widget): if not self.button_apply.is_sensitive(): self.button_apply.set_sensitive(True) diff --git a/deluge/ui/gtkui/peers_tab.py b/deluge/ui/gtkui/peers_tab.py index e95916bec..5e7452227 100644 --- a/deluge/ui/gtkui/peers_tab.py +++ b/deluge/ui/gtkui/peers_tab.py @@ -37,9 +37,7 @@ class PeersTab(Tab): self._child_widget = main_builder.get_object('peers_tab') self._tab_label = main_builder.get_object('peers_tab_label') self.peer_menu = main_builder.get_object('menu_peer_tab') - component.get('MainWindow').connect_signals({ - 'on_menuitem_add_peer_activate': self._on_menuitem_add_peer_activate, - }) + component.get('MainWindow').connect_signals(self) self.listview = main_builder.get_object('peers_listview') self.listview.props.has_tooltip = True @@ -325,7 +323,7 @@ class PeersTab(Tab): else: return False - def _on_menuitem_add_peer_activate(self, menuitem): + def on_menuitem_add_peer_activate(self, menuitem): """This is a callback for manually adding a peer""" log.debug('on_menuitem_add_peer') builder = Builder() diff --git a/deluge/ui/gtkui/preferences.py b/deluge/ui/gtkui/preferences.py index c9d1a39db..51aab29de 100644 --- a/deluge/ui/gtkui/preferences.py +++ b/deluge/ui/gtkui/preferences.py @@ -103,7 +103,7 @@ class Preferences(component.Component): password_column.set_visible(False) self.accounts_listview.set_model(self.accounts_liststore) - self.accounts_listview.get_selection().connect('changed', self._on_accounts_selection_changed) + self.accounts_listview.get_selection().connect('changed', self.on_accounts_selection_changed) self.accounts_frame = self.builder.get_object('AccountsFrame') # Setup plugin tab listview @@ -124,36 +124,7 @@ class Preferences(component.Component): self.plugin_listview.get_selection().connect('changed', self.on_plugin_selection_changed) - self.builder.connect_signals({ - 'on_pref_dialog_delete_event': self.on_pref_dialog_delete_event, - 'on_button_ok_clicked': self.on_button_ok_clicked, - 'on_button_apply_clicked': self.on_button_apply_clicked, - 'on_button_cancel_clicked': self.on_button_cancel_clicked, - 'on_toggle': self.on_toggle, - 'on_test_port_clicked': self.on_test_port_clicked, - 'on_button_plugin_install_clicked': self._on_button_plugin_install_clicked, - 'on_button_rescan_plugins_clicked': self._on_button_rescan_plugins_clicked, - 'on_button_find_plugins_clicked': self._on_button_find_plugins_clicked, - 'on_button_cache_refresh_clicked': self._on_button_cache_refresh_clicked, - 'on_combo_encryption_changed': self._on_combo_encryption_changed, - 'on_combo_proxy_type_changed': self._on_combo_proxy_type_changed, - 'on_entry_proxy_host_paste_clipboard': self._on_entry_proxy_host_paste_clipboard, - 'on_button_associate_magnet_clicked': self._on_button_associate_magnet_clicked, - 'on_accounts_add_clicked': self._on_accounts_add_clicked, - 'on_accounts_delete_clicked': self._on_accounts_delete_clicked, - 'on_accounts_edit_clicked': self._on_accounts_edit_clicked, - 'on_piecesbar_toggle_toggled': self._on_piecesbar_toggle_toggled, - 'on_completed_color_set': self._on_completed_color_set, - 'on_revert_color_completed_clicked': self._on_revert_color_completed_clicked, - 'on_downloading_color_set': self._on_downloading_color_set, - 'on_revert_color_downloading_clicked': self._on_revert_color_downloading_clicked, - 'on_waiting_color_set': self._on_waiting_color_set, - 'on_revert_color_waiting_clicked': self._on_revert_color_waiting_clicked, - 'on_missing_color_set': self._on_missing_color_set, - 'on_revert_color_missing_clicked': self._on_revert_color_missing_clicked, - 'on_pref_dialog_configure_event': self.on_pref_dialog_configure_event, - 'on_checkbutton_language_toggled': self._on_checkbutton_language_toggled, - }) + self.builder.connect_signals(self) # Radio buttons to choose between systray and appindicator self.builder.get_object('alignment_tray_type').set_visible(bool(appindicator)) @@ -283,28 +254,28 @@ class Preferences(component.Component): if client.connected(): self._get_accounts_tab_data() - def _on_get_config(config): + def on_get_config(config): self.core_config = config - client.core.get_available_plugins().addCallback(_on_get_available_plugins) + client.core.get_available_plugins().addCallback(on_get_available_plugins) - def _on_get_available_plugins(plugins): + def on_get_available_plugins(plugins): self.all_plugins = plugins - client.core.get_enabled_plugins().addCallback(_on_get_enabled_plugins) + client.core.get_enabled_plugins().addCallback(on_get_enabled_plugins) - def _on_get_enabled_plugins(plugins): + def on_get_enabled_plugins(plugins): self.enabled_plugins = plugins - client.core.get_listen_port().addCallback(_on_get_listen_port) + client.core.get_listen_port().addCallback(on_get_listen_port) - def _on_get_listen_port(port): + def on_get_listen_port(port): self.active_port = port - client.core.get_session_status(DISK_CACHE_KEYS).addCallback(_on_get_session_status) + client.core.get_session_status(DISK_CACHE_KEYS).addCallback(on_get_session_status) - def _on_get_session_status(status): + def on_get_session_status(status): self.cache_status = status self._show() # This starts a series of client.core requests prior to showing the window - client.core.get_config().addCallback(_on_get_config) + client.core.get_config().addCallback(on_get_config) else: self._show() @@ -742,7 +713,7 @@ class Preferences(component.Component): widget.set_text(value) - def _on_button_cache_refresh_clicked(self, widget): + def on_button_cache_refresh_clicked(self, widget): def on_get_session_status(status): self.cache_status = status self.__update_cache_status() @@ -890,8 +861,8 @@ class Preferences(component.Component): self.builder.get_object('label_plugin_homepage').set_text(plugin_info['Home-page']) self.builder.get_object('label_plugin_details').set_text(plugin_info['Description']) - def _on_button_plugin_install_clicked(self, widget): - log.debug('_on_button_plugin_install_clicked') + def on_button_plugin_install_clicked(self, widget): + log.debug('on_button_plugin_install_clicked') chooser = gtk.FileChooserDialog( _('Select the Plugin'), self.pref_dialog, @@ -937,16 +908,16 @@ class Preferences(component.Component): # We need to re-show the preferences dialog to show the new plugins self.show() - def _on_button_rescan_plugins_clicked(self, widget): + def on_button_rescan_plugins_clicked(self, widget): component.get('PluginManager').scan_for_plugins() if client.connected(): client.core.rescan_plugins() self.show() - def _on_button_find_plugins_clicked(self, widget): + def on_button_find_plugins_clicked(self, widget): deluge.common.open_url_in_browser('http://dev.deluge-torrent.org/wiki/Plugins') - def _on_combo_encryption_changed(self, widget): + def on_combo_encryption_changed(self, widget): combo_encin = self.builder.get_object('combo_encin').get_active() combo_encout = self.builder.get_object('combo_encout').get_active() combo_enclevel = self.builder.get_object('combo_enclevel') @@ -957,7 +928,7 @@ class Preferences(component.Component): elif self.is_connected: combo_enclevel.set_sensitive(True) - def _on_combo_proxy_type_changed(self, widget): + def on_combo_proxy_type_changed(self, widget): proxy_type = self.builder.get_object('combo_proxy_type').get_active() proxy_entries = [ 'label_proxy_host', 'entry_proxy_host', 'label_proxy_port', 'spin_proxy_port', @@ -982,7 +953,7 @@ class Preferences(component.Component): else: self.builder.get_object(entry).hide() - def _on_entry_proxy_host_paste_clipboard(self, widget): + def on_entry_proxy_host_paste_clipboard(self, widget): text = gtk.clipboard_get().wait_for_text().strip() log.debug('on_entry_proxy_host_paste-clipboard: got paste: %s', text) text = text if '//' in text else '//' + text @@ -997,13 +968,13 @@ class Preferences(component.Component): if parsed.password: self.builder.get_object('entry_proxy_pass').set_text(parsed.password) - def _on_button_associate_magnet_clicked(self, widget): + def on_button_associate_magnet_clicked(self, widget): associate_magnet_links(True) def _get_accounts_tab_data(self): def on_ok(accounts): self.accounts_frame.show() - self._on_get_known_accounts(accounts) + self.on_get_known_accounts(accounts) def on_fail(failure): if failure.type == NotAuthorizedError: @@ -1016,7 +987,7 @@ class Preferences(component.Component): ).run() client.core.get_known_accounts().addCallback(on_ok).addErrback(on_fail) - def _on_get_known_accounts(self, known_accounts): + def on_get_known_accounts(self, known_accounts): known_accounts_to_log = [] for account in known_accounts: account_to_log = {} @@ -1025,7 +996,7 @@ class Preferences(component.Component): value = '*' * len(value) account_to_log[key] = value known_accounts_to_log.append(account_to_log) - log.debug('_on_known_accounts: %s', known_accounts_to_log) + log.debug('on_known_accounts: %s', known_accounts_to_log) self.accounts_liststore.clear() @@ -1035,8 +1006,8 @@ class Preferences(component.Component): self.accounts_liststore.set_value(accounts_iter, ACCOUNTS_LEVEL, account['authlevel']) self.accounts_liststore.set_value(accounts_iter, ACCOUNTS_PASSWORD, account['password']) - def _on_accounts_selection_changed(self, treeselection): - log.debug('_on_accounts_selection_changed') + def on_accounts_selection_changed(self, treeselection): + log.debug('on_accounts_selection_changed') (model, itr) = treeselection.get_selected() if not itr: return @@ -1048,7 +1019,7 @@ class Preferences(component.Component): self.builder.get_object('accounts_edit').set_sensitive(False) self.builder.get_object('accounts_delete').set_sensitive(False) - def _on_accounts_add_clicked(self, widget): + def on_accounts_add_clicked(self, widget): dialog = AccountDialog(levels_mapping=client.auth_levels_mapping, parent=self.pref_dialog) def dialog_finished(response_id): @@ -1083,7 +1054,7 @@ class Preferences(component.Component): dialog.run().addCallback(dialog_finished) - def _on_accounts_edit_clicked(self, widget): + def on_accounts_edit_clicked(self, widget): (model, itr) = self.accounts_listview.get_selection().get_selected() if not itr: return @@ -1118,7 +1089,7 @@ class Preferences(component.Component): dialog.run().addCallback(dialog_finished) - def _on_accounts_delete_clicked(self, widget): + def on_accounts_delete_clicked(self, widget): (model, itr) = self.accounts_listview.get_selection().get_selected() if not itr: return @@ -1152,36 +1123,36 @@ class Preferences(component.Component): ).addCallback(remove_ok).addErrback(remove_fail) dialog.run().addCallback(dialog_finished) - def _on_piecesbar_toggle_toggled(self, widget): + def on_piecesbar_toggle_toggled(self, widget): self.gtkui_config['show_piecesbar'] = widget.get_active() colors_widget = self.builder.get_object('piecebar_colors_expander') colors_widget.set_visible(widget.get_active()) - def _on_checkbutton_language_toggled(self, widget): + def on_checkbutton_language_toggled(self, widget): self.language_combo.set_visible(not self.language_checkbox.get_active()) - def _on_completed_color_set(self, widget): + def on_completed_color_set(self, widget): self.__set_color('completed') - def _on_revert_color_completed_clicked(self, widget): + def on_revert_color_completed_clicked(self, widget): self.__revert_color('completed') - def _on_downloading_color_set(self, widget): + def on_downloading_color_set(self, widget): self.__set_color('downloading') - def _on_revert_color_downloading_clicked(self, widget): + def on_revert_color_downloading_clicked(self, widget): self.__revert_color('downloading') - def _on_waiting_color_set(self, widget): + def on_waiting_color_set(self, widget): self.__set_color('waiting') - def _on_revert_color_waiting_clicked(self, widget): + def on_revert_color_waiting_clicked(self, widget): self.__revert_color('waiting') - def _on_missing_color_set(self, widget): + def on_missing_color_set(self, widget): self.__set_color('missing') - def _on_revert_color_missing_clicked(self, widget): + def on_revert_color_missing_clicked(self, widget): self.__revert_color('missing') def __set_color(self, state, from_config=False): diff --git a/deluge/ui/gtkui/queuedtorrents.py b/deluge/ui/gtkui/queuedtorrents.py index c48e481a9..64cf4baad 100644 --- a/deluge/ui/gtkui/queuedtorrents.py +++ b/deluge/ui/gtkui/queuedtorrents.py @@ -38,13 +38,7 @@ class QueuedTorrents(component.Component): self.dialog = self.builder.get_object('queued_torrents_dialog') self.dialog.set_icon(get_logo(32)) - self.builder.connect_signals({ - 'on_button_remove_clicked': self.on_button_remove_clicked, - 'on_button_clear_clicked': self.on_button_clear_clicked, - 'on_button_close_clicked': self.on_button_close_clicked, - 'on_button_add_clicked': self.on_button_add_clicked, - 'on_chk_autoadd_toggled': self.on_chk_autoadd_toggled - }) + self.builder.connect_signals(self) self.treeview = self.builder.get_object('treeview') self.treeview.append_column(TreeViewColumn(_('Torrent'), CellRendererText(), text=0)) diff --git a/deluge/ui/gtkui/removetorrentdialog.py b/deluge/ui/gtkui/removetorrentdialog.py index 7b850dbf5..4d2893278 100644 --- a/deluge/ui/gtkui/removetorrentdialog.py +++ b/deluge/ui/gtkui/removetorrentdialog.py @@ -48,9 +48,7 @@ class RemoveTorrentDialog(object): self.__dialog = self.builder.get_object('remove_torrent_dialog') self.__dialog.set_transient_for(component.get('MainWindow').window) - self.builder.connect_signals({ - 'on_delete_files_toggled': self.on_delete_files_toggled - }) + self.builder.connect_signals(self) self.builder.get_object('delete_files').set_active(delete_files) label_title = self.builder.get_object('label_title') label_torrents = self.builder.get_object('label_torrents') diff --git a/deluge/ui/gtkui/systemtray.py b/deluge/ui/gtkui/systemtray.py index 5cae9a097..6212d969e 100644 --- a/deluge/ui/gtkui/systemtray.py +++ b/deluge/ui/gtkui/systemtray.py @@ -69,14 +69,7 @@ class SystemTray(component.Component): self.builder.add_from_file(resource_filename('deluge.ui.gtkui', os.path.join( 'glade', 'tray_menu.ui'))) - self.builder.connect_signals({ - 'on_menuitem_show_deluge_activate': self.on_menuitem_show_deluge_activate, - 'on_menuitem_add_torrent_activate': self.on_menuitem_add_torrent_activate, - 'on_menuitem_pause_session_activate': self.on_menuitem_pause_session_activate, - 'on_menuitem_resume_session_activate': self.on_menuitem_resume_session_activate, - 'on_menuitem_quit_activate': self.on_menuitem_quit_activate, - 'on_menuitem_quitdaemon_activate': self.on_menuitem_quitdaemon_activate - }) + self.builder.connect_signals(self) self.tray_menu = self.builder.get_object('tray_menu') diff --git a/deluge/ui/gtkui/toolbar.py b/deluge/ui/gtkui/toolbar.py index 921779bf7..75512568d 100644 --- a/deluge/ui/gtkui/toolbar.py +++ b/deluge/ui/gtkui/toolbar.py @@ -28,16 +28,7 @@ class ToolBar(component.Component): self.toolbar = self.main_builder.get_object('toolbar') self.config = ConfigManager('gtkui.conf') # Connect main window Signals # - mainwindow.connect_signals({ - 'on_toolbutton_add_clicked': self.on_toolbutton_add_clicked, - 'on_toolbutton_remove_clicked': self.on_toolbutton_remove_clicked, - 'on_toolbutton_pause_clicked': self.on_toolbutton_pause_clicked, - 'on_toolbutton_resume_clicked': self.on_toolbutton_resume_clicked, - 'on_toolbutton_preferences_clicked': self.on_toolbutton_preferences_clicked, - 'on_toolbutton_connectionmanager_clicked': self.on_toolbutton_connectionmanager_clicked, - 'on_toolbutton_queue_up_clicked': self.on_toolbutton_queue_up_clicked, - 'on_toolbutton_queue_down_clicked': self.on_toolbutton_queue_down_clicked - }) + mainwindow.connect_signals(self) self.change_sensitivity = [ 'toolbutton_add', 'toolbutton_remove', diff --git a/deluge/ui/gtkui/trackers_tab.py b/deluge/ui/gtkui/trackers_tab.py index 82f08ba7f..7c9dc800f 100644 --- a/deluge/ui/gtkui/trackers_tab.py +++ b/deluge/ui/gtkui/trackers_tab.py @@ -40,9 +40,7 @@ class TrackersTab(Tab): self.status_keys = [status for widget in self.label_widgets for status in widget[2]] - component.get('MainWindow').connect_signals({ - 'on_button_edit_trackers_clicked': self._on_button_edit_trackers_clicked, - }) + component.get('MainWindow').connect_signals(self) def update(self): # Get the first selected torrent @@ -73,7 +71,7 @@ class TrackersTab(Tab): for widget in self.label_widgets: widget[0].set_text('') - def _on_button_edit_trackers_clicked(self, button): + def on_button_edit_trackers_clicked(self, button): torrent_id = component.get('TorrentView').get_selected_torrent() if torrent_id: from deluge.ui.gtkui.edittrackersdialog import EditTrackersDialog