From 121b4f52ea7a343c969a6e733cb2ac91fcbce88d Mon Sep 17 00:00:00 2001
From: alexmitchell <alcomposer@gmail.com>
Date: Sun, 10 Feb 2019 15:27:48 +1030
Subject: [PATCH 1/2] Add "yes to all" to proxydialog.cpp

change
---
 dialogs/proxydialog.cpp | 26 +++++++++++++++++---------
 1 file changed, 17 insertions(+), 9 deletions(-)

diff --git a/dialogs/proxydialog.cpp b/dialogs/proxydialog.cpp
index 5e646abd7..9f85b09d8 100644
--- a/dialogs/proxydialog.cpp
+++ b/dialogs/proxydialog.cpp
@@ -70,6 +70,7 @@ ProxyDialog::ProxyDialog(QWidget *parent, const QVector<Footage *> &footage) :
 
 void ProxyDialog::accept() {
 	QVector<ProxyInfo> info_list;
+	bool yesForAll = false;
 
 	for (int i=0;i<selected_footage.size();i++) {
 		// loop through selected footage and send info to the proxy queue
@@ -97,19 +98,26 @@ void ProxyDialog::accept() {
 			info.path = QDir(custom_location).filePath(base_footage_fn);
 		}
 
-		// if the proposed proxy file already exists
-		if (QFileInfo::exists(info.path) && QMessageBox::warning(this,
-																 tr("Proxy file exists"),
-																 tr("The file \"%1\" already exists. Do you wish to replace it?").arg(info.path),
-																 QMessageBox::Yes | QMessageBox::No) == QMessageBox::No) {
-			// return to dialog without closing or starting any proxy generation
-			return;
+		// if the proposed proxy file already exists & user didn't select YesToAll box
+		if (QFileInfo::exists(info.path) && !yesForAll){
+			int rtn = QMessageBox::warning(	this,
+								tr("Proxy file exists"),
+								tr("The file \"%1\" already exists. Do you wish to replace it?").arg(info.path),
+								QMessageBox::YesToAll | QMessageBox::Yes | QMessageBox::No);
+								// return to dialog without closing or starting any proxy generation
+			switch (rtn){
+				case QMessageBox::Yes:
+					break;
+				case QMessageBox::YesToAll:
+					yesForAll = true;
+					break;
+				case QMessageBox::No:
+					return;
+			}
 		}
-
 		// send to proxy generator thread
 		info_list.append(info);
 	}
-
 	// all proxy info checks out, queue it with the proxy generator
 	for (int i=0;i<info_list.size();i++) {
 		info_list.at(i).footage->proxy = true;

From 9fd320de45ca2207a2e9c6ef86942b6f8d55cf8c Mon Sep 17 00:00:00 2001
From: itsmattkc <itsmattkc@gmail.com>
Date: Tue, 12 Feb 2019 16:11:07 -0800
Subject: [PATCH 2/2] slight conformity changes

---
 dialogs/proxydialog.cpp | 27 +++++++++++++++++----------
 1 file changed, 17 insertions(+), 10 deletions(-)

diff --git a/dialogs/proxydialog.cpp b/dialogs/proxydialog.cpp
index 9f85b09d8..9aef33dcb 100644
--- a/dialogs/proxydialog.cpp
+++ b/dialogs/proxydialog.cpp
@@ -70,7 +70,9 @@ ProxyDialog::ProxyDialog(QWidget *parent, const QVector<Footage *> &footage) :
 
 void ProxyDialog::accept() {
 	QVector<ProxyInfo> info_list;
-	bool yesForAll = false;
+
+    // set to TRUE if any existing proxies exist and the user chooses to overwrite all of them
+    bool overwrite_all_existing = false;
 
 	for (int i=0;i<selected_footage.size();i++) {
 		// loop through selected footage and send info to the proxy queue
@@ -99,25 +101,30 @@ void ProxyDialog::accept() {
 		}
 
 		// if the proposed proxy file already exists & user didn't select YesToAll box
-		if (QFileInfo::exists(info.path) && !yesForAll){
+        if (QFileInfo::exists(info.path) && !overwrite_all_existing){
 			int rtn = QMessageBox::warning(	this,
 								tr("Proxy file exists"),
 								tr("The file \"%1\" already exists. Do you wish to replace it?").arg(info.path),
 								QMessageBox::YesToAll | QMessageBox::Yes | QMessageBox::No);
-								// return to dialog without closing or starting any proxy generation
+
 			switch (rtn){
-				case QMessageBox::Yes:
-					break;
-				case QMessageBox::YesToAll:
-					yesForAll = true;
-					break;
-				case QMessageBox::No:
-					return;
+            case QMessageBox::Yes:
+                // continue as normal, proxy generator will automatically overwrite this file
+                break;
+            case QMessageBox::YesToAll:
+                // continue as normal, also set variable so that above messagebox is not shown again
+                overwrite_all_existing = true;
+                break;
+            case QMessageBox::No:
+                // return to dialog without closing or starting any proxy generation
+                return;
 			}
 		}
+
 		// send to proxy generator thread
 		info_list.append(info);
 	}
+
 	// all proxy info checks out, queue it with the proxy generator
 	for (int i=0;i<info_list.size();i++) {
 		info_list.at(i).footage->proxy = true;