Fixed filter widgets not being destructed

* Changed PipelineItem::setFilterInputWidget to private Signed-off-by: Jared Duffey <jared.duffey@bluequartz.net>
BlueQuartzSoftware · Feb 18, 2021 · 1d8ba0b · 1d8ba0b
1 parent 40e9e53
commit 1d8ba0b
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 4 deletions.
diff --git a/Source/SVWidgetsLib/Widgets/PipelineItem.cpp b/Source/SVWidgetsLib/Widgets/PipelineItem.cpp
@@ -52,6 +52,7 @@ PipelineItem::PipelineItem(const QVector<QVariant>& data, PipelineItem* parent)
 PipelineItem::~PipelineItem()
 {
   qDeleteAll(m_ChildItems);
+  m_FilterInputWidget->deleteLater();
 }
 
 // -----------------------------------------------------------------------------

diff --git a/Source/SVWidgetsLib/Widgets/PipelineItem.h b/Source/SVWidgetsLib/Widgets/PipelineItem.h
@@ -49,10 +49,6 @@ class SVWidgetsLib_EXPORT PipelineItem
   PipelineItem(const QVector<QVariant>& data, PipelineItem* parent = nullptr);
   virtual ~PipelineItem();
 
-  /**
-   * @brief Setter property for FilterInputWidget
-   */
-  void setFilterInputWidget(FilterInputWidget* value);
   /**
    * @brief Getter property for FilterInputWidget
    * @return Value of FilterInputWidget
@@ -323,6 +319,11 @@ class SVWidgetsLib_EXPORT PipelineItem
 
   void setupFilterInputWidget();
 
+  /**
+   * @brief Setter property for FilterInputWidget
+   */
+  void setFilterInputWidget(FilterInputWidget* value);
+
 public:
   PipelineItem(const PipelineItem&) = delete;            // Copy Constructor Not Implemented
   PipelineItem(PipelineItem&&) = delete;                 // Move Constructor Not Implemented