From 9bc5cb8c08161ee5f5424bc689d9ac9bf5238f90 Mon Sep 17 00:00:00 2001
From: ed <eddavies95@gmail.com>
Date: Wed, 8 Jul 2020 15:03:11 +0100
Subject: [PATCH] Projucer:  Support opening multiple files from macOS and
 don't re-open all last open project windows if started from a file open

---
 .../Source/Application/jucer_Application.cpp  | 28 ++++++++++++-------
 .../Source/Application/jucer_Application.h    |  1 +
 2 files changed, 19 insertions(+), 10 deletions(-)

diff --git a/extras/Projucer/Source/Application/jucer_Application.cpp b/extras/Projucer/Source/Application/jucer_Application.cpp
index f6f89cab22..56d22e4baf 100644
--- a/extras/Projucer/Source/Application/jucer_Application.cpp
+++ b/extras/Projucer/Source/Application/jucer_Application.cpp
@@ -100,6 +100,8 @@ void ProjucerApplication::initialise (const String& commandLine)
             return;
         }
 
+        doBasicApplicationSetup();
+
         // do further initialisation in a moment when the message loop has started
         triggerAsyncUpdate();
     }
@@ -130,7 +132,7 @@ void ProjucerApplication::initialiseWindows (const String& commandLine)
 
     if (commandLineWithoutNSDebug.trim().isNotEmpty() && ! commandLineWithoutNSDebug.trim().startsWithChar ('-'))
         anotherInstanceStarted (commandLine);
-    else
+    else if (mainWindowList.windows.size() == 0)
         mainWindowList.reopenLastProjects();
 
     mainWindowList.createWindowIfNoneAreOpen();
@@ -138,17 +140,11 @@ void ProjucerApplication::initialiseWindows (const String& commandLine)
 
 void ProjucerApplication::handleAsyncUpdate()
 {
-    licenseController = std::make_unique<LicenseController>();
-
-    LookAndFeel::setDefaultLookAndFeel (&lookAndFeel);
-
     rescanJUCEPathModules();
     rescanUserPathModules();
 
     openDocumentManager.registerType (new ProjucerAppClasses::LiveBuildCodeEditorDocument::Type(), 2);
-    childProcessCache.reset (new ChildProcessCache());
 
-    initCommandManager();
     menuModel.reset (new MainMenuModel());
 
    #if JUCE_MAC
@@ -162,7 +158,6 @@ void ProjucerApplication::handleAsyncUpdate()
     updateEditorColourSchemeIfNeeded();
 
     ImageCache::setCacheTimeout (30 * 1000);
-    icons = std::make_unique<Icons>();
     tooltipWindow = std::make_unique<TooltipWindow> (nullptr, 1200);
 
     if (isAutomaticVersionCheckingEnabled())
@@ -171,6 +166,15 @@ void ProjucerApplication::handleAsyncUpdate()
     initialiseWindows (getCommandLineParameters());
 }
 
+void ProjucerApplication::doBasicApplicationSetup()
+{
+    licenseController = std::make_unique<LicenseController>();
+    LookAndFeel::setDefaultLookAndFeel (&lookAndFeel);
+    initCommandManager();
+    childProcessCache = std::make_unique<ChildProcessCache>();
+    icons = std::make_unique<Icons>();
+}
+
 static void deleteTemporaryFiles()
 {
     auto tempDirectory = File::getSpecialLocation (File::SpecialLocationType::tempDirectory).getChildFile ("PIPs");
@@ -272,7 +276,12 @@ String ProjucerApplication::getVersionDescription() const
 void ProjucerApplication::anotherInstanceStarted (const String& commandLine)
 {
     if (server == nullptr && ! commandLine.trim().startsWithChar ('-'))
-        openFile (File (commandLine.unquoted()));
+    {
+        ArgumentList list ({}, commandLine);
+
+        for (auto& arg : list.arguments)
+            openFile (arg.resolveAsFile());
+    }
 }
 
 ProjucerApplication& ProjucerApplication::getApp()
@@ -1195,7 +1204,6 @@ void ProjucerApplication::askUserToOpenFile()
 
 bool ProjucerApplication::openFile (const File& file)
 {
-    handleUpdateNowIfNeeded();
     return mainWindowList.openFile (file);
 }
 
diff --git a/extras/Projucer/Source/Application/jucer_Application.h b/extras/Projucer/Source/Application/jucer_Application.h
index a7f80d717d..038b4ac071 100644
--- a/extras/Projucer/Source/Application/jucer_Application.h
+++ b/extras/Projucer/Source/Application/jucer_Application.h
@@ -112,6 +112,7 @@ public:
 private:
     //==============================================================================
     void handleAsyncUpdate() override;
+    void doBasicApplicationSetup();
 
     void initCommandManager();
     bool initialiseLogger (const char* filePrefix);