Add warning messages in FileReader

Author: anjaldoshi

Source/Processors/FileReader/FileReader.cpp

@@ -103,6 +103,7 @@ void FileReader::parameterValueChanged (Parameter* p)
 {
     if (p->getName() == "selected_file")
     {
+        LOGC ("FileReader::parameterValueChanged - selected_file changed to: ", p->getValue().toString());
         setFile (p->getValue(), false);
     }
     else if (p->getName() == "active_stream")
@@ -239,6 +240,8 @@ bool FileReader::setFile (String fullpath, bool shouldUpdateSignalChain)
         }
     }
 
+    LOGC ("[FileReader] set file: ", fullpath);
+
     //Open file
     File file (fullpath);
 
@@ -262,28 +265,31 @@ bool FileReader::setFile (String fullpath, bool shouldUpdateSignalChain)
         if (! input)
         {
             LOGE ("Error creating file source for extension ", ext);
+            showWarningAsync ("Failed to open file", "Error creating file source for extension " + ext);
             return false;
         }
     }
     else
     {
         input = nullptr;
+        showWarningAsync ("Failed to open file", "File type \"" + ext + "\" not supported");
         CoreServices::sendStatusMessage ("File type not supported");
         return false;
     }
 
     if (! input->openFile (file))
     {
         input = nullptr;
+        showWarningAsync ("Invalid file", "The selected file is invalid. Please make sure the file is not corrupted and has valid format.");
         CoreServices::sendStatusMessage ("Invalid file");
-
         return false;
     }
 
     const bool isEmptyFile = input->getNumRecords() <= 0;
     if (isEmptyFile)
     {
         input = nullptr;
+        showWarningAsync ("Failed to open file", "Continuous data file is missing or empty.");
         CoreServices::sendStatusMessage ("Empty file. Ignoring open operation");
 
         return false;
@@ -318,6 +324,17 @@ bool FileReader::setFile (String fullpath, bool shouldUpdateSignalChain)
     return true;
 }
 
+void FileReader::showWarningAsync (const String& title, const String& message) const
+{
+    if (headlessMode)
+        return;
+
+    MessageManager::callAsync ([title, message]() 
+                               { AlertWindow::showMessageBoxAsync (AlertWindow::WarningIcon,
+                                                                   title,
+                                                                   message); });
+}
+
 void FileReader::setActiveStream (int index, bool reset)
 {
     //Resets the stream to the beginning if reset flag is true

Source/Processors/FileReader/FileReader.h

@@ -242,6 +242,9 @@ class FileReader : public GenericProcessor,
     /** Returns a new FileSource object for a given file source */
     FileSource* createBuiltInFileSource (int index) const;
 
+    /** Shows a warning message asynchronously */
+    void showWarningAsync (const String& title, const String& message) const;
+
     /** Holds a path to the default file */
     File defaultFile;
 

Source/Processors/FileReader/FileReaderActions.cpp

@@ -47,10 +47,9 @@ bool SelectFile::perform()
     // Set the new path - this will trigger the linked parameter changes
     pathParam->setNextValue (newPath, false);
 
-    // Load the new file
-    bool success = processor->setFile (newPath, false);
-
-    if (!success)
+    // Check if the file was loaded successfully
+    if (processor->getFile().isEmpty()
+        || ! processor->getFile().equalsIgnoreCase (newPath.toString()))
     {
         // If loading the file failed, revert the path parameter to the original path
         pathParam->setNextValue (originalPath, false);