NewPublisher: Keep plugins with mismatch target in report

openpype/pipeline/create/context.py

@@ -709,6 +709,7 @@ def __init__(
  self.manual_creators = {}
 
  self.publish_discover_result = None
+ self.publish_plugins_mismatch_targets = []
  self.publish_plugins = []
  self.plugins_with_defs = []
  self._attr_plugins_by_family = {}
@@ -831,6 +832,7 @@ def reset_plugins(self, discover_publish_plugins=True):
  discover_result = DiscoverResult()
  plugins_with_defs = []
  plugins_by_targets = []
+ plugins_mismatch_targets = []
  if discover_publish_plugins:
  discover_result = publish_plugins_discover()
  publish_plugins = discover_result.plugins
@@ -840,11 +842,19 @@ def reset_plugins(self, discover_publish_plugins=True):
  plugins_by_targets = pyblish.logic.plugins_by_targets(
  publish_plugins, list(targets)
  )
+
  # Collect plugins that can have attribute definitions
  for plugin in publish_plugins:
  if OpenPypePyblishPluginMixin in inspect.getmro(plugin):
  plugins_with_defs.append(plugin)
 
+ plugins_mismatch_targets = [
+ plugin
+ for plugin in publish_plugins
+ if plugin not in plugins_by_targets
+ ]
+
+ self.publish_plugins_mismatch_targets = plugins_mismatch_targets
  self.publish_discover_result = discover_result
  self.publish_plugins = plugins_by_targets
  self.plugins_with_defs = plugins_with_defs

openpype/tools/publisher/control.py

@@ -154,15 +154,20 @@ def __init__(self, controller):
  self._all_instances_by_id = {}
  self._current_context = None
 
- def reset(self, context, publish_discover_result=None):
+ def reset(self, context, create_context):
  """Reset report and clear all data."""
- self._publish_discover_result = publish_discover_result
+
+ self._publish_discover_result = create_context.publish_discover_result
  self._plugin_data = []
  self._plugin_data_with_plugin = []
  self._current_plugin_data = {}
  self._all_instances_by_id = {}
  self._current_context = context
 
+ for plugin in create_context.publish_plugins_mismatch_targets:
+ plugin_data = self._add_plugin_data_item(plugin)
+ plugin_data["skipped"] = True
+
  def add_plugin_iter(self, plugin, context):
  """Add report about single iteration of plugin."""
  for instance in context:
@@ -205,6 +210,7 @@ def _create_plugin_data_item(self, plugin):
  "name": plugin.__name__,
  "label": label,
  "order": plugin.order,
+ "targets": list(plugin.targets),
  "instances_data": [],
  "actions_data": [],
  "skipped": False,
@@ -777,10 +783,7 @@ def _reset_publish(self):
  # - pop the key after first collector using it would be safest option?
  self._publish_context.data["create_context"] = self.create_context
 
- self._publish_report.reset(
- self._publish_context,
- self.create_context.publish_discover_result
- )
+ self._publish_report.reset(self._publish_context, self.create_context)
  self._publish_validation_errors = []
  self._publish_current_plugin_validation_errors = None
  self._publish_error = None