From 7a0179d41a3c3dc3acebe13455c0e65efc869656 Mon Sep 17 00:00:00 2001 From: iLLiCiTiT Date: Fri, 10 Sep 2021 10:37:52 +0200 Subject: [PATCH] active changed signal does not require value --- openpype/tools/new_publisher/widgets/card_view_widgets.py | 6 ++---- openpype/tools/new_publisher/widgets/list_view_widgets.py | 4 ++-- openpype/tools/new_publisher/widgets/widgets.py | 2 +- openpype/tools/new_publisher/window.py | 2 +- 4 files changed, 6 insertions(+), 8 deletions(-) diff --git a/openpype/tools/new_publisher/widgets/card_view_widgets.py b/openpype/tools/new_publisher/widgets/card_view_widgets.py index 36e8891fca..bec28aed4a 100644 --- a/openpype/tools/new_publisher/widgets/card_view_widgets.py +++ b/openpype/tools/new_publisher/widgets/card_view_widgets.py @@ -241,10 +241,8 @@ class InstanceCardView(AbstractInstanceView): for widget in self._widgets_by_id.values(): widget.update_instance_values() - def _on_active_changed(self, changed_instance_id, new_value): - changed_ids = set() - if changed_ids: - self.active_changed.emit(changed_ids) + def _on_active_changed(self): + self.active_changed.emit() def _on_widget_selection(self, widget_id): if widget_id == CONTEXT_ID: diff --git a/openpype/tools/new_publisher/widgets/list_view_widgets.py b/openpype/tools/new_publisher/widgets/list_view_widgets.py index 72b0d0fc28..aefcc97236 100644 --- a/openpype/tools/new_publisher/widgets/list_view_widgets.py +++ b/openpype/tools/new_publisher/widgets/list_view_widgets.py @@ -66,7 +66,7 @@ class InstanceListItemWidget(QtWidgets.QWidget): return self.instance.data["active"] = new_value - self.active_changed.emit(self.instance.data["uuid"], new_value) + self.active_changed.emit(self.instance.id, new_value) class ListContextWidget(QtWidgets.QFrame): @@ -346,7 +346,7 @@ class InstanceListView(AbstractInstanceView): widget.set_active(new_value) if changed_ids: - self.active_changed.emit(changed_ids) + self.active_changed.emit() def get_selected_items(self): instances = [] diff --git a/openpype/tools/new_publisher/widgets/widgets.py b/openpype/tools/new_publisher/widgets/widgets.py index d246d6d73a..158e69eefa 100644 --- a/openpype/tools/new_publisher/widgets/widgets.py +++ b/openpype/tools/new_publisher/widgets/widgets.py @@ -126,7 +126,7 @@ class PublishBtn(PublishIconBtn): class AbstractInstanceView(QtWidgets.QWidget): selection_changed = QtCore.Signal() - active_changed = QtCore.Signal(set) + active_changed = QtCore.Signal() refreshed = False def set_refreshed(self, refreshed): diff --git a/openpype/tools/new_publisher/window.py b/openpype/tools/new_publisher/window.py index 2361219932..faf913d704 100644 --- a/openpype/tools/new_publisher/window.py +++ b/openpype/tools/new_publisher/window.py @@ -376,7 +376,7 @@ class PublisherWindow(QtWidgets.QDialog): instances, context_selected ) - def _on_active_changed(self, _instance_ids): + def _on_active_changed(self): if self._refreshing_instances: return self._validate_create_instances()