Unify quote style from single quotes to double quotes (only in tools)

Author: BigRoy

avalon/tools/creator/app.py

@@ -139,8 +139,8 @@ def __init__(self, parent=None):
         create_btn.setEnabled(False)
 
     def _on_state_changed(self, state):
-        self.state['valid'] = state
-        self.data['Create Button'].setEnabled(state)
+        self.state["valid"] = state
+        self.data["Create Button"].setEnabled(state)
 
     def _build_menu(self, default_names):
         """Create optional predefined subset names
@@ -311,7 +311,7 @@ def refresh(self):
     def on_create(self):
 
         # Do not allow creation in an invalid state
-        if not self.state['valid']:
+        if not self.state["valid"]:
             return
 
         asset = self.data["Asset"]

avalon/tools/delegates.py

@@ -50,15 +50,15 @@ def setEditorData(self, editor, index):
 
         # Add all available versions to the editor
         item = index.data(TreeModel.ItemRole)
-        parent_id = item['version_document']['parent']
+        parent_id = item["version_document"]["parent"]
         versions = io.find({"type": "version", "parent": parent_id},
                            sort=[("name", 1)])
         index = 0
         for i, version in enumerate(versions):
-            label = self._format_version(version['name'])
+            label = self._format_version(version["name"])
             editor.addItem(label, userData=version)
 
-            if version['name'] == value:
+            if version["name"] == value:
                 index = i
 
         editor.setCurrentIndex(index)  # Will trigger index-change signal
@@ -68,4 +68,4 @@ def setEditorData(self, editor, index):
     def setModelData(self, editor, model, index):
         """Apply the integer version back in the model"""
         version = editor.itemData(editor.currentIndex())
-        model.setData(index, version['name'])
+        model.setData(index, version["name"])

avalon/tools/lib.py

@@ -256,8 +256,8 @@ def refresh_family_config_cache():
                           projection={"config.families": True})
 
     assert project, "Project not found!"
-    families = project['config'].get("families", [])
-    families = {family['name']: family for family in families}
+    families = project["config"].get("families", [])
+    families = {family["name"]: family for family in families}
 
     # Check if any family state are being overwritten by the configuration
     default_state = api.data.get("familiesStateDefault", True)
@@ -269,10 +269,10 @@ def refresh_family_config_cache():
         # Set family icon
         icon = family.get("icon", None)
         if icon:
-            family['icon'] = qtawesome.icon("fa.{}".format(icon),
+            family["icon"] = qtawesome.icon("fa.{}".format(icon),
                                             color=FAMILY_ICON_COLOR)
         else:
-            family['icon'] = default_icon
+            family["icon"] = default_icon
 
         # Update state
         state = not default_state if name in toggled else default_state

avalon/tools/loader/app.py

@@ -180,10 +180,10 @@ def _assetschanged(self):
             return
 
         document = asset_item.data(DocumentRole)
-        subsets_model.set_asset(document['_id'])
+        subsets_model.set_asset(document["_id"])
 
         # Clear the version information on asset change
-        self.data['model']['version'].set_version(None)
+        self.data["model"]["version"].set_version(None)
 
         self.data["state"]["context"]["asset"] = document["name"]
         self.data["state"]["context"]["assetId"] = document["_id"]
@@ -204,9 +204,9 @@ def _versionschanged(self):
             if active in rows:
                 node = active.data(subsets.model.ItemRole)
                 if node is not None and not node.get("isGroup"):
-                    version = node['version_document']['_id']
+                    version = node["version_document"]["_id"]
 
-        self.data['model']['version'].set_version(version)
+        self.data["model"]["version"].set_version(version)
 
     def _set_context(self, context, refresh=True):
         """Set the selection in the interface using a context.
@@ -243,7 +243,7 @@ def _set_context(self, context, refresh=True):
             # scheduled refresh and the silo tabs are not shown.
             self._refresh()
 
-        asset_widget = self.data['model']['assets']
+        asset_widget = self.data["model"]["assets"]
         asset_widget.set_silo(silo)
         asset_widget.select_assets([asset], expand=True)
 
@@ -441,8 +441,8 @@ def show(debug=False, parent=None, use_context=False):
         window.show()
 
         if use_context:
-            context = {"asset": api.Session['AVALON_ASSET'],
-                       "silo": api.Session['AVALON_SILO']}
+            context = {"asset": api.Session["AVALON_ASSET"],
+                       "silo": api.Session["AVALON_SILO"]}
             window.set_context(context, refresh=True)
         else:
             window.refresh()

avalon/tools/loader/delegates.py

@@ -34,7 +34,7 @@ def pretty_date(t, now=None, strftime="%b %d %Y %H:%M"):
 
     # future (consider as just now)
     if day_diff < 0:
-        return 'just now'
+        return "just now"
 
     # history
     if day_diff == 0:

avalon/tools/loader/model.py

@@ -74,7 +74,7 @@ def set_version(self, index, version):
             return
 
         item = index.internalPointer()
-        assert version['parent'] == item['_id'], ("Version does not "
+        assert version["parent"] == item["_id"], ("Version does not "
                                                   "belong to subset")
 
         # Get the data from the version
@@ -102,8 +102,8 @@ def set_version(self, index, version):
         if frame_start is not None and frame_end is not None:
             # Remove superfluous zeros from numbers (3.0 -> 3) to improve
             # readability for most frame ranges
-            start_clean = ('%f' % frame_start).rstrip('0').rstrip('.')
-            end_clean = ('%f' % frame_end).rstrip('0').rstrip('.')
+            start_clean = ("%f" % frame_start).rstrip("0").rstrip(".")
+            end_clean = ("%f" % frame_end).rstrip("0").rstrip(".")
             frames = "{0}-{1}".format(start_clean, end_clean)
             duration = frame_end - frame_start + 1
         else:
@@ -115,13 +115,13 @@ def set_version(self, index, version):
         family_config = tools_lib.get_family_cached_config(family)
 
         item.update({
-            "version": version['name'],
+            "version": version["name"],
             "version_document": version,
             "author": version_data.get("author", None),
             "time": version_data.get("time", None),
             "family": family,
             "familyLabel": family_config.get("label", family),
-            "familyIcon": family_config.get('icon', None),
+            "familyIcon": family_config.get("icon", None),
             "families": set(families),
             "frameStart": frame_start,
             "frameEnd": frame_end,

avalon/tools/loader/widgets.py

@@ -141,13 +141,13 @@ def on_context_menu(self, point):
         available_loaders = api.discover(api.Loader)
         loaders = list()
 
-        version_id = node['version_document']['_id']
+        version_id = node["version_document"]["_id"]
         representations = io.find({"type": "representation",
                                    "parent": version_id})
         for representation in representations:
             for loader in api.loaders_from_representation(
                     available_loaders,
-                    representation['_id']
+                    representation["_id"]
             ):
                 loaders.append((representation, loader))
 
@@ -171,7 +171,7 @@ def sorter(value):
                 label = loader.__name__
 
             # Add the representation as suffix
-            label = "{0} ({1})".format(label, representation['name'])
+            label = "{0} ({1})".format(label, representation["name"])
 
             action = QtWidgets.QAction(label, menu)
             action.setData((representation, loader))
@@ -204,7 +204,7 @@ def sorter(value):
 
         # Find the representation name and loader to trigger
         action_representation, loader = action.data()
-        representation_name = action_representation['name']  # extension
+        representation_name = action_representation["name"]  # extension
 
         # Run the loader for all selected indices, for those that have the
         # same representation available
@@ -320,26 +320,26 @@ def set_version(self, version_id):
         version = io.find_one({"_id": version_id, "type": "version"})
         assert version, "Not a valid version id"
 
-        subset = io.find_one({"_id": version['parent'], "type": "subset"})
+        subset = io.find_one({"_id": version["parent"], "type": "subset"})
         assert subset, "No valid subset parent for version"
 
         # Define readable creation timestamp
         created = version["data"]["time"]
         created = datetime.datetime.strptime(created, "%Y%m%dT%H%M%SZ")
         created = datetime.datetime.strftime(created, "%b %d %Y %H:%M")
 
-        comment = version['data'].get("comment", None) or "No comment"
+        comment = version["data"].get("comment", None) or "No comment"
 
-        source = version['data'].get("source", None)
+        source = version["data"].get("source", None)
         source_label = source if source else "No source"
 
         # Store source and raw data
-        self.data['source'] = source
-        self.data['raw'] = version
+        self.data["source"] = source
+        self.data["raw"] = version
 
         data = {
-            "subset": subset['name'],
-            "version": version['name'],
+            "subset": subset["name"],
+            "version": version["name"],
             "comment": comment,
             "created": created,
             "source": source_label

avalon/tools/models.py

@@ -203,7 +203,7 @@ def __init__(self):
     def _get_task_icons(self):
         # Get the project configured icons from database
         project = io.find_one({"type": "project"})
-        tasks = project['config'].get('tasks', [])
+        tasks = project["config"].get("tasks", [])
         for task in tasks:
             icon_name = task.get("icon", None)
             if icon_name:
@@ -280,7 +280,7 @@ def data(self, index, role):
         # Add icon to the first column
         if role == QtCore.Qt.DecorationRole:
             if index.column() == 0:
-                return index.internalPointer()['icon']
+                return index.internalPointer()["icon"]
 
         return super(TasksModel, self).data(index, role)
 
@@ -326,30 +326,30 @@ def _add_hierarchy(self, parent=None):
         if parent is None:
             # if not a parent find all that are parented to the project
             # or do *not* have a visualParent field at all
-            find_data['$or'] = [
-                {'data.visualParent': {'$exists': False}},
-                {'data.visualParent': None}
+            find_data["$or"] = [
+                {"data.visualParent": {"$exists": False}},
+                {"data.visualParent": None}
             ]
         else:
-            find_data["data.visualParent"] = parent['_id']
+            find_data["data.visualParent"] = parent["_id"]
 
-        assets = io.find(find_data).sort('name', 1)
+        assets = io.find(find_data).sort("name", 1)
 
         for asset in assets:
 
             # get label from data, otherwise use name
             data = asset.get("data", {})
-            label = data.get("label", asset['name'])
+            label = data.get("label", asset["name"])
             tags = data.get("tags", [])
 
             # store for the asset for optimization
             deprecated = "deprecated" in tags
 
             item = Item({
-                "_id": asset['_id'],
+                "_id": asset["_id"],
                 "name": asset["name"],
                 "label": label,
-                "type": asset['type'],
+                "type": asset["type"],
                 "tags": ", ".join(tags),
                 "deprecated": deprecated,
                 "_document": asset

avalon/tools/projectmanager/app.py

@@ -134,7 +134,7 @@ def _on_asset_created(data):
             # This is to allow quick continuing of typing a new asset name
             # whenever the user created one; this way we can press the "ENTER"
             # key to add an asset and continue typing for the next.
-            dialog.data['label']['label'].setFocus()
+            dialog.data["label"]["label"].setFocus()
 
         def _on_current_asset_changed():
             """Callback on current asset changed in item widget.
@@ -184,7 +184,7 @@ def on_add_task(self):
                     asset_tasks.append(task)
 
             # Update the field
-            asset['data']['tasks'] = asset_tasks
+            asset["data"]["tasks"] = asset_tasks
 
             schema.validate(asset)
             io.replace_one(_filter, asset)
@@ -203,7 +203,7 @@ def on_asset_changed(self):
 
         model = self.data["model"]["assets"]
         selected = model.get_selected_assets()
-        self.data['model']['tasks'].set_assets(selected)
+        self.data["model"]["tasks"].set_assets(selected)
 
     def on_silo_changed(self, silo):
         """Callback on asset silo changed"""

avalon/tools/projectmanager/dialogs.py

@@ -122,7 +122,7 @@ def set_parent(self, parent_id):
         if parent_id:
             parent_asset = io.find_one({"_id": parent_id, "type": "asset"})
             assert parent_asset, "Parent asset does not exist."
-            parent_name = parent_asset['name']
+            parent_name = parent_asset["name"]
 
             self.parent_name = parent_name
             self.parent_id = parent_id
@@ -131,7 +131,7 @@ def set_parent(self, parent_id):
             self.parent_name = ""
             self.parent_id = None
 
-        self.data['label']['parent'].setText(parent_name)
+        self.data["label"]["parent"].setText(parent_name)
 
     def update_name(self):
         """Force an update on the long name.
@@ -141,20 +141,20 @@ def update_name(self):
 
         """
 
-        label = self.data['label']['label'].text()
+        label = self.data["label"]["label"].text()
         name = label
 
         # Prefix with parent name (if parent)
         if self.parent_name:
             name = self.parent_name + "_" + name
 
-        self.data['label']['name'].setText(name)
+        self.data["label"]["name"].setText(name)
 
     def on_add_asset(self):
 
         parent_id = self.parent_id
-        name = self.data['label']['name'].text()
-        label = self.data['label']['label'].text()
+        name = self.data["label"]["name"].text()
+        label = self.data["label"]["label"].text()
         silo = self.silo
 
         if not label:
@@ -184,8 +184,8 @@ def on_add_asset(self):
         if parent_id:
             parent = io.find_one({"_id": io.ObjectId(parent_id)})
             if parent:
-                group = parent['name']
-                data['group'] = group
+                group = parent["name"]
+                data["group"] = group
 
         try:
             lib.create_asset(data)