Cleanup and add warning message.

app/widget/projectexplorer/projectexplorer.cpp

@@ -23,6 +23,7 @@
 #include <QDebug>
 #include <QDesktopServices>
 #include <QDir>
+#include <QMessageBox>
 #include <QProcess>
 #include <QUrl>
 #include <QVBoxLayout>
@@ -558,8 +559,7 @@ QList<Node*> ProjectExplorer::GetItemNodes(Item* item, Item::Type type)
           // Loop through nodes to find our Footage node
           foreach (Node* node, s->nodes()) {
             // Check if node is of the right type
-            if (node->IsMedia() && static_cast<MediaInput*>(node)->type() == stream.get()->type() ||
-                static_cast<MediaInput*>(node)->type() == Stream::kImage) {
+            if (node->IsMedia() && static_cast<MediaInput*>(node)->type() == stream.get()->type()) {
               // Check the streams are the same
               if (static_cast<MediaInput*>(node)->footage() == stream) {
                 nodes.append(node);
@@ -573,6 +573,28 @@ QList<Node*> ProjectExplorer::GetItemNodes(Item* item, Item::Type type)
   return nodes;
 }
 
+ProjectExplorer::FootageDeleteResponse ProjectExplorer::DeleteWarningMessage()
+{
+  QMessageBox msgBox;
+  msgBox.setText(tr("This footage is in use."));
+  msgBox.setInformativeText(tr("Do you want to offline the footage or entirely delete it from the timeline?"));
+  QPushButton* offline = msgBox.addButton(tr("Offline Footage"), QMessageBox::ApplyRole);
+  QPushButton* deleteClips = msgBox.addButton(tr("Delete Clips"), QMessageBox::ApplyRole);
+  msgBox.setStandardButtons(QMessageBox::Cancel);
+  msgBox.setIcon(QMessageBox::Warning);
+
+  msgBox.exec();
+
+  if (msgBox.clickedButton() == offline) {
+    return kOffline;
+  }
+  if (msgBox.clickedButton() == deleteClips) {
+    return kDelete;
+  }
+
+  return kCancel;
+}
+
 void ProjectExplorer::DeleteSelected()
 {
   QList<Item*> selected = SelectedItems();
@@ -601,38 +623,16 @@ void ProjectExplorer::DeleteSelected()
       // Check if nodes exists
       QList<Node*> nodes = GetItemNodes(item, Item::kFootage);
       if (!nodes.isEmpty()){
-        // Loop through Footage nodes
-        foreach (Node* node, nodes) {
-
-          // Loop through sequences
-          QList<ItemPtr> sequences = model_.project()->get_items_of_type(Item::kSequence);
-          foreach (ItemPtr seq, sequences) {
-            Sequence* s = static_cast<Sequence*>(seq.get());
-
-            // Check all nodes to see if they're linked
-            foreach (Node* check_node, s->nodes()) {
-              // Skip itself
-              if (nodes.contains(check_node)) {
-                continue;
-              }
-              if (check_node->GetImmediateDependencies().contains(node)) {
-                QList<NodeInput*> inputs = check_node->GetInputsIncludingArrays();
-
-                foreach (NodeInput* input, inputs) {
-                  foreach (NodeEdgePtr edge, input->edges()) {
-                    Node* connected = edge->output()->parentNode();
-
-                    if (connected == node) {
-                      input->DisconnectEdge(edge);
-                    }
-                  }
-                }
-              }
-            }
+        FootageDeleteResponse response = DeleteWarningMessage();
+        if (response == kOffline) {
+          // Loop through Footage nodes
+          foreach (Node* node, nodes) {
+            // Set footage to be null
+            static_cast<MediaInput*>(node)->SetFootage(nullptr);
           }
-
-          // Set footage to be null
-          static_cast<MediaInput*>(node)->SetFootage(nullptr);
+        }
+        if (response == kCancel) {
+          return;
         }
       }
     }

app/widget/projectexplorer/projectexplorer.h

@@ -85,12 +85,6 @@ public:
 
   void DeleteSelected();
 
-  /**
-   * @brief Check if an item is in use anywhere and return any relevant input nodes
-   * kFootage has two streams that need to be handled
-   */
-  QList<Node*> GetItemNodes(Item* item, Item::Type type);
-
 public slots:
   void set_view_type(ProjectToolbar::ViewType type);
 
@@ -107,6 +101,26 @@ signals:
   void DoubleClickedItem(Item* item);
 
 private:
+  enum FootageDeleteResponse {
+    kDelete,
+    kOffline,
+    kCancel
+  };
+
+
+  /**
+   * @brief Pop up a QMessageBox to warn the user if the deleted clips are in use
+   * 
+   * Returns a FootageDeleteResponse
+   */
+  FootageDeleteResponse DeleteWarningMessage();
+
+  /**
+   * @brief Check if an item is in use anywhere and return any relevant input nodes
+   * kFootage has two streams that need to be handled
+   */
+  QList<Node*> GetItemNodes(Item* item, Item::Type type);
+
   /**
    * @brief Simple convenience function for adding a view to this stacked widget
    *