[Index template] Fix editor should support mappings types

x-pack/legacy/plugins/index_management/public/app/components/mappings_editor/index.ts

@@ -11,3 +11,5 @@ export * from './mappings_editor';
 export * from './components/load_mappings';
 
 export { OnUpdateHandler, Types } from './mappings_state';
+
+export { doesMappingsHasType } from './lib';

x-pack/legacy/plugins/index_management/public/app/components/mappings_editor/lib/extract_mappings_definition.test.ts

@@ -73,7 +73,10 @@ describe('extractMappingsDefinition', () => {
       },
     };
 
-    expect(extractMappingsDefinition(mappings)).toBe(mappings.type2);
+    expect(extractMappingsDefinition(mappings)).toEqual({
+      type: 'type2',
+      mappings: mappings.type2,
+    });
   });
 
   test('should detect that the mappings has one type and return its mapping definition', () => {
@@ -97,7 +100,10 @@ describe('extractMappingsDefinition', () => {
       },
     };
 
-    expect(extractMappingsDefinition(mappings)).toBe(mappings.myType);
+    expect(extractMappingsDefinition(mappings)).toEqual({
+      type: 'myType',
+      mappings: mappings.myType,
+    });
   });
 
   test('should detect that the mappings has one type at root level', () => {
@@ -123,6 +129,6 @@ describe('extractMappingsDefinition', () => {
       },
     };
 
-    expect(extractMappingsDefinition(mappings)).toBe(mappings);
+    expect(extractMappingsDefinition(mappings)).toEqual({ mappings });
   });
 });

x-pack/legacy/plugins/index_management/public/app/components/mappings_editor/lib/extract_mappings_definition.ts

@@ -11,6 +11,11 @@ import {
   mappingsConfigurationSchemaKeys,
 } from './mappings_validator';
 
+interface MappingsWithType {
+  type?: string;
+  mappings: GenericObject;
+}
+
 const ALLOWED_PARAMETERS = [...mappingsConfigurationSchemaKeys, 'dynamic_templates', 'properties'];
 
 const isMappingDefinition = (obj: GenericObject): boolean => {
@@ -32,6 +37,29 @@ const isMappingDefinition = (obj: GenericObject): boolean => {
   return isConfigurationValid && isPropertiesValid && isDynamicTemplatesValid;
 };
 
+const getMappingsDefinitionWithType = (mappings: GenericObject): MappingsWithType[] => {
+  if (isMappingDefinition(mappings)) {
+    // No need to go any further
+    return [{ mappings }];
+  }
+
+  // At this point there must be one or more type mappings
+  const typedMappings = Object.entries(mappings).reduce(
+    (acc: Array<{ type: string; mappings: GenericObject }>, [type, value]) => {
+      if (isMappingDefinition(value)) {
+        acc.push({ type, mappings: value as GenericObject });
+      }
+      return acc;
+    },
+    []
+  );
+
+  return typedMappings;
+};
+
+export const doesMappingsHasType = (mappings: GenericObject = {}): boolean =>
+  getMappingsDefinitionWithType(mappings).filter(({ type }) => type !== undefined).length > 0;
+
 /**
  * 5.x index templates can be created with multiple types.
  * e.g.
@@ -72,19 +100,10 @@ const isMappingDefinition = (obj: GenericObject): boolean => {
  *
  * @param mappings The mappings object to validate
  */
-export const extractMappingsDefinition = (mappings: GenericObject = {}): GenericObject | null => {
-  if (isMappingDefinition(mappings)) {
-    // No need to go any further
-    return mappings;
-  }
-
-  // At this point there must be one or more type mappings
-  const typedMappings = Object.values(mappings).reduce((acc: GenericObject[], value) => {
-    if (isMappingDefinition(value)) {
-      acc.push(value as GenericObject);
-    }
-    return acc;
-  }, []);
+export const extractMappingsDefinition = (
+  mappings: GenericObject = {}
+): MappingsWithType | null => {
+  const typedMappings = getMappingsDefinitionWithType(mappings);
 
   // If there are no typed mappings found this means that one of the type must did not pass
   // the "isMappingDefinition()" validation.

x-pack/legacy/plugins/index_management/public/app/components/mappings_editor/mappings_editor.tsx

@@ -31,7 +31,7 @@ type TabName = 'fields' | 'advanced' | 'templates';
 export const MappingsEditor = React.memo(({ onUpdate, defaultValue, indexSettings }: Props) => {
   const [selectedTab, selectTab] = useState<TabName>('fields');
 
-  const { parsedDefaultValue, multipleMappingsDeclared } = useMemo(() => {
+  const { parsedDefaultValue, multipleMappingsDeclared, mappingsType } = useMemo(() => {
     const mappingsDefinition = extractMappingsDefinition(defaultValue);
 
     if (mappingsDefinition === null) {
@@ -48,7 +48,7 @@ export const MappingsEditor = React.memo(({ onUpdate, defaultValue, indexSetting
       dynamic_date_formats,
       properties = {},
       dynamic_templates,
-    } = mappingsDefinition;
+    } = mappingsDefinition.mappings;
 
     const parsed = {
       configuration: {
@@ -66,7 +66,11 @@ export const MappingsEditor = React.memo(({ onUpdate, defaultValue, indexSetting
       },
     };
 
-    return { parsedDefaultValue: parsed, multipleMappingsDeclared: false };
+    return {
+      parsedDefaultValue: parsed,
+      multipleMappingsDeclared: false,
+      mappingsType: mappingsDefinition.type,
+    };
   }, [defaultValue]);
 
   useEffect(() => {
@@ -108,7 +112,11 @@ export const MappingsEditor = React.memo(({ onUpdate, defaultValue, indexSetting
         <MultipleMappingsWarning />
       ) : (
         <IndexSettingsProvider indexSettings={indexSettings}>
-          <MappingsState onUpdate={onUpdate} defaultValue={parsedDefaultValue!}>
+          <MappingsState
+            onUpdate={onUpdate}
+            defaultValue={parsedDefaultValue!}
+            mappingsType={mappingsType}
+          >
             {({ state }) => {
               const tabToContentMap = {
                 fields: <DocumentFields />,