Added global "On update failed"

Added global "On update failed" command
Added .editorconfig

See also. https://ketarin.userecho.com/communities/1/topics/40-onerror-global-events

.editorconfig

@@ -0,0 +1,101 @@
+# You can learn more about editorconfig here: https://docs.microsoft.com/en-us/visualstudio/ide/editorconfig-code-style-settings-reference
+[*.cs]
+
+#Core editorconfig formatting - indentation
+
+#use soft tabs (spaces) for indentation
+indent_style = space
+
+#Formatting - indentation options
+
+#indent switch case contents.
+csharp_indent_case_contents = true
+#indent switch labels
+csharp_indent_switch_labels = true
+
+#Formatting - new line options
+
+#place catch statements on a new line
+csharp_new_line_before_catch = true
+#place else statements on a new line
+csharp_new_line_before_else = true
+#require finally statements to be on a new line after the closing brace
+csharp_new_line_before_finally = true
+#require braces to be on a new line for methods, properties, accessors, anonymous_methods, object_collection_array_initializers, types, and control_blocks (also known as "Allman" style)
+csharp_new_line_before_open_brace = methods, properties, accessors, anonymous_methods, object_collection_array_initializers, types, control_blocks
+
+#Formatting - organize using options
+
+#sort System.* using directives alphabetically, and place them before other usings
+dotnet_sort_system_directives_first = true
+
+#Formatting - spacing options
+
+#require NO space between a cast and the value
+csharp_space_after_cast = false
+#require a space before the colon for bases or interfaces in a type declaration
+csharp_space_after_colon_in_inheritance_clause = true
+#require a space after a keyword in a control flow statement such as a for loop
+csharp_space_after_keywords_in_control_flow_statements = true
+#require a space before the colon for bases or interfaces in a type declaration
+csharp_space_before_colon_in_inheritance_clause = true
+#remove space within empty argument list parentheses
+csharp_space_between_method_call_empty_parameter_list_parentheses = false
+#remove space between method call name and opening parenthesis
+csharp_space_between_method_call_name_and_opening_parenthesis = false
+#do not place space characters after the opening parenthesis and before the closing parenthesis of a method call
+csharp_space_between_method_call_parameter_list_parentheses = false
+#remove space within empty parameter list parentheses for a method declaration
+csharp_space_between_method_declaration_empty_parameter_list_parentheses = false
+#place a space character after the opening parenthesis and before the closing parenthesis of a method declaration parameter list.
+csharp_space_between_method_declaration_parameter_list_parentheses = false
+
+#Formatting - wrapping options
+
+#leave code block on single line
+csharp_preserve_single_line_blocks = true
+#leave statements and member declarations on the same line
+csharp_preserve_single_line_statements = true
+
+#Style - expression bodied member options
+
+csharp_style_expression_bodied_accessors = true:suggestion
+#prefer block bodies for constructors
+csharp_style_expression_bodied_constructors = false:suggestion
+#prefer block bodies for methods
+csharp_style_expression_bodied_methods = false:suggestion
+#prefer block bodies for properties
+csharp_style_expression_bodied_properties = true:suggestion
+
+#Style - expression level options
+
+#prefer out variables to be declared before the method call
+csharp_style_inlined_variable_declaration = true:suggestion
+#prefer the language keyword for member access expressions, instead of the type name, for types that have a keyword to represent them
+dotnet_style_predefined_type_for_member_access = true:suggestion
+
+#Style - implicit and explicit types
+
+#prefer explicit type over var to declare variables with built-in system types such as int
+csharp_style_var_for_built_in_types = false:suggestion
+#prefer explicit type over var when the type is already mentioned on the right-hand side of a declaration
+csharp_style_var_when_type_is_apparent = false:suggestion
+
+#Style - language keyword and framework type options
+
+#prefer the language keyword for local variables, method parameters, and class members, instead of the type name, for types that have a keyword to represent them
+dotnet_style_predefined_type_for_locals_parameters_members = true:suggestion
+
+#Style - qualification options
+
+#prefer events not to be prefaced with this. or Me. in Visual Basic
+dotnet_style_qualification_for_event = false:suggestion
+#prefer fields not to be prefaced with this. or Me. in Visual Basic
+dotnet_style_qualification_for_field = false:suggestion
+#prefer methods not to be prefaced with this. or Me. in Visual Basic
+dotnet_style_qualification_for_method = true:suggestion
+#prefer properties not to be prefaced with this. or Me. in Visual Basic
+dotnet_style_qualification_for_property = true:suggestion
+
+# Modifiers should always be specified
+dotnet_style_require_accessibility_modifiers = for_non_interface_members:suggestion

.gitignore

@@ -2,4 +2,5 @@
 /bin
 /.vs
 /packages
-*.csproj.user
+*.csproj.user
+*.cache

ApplicationJob.cs

@@ -695,10 +695,11 @@ public bool TargetIsFolder
         }
 
         [XmlElement("TargetPath")]
-        public string TargetPath {
-	    get { return m_TargetPath; }
-	    set { m_TargetPath = PathEx.FixDirectorySeparator(value); }
-       	}
+        public string TargetPath
+        {
+            get { return m_TargetPath; }
+            set { m_TargetPath = PathEx.FixDirectorySeparator(value); }
+        }
 
         [XmlElement("FixedDownloadUrl")]
         public string FixedDownloadUrl { get; set; } = string.Empty;
@@ -707,7 +708,8 @@ public bool TargetIsFolder
         public string Name
         {
             get { return this.m_Name; }
-            set {
+            set
+            {
                 this.m_Name = value.Length > 255 ? value.Substring(0, 255) : value;
             }
         }

Command.cs

@@ -3,14 +3,15 @@
 using System.IO;
 using System.Text;
 using System.Windows.Forms;
+using CDBurnerXP;
 using Ketarin.Forms;
 
 namespace Ketarin
 {
     /// <summary>
     /// Represents a user command, which can be any kind of scripting language.
     /// </summary>
-    class Command
+    internal class Command
     {
         /// <summary>
         /// Gets or sets the scripting language.
@@ -34,19 +35,11 @@ public Command(string text, string type)
             this.Type = ConvertToScriptType(type);
         }
 
-        /// <summary>
-        /// Executes the command.
-        /// </summary>
-        public virtual int Execute(ApplicationJob application)
-        {
-            return Execute(application, null);
-        }
-
         /// <summary>
         /// Executes the command.
         /// </summary>
         /// <param name="targetFileName">Content for variable "{url:...}"</param>
-        public virtual int Execute(ApplicationJob application, string targetFileName)
+        public virtual int Execute(ApplicationJob application, string targetFileName = null, ApplicationJobError errorInfo = null)
         {
             switch (Type)
             {
@@ -57,8 +50,8 @@ public virtual int Execute(ApplicationJob application, string targetFileName)
 
                 case ScriptType.PowerShell:
                     PowerShellScript psScript = new PowerShellScript(this.Text);
-                    psScript.Execute(application);
-                    break;
+                    psScript.Execute(application, errorInfo);
+                    return Conversion.ToInt(psScript.LastOutput);
 
                 default:
                     return ExecuteBatchCommand(application, this.Text, targetFileName);

CommandErrorException.cs

@@ -7,7 +7,7 @@ namespace Ketarin
     /// command returns the code "1" to indicate an error
     /// and prevent downloading.
     /// </summary>
-    class CommandErrorException : ApplicationException
+    internal class CommandErrorException : ApplicationException
     {
         public CommandErrorException()
             : base("The custom command failed (exit code 1), download aborted")

DbManager.cs

@@ -15,7 +15,7 @@ namespace Ketarin
     /// This class contains a collection of functions
     /// for reading from the database.
     /// </summary>
-    class DbManager
+    internal class DbManager
     {
         #region SettingsProvider
 

Downloader/Protocols/HttpFtpProtocolParametersSettingsProxy.cs

@@ -7,7 +7,7 @@
 
 namespace MyDownloader.Extension.Protocols
 {
-    class HttpFtpProtocolParametersSettingsProxy : IHttpFtpProtocolParameters
+    internal class HttpFtpProtocolParametersSettingsProxy : IHttpFtpProtocolParameters
     {
         #region IHttpFtpProtocolParameters Members
 

ExternalServices.cs

@@ -10,7 +10,7 @@ namespace Ketarin
     /// <summary>
     /// Provides a couple of functions for external services like FileHippo.
     /// </summary>
-    static class ExternalServices
+    internal static class ExternalServices
     {
         /// <summary>
         /// If the given URL contains a FileHippo ID, it is returned.

Forms/HintTextBox.cs

@@ -9,7 +9,7 @@ namespace Ketarin
     /// <summary>
     /// A text box which displays a gray coloured hint.
     /// </summary>
-    class HintTextBox : PaintableTextBoxBase
+    internal class HintTextBox : PaintableTextBoxBase
     {
         private string hintText = string.Empty;
         private HorizontalAlign hintTextAlign = HorizontalAlign.Left;

Forms/PaintableTextBoxBase.cs

@@ -8,7 +8,7 @@ namespace Ketarin.Forms
     /// <summary>
     /// Allows custom drawing within a text box.
     /// </summary>
-    class PaintableTextBoxBase : TextBox
+    internal class PaintableTextBoxBase : TextBox
     {
         public new event PaintEventHandler Paint;
 

Forms/SettingsDialog.cs

@@ -312,6 +312,11 @@ private void SaveCurrentCommand()
                     Settings.SetValue("PostUpdateCommand", this.commandControl.Text);
                     Settings.SetValue("PostUpdateCommandType", this.commandControl.CommandType.ToString());
                     break;
+
+                case 3:
+                    Settings.SetValue("UpdateFailedCommand", this.commandControl.Text);
+                    Settings.SetValue("UpdateFailedCommandType", this.commandControl.CommandType.ToString());
+                    break;
             }            
         }
 
@@ -344,6 +349,11 @@ private void LoadCommand()
                     this.commandControl.Text = Settings.GetValue("PostUpdateCommand", "") as string;
                     this.commandControl.CommandType = Command.ConvertToScriptType(Settings.GetValue("PostUpdateCommandType", ScriptType.Batch.ToString()) as string);
                     break;
+
+                case 3:
+                    this.commandControl.Text = Settings.GetValue("UpdateFailedCommand", "") as string;
+                    this.commandControl.CommandType = Command.ConvertToScriptType(Settings.GetValue("UpdateFailedCommandType", ScriptType.Batch.ToString()) as string);
+                    break;
             }
         }
 

Forms/VariableTextBox.cs

@@ -8,7 +8,7 @@ namespace Ketarin.Forms
     /// <summary>
     /// A usual textbox plus variable selection tool.
     /// </summary>
-    class VariableTextBox : TextBox
+    internal class VariableTextBox : TextBox
     {
         private string[] m_VariableNames = new string[0];
         private ContextMenuCustomiser m_Customiser;

Ketarin.csproj

@@ -498,6 +498,7 @@
       <DependentUpon>Resources.resx</DependentUpon>
       <DesignTime>True</DesignTime>
     </Compile>
+    <None Include=".editorconfig" />
     <None Include="app.manifest" />
     <None Include="packages.config" />
     <None Include="Properties\Settings.settings">
@@ -669,4 +670,4 @@
   <Target Name="AfterBuild">
   </Target>
   -->
-</Project>
+</Project>

NonBinaryFileException.cs

@@ -7,7 +7,7 @@ namespace Ketarin
     /// This exception should be thrown, if the file to download is
     /// not a binary file type.
     /// </summary>
-    class NonBinaryFileException : Exception
+    internal class NonBinaryFileException : Exception
     {
         private static string m_Message = "The downloaded file is not a binary file type ({0}). Possibly there is an error page. Status code: {1} ({2})";