[ML] Fixing optional plugin dependency types (elastic#64450)

Co-authored-by: Elastic Machine <[email protected]>
jgowdyelastic · May 15, 2020 · 44d4ddc · 44d4ddc
1 parent 7827fa5
commit 44d4ddc
Show file tree

Hide file tree

Showing 5 changed files with 14 additions and 9 deletions.
diff --git a/x-pack/plugins/ml/public/application/contexts/kibana/kibana_context.ts b/x-pack/plugins/ml/public/application/contexts/kibana/kibana_context.ts
@@ -15,7 +15,7 @@ import { LicenseManagementUIPluginSetup } from '../../../../../license_managemen
 
 interface StartPlugins {
   data: DataPublicPluginStart;
-  security: SecurityPluginSetup;
+  security?: SecurityPluginSetup;
   licenseManagement?: LicenseManagementUIPluginSetup;
 }
 export type StartServices = CoreStart & StartPlugins;

diff --git a/...on/datavisualizer/file_based/components/filebeat_config_flyout/filebeat_config_flyout.tsx b/...on/datavisualizer/file_based/components/filebeat_config_flyout/filebeat_config_flyout.tsx
@@ -55,9 +55,11 @@ export const FilebeatConfigFlyout: FC<Props> = ({
   } = useMlKibana();
 
   useEffect(() => {
-    security.authc.getCurrentUser().then(user => {
-      setUsername(user.username === undefined ? null : user.username);
-    });
+    if (security !== undefined) {
+      security.authc.getCurrentUser().then(user => {
+        setUsername(user.username === undefined ? null : user.username);
+      });
+    }
   }, []);
 
   useEffect(() => {

diff --git a/x-pack/plugins/ml/public/application/management/index.ts b/x-pack/plugins/ml/public/application/management/index.ts
@@ -25,8 +25,11 @@ export function initManagementSection(
 ) {
   const licensing = pluginsSetup.licensing.license$.pipe(take(1));
   licensing.subscribe(license => {
-    if (license.check(PLUGIN_ID, MINIMUM_FULL_LICENSE).state === 'valid') {
-      const management = pluginsSetup.management;
+    const management = pluginsSetup.management;
+    if (
+      management !== undefined &&
+      license.check(PLUGIN_ID, MINIMUM_FULL_LICENSE).state === 'valid'
+    ) {
       const mlSection = management.sections.register({
         id: PLUGIN_ID,
         title: i18n.translate('xpack.ml.management.mlTitle', {

diff --git a/x-pack/plugins/ml/public/application/util/dependency_cache.ts b/x-pack/plugins/ml/public/application/util/dependency_cache.ts
@@ -39,7 +39,7 @@ export interface DependencyCache {
   savedObjectsClient: SavedObjectsClientContract | null;
   application: ApplicationStart | null;
   http: HttpStart | null;
-  security: SecurityPluginSetup | null;
+  security: SecurityPluginSetup | undefined | null;
   i18n: I18nStart | null;
   urlGenerators: SharePluginStart['urlGenerators'] | null;
 }

diff --git a/x-pack/plugins/ml/public/plugin.ts b/x-pack/plugins/ml/public/plugin.ts
@@ -25,9 +25,9 @@ export interface MlStartDependencies {
   share: SharePluginStart;
 }
 export interface MlSetupDependencies {
-  security: SecurityPluginSetup;
+  security?: SecurityPluginSetup;
   licensing: LicensingPluginSetup;
-  management: ManagementSetup;
+  management?: ManagementSetup;
   usageCollection: UsageCollectionSetup;
   licenseManagement?: LicenseManagementUIPluginSetup;
   home: HomePublicPluginSetup;