fix(base): IsNamespaceStore and IsClusterStore logic simplified.

spbsoluble · spbsoluble · commit a310ea042bae · 2025-03-12T12:18:58.000-07:00
diff --git a/kubernetes-orchestrator-extension/Jobs/JobBase.cs b/kubernetes-orchestrator-extension/Jobs/JobBase.cs
@@ -443,13 +443,13 @@ protected K8SJobCertificate InitJobCertificate(dynamic config)
 
     private static bool IsNamespaceStore(string capability)
     {
-        return capability != null && string.IsNullOrEmpty(capability) &&
+        return !string.IsNullOrEmpty(capability) &&
                capability.Contains("K8SNS", StringComparison.OrdinalIgnoreCase);
     }
 
     private static bool IsClusterStore(string capability)
     {
-        return capability != null && string.IsNullOrEmpty(capability) &&
+        return !string.IsNullOrEmpty(capability) &&
                capability.Contains("K8SCLUSTER", StringComparison.OrdinalIgnoreCase);
     }
 

Original file line number	Diff line number	Diff line change
`@@ -443,13 +443,13 @@ protected K8SJobCertificate InitJobCertificate(dynamic config)`
`443`	`443`
`444`	`444`	`private static bool IsNamespaceStore(string capability)`
`445`	`445`	`{`
`446`		`- return capability != null && string.IsNullOrEmpty(capability) &&`
	`446`	`+ return !string.IsNullOrEmpty(capability) &&`
`447`	`447`	`capability.Contains("K8SNS", StringComparison.OrdinalIgnoreCase);`
`448`	`448`	`}`
`449`	`449`
`450`	`450`	`private static bool IsClusterStore(string capability)`
`451`	`451`	`{`
`452`		`- return capability != null && string.IsNullOrEmpty(capability) &&`
	`452`	`+ return !string.IsNullOrEmpty(capability) &&`
`453`	`453`	`capability.Contains("K8SCLUSTER", StringComparison.OrdinalIgnoreCase);`
`454`	`454`	`}`
`455`	`455`