WinGui:

- Removed the Question Alert box when adding an item to the queue where the destination folder does not exist. The folder is automatically created before encode anyway, so no point in asking the user pre-Encode.cs
- Better error handling in the encode service.

git-svn-id: svn://svn.handbrake.fr/HandBrake/trunk@3979 b64f7644-9d1e-0410-96f1-a4d463321fa5

1 files changed, 8 insertions, 4 deletions

diff --git a/win/CS/HandBrake.ApplicationServices/Services/QueueProcessor.cs b/win/CS/HandBrake.ApplicationServices/Services/QueueProcessor.cs
index f2ca04c6d..fe3eb8549 100644
--- a/win/CS/HandBrake.ApplicationServices/Services/QueueProcessor.cs
+++ b/win/CS/HandBrake.ApplicationServices/Services/QueueProcessor.cs
@@ -10,6 +10,7 @@ namespace HandBrake.ApplicationServices.Services
     using System.Windows.Forms;

 

     using HandBrake.ApplicationServices.EventArgs;

+    using HandBrake.ApplicationServices.Exceptions;

     using HandBrake.ApplicationServices.Functions;

     using HandBrake.ApplicationServices.Model;

     using HandBrake.ApplicationServices.Services.Interfaces;

@@ -200,15 +201,18 @@ namespace HandBrake.ApplicationServices.Services
             if (!e.Successful)

             {

                 this.Pause();

-                MessageBox.Show(e.Exception + e.ErrorInformation);

+                throw new GeneralApplicationException(e.ErrorInformation, e.Exception.Message, e.Exception);

             }

 

-            // Post-Processing

-            SendToApplication(this.QueueManager.LastProcessedJob.Destination);

-

             // Handling Log Data 

             this.EncodeService.ProcessLogs(this.QueueManager.LastProcessedJob.Destination);

 

+            // Post-Processing

+            if (e.Successful)

+            {

+                SendToApplication(this.QueueManager.LastProcessedJob.Destination);

+            }

+

             // Move onto the next job.

             this.ProcessNextJob();

         }