Code review fix - Using boolean flag to suppress ResourceCheck instead of override

Author: ramyapri

src/ServiceManagement/StorSimple/Commands.StorSimple/Cmdlets/GetAzureStorSimpleResource.cs

@@ -31,11 +31,8 @@ public class GetAzureStorSimpleResource : StorSimpleCmdletBase
         [ValidateNotNullOrEmpty]
         public string ResourceName { get; set; }
 
-        protected override void BeginProcessing()
-        {
-            //to prevent resource checking in StorSimpleCmdletbase.BeginProcessing()
-            return;
-        }
+        //suppress resource check for this commandlet
+        public GetAzureStorSimpleResource() : base(false) { }
 
         public override void ExecuteCmdlet()
         {

src/ServiceManagement/StorSimple/Commands.StorSimple/Cmdlets/GetAzureStorSimpleResourceContext.cs

@@ -25,12 +25,8 @@ namespace Microsoft.WindowsAzure.Commands.StorSimple.Cmdlets
     [Cmdlet(VerbsCommon.Get, "AzureStorSimpleResourceContext"),OutputType(typeof(StorSimpleResourceContext))]
     public class GetAzureStorSimpleResourceContext : StorSimpleCmdletBase
     {
-        protected override void BeginProcessing()
-        {
-            //we expliclity override BeginProcessing() so that it doesnt verify resource selection as part of StorSimpleCmdletBase
-            //class's BeginProcessing method
-            return;
-        }
+        //suppress resource check for this commandlet
+        public GetAzureStorSimpleResourceContext() : base(false) { }
 
         public override void ExecuteCmdlet()
         {

src/ServiceManagement/StorSimple/Commands.StorSimple/Cmdlets/SelectAzureStorSimpleResource.cs

@@ -35,11 +35,8 @@ public class SelectAzureStorSimpleResource : StorSimpleCmdletBase
         [ValidateNotNullOrEmpty]
         public string RegistrationKey { get; set; }
 
-        protected override void BeginProcessing()
-        {
-            //we dont have to verify that resource is selected
-            return;
-        }
+        //suppress resource check for this commandlet
+        public SelectAzureStorSimpleResource() : base(false) { }
 
         /// <summary>
         /// ProcessRecord of the command.

src/ServiceManagement/StorSimple/Commands.StorSimple/StorSimpleCmdletBase.cs

@@ -31,6 +31,23 @@ namespace Microsoft.WindowsAzure.Commands.StorSimple
 {
     public class StorSimpleCmdletBase : AzurePSCmdlet
     {
+        //this property will determine whether before running the actual commandlet logic, should resource selection be verified
+        protected bool verifyResourceBeforeCmdletExecute;
+
+        /// <summary>
+        /// default constructor for most commandlets. In this case, Resource check will be verified
+        /// </summary>
+        public StorSimpleCmdletBase() : this(true) { }
+
+        /// <summary>
+        /// constructor variant if you want to suppress the resource check for your commandlet
+        /// </summary>
+        /// <param name="performResourceCheck"></param>
+        public StorSimpleCmdletBase(bool performResourceCheck):base()
+        {
+            verifyResourceBeforeCmdletExecute = performResourceCheck;
+        }
+
         private StorSimpleClient storSimpleClient;
 
         internal StorSimpleClient StorSimpleClient
@@ -200,7 +217,8 @@ internal virtual void HandleException(Exception exception)
         protected override void BeginProcessing()
         {
             base.BeginProcessing();
-            VerifyResourceContext();
+            if(verifyResourceBeforeCmdletExecute)
+                VerifyResourceContext();
         }
         /// <summary>
         /// this method verifies that a resource has been selected before this commandlet is executed