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

mergify[bot] · ahcorde · web-flow · commit 1a901728780d · 2025-07-01T15:55:47.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
@@ -128,6 +128,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
`@@ -128,6 +128,10 @@ class ImageTransportDisplay : public rviz_common::_RosTopicDisplay`
`128`	`128`	`setStatus(`
`129`	`129`	`rviz_common::properties::StatusProperty::Error, "Topic",`
`130`	`130`	`QString("Error subscribing: ") + e.what());`
	`131`	`+ } catch (image_transport::TransportLoadException & e) {`
	`132`	`+ setStatus(`
	`133`	`+ rviz_common::properties::StatusProperty::Error, "Topic",`
	`134`	`+ QString("Error subscribing: ") + e.what());`
`131`	`135`	`}`
`132`	`136`	`}`
`133`	`137`