finish config migration

docs/setup/settings.asciidoc

@@ -447,7 +447,7 @@ us improve your user experience. Your data is never shared with anyone. Set to
 `false` to disable telemetry capabilities entirely. You can alternatively opt
 out through the *Advanced Settings* in {kib}.
 
-`vega.enableExternalUrls:`:: *Default: false* Set this value to true to allow Vega to use any URL to access external data sources and images. If false, Vega can only get data from Elasticsearch.
+`vis_type_vega.enableExternalUrls:`:: *Default: false* Set this value to true to allow Vega to use any URL to access external data sources and images. If false, Vega can only get data from Elasticsearch.
 
 `xpack.license_management.enabled`:: *Default: true* Set this value to false to
 disable the License Management user interface.

src/legacy/core_plugins/vis_type_vega/index.ts

@@ -43,11 +43,6 @@ const vegaPluginInitializer: LegacyPluginInitializer = ({ Plugin }: LegacyPlugin
       },
     },
     init: (server: Legacy.Server) => ({}),
-    config(Joi: any) {
-      return Joi.object({
-        enabled: Joi.boolean().default(true),
-      }).default();
-    },
   } as Legacy.PluginSpecOptions);
 
 // eslint-disable-next-line import/no-default-export

src/legacy/core_plugins/vis_type_vega/public/legacy.ts

@@ -26,17 +26,16 @@ import { LegacyDependenciesPlugin } from './shim';
 import { plugin } from '.';
 
 const setupPlugins: Readonly<VegaPluginSetupDependencies> = {
-  expressions: npSetup.plugins.expressions,
+  ...npSetup.plugins,
   visualizations: visualizationsSetup,
-  data: npSetup.plugins.data,
 
   // Temporary solution
   // It will be removed when all dependent services are migrated to the new platform.
   __LEGACY: new LegacyDependenciesPlugin(),
 };
 
 const startPlugins: Readonly<VegaPluginStartDependencies> = {
-  data: npStart.plugins.data,
+  ...npStart.plugins,
 };
 
 const pluginInstance = plugin({} as PluginInitializerContext);

src/legacy/ui/public/new_platform/new_platform.ts

@@ -43,6 +43,7 @@ import {
   NavigationPublicPluginSetup,
   NavigationPublicPluginStart,
 } from '../../../../plugins/navigation/public';
+import { VisTypeVegaSetup } from '../../../../plugins/vis_type_vega/public';
 
 export interface PluginsSetup {
   bfetch: BfetchPublicSetup;
@@ -60,6 +61,7 @@ export interface PluginsSetup {
   usageCollection: UsageCollectionSetup;
   advancedSettings: AdvancedSettingsSetup;
   management: ManagementSetup;
+  visTypeVega: VisTypeVegaSetup;
 }
 
 export interface PluginsStart {

src/plugins/vis_type_vega/config.ts

@@ -20,6 +20,7 @@
 import { schema, TypeOf } from '@kbn/config-schema';
 
 export const configSchema = schema.object({
+  enabled: schema.boolean({ defaultValue: true }),
   enableExternalUrls: schema.boolean({ defaultValue: false }),
 });
 

src/plugins/vis_type_vega/server/index.ts

@@ -27,6 +27,12 @@ export const config: PluginConfigDescriptor<ConfigSchema> = {
   },
   schema: configSchema,
   deprecations: ({ renameFromRoot }) => [
-    renameFromRoot('vega.enableExternalUrls', 'vis_tye_vega.enableExternalUrls'),
+    renameFromRoot('vega.enableExternalUrls', 'vis_type_vega.enableExternalUrls'),
+    renameFromRoot('vega.enabled', 'vis_type_vega.enabled'),
   ],
 };
+
+export const plugin = () => ({
+  setup() {},
+  start() {},
+});

x-pack/legacy/plugins/graph/index.ts

@@ -44,14 +44,6 @@ export const graph: LegacyPluginInitializer = kibana => {
     },
 
     init(server) {
-      server.injectUiAppVars('graph', () => {
-        const config = server.config();
-        return {
-          graphSavePolicy: config.get('xpack.graph.savePolicy'),
-          canEditDrillDownUrls: config.get('xpack.graph.canEditDrillDownUrls'),
-        };
-      });
-
       server.plugins.xpack_main.registerFeature({
         id: 'graph',
         name: i18n.translate('xpack.graph.featureRegistry.graphFeatureName', {

x-pack/legacy/plugins/graph/public/index.ts

@@ -7,9 +7,11 @@
 import { npSetup, npStart } from 'ui/new_platform';
 import { LicensingPluginSetup } from '../../../../plugins/licensing/public';
 import { GraphPlugin } from './plugin';
+import { GraphSetup } from '../../../../plugins/graph/public';
 
 type XpackNpSetupDeps = typeof npSetup.plugins & {
   licensing: LicensingPluginSetup;
+  graph: GraphSetup;
 };
 
 (async () => {

x-pack/legacy/plugins/graph/public/plugin.ts

@@ -11,6 +11,7 @@ import { Storage } from '../../../../../src/plugins/kibana_utils/public';
 import { LicensingPluginSetup } from '../../../../plugins/licensing/public';
 import { NavigationPublicPluginStart as NavigationStart } from '../../../../../src/plugins/navigation/public';
 import { initAngularBootstrap } from '../../../../../src/plugins/kibana_legacy/public';
+import { GraphSetup } from '../../../../plugins/graph/public';
 
 export interface GraphPluginStartDependencies {
   npData: ReturnType<DataPlugin['start']>;
@@ -19,14 +20,15 @@ export interface GraphPluginStartDependencies {
 
 export interface GraphPluginSetupDependencies {
   licensing: LicensingPluginSetup;
+  graph: GraphSetup;
 }
 
 export class GraphPlugin implements Plugin {
   private navigationStart: NavigationStart | null = null;
   private npDataStart: ReturnType<DataPlugin['start']> | null = null;
   private savedObjectsClient: SavedObjectsClientContract | null = null;
 
-  setup(core: CoreSetup, { licensing }: GraphPluginSetupDependencies) {
+  setup(core: CoreSetup, { licensing, graph }: GraphPluginSetupDependencies) {
     initAngularBootstrap();
     core.application.register({
       id: 'graph',
@@ -41,10 +43,8 @@ export class GraphPlugin implements Plugin {
           savedObjectsClient: this.savedObjectsClient!,
           addBasePath: core.http.basePath.prepend,
           getBasePath: core.http.basePath.get,
-          canEditDrillDownUrls: core.injectedMetadata.getInjectedVar(
-            'canEditDrillDownUrls'
-          ) as boolean,
-          graphSavePolicy: core.injectedMetadata.getInjectedVar('graphSavePolicy') as string,
+          canEditDrillDownUrls: graph.config.canEditDrillDownUrls,
+          graphSavePolicy: graph.config.savePolicy,
           storage: new Storage(window.localStorage),
           capabilities: contextCore.application.capabilities.graph,
           coreStart: contextCore,

x-pack/plugins/graph/config.ts

@@ -0,0 +1,23 @@
+/*
+ * Copyright Elasticsearch B.V. and/or licensed to Elasticsearch B.V. under one
+ * or more contributor license agreements. Licensed under the Elastic License;
+ * you may not use this file except in compliance with the Elastic License.
+ */
+
+import { schema, TypeOf } from '@kbn/config-schema';
+
+export const configSchema = schema.object({
+  enabled: schema.boolean({ defaultValue: true }),
+  savePolicy: schema.oneOf(
+    [
+      schema.literal('none'),
+      schema.literal('config'),
+      schema.literal('configAndData'),
+      schema.literal('configAndDataWithConsent'),
+    ],
+    { defaultValue: 'configAndData' }
+  ),
+  canEditDrillDownUrls: schema.boolean({ defaultValue: true }),
+});
+
+export type ConfigSchema = TypeOf<typeof configSchema>;

x-pack/plugins/graph/public/index.ts

@@ -4,6 +4,16 @@
  * you may not use this file except in compliance with the Elastic License.
  */
 
+import { PluginConfigDescriptor } from 'kibana/server';
+import { PluginInitializerContext } from 'kibana/public';
 import { GraphPlugin } from './plugin';
+import { configSchema, ConfigSchema } from '../config';
 
-export const plugin = () => new GraphPlugin();
+export const plugin = (initializerContext: PluginInitializerContext) =>
+  new GraphPlugin(initializerContext);
+
+export { GraphSetup } from './plugin';
+
+export const config: PluginConfigDescriptor<ConfigSchema> = {
+  schema: configSchema,
+};

x-pack/plugins/graph/public/plugin.ts

@@ -7,22 +7,26 @@
 import { i18n } from '@kbn/i18n';
 import { CoreSetup, CoreStart } from 'kibana/public';
 import { Plugin } from 'src/core/public';
+import { PluginInitializerContext } from 'kibana/public';
 import { toggleNavLink } from './services/toggle_nav_link';
 import { LicensingPluginSetup } from '../../licensing/public';
 import { checkLicense } from '../common/check_license';
 import {
   FeatureCatalogueCategory,
   HomePublicPluginSetup,
 } from '../../../../src/plugins/home/public';
+import { ConfigSchema } from '../config';
 
 export interface GraphPluginSetupDependencies {
   licensing: LicensingPluginSetup;
   home?: HomePublicPluginSetup;
 }
 
-export class GraphPlugin implements Plugin {
+export class GraphPlugin implements Plugin<{ config: Readonly<ConfigSchema> }, void> {
   private licensing: LicensingPluginSetup | null = null;
 
+  constructor(private initializerContext: PluginInitializerContext<ConfigSchema>) {}
+
   setup(core: CoreSetup, { licensing, home }: GraphPluginSetupDependencies) {
     this.licensing = licensing;
 
@@ -39,6 +43,16 @@ export class GraphPlugin implements Plugin {
         category: FeatureCatalogueCategory.DATA,
       });
     }
+
+    return {
+      /**
+       * The configuration is temporarily exposed to allow the legacy graph plugin to consume
+       * the setting. Once the graph plugin is migrated completely, this will become an implementation
+       * detail.
+       * @deprecated
+       */
+      config: this.initializerContext.config.get(),
+    };
   }
 
   start(core: CoreStart) {
@@ -52,3 +66,5 @@ export class GraphPlugin implements Plugin {
 
   stop() {}
 }
+
+export type GraphSetup = ReturnType<GraphPlugin['setup']>;

x-pack/plugins/graph/server/index.ts

@@ -4,6 +4,17 @@
  * you may not use this file except in compliance with the Elastic License.
  */
 
+import { PluginConfigDescriptor } from 'kibana/server';
+
+import { configSchema, ConfigSchema } from '../config';
 import { GraphPlugin } from './plugin';
 
 export const plugin = () => new GraphPlugin();
+
+export const config: PluginConfigDescriptor<ConfigSchema> = {
+  exposeToBrowser: {
+    canEditDrillDownUrls: true,
+    savePolicy: true,
+  },
+  schema: configSchema,
+};