Use NP registry instead of ui/registry/field_formats

docs/developer/plugin/development-uiexports.asciidoc

@@ -8,7 +8,6 @@ An aggregate list of available UiExport types:
 | Type | Purpose
 | hacks | Any module that should be included in every application
 | visTypes | Modules that register providers with the `ui/registry/vis_types` registry.
-| fieldFormats | Modules that register providers with the `ui/registry/field_formats` registry.
 | inspectorViews | Modules that register custom inspector views via the `viewRegistry` in `ui/inspector`.
 | chromeNavControls | Modules that register providers with the `ui/registry/chrome_nav_controls` registry.
 | navbarExtensions | Modules that register providers with the `ui/registry/navbar_extensions` registry.

src/legacy/core_plugins/data/public/index_patterns/fields/field.ts

@@ -17,16 +17,14 @@
  * under the License.
  */
 
-// @ts-ignore
-import { fieldFormats } from 'ui/registry/field_formats';
 import { i18n } from '@kbn/i18n';
 // @ts-ignore
 import { ObjDefine } from './obj_define';
-import { FieldFormat } from '../../../../../../plugins/data/common/field_formats';
+import { FieldFormat } from '../../../../../../plugins/data/public';
 // @ts-ignore
 import { shortenDottedString } from '../../../../../core_plugins/kibana/common/utils/shorten_dotted_string';
 import { IndexPattern } from '../index_patterns';
-import { getNotifications } from '../services';
+import { getNotifications, getFieldFormats } from '../services';
 
 import { getKbnFieldType } from '../../../../../../plugins/data/public';
 
@@ -120,6 +118,8 @@ export class Field implements FieldType {
 
     let format = spec.format;
     if (!format || !(format instanceof FieldFormat)) {
+      const fieldFormats = getFieldFormats();
+
       format =
         indexPattern.fieldFormatMap[spec.name] ||
         fieldFormats.getDefaultInstance(spec.type, spec.esTypes);

src/legacy/core_plugins/data/public/index_patterns/index_patterns/index_pattern.test.ts

@@ -27,17 +27,14 @@ import mockLogStashFields from '../../../../../../fixtures/logstash_fields';
 
 import { stubbedSavedObjectIndexPattern } from '../../../../../../fixtures/stubbed_saved_object_index_pattern';
 import { Field } from '../index_patterns_service';
-import { setNotifications } from '../services';
+import { setNotifications, setFieldFormats } from '../services';
 
 // Temporary disable eslint, will be removed after moving to new platform folder
 // eslint-disable-next-line @kbn/eslint/no-restricted-paths
 import { notificationServiceMock } from '../../../../../../core/public/notifications/notifications_service.mock';
+import { FieldFormatRegisty } from '../../../../../../plugins/data/public';
 
-jest.mock('ui/registry/field_formats', () => ({
-  fieldFormats: {
-    getDefaultInstance: jest.fn(),
-  },
-}));
+jest.mock('ui/new_platform');
 
 jest.mock('../../../../../../plugins/kibana_utils/public', () => {
   const originalModule = jest.requireActual('../../../../../../plugins/kibana_utils/public');
@@ -142,6 +139,9 @@ describe('IndexPattern', () => {
   // create an indexPattern instance for each test
   beforeEach(() => {
     setNotifications(notifications);
+    setFieldFormats(({
+      getDefaultInstance: jest.fn(),
+    } as unknown) as FieldFormatRegisty);
 
     return create(indexPatternId).then((pattern: IndexPattern) => {
       indexPattern = pattern;

src/legacy/core_plugins/data/public/index_patterns/index_patterns/index_pattern.ts

@@ -19,8 +19,6 @@
 
 import _, { each, reject } from 'lodash';
 import { i18n } from '@kbn/i18n';
-// @ts-ignore
-import { fieldFormats } from 'ui/registry/field_formats';
 import { SavedObjectsClientContract } from 'src/core/public';
 import {
   DuplicateField,
@@ -37,8 +35,8 @@ import { createFieldsFetcher } from './_fields_fetcher';
 import { formatHitProvider } from './format_hit';
 import { flattenHitWrapper } from './flatten_hit';
 import { IIndexPatternsApiClient } from './index_patterns_api_client';
-import { ES_FIELD_TYPES } from '../../../../../../plugins/data/common';
-import { getNotifications } from '../services';
+import { ES_FIELD_TYPES, KBN_FIELD_TYPES } from '../../../../../../plugins/data/common';
+import { getNotifications, getFieldFormats } from '../services';
 
 const MAX_ATTEMPTS_TO_RESOLVE_CONFLICTS = 3;
 const type = 'index-pattern';
@@ -117,7 +115,10 @@ export class IndexPattern implements StaticIndexPattern {
     this.fields = new FieldList(this, [], this.shortDotsEnable);
     this.fieldsFetcher = createFieldsFetcher(this, apiClient, this.getConfig('metaFields'));
     this.flattenHit = flattenHitWrapper(this, this.getConfig('metaFields'));
-    this.formatHit = formatHitProvider(this, fieldFormats.getDefaultInstance('string'));
+    this.formatHit = formatHitProvider(
+      this,
+      getFieldFormats().getDefaultInstance(KBN_FIELD_TYPES.STRING)
+    );
     this.formatField = this.formatHit.formatField;
   }
 
@@ -128,12 +129,14 @@ export class IndexPattern implements StaticIndexPattern {
   }
 
   private deserializeFieldFormatMap(mapping: any) {
-    const FieldFormat = fieldFormats.getType(mapping.id);
+    const FieldFormat = getFieldFormats().getType(mapping.id);
+
     return FieldFormat && new FieldFormat(mapping.params, this.getConfig);
   }
 
   private initFields(input?: any) {
     const newValue = input || this.fields;
+
     this.fields = new FieldList(this, newValue, this.shortDotsEnable);
   }
 
@@ -454,6 +457,7 @@ export class IndexPattern implements StaticIndexPattern {
               const { toasts } = getNotifications();
 
               toasts.addDanger(message);
+
               throw err;
             }
 
@@ -495,6 +499,7 @@ export class IndexPattern implements StaticIndexPattern {
 
         if (err instanceof IndexPatternMissingIndices) {
           toasts.addDanger((err as any).message);
+
           return [];
         }
 

src/legacy/core_plugins/data/public/index_patterns/index_patterns/index_patterns.test.ts

@@ -29,12 +29,6 @@ jest.mock('../errors', () => ({
   IndexPatternMissingIndices: jest.fn(),
 }));
 
-jest.mock('ui/registry/field_formats', () => ({
-  fieldFormats: {
-    getDefaultInstance: jest.fn(),
-  },
-}));
-
 jest.mock('./index_pattern', () => {
   class IndexPattern {
     init = async () => {

src/legacy/core_plugins/data/public/index_patterns/index_patterns/index_patterns.ts

@@ -24,8 +24,6 @@ import {
   UiSettingsClientContract,
   HttpServiceBase,
 } from 'src/core/public';
-// @ts-ignore
-import { fieldFormats } from 'ui/registry/field_formats';
 
 import { createIndexPatternCache } from './_pattern_cache';
 import { IndexPattern } from './index_pattern';
@@ -34,8 +32,6 @@ import { IndexPatternsApiClient, GetFieldsOptions } from './index_patterns_api_c
 const indexPatternCache = createIndexPatternCache();
 
 export class IndexPatterns {
-  fieldFormats: fieldFormats;
-
   private config: UiSettingsClientContract;
   private savedObjectsClient: SavedObjectsClientContract;
   private savedObjectsCache?: Array<SimpleSavedObject<Record<string, any>>> | null;

src/legacy/core_plugins/data/public/index_patterns/index_patterns_service.ts

@@ -23,10 +23,11 @@ import {
   HttpServiceBase,
   NotificationsStart,
 } from 'src/core/public';
+import { FieldFormatRegisty } from '../../../../../plugins/data/public';
 import { Field, FieldList, FieldListInterface, FieldType } from './fields';
 import { createFlattenHitWrapper } from './index_patterns';
 import { createIndexPatternSelect } from './components';
-import { setNotifications } from './services';
+import { setNotifications, setFieldFormats } from './services';
 
 import {
   formatHitProvider,
@@ -40,6 +41,7 @@ export interface IndexPatternDependencies {
   savedObjectsClient: SavedObjectsClientContract;
   http: HttpServiceBase;
   notifications: NotificationsStart;
+  fieldFormats: FieldFormatRegisty;
 }
 
 /**
@@ -64,8 +66,15 @@ export class IndexPatternsService {
     return this.setupApi;
   }
 
-  public start({ uiSettings, savedObjectsClient, http, notifications }: IndexPatternDependencies) {
+  public start({
+    uiSettings,
+    savedObjectsClient,
+    http,
+    notifications,
+    fieldFormats,
+  }: IndexPatternDependencies) {
     setNotifications(notifications);
+    setFieldFormats(fieldFormats);
 
     return {
       ...this.setupApi,

src/legacy/core_plugins/data/public/index_patterns/services.ts

@@ -19,7 +19,12 @@
 
 import { NotificationsStart } from 'src/core/public';
 import { createGetterSetter } from '../../../../../plugins/kibana_utils/public';
+import { FieldFormatRegisty } from '../../../../../plugins/data/public';
 
 export const [getNotifications, setNotifications] = createGetterSetter<NotificationsStart>(
   'Notifications'
 );
+
+export const [getFieldFormats, setFieldFormats] = createGetterSetter<FieldFormatRegisty>(
+  'FieldFormats'
+);

src/legacy/core_plugins/data/public/plugin.ts

@@ -99,6 +99,7 @@ export class DataPlugin implements Plugin<DataSetup, DataStart, {}, DataPluginSt
       savedObjectsClient: savedObjects.client,
       http,
       notifications,
+      fieldFormats: data.fieldFormats,
     });
 
     initLegacyModule(indexPatternsService.indexPatterns);

src/legacy/core_plugins/kibana/index.js

@@ -64,7 +64,6 @@ export default function (kibana) {
       hacks: [
         'plugins/kibana/dev_tools/hacks/hide_empty_tools',
       ],
-      fieldFormats: ['plugins/kibana/field_formats/register'],
       savedObjectTypes: [
         'plugins/kibana/visualize/saved_visualizations/saved_visualization_register',
         'plugins/kibana/discover/saved_searches/saved_search_register',

src/legacy/core_plugins/kibana/public/dashboard/dashboard_state.test.ts

@@ -27,12 +27,6 @@ import { TimeRange, TimefilterContract } from 'src/plugins/data/public';
 import { ViewMode } from 'src/plugins/embeddable/public';
 import { InputTimeRange } from 'ui/timefilter';
 
-jest.mock('ui/registry/field_formats', () => ({
-  fieldFormats: {
-    getDefaultInstance: jest.fn(),
-  },
-}));
-
 describe('DashboardState', function() {
   let dashboardState: DashboardStateManager;
   const savedDashboard = getSavedDashboardMock();

src/legacy/core_plugins/kibana/public/kibana.js

@@ -31,7 +31,6 @@ import 'uiExports/visTypes';
 import 'uiExports/visEditorTypes';
 import 'uiExports/visualize';
 import 'uiExports/savedObjectTypes';
-import 'uiExports/fieldFormats';
 import 'uiExports/fieldFormatEditors';
 import 'uiExports/navbarExtensions';
 import 'uiExports/contextMenuActions';