fire off send_fatal_notification() on unrecoverable errors for ScreenshotException.

Author: jw098

SerialPrograms/Source/CommonFramework/Exceptions/ScreenshotException.cpp

@@ -111,6 +111,41 @@ void ScreenshotException::send_notification(ProgramEnvironment& env, EventNotifi
     );
 }
 
+void ScreenshotException::send_fatal_notification(ProgramEnvironment& env) const{
+    std::vector<std::pair<std::string, std::string>> embeds;
+    if (!m_message.empty()){
+        embeds.emplace_back(std::pair<std::string, std::string>("Message:", m_message));
+    }
+
+    std::string title = "Fatal Error: ";
+    title.append(name());
+
+    if (m_send_error_report == ErrorReport::SEND_ERROR_REPORT){
+        report_error(
+            &env.logger(),
+            env.program_info(),
+            title,
+            embeds,
+            screenshot_view(),
+            m_stream ? &m_stream->history() : nullptr
+        );
+    }
+
+    EventNotificationOption fatal_notification = EventNotificationOption(
+        "Program Error (Fatal)",
+        true, true,
+        ImageAttachmentMode::JPG,
+        {"Notifs"}
+    );
+
+    send_program_notification(
+        env, fatal_notification,
+        color(),
+        name(),
+        std::move(embeds), "",
+        screenshot_view()
+    );
+}
 
 
 

SerialPrograms/Source/CommonFramework/Exceptions/ScreenshotException.h

@@ -72,6 +72,7 @@ class ScreenshotException : public Exception{
 
     virtual Color color() const{ return COLOR_RED; }
     virtual void send_notification(ProgramEnvironment& env, EventNotificationOption& notification) const;
+    void send_fatal_notification(ProgramEnvironment& env) const;
 
 public:
     ErrorReport m_send_error_report;

SerialPrograms/Source/NintendoSwitch/Framework/NintendoSwitch_MultiSwitchProgramSession.cpp

@@ -233,7 +233,7 @@ void MultiSwitchProgramSession::internal_run_program(){
             message = e.name();
         }
         report_error(message);
-        e.send_notification(env, m_option.instance().NOTIFICATION_ERROR_FATAL);
+        e.send_fatal_notification(env);
     }catch (Exception& e){
         logger().log("Program stopped with an exception!", COLOR_RED);
         env.add_overlay_log_to_all_consoles("- Program Error -", COLOR_RED);

SerialPrograms/Source/NintendoSwitch/Framework/NintendoSwitch_SingleSwitchProgramSession.cpp

@@ -194,7 +194,7 @@ void SingleSwitchProgramSession::internal_run_program(){
             message = e.name();
         }
         report_error(message);
-        e.send_notification(env, m_option.instance().NOTIFICATION_ERROR_FATAL);
+        e.send_fatal_notification(env);
     }catch (Exception& e){
         logger().log("Program stopped with an exception!", COLOR_RED);
         env.console.overlay().add_log("- Program Error -", COLOR_RED);

SerialPrograms/Source/PokemonSV/Programs/Eggs/PokemonSV_EggAutonomous.cpp

@@ -767,7 +767,6 @@ bool EggAutonomous::handle_recoverable_error(
     auto& stats = env.current_stats<EggAutonomous_Descriptor::Stats>();
     stats.m_errors++;
     env.update_stats();
-    e.send_notification(env, notification);
 
     if (SAVE_DEBUG_VIDEO){
         // Take a video to give more context for debugging
@@ -797,6 +796,7 @@ bool EggAutonomous::handle_recoverable_error(
             env.console
         );
     }
+    e.send_notification(env, notification);
 
     env.log("Reset game to handle recoverable error");
     reset_game(env.program_info(), env.console, context);