Added a way to disable destinations in the configuration, for now OneNote is disabled by default. (As I am not satisfied with the way it works)

git-svn-id: http://svn.code.sf.net/p/greenshot/code/trunk@1830 7dccd23d-a4a3-4e1f-8c07-b4c1b4018ab4

Greenshot-OCR-Plugin/OCRDestination.cs

@@ -50,12 +50,6 @@ namespace GreenshotOCR {
 			}
 		}
 		
-		public override bool isActive {
-			get {
-				return true;
-			}
-		}
-		
 		public override bool ExportCapture(bool manuallyInitiated, ISurface surface, ICaptureDetails captureDetails) {
 			OcrPlugin.DoOCR(surface);
 			return true;

Greenshot/Destinations/EmailDestination.cs

@@ -128,7 +128,7 @@ namespace Greenshot.Destinations {
 
 		public override bool isActive {
 			get {
-				return isActiveFlag;
+				return base.isActive && isActiveFlag;
 			}
 		}
 

Greenshot/Destinations/ExcelDestination.cs

@@ -90,7 +90,7 @@ namespace Greenshot.Destinations {
 
 		public override bool isActive {
 			get {
-				return exePath != null;
+				return base.isActive && exePath != null;
 			}
 		}
 

Greenshot/Destinations/OneNoteDestination.cs

@@ -35,6 +35,7 @@ using Greenshot.IniFile;
 namespace Greenshot.Destinations {
 	public class OneNoteDestination : AbstractDestination {
 		private static log4net.ILog LOG = log4net.LogManager.GetLogger(typeof(WordDestination));
+		public const string DESIGNATION = "OneNote";
 		private static CoreConfiguration conf = IniConfig.GetIniSection<CoreConfiguration>();
 		private static string exePath = null;
 		private static Image applicationIcon = null;
@@ -61,7 +62,7 @@ namespace Greenshot.Destinations {
 
 		public override string Designation {
 			get {
-				return "OneNote";
+				return DESIGNATION;
 			}
 		}
 
@@ -89,7 +90,7 @@ namespace Greenshot.Destinations {
 
 		public override bool isActive {
 			get {
-				return exePath != null;
+				return base.isActive && exePath != null;
 			}
 		}
 

Greenshot/Destinations/PickerDestination.cs

@@ -58,12 +58,6 @@ namespace Greenshot.Destinations {
 			}
 		}
 		
-		public override bool isActive {
-			get {
-				return true;
-			}
-		}
-
 		public static ContextMenuStrip CreatePickerMenu(bool addDynamics, ISurface surface, ICaptureDetails captureDetails, IEnumerable<IDestination> destinations) {
 			ContextMenuStrip menu = new ContextMenuStrip();
 			menu.Closing += delegate(object source, ToolStripDropDownClosingEventArgs eventArgs) {

Greenshot/Destinations/PowerpointDestination.cs

@@ -90,7 +90,7 @@ namespace Greenshot.Destinations {
 
 		public override bool isActive {
 			get {
-				return exePath != null;
+				return base.isActive && exePath != null;
 			}
 		}
 

Greenshot/Destinations/WordDestination.cs

@@ -92,7 +92,7 @@ namespace Greenshot.Destinations {
 
 		public override bool isActive {
 			get {
-				return exePath != null;
+				return base.isActive && exePath != null;
 			}
 		}
 

GreenshotConfluencePlugin/ConfluenceDestination.cs

@@ -81,7 +81,7 @@ namespace GreenshotConfluencePlugin {
 		
 		public override bool isActive {
 			get {
-				return !string.IsNullOrEmpty(config.Url);
+				return base.isActive && !string.IsNullOrEmpty(config.Url);
 			}
 		}
 

GreenshotExternalCommandPlugin/ExternalCommandDestination.cs

@@ -55,12 +55,6 @@ namespace ExternalCommand {
 				return presetCommand;
 			}
 		}
-		
-		public override bool isActive {
-			get {
-				return true;
-			}
-		}
 
 		public override IEnumerable<IDestination> DynamicDestinations() {
 			yield break;

GreenshotJiraPlugin/JiraDestination.cs

@@ -73,7 +73,7 @@ namespace GreenshotJiraPlugin {
 		
 		public override bool isActive {
 			get {
-				return !string.IsNullOrEmpty(config.Url);
+				return base.isActive && !string.IsNullOrEmpty(config.Url);
 			}
 		}
 

GreenshotPlugin/Core/AbstractDestination.cs

@@ -29,6 +29,7 @@ using System.Reflection;
 using Microsoft.Win32;
 
 using Greenshot.Plugin;
+using Greenshot.IniFile;
 
 namespace GreenshotPlugin.Core {
 	/// <summary>
@@ -37,6 +38,7 @@ namespace GreenshotPlugin.Core {
 	public abstract class AbstractDestination : IDestination	{
 		private const string PATH_KEY = @"SOFTWARE\Microsoft\Windows\CurrentVersion\App Paths\";
 		private static readonly log4net.ILog LOG = log4net.LogManager.GetLogger(typeof(AbstractDestination));
+		private static CoreConfiguration configuration = IniConfig.GetIniSection<CoreConfiguration>();
 
 		public static string GetExePath(string exeName) {
 			using (RegistryKey key = Registry.LocalMachine.OpenSubKey(PATH_KEY + exeName, false)) {
@@ -168,6 +170,9 @@ namespace GreenshotPlugin.Core {
 
 		public virtual bool isActive {
 			get {
+				if (configuration.ExcludeDestinations != null && configuration.ExcludeDestinations.Contains(Designation)) {
+					return false;
+				}
 				return true;
 			}
 		}

GreenshotPlugin/Core/CoreConfiguration.cs

@@ -148,6 +148,8 @@ namespace GreenshotPlugin.Core {
 		public List<string> IncludePlugins;
 		[IniProperty("ExcludePlugins", Description="Comma separated list of Plugins which are NOT allowed.")]
 		public List<string> ExcludePlugins;
+		[IniProperty("ExcludeDestinations", Description = "Comma separated list of destinations which should be disabled.", DefaultValue = "OneNote")]
+		public List<string> ExcludeDestinations;
 
 		[IniProperty("UpdateCheckInterval", Description="How many days between every update check? (0=no checks)", DefaultValue="1")]
 		public int UpdateCheckInterval;

PluginExample/SimpleOutputDestination.cs

@@ -55,12 +55,6 @@ namespace PluginExample {
 			}
 		}
 		
-		public override bool isActive {
-			get {
-				return true;
-			}
-		}
-		
 		public override bool ExportCapture(bool manuallyInitiated, ISurface surface, ICaptureDetails captureDetails) {
 			CoreConfiguration config = IniConfig.GetIniSection<CoreConfiguration>();
 			OutputSettings outputSettings = new OutputSettings();