diff --git a/deluge/config.py b/deluge/config.py index 6133e2f5c..9fe700159 100644 --- a/deluge/config.py +++ b/deluge/config.py @@ -125,7 +125,7 @@ class Config(object): self._save_timer = None if defaults: - for key, value in defaults.items(): + for key, value in defaults.iteritems(): self.set_item(key, value) # Load the config from file in the config_dir @@ -363,7 +363,7 @@ what is currently in the config and it could not convert the value """ log.debug("Calling all set functions..") - for key, value in self.__set_functions.items(): + for key, value in self.__set_functions.iteritems(): for func in value: func(key, self.__config[key]) diff --git a/deluge/core/authmanager.py b/deluge/core/authmanager.py index 105a0905e..ee94f003e 100644 --- a/deluge/core/authmanager.py +++ b/deluge/core/authmanager.py @@ -29,7 +29,7 @@ AUTH_LEVELS_MAPPING = { } AUTH_LEVELS_MAPPING_REVERSE = {} -for key, value in AUTH_LEVELS_MAPPING.items(): +for key, value in AUTH_LEVELS_MAPPING.iteritems(): AUTH_LEVELS_MAPPING_REVERSE[value] = key diff --git a/deluge/core/filtermanager.py b/deluge/core/filtermanager.py index babf7d5ad..cce367826 100644 --- a/deluge/core/filtermanager.py +++ b/deluge/core/filtermanager.py @@ -173,7 +173,7 @@ class FilterManager(component.Component): # Leftover filter arguments, default filter on status fields. for torrent_id in list(torrent_ids): status = self.core.create_torrent_status(torrent_id, torrent_keys, plugin_keys) - for field, values in filter_dict.items(): + for field, values in filter_dict.iteritems(): if field in status and status[field] in values: continue elif torrent_id in torrent_ids: @@ -212,7 +212,7 @@ class FilterManager(component.Component): # Return a dict of tuples: sorted_items = {} for field in tree_keys: - sorted_items[field] = sorted(items[field].items()) + sorted_items[field] = sorted(items[field].iteritems()) if "state" in tree_keys: sorted_items["state"].sort(self._sort_state_items) diff --git a/deluge/core/torrent.py b/deluge/core/torrent.py index 43dc1299b..97c03b283 100644 --- a/deluge/core/torrent.py +++ b/deluge/core/torrent.py @@ -160,7 +160,7 @@ class TorrentOptions(dict): "super_seeding": "super_seeding", "priority": "priority", } - for opt_k, conf_k in options_conf_map.items(): + for opt_k, conf_k in options_conf_map.iteritems(): self[opt_k] = config[conf_k] self["file_priorities"] = [] self["mapped_files"] = {} @@ -1323,7 +1323,7 @@ class Torrent(object): torrent.waiting_on_folder_rename = [_dir for _dir in torrent.waiting_on_folder_rename if _dir] component.get("TorrentManager").save_resume_data((self.torrent_id,)) - d = DeferredList(list(wait_on_folder.values())) + d = DeferredList(wait_on_folder.values()) d.addBoth(on_folder_rename_complete, self, folder, new_folder) return d diff --git a/deluge/core/torrentmanager.py b/deluge/core/torrentmanager.py index 6ec76b801..b9d0b00aa 100644 --- a/deluge/core/torrentmanager.py +++ b/deluge/core/torrentmanager.py @@ -746,7 +746,7 @@ class TorrentManager(component.Component): """ if torrent_ids is None: - torrent_ids = (t[0] for t in self.torrents.items() if t[1].handle.need_save_resume_data()) + torrent_ids = (t[0] for t in self.torrents.iteritems() if t[1].handle.need_save_resume_data()) def on_torrent_resume_save(dummy_result, torrent_id): """Recieved torrent resume_data alert so remove from waiting list""" @@ -918,7 +918,7 @@ class TorrentManager(component.Component): def cleanup_torrents_prev_status(self): """Run cleanup_prev_status for each registered torrent""" - for torrent in self.torrents.items(): + for torrent in self.torrents.iteritems(): torrent[1].cleanup_prev_status() def on_set_max_connections_per_torrent(self, key, value): diff --git a/deluge/plugins/AutoAdd/deluge/plugins/autoadd/core.py b/deluge/plugins/AutoAdd/deluge/plugins/autoadd/core.py index ba9398f6c..9487024f9 100644 --- a/deluge/plugins/AutoAdd/deluge/plugins/autoadd/core.py +++ b/deluge/plugins/AutoAdd/deluge/plugins/autoadd/core.py @@ -99,7 +99,7 @@ class Core(CorePluginBase): def enable_looping(self): # Enable all looping calls for enabled watchdirs here - for watchdir_id, watchdir in self.watchdirs.items(): + for watchdir_id, watchdir in self.watchdirs.iteritems(): if watchdir["enabled"]: self.enable_watchdir(watchdir_id) @@ -108,7 +108,7 @@ class Core(CorePluginBase): component.get("EventManager").deregister_event_handler( "PreTorrentRemovedEvent", self.__on_pre_torrent_removed ) - for loopingcall in self.update_timers.values(): + for loopingcall in self.update_timers.itervalues(): loopingcall.stop() self.config.save() @@ -128,7 +128,7 @@ class Core(CorePluginBase): check_input( os.path.isdir(options["abspath"]), _("Path does not exist.") ) - for w_id, w in self.watchdirs.items(): + for w_id, w in self.watchdirs.iteritems(): if options["abspath"] == w["abspath"] and watchdir_id != w_id: raise Exception("Path is already being watched.") for key in options: @@ -223,7 +223,7 @@ class Core(CorePluginBase): watchdir["stop_ratio_toggle"] = watchdir["stop_at_ratio_toggle"] # We default to True when reading _toggle values, so a config # without them is valid, and applies all its settings. - for option, value in watchdir.items(): + for option, value in watchdir.iteritems(): if OPTIONS_AVAILABLE.get(option): if watchdir.get(option + "_toggle", True) or option in ["owner", "seed_mode"]: opts[option] = value @@ -381,7 +381,7 @@ class Core(CorePluginBase): return self.watchdirs watchdirs = {} - for watchdir_id, watchdir in self.watchdirs.items(): + for watchdir_id, watchdir in self.watchdirs.iteritems(): if watchdir.get("owner", "localclient") == session_user: watchdirs[watchdir_id] = watchdir @@ -409,7 +409,7 @@ class Core(CorePluginBase): os.access(abswatchdir, os.R_OK | os.W_OK), "You must have read and write access to watch folder." ) - if abswatchdir in [wd["abspath"] for wd in self.watchdirs.values()]: + if abswatchdir in [wd["abspath"] for wd in self.watchdirs.itervalues()]: raise Exception("Path is already being watched.") options.setdefault("enabled", False) options["abspath"] = abswatchdir @@ -447,7 +447,7 @@ class Core(CorePluginBase): torrent_id) return torrent_fname = torrent.filename - for watchdir in self.watchdirs.values(): + for watchdir in self.watchdirs.itervalues(): if not watchdir.get("copy_torrent_toggle", False): # This watchlist does copy torrents continue diff --git a/deluge/plugins/AutoAdd/deluge/plugins/autoadd/gtkui.py b/deluge/plugins/AutoAdd/deluge/plugins/autoadd/gtkui.py index a4f6dc226..9a569ed73 100644 --- a/deluge/plugins/AutoAdd/deluge/plugins/autoadd/gtkui.py +++ b/deluge/plugins/AutoAdd/deluge/plugins/autoadd/gtkui.py @@ -401,7 +401,7 @@ class GtkUI(GtkPluginBase): def create_model(self): store = gtk.ListStore(str, bool, str, str) - for watchdir_id, watchdir in self.watchdirs.items(): + for watchdir_id, watchdir in self.watchdirs.iteritems(): store.append([ watchdir_id, watchdir['enabled'], watchdir.get('owner', 'localclient'), watchdir['path'] @@ -474,7 +474,7 @@ class GtkUI(GtkPluginBase): def on_apply_prefs(self): log.debug("applying prefs for AutoAdd") - for watchdir_id, watchdir in self.watchdirs.items(): + for watchdir_id, watchdir in self.watchdirs.iteritems(): client.autoadd.set_options(watchdir_id, watchdir) def on_show_prefs(self): @@ -488,7 +488,7 @@ class GtkUI(GtkPluginBase): log.trace("Got whatchdirs from core: %s", watchdirs) self.watchdirs = watchdirs or {} self.store.clear() - for watchdir_id, watchdir in self.watchdirs.items(): + for watchdir_id, watchdir in self.watchdirs.iteritems(): self.store.append([ watchdir_id, watchdir['enabled'], watchdir.get('owner', 'localclient'), watchdir['path'] diff --git a/deluge/plugins/Execute/deluge/plugins/execute/core.py b/deluge/plugins/Execute/deluge/plugins/execute/core.py index d91f91770..6fc257138 100644 --- a/deluge/plugins/Execute/deluge/plugins/execute/core.py +++ b/deluge/plugins/Execute/deluge/plugins/execute/core.py @@ -133,7 +133,7 @@ class Core(CorePluginBase): def disable(self): self.config.save() event_manager = component.get("EventManager") - for event, handler in self.registered_events.items(): + for event, handler in self.registered_events.iteritems(): event_manager.deregister_event_handler(event, handler) log.debug("Execute core plugin disabled!") diff --git a/deluge/plugins/Label/deluge/plugins/label/core.py b/deluge/plugins/Label/deluge/plugins/label/core.py index bd01b815c..5ca933375 100644 --- a/deluge/plugins/Label/deluge/plugins/label/core.py +++ b/deluge/plugins/Label/deluge/plugins/label/core.py @@ -118,7 +118,7 @@ class Core(CorePluginBase): log.debug("post_torrent_add") torrent = self.torrents[torrent_id] - for label_id, options in self.labels.items(): + for label_id, options in self.labels.iteritems(): if options["auto_add"]: if self._has_auto_match(torrent, options): self.set_torrent(torrent_id, label_id) @@ -132,7 +132,7 @@ class Core(CorePluginBase): # Utils # def clean_config(self): """remove invalid data from config-file""" - for torrent_id, label_id in list(self.torrent_labels.items()): + for torrent_id, label_id in list(self.torrent_labels.iteritems()): if (label_id not in self.labels) or (torrent_id not in self.torrents): log.debug("label: rm %s:%s", torrent_id, label_id) del self.torrent_labels[torrent_id] @@ -266,14 +266,14 @@ class Core(CorePluginBase): self.labels[label_id].update(options_dict) # apply - for torrent_id, label in self.torrent_labels.items(): + for torrent_id, label in self.torrent_labels.iteritems(): if label_id == label and torrent_id in self.torrents: self._set_torrent_options(torrent_id, label_id) # auto add options = self.labels[label_id] if options["auto_add"]: - for torrent_id, torrent in self.torrents.items(): + for torrent_id, torrent in self.torrents.iteritems(): if self._has_auto_match(torrent, options): self.set_torrent(torrent_id, label_id) diff --git a/deluge/plugins/Stats/deluge/plugins/stats/core.py b/deluge/plugins/Stats/deluge/plugins/stats/core.py index fcbe335ae..8bdc75570 100644 --- a/deluge/plugins/Stats/deluge/plugins/stats/core.py +++ b/deluge/plugins/Stats/deluge/plugins/stats/core.py @@ -138,7 +138,7 @@ class Core(CorePluginBase): # extract the ones we are interested in # adding them to the 1s array - for stat, stat_list in self.stats[1].items(): + for stat, stat_list in self.stats[1].iteritems(): if stat in stats: stat_list.insert(0, int(stats[stat])) else: @@ -152,7 +152,7 @@ class Core(CorePluginBase): self.last_update[interval] = update_time self.count[interval] = 0 current_stats = self.stats[interval] - for stat, stat_list in self.stats[base].items(): + for stat, stat_list in self.stats[base].iteritems(): try: avg = mean(stat_list[0:multiplier]) except ValueError: diff --git a/deluge/plugins/Stats/deluge/plugins/stats/graph.py b/deluge/plugins/Stats/deluge/plugins/stats/graph.py index 974f8c9e2..86682b9cc 100644 --- a/deluge/plugins/Stats/deluge/plugins/stats/graph.py +++ b/deluge/plugins/Stats/deluge/plugins/stats/graph.py @@ -245,7 +245,7 @@ class Graph(object): self.draw_y_text(y_tick_text[i], left, y) self.draw_line(gray, left, top, left, bottom) - for stat, info in stats.items(): + for stat, info in stats.iteritems(): if len(info['values']) > 0: self.draw_value_poly(info['values'], info['color'], max_value, bounds) self.draw_value_poly(info['values'], info['fill_color'], max_value, bounds, info['fill']) diff --git a/deluge/plugins/Stats/deluge/plugins/stats/tests/test_stats.py b/deluge/plugins/Stats/deluge/plugins/stats/tests/test_stats.py index 96cbd049f..a9ca8dc6a 100644 --- a/deluge/plugins/Stats/deluge/plugins/stats/tests/test_stats.py +++ b/deluge/plugins/Stats/deluge/plugins/stats/tests/test_stats.py @@ -18,7 +18,7 @@ from deluge.ui.client import client def print_totals(totals): - for name, value in totals.items(): + for name, value in totals.iteritems(): print(name, fsize(value)) print("overhead:") diff --git a/deluge/ui/gtkui/addtorrentdialog.py b/deluge/ui/gtkui/addtorrentdialog.py index de4c55833..6b7f60efa 100644 --- a/deluge/ui/gtkui/addtorrentdialog.py +++ b/deluge/ui/gtkui/addtorrentdialog.py @@ -302,7 +302,7 @@ class AddTorrentDialog(component.Component): def add_files(self, parent_iter, split_files): ret = 0 - for key, value in split_files.items(): + for key, value in split_files.iteritems(): if key.endswith(os.path.sep): chunk_iter = self.files_treestore.append( parent_iter, [True, key, 0, -1, False, gtk.STOCK_DIRECTORY]) diff --git a/deluge/ui/gtkui/files_tab.py b/deluge/ui/gtkui/files_tab.py index a9e77bc7e..8014cf03d 100644 --- a/deluge/ui/gtkui/files_tab.py +++ b/deluge/ui/gtkui/files_tab.py @@ -364,7 +364,7 @@ class FilesTab(Tab): def add_files(self, parent_iter, split_files): chunk_size_total = 0 - for key, value in split_files.items(): + for key, value in split_files.iteritems(): if key.endswith("/"): chunk_iter = self.treestore.append(parent_iter, [key, 0, "", 0, 0, -1, gtk.STOCK_DIRECTORY]) diff --git a/deluge/ui/gtkui/filtertreeview.py b/deluge/ui/gtkui/filtertreeview.py index 500a08b4d..c788048ec 100644 --- a/deluge/ui/gtkui/filtertreeview.py +++ b/deluge/ui/gtkui/filtertreeview.py @@ -159,7 +159,7 @@ class FilterTreeView(component.Component): # update rows visible_filters = [] - for cat, filters in filter_items.items(): + for cat, filters in filter_items.iteritems(): for value, count in filters: self.update_row(cat, value, count) visible_filters.append((cat, value)) diff --git a/deluge/ui/gtkui/listview.py b/deluge/ui/gtkui/listview.py index 3ff769074..004560973 100644 --- a/deluge/ui/gtkui/listview.py +++ b/deluge/ui/gtkui/listview.py @@ -118,7 +118,7 @@ class ListView(object): def set_col_attributes(self, renderer, add=True, **kw): if add is True: - for attr, value in kw.items(): + for attr, value in kw.iteritems(): self.add_attribute(renderer, attr, value) else: self.set_attributes(renderer, **kw) diff --git a/deluge/ui/gtkui/mainwindow.py b/deluge/ui/gtkui/mainwindow.py index 75508707d..cd2e1c137 100644 --- a/deluge/ui/gtkui/mainwindow.py +++ b/deluge/ui/gtkui/mainwindow.py @@ -41,7 +41,7 @@ class _GtkBuilderSignalsHolder(object): def connect_signals(self, mapping_or_class): if isinstance(mapping_or_class, dict): - for name, handler in mapping_or_class.items(): + for name, handler in mapping_or_class.iteritems(): if hasattr(self, name): raise RuntimeError( "A handler for signal %r has already been registered: %s" % diff --git a/deluge/ui/gtkui/menubar.py b/deluge/ui/gtkui/menubar.py index a3280e7ce..c48eebc54 100644 --- a/deluge/ui/gtkui/menubar.py +++ b/deluge/ui/gtkui/menubar.py @@ -496,7 +496,7 @@ class MenuBar(component.Component): known_accounts_to_log = [] for account in known_accounts: account_to_log = {} - for key, value in account.copy().items(): + for key, value in account.copy().iteritems(): if key == 'password': value = '*' * len(value) account_to_log[key] = value @@ -539,7 +539,7 @@ class MenuBar(component.Component): return torrent_owner = component.get("TorrentView").get_torrent_status(selected[0])["owner"] - for username, item in self.change_owner_submenu_items.items(): + for username, item in self.change_owner_submenu_items.iteritems(): item.set_active(username == torrent_owner) def _on_change_owner_toggled(self, widget, username): diff --git a/deluge/ui/gtkui/preferences.py b/deluge/ui/gtkui/preferences.py index cb22becf5..03f2d9d0c 100644 --- a/deluge/ui/gtkui/preferences.py +++ b/deluge/ui/gtkui/preferences.py @@ -1005,7 +1005,7 @@ class Preferences(component.Component): known_accounts_to_log = [] for account in known_accounts: account_to_log = {} - for key, value in account.copy().items(): + for key, value in account.copy().iteritems(): if key == 'password': value = '*' * len(value) account_to_log[key] = value diff --git a/deluge/ui/gtkui/torrentdetails.py b/deluge/ui/gtkui/torrentdetails.py index b71a7e872..0db1023cb 100644 --- a/deluge/ui/gtkui/torrentdetails.py +++ b/deluge/ui/gtkui/torrentdetails.py @@ -129,7 +129,7 @@ class TorrentDetails(component.Component): # We need to rename the tab in the state for backwards compat self.state = [(tab_name.replace("Statistics", "Status"), visible) for tab_name, visible in state] - for tab in default_tabs.values(): + for tab in default_tabs.itervalues(): self.add_tab(tab(), generate_menu=False) # Generate the checklist menu @@ -140,7 +140,7 @@ class TorrentDetails(component.Component): # Determine insert position based on weight # weights is a list of visible tab names in weight order - weights = sorted([(tab.weight, name) for name, tab in self.tabs.items() if tab.is_visible]) + weights = sorted([(tab.weight, name) for name, tab in self.tabs.iteritems() if tab.is_visible]) log.debug("weights: %s", weights) log.debug("weight of tab: %s", weight) @@ -243,7 +243,7 @@ class TorrentDetails(component.Component): self.generate_menu() show = False - for name, tab in self.tabs.items(): + for name, tab in self.tabs.iteritems(): show = show or tab.is_visible self.visible(show) diff --git a/deluge/ui/sessionproxy.py b/deluge/ui/sessionproxy.py index 76f0d8c17..d06fba72d 100644 --- a/deluge/ui/sessionproxy.py +++ b/deluge/ui/sessionproxy.py @@ -178,7 +178,7 @@ class SessionProxy(component.Component): def on_status(result, torrent_ids, keys): # Update the internal torrent status dict with the update values t = time() - for key, value in result.items(): + for key, value in result.iteritems(): try: self.torrents[key][0] = t self.torrents[key][1].update(value)