Better handling of missing transport plugins (#1488) (#1515)

mergify[bot] · ahcorde · web-flow · commit 9a259bb8f16d · 2025-07-01T16:37:49.000+02:00
(cherry picked from commit 3c19b34) Signed-off-by: Alejandro Hernandez Cordero <ahcorde@gmail.com> Co-authored-by: Alejandro Hernández Cordero <alejandro@openrobotics.org>
diff --git a/rviz_default_plugins/include/rviz_default_plugins/displays/image/image_transport_display.hpp b/rviz_default_plugins/include/rviz_default_plugins/displays/image/image_transport_display.hpp
@@ -126,6 +126,10 @@ class ImageTransportDisplay : public rviz_common::_RosTopicDisplay
       setStatus(
         rviz_common::properties::StatusProperty::Error, "Topic",
         QString("Error subscribing: ") + e.what());
+    } catch (image_transport::TransportLoadException & e) {
+      setStatus(
+        rviz_common::properties::StatusProperty::Error, "Topic",
+        QString("Error subscribing: ") + e.what());
     }
   }
 

Original file line number	Diff line number	Diff line change
`@@ -126,6 +126,10 @@ class ImageTransportDisplay : public rviz_common::_RosTopicDisplay`
`126`	`126`	`setStatus(`
`127`	`127`	`rviz_common::properties::StatusProperty::Error, "Topic",`
`128`	`128`	`QString("Error subscribing: ") + e.what());`
	`129`	`+ } catch (image_transport::TransportLoadException & e) {`
	`130`	`+ setStatus(`
	`131`	`+ rviz_common::properties::StatusProperty::Error, "Topic",`
	`132`	`+ QString("Error subscribing: ") + e.what());`
`129`	`133`	`}`
`130`	`134`	`}`
`131`	`135`