From 5d97e84f19aff19dd9f002010d6d9bc2f7d6961d Mon Sep 17 00:00:00 2001 From: Veljko Simakovic Date: Tue, 22 Aug 2023 18:30:20 +0200 Subject: [PATCH] Bug/sc 10636/fix parentid issue when setting an isolated (#92) * Fix:parentId when creating an item * Fix:parentId when creating an item --- .changeset/curly-pots-rhyme.md | 5 + src/bootstrap-tenant/bootstrapper/items.ts | 2186 ----------------- .../bootstrapper/items/handleItem.ts | 2 +- 3 files changed, 6 insertions(+), 2187 deletions(-) create mode 100644 .changeset/curly-pots-rhyme.md delete mode 100644 src/bootstrap-tenant/bootstrapper/items.ts diff --git a/.changeset/curly-pots-rhyme.md b/.changeset/curly-pots-rhyme.md new file mode 100644 index 0000000..3f03a47 --- /dev/null +++ b/.changeset/curly-pots-rhyme.md @@ -0,0 +1,5 @@ +--- +'@crystallize/import-utilities': patch +--- + +Hotfix of parentId when creating an item diff --git a/src/bootstrap-tenant/bootstrapper/items.ts b/src/bootstrap-tenant/bootstrapper/items.ts deleted file mode 100644 index b952b03..0000000 --- a/src/bootstrap-tenant/bootstrapper/items.ts +++ /dev/null @@ -1,2186 +0,0 @@ -import { Shape, ShapeComponent } from '@crystallize/schema' -import gql from 'graphql-tag' -// @ts-expect-error no types for this -import fromHTML from '@crystallize/content-transformer/fromHTML' - -import { - BooleanComponentContentInput, - ComponentChoiceComponentContentInput, - ComponentContentInput, - ContentChunkComponentContentInput, - DateTimeComponentContentInput, - GridRelationsComponentContentInput, - ImageComponentContentInput, - ImagesComponentContentInput, - ItemRelationsComponentContentInput, - ItemType, - KeyValuePairInput, - LocationComponentContentInput, - NumericComponentContentInput, - ParagraphCollectionComponentContentInput, - PropertiesTableComponentContentInput, - RichTextContentInput, - SelectionComponentContentInput, - SingleLineComponentContentInput, - VideoContentInput, - VideosComponentContentInput, -} from '../../types' -import { - buildMoveItemMutation, - buildCreateItemQueryAndVariables, - buildUpdateItemQueryAndVariables, - buildUpdateItemComponentQueryAndVariables, -} from '../../graphql' - -import { - JSONItem, - JSONParagraphCollection, - JsonSpec, - JSONImages, - JSONFolder, - JSONProduct, - JSONPropertiesTable, - JSONNumeric, - JSONLocation, - JSONDateTime, - JSONComponentContent, - JSONComponentChoice, - JSONContentChunk, - JSONItemRelation, - JSONSelection, - JSONVideos, - JSONGrid, - JSONProductVariant, - JSONProductSubscriptionPlanPricing, - JSONProductVariantPriceVariants, - JSONProductVariantSubscriptionPlanMeteredVariable, - JSONProductVariantStockLocations, - JSONRichTextTranslated, - JSONFiles, - JSONItemReference, -} from '../json-spec' -import { - getTranslation, - AreaUpdate, - BootstrapperContext, - validShapeIdentifier, - ItemVersionDescription, - getItemVersionsForLanguages, - getItemId, - EVENT_NAMES, - getTenantRootItemId, - BootstrapperError, - ItemEventPayload, - ItemEventPayloadCreatedOrUpdated, -} from './utils' -import { getAllGrids } from './utils/get-all-grids' -import { ffmpegAvailable } from './utils/remote-file-upload' -import { - CreateProductVariantInput, - FileContentInput, - FileInput, - PriceVariantReferenceInput, - ProductPriceVariant, - ProductStockLocation, - ProductVariant, - ProductVariantAttributeInput, - StockLocationReferenceInput, - SubscriptionPlanMeteredVariable, - SubscriptionPlanMeteredVariableReferenceInput, - SubscriptionPlanPriceInput, -} from '../../generated/graphql' -import { getProduct } from './utils/get-product' -import { getTopicIds } from './utils/get-topic-id' -import { - ComponentChoiceComponentConfig, - ContentChunkComponentConfig, -} from '@crystallize/schema' -import { buildUpdateVariantComponentQueryAndVariables } from '../../graphql/build-update-variant-component-mutation' -import { hasItemRelationsComponent } from './utils/has-item-relations-component' - -export interface Props { - spec: JsonSpec | null - onUpdate(t: AreaUpdate): any - context: BootstrapperContext -} - -function publishItem( - language: string, - id: string, - context: BootstrapperContext -) { - if (!id) { - return Promise.resolve() - } - - return context.callPIM({ - query: gql` - mutation PUBLISH_ITEM($id: ID!, $language: String!) { - item { - publish(id: $id, language: $language) { - id - } - } - } - `, - variables: { - id, - language, - }, - }) -} - -function createRichTextInput( - content: JSONRichTextTranslated, - language: string -) { - function stringToJson(str: string) { - return [ - JSON.parse( - JSON.stringify({ - kind: 'block', - type: 'paragraph', - children: [ - { - kind: 'inline', - type: 'span', - textContent: str, - }, - ], - }) - ), - ] - } - - const inp: RichTextContentInput = {} - if (typeof content === 'string') { - inp.json = stringToJson(content) - } else if (typeof content === 'object') { - /** - * Determine if the rich text content is one of - * { - * json: ..., - * html: ..., - * plainText: ... - * } - * or - * { - * [lang]: { - * json: ... - * ... - * } - * } - * - **/ - const c = content as any - - const keys = Object.keys(content || {}) - - const isNotTranslated = ['json', 'html', 'plainText'].includes(keys[0]) - const translatedContent = isNotTranslated ? c : getTranslation(c, language) - - if (translatedContent?.html) { - inp.json = fromHTML(translatedContent?.html) - } else { - if (typeof translatedContent === 'string') { - inp.json = stringToJson(translatedContent) - } else { - if (translatedContent.json) { - inp.json = translatedContent.json - } else if (translatedContent.html) { - inp.json = fromHTML(translatedContent.html) - } else if (translatedContent.plainText) { - inp.json = stringToJson(translatedContent.plainText) - } - } - } - } - - return inp -} - -interface ICreateMediaInput { - language: string - context: BootstrapperContext - onUpdate(t: AreaUpdate): any -} - -interface ICreateImagesInput extends ICreateMediaInput { - images: JSONImages -} - -async function createImagesInput( - props: ICreateImagesInput -): Promise { - const { images, language, onUpdate, context } = props - const imgs: ImageComponentContentInput[] = [] - - for (let i = 0; i < images?.length; i++) { - const image = images[i] - let { key, mimeType } = image - - if (!key) { - try { - const uploadResult = await context.uploadFileFromUrl( - image.src, - image.fileName - ) - if (uploadResult) { - key = uploadResult.key - mimeType = uploadResult.mimeType - - // Store the values so that we don't re-upload again during import - image.key = uploadResult.key - image.mimeType = uploadResult.mimeType - } - } catch (e) { - onUpdate({ - error: { - code: 'UPLOAD_FAILED', - message: `Could not upload image "${JSON.stringify(image)}"`, - }, - }) - } - } - - if (key) { - imgs.push({ - key, - mimeType, - altText: getTranslation(image.altText, language), - ...(image.caption && { - caption: createRichTextInput(image.caption, language), - }), - }) - } - } - - return imgs -} - -interface ICreateVideosInput extends ICreateMediaInput { - videos: JSONVideos -} - -async function createVideosInput( - props: ICreateVideosInput -): Promise { - const { videos, language, context, onUpdate } = props - - const vids: VideoContentInput[] = [] - - for (let i = 0; i < videos?.length; i++) { - const video = videos[i] - let { key } = video - - if (!key) { - try { - const uploadResult = await context.uploadFileFromUrl(video.src) - if (uploadResult) { - key = uploadResult.key - - // Store the values so that we don't re-upload again during import - video.key = uploadResult.key - } - } catch (e) { - onUpdate({ - error: { - code: 'UPLOAD_FAILED', - message: `${e} - Could not upload video "${JSON.stringify(video)}"`, - }, - }) - } - } - - if (key) { - vids.push({ - key, - title: getTranslation(video.title, language), - ...(video.thumbnails && { - thumbnails: await createImagesInput({ - images: video.thumbnails, - language, - onUpdate, - context, - }), - }), - }) - } - } - - return vids -} - -interface ICreateFilesInput extends ICreateMediaInput { - files: JSONFiles -} -async function createFilesInput( - props: ICreateFilesInput -): Promise { - const { files, language, context, onUpdate } = props - - const fs: FileInput[] = [] - - for (let i = 0; i < files?.length; i++) { - const file = files[i] - let { key } = file - - if (!key) { - try { - const uploadResult = await context.uploadFileFromUrl(file.src) - if (uploadResult) { - key = uploadResult.key - - // Store the values so that we don't re-upload again during import - file.key = uploadResult.key - } - } catch (e) { - onUpdate({ - error: { - code: 'UPLOAD_FAILED', - message: `Could not upload file "${JSON.stringify(file)}"`, - }, - }) - } - } - - if (key) { - fs.push({ - key, - title: getTranslation(file.title, language), - }) - } - } - - return fs -} - -type ItemComponents = Record< - string, - JSONComponentContent | JSONComponentChoice -> | null - -interface ICreateComponentsInput { - components?: ItemComponents - componentDefinitions?: ShapeComponent[] - language: string - grids: JSONGrid[] - context: BootstrapperContext - onUpdate(t: AreaUpdate): any -} - -async function createComponentsInput( - props: ICreateComponentsInput -): Promise | null | undefined> { - const { - components, - componentDefinitions, - language, - grids, - context, - onUpdate, - } = props - - // Explicit `null` clears all components data - if (components === null) { - return null - } - - // Undefined implies no action to be taken on components - if (!components) { - return undefined - } - - const input: Record = {} - - async function createComponentInput( - componentDefinition: ShapeComponent, - component: JSONComponentContent, - context: BootstrapperContext - ) { - switch (componentDefinition?.type) { - case 'boolean': { - const b = component as boolean - - if (!b) { - return { - boolean: null, - } - } - - const inp: BooleanComponentContentInput = { - boolean: { - value: b, - }, - } - return inp - } - case 'singleLine': { - const inp: SingleLineComponentContentInput = { - singleLine: { - text: getTranslation(component, language), - }, - } - return inp - } - case 'numeric': { - const n = component as JSONNumeric - - if (!n) { - return { - numeric: null, - } - } - - const inp: NumericComponentContentInput = { - numeric: { - number: n.number, - unit: n.unit ?? '', - }, - } - return inp - } - case 'location': { - const l = component as JSONLocation - - if (!l) { - return { - location: null, - } - } - - const inp: LocationComponentContentInput = { - location: { - lat: l.lat, - long: l.long, - }, - } - return inp - } - case 'datetime': { - const d = component as JSONDateTime - - if (!d) { - return { - datetime: null, - } - } - - const parsedDate = new Date(d) - const inp: DateTimeComponentContentInput = { - datetime: { - datetime: parsedDate.toISOString(), - }, - } - - return inp - } - case 'images': { - const images = component as JSONImages - - const inp: ImagesComponentContentInput = { - images: await createImagesInput({ - images, - language, - onUpdate, - context, - }), - } - - return inp - } - case 'videos': { - const videos = component as JSONVideos - - const inp: VideosComponentContentInput = { - videos: await createVideosInput({ - videos, - language, - onUpdate, - context, - }), - } - return inp - } - case 'files': { - const files = component as JSONFiles - - const inp: FileContentInput = { - files: await createFilesInput({ - files, - language, - onUpdate, - context, - }), - } - return inp - } - case 'richText': { - const inp: RichTextContentInput = createRichTextInput( - component as JSONRichTextTranslated, - language - ) - - if (Object.keys(inp).length > 0) { - return { - richText: inp, - } - } - break - } - case 'selection': { - const inp: SelectionComponentContentInput = { - selection: { - keys: [], // Todo - }, - } - const s = component as JSONSelection - if (typeof s === 'string') { - inp.selection.keys.push(s as string) - } else if (Array.isArray(s)) { - s.forEach((key: string) => inp.selection.keys.push(key)) - } - return inp - } - case 'paragraphCollection': { - const inp: ParagraphCollectionComponentContentInput = { - paragraphCollection: { - paragraphs: [], - }, - } - - const paragraphs = component as JSONParagraphCollection[] - for (let i = 0; i < paragraphs?.length; i++) { - const { title, body, images, videos } = paragraphs[i] - - inp.paragraphCollection.paragraphs.push({ - title: { - text: getTranslation(title, language), - }, - ...(body && { body: createRichTextInput(body, language) }), - ...(images && { - images: await createImagesInput({ - images, - language, - context, - onUpdate, - }), - }), - ...(videos && { - videos: await createVideosInput({ - videos, - language, - context, - onUpdate, - }), - }), - }) - } - - if (Object.keys(inp).length > 0) { - return inp - } - - break - } - case 'propertiesTable': { - const inp: PropertiesTableComponentContentInput = { - propertiesTable: { - sections: [], - }, - } - - const sections = (component as JSONPropertiesTable) || [] - sections.forEach((section) => { - const properties: KeyValuePairInput[] = [] - if (section.properties) { - Object.keys(section.properties).forEach((key) => { - properties.push({ - key, - value: section.properties?.[key], - } as KeyValuePairInput) - }) - } - - inp.propertiesTable.sections?.push({ - title: section.title, - properties, - }) - }) - - return inp - } - case 'componentChoice': { - const choice = component as JSONComponentChoice - - const [selectedComponentId] = Object.keys(choice) - - if (!selectedComponentId) { - return { - componentChoice: null, - } - } - - const componentConfig = componentDefinition.config as any - const selectedComponentDefinition = componentConfig.choices.find( - (c: any) => c.id === selectedComponentId - ) - - const content = await createComponentInput( - selectedComponentDefinition, - choice[selectedComponentId], - context - ) - - const inp: ComponentChoiceComponentContentInput = { - componentChoice: { - componentId: selectedComponentId, - ...content, - }, - } - return inp - } - case 'itemRelations': { - const inp: ItemRelationsComponentContentInput = { - itemRelations: { - itemIds: [], - skus: [], // Will be populated later - }, - } - return inp - - return undefined - } - case 'gridRelations': { - const gridsComponent = component as JSONGrid[] - const gridIds: string[] = [] - - gridsComponent.forEach((g) => { - const found = grids.find( - (a) => - getTranslation(a.name, language) === - getTranslation(g.name, language) - ) - if (found?.id) { - gridIds.push(found.id) - } else { - const err: BootstrapperError = { - error: `Could not find grid with name "${g.name}". Skipping the grid relation.`, - willRetry: false, - type: 'error', - } - context.emit(EVENT_NAMES.ERROR, err) - } - }) - - const inp: GridRelationsComponentContentInput = { - gridRelations: { - gridIds, - }, - } - return inp - } - case 'contentChunk': { - const chunks = (component as JSONContentChunk) || [] - const inp: ContentChunkComponentContentInput = { - contentChunk: { - chunks: [], - }, - } - - for (let i = 0; i < chunks?.length; i++) { - const newChunk: ComponentContentInput[] = [] - const chunk = chunks[i] - const chunkKeys = Object.keys(chunk) - for (let x = 0; x < chunkKeys.length; x++) { - const componentId = chunkKeys[x] - const selectedComponentDefinition = ( - componentDefinition.config as ContentChunkComponentConfig - )?.components?.find((c: any) => c.id === componentId) - - if (selectedComponentDefinition) { - const content: any = await createComponentInput( - selectedComponentDefinition, - chunk[componentId], - context - ) - if (content) { - newChunk.push({ - componentId, - ...content, - }) - } - } - } - if (newChunk.length > 0) { - inp.contentChunk.chunks.push(newChunk) - } - } - - return inp - } - } - } - - const componentIds = Object.keys(components) - - /** - * If you don't supply any components, let's not continue. - * The intention of the user is probably to only update - * any provided components - */ - if (componentIds.length === 0) { - return undefined - } - - for (let i = 0; i < componentIds.length; i++) { - const componentId = componentIds[i] - const componentDefinition = componentDefinitions?.find( - (c) => c.id === componentId - ) - if (componentDefinition && components) { - if (componentId in components) { - const component = components?.[componentId] - /** - * Make sure we don't just do a truthy check here, - * because true & "" & 0 are all valid content - */ - if (typeof component !== 'undefined') { - const content = await createComponentInput( - componentDefinition, - component, - context - ) - - if (typeof content !== 'undefined') { - input[componentId] = { - componentId, - ...content, - } - } - } - } - } - } - - return input -} - -async function getExistingTopicIdsForItem( - itemId: string, - language: string, - context: BootstrapperContext -): Promise { - const result = await context.callPIM({ - query: gql` - query GET_ITEM_TOPICS($itemId: ID!, $language: String!) { - item { - get(id: $itemId, language: $language) { - topics { - id - } - } - } - } - `, - variables: { - itemId, - language, - }, - }) - - return result.data?.item?.get?.topics?.map((t: any) => t.id) || [] -} - -function getSubscriptionPlanMeteredVariables({ - planIdentifier, - context, -}: { - planIdentifier: string - context: BootstrapperContext -}): SubscriptionPlanMeteredVariable[] { - const plan = context.subscriptionPlans?.find( - (p) => p.identifier === planIdentifier - ) - return plan?.meteredVariables || [] -} - -function getSubscriptionPlanPeriodId({ - planIdentifier, - periodName, - context, -}: { - planIdentifier: string - periodName: string - context: BootstrapperContext -}): null | string { - const plan = context.subscriptionPlans?.find( - (p) => p.identifier === planIdentifier - ) - if (plan) { - return plan.periods?.find((p) => p.name === periodName)?.id || null - } - - return null -} - -function subscriptionPlanPrincingJsonToInput( - pricing: JSONProductSubscriptionPlanPricing, - meteredVaribles: SubscriptionPlanMeteredVariable[] -): SubscriptionPlanPriceInput { - function handleMeteredVariable( - mv: JSONProductVariantSubscriptionPlanMeteredVariable - ): SubscriptionPlanMeteredVariableReferenceInput { - const id = meteredVaribles.find((m) => m.identifier === mv.identifier)?.id - if (!id) { - throw new Error('Cannot find id for metered variable ' + mv.identifier) - } - - return { - id, - tierType: mv.tierType, - tiers: mv.tiers?.map((t) => ({ - threshold: t.threshold, - priceVariants: handleJsonPriceToPriceInput({ - jsonPrice: t.price, - }), - })), - } - } - - return { - priceVariants: handleJsonPriceToPriceInput({ - jsonPrice: pricing.price, - }), - meteredVariables: - pricing?.meteredVariables?.length > 0 - ? pricing.meteredVariables.map(handleMeteredVariable) - : undefined, - } -} - -function handleJsonPriceToPriceInput({ - jsonPrice, - existingProductVariantPriceVariants, -}: { - jsonPrice?: number | JSONProductVariantPriceVariants - existingProductVariantPriceVariants?: ProductPriceVariant[] | null -}): PriceVariantReferenceInput[] { - if ( - typeof jsonPrice === 'undefined' && - !existingProductVariantPriceVariants - ) { - return [ - { - identifier: 'default', - price: 0, - }, - ] - } - - const priceVariants: PriceVariantReferenceInput[] = - existingProductVariantPriceVariants || [] - - if (jsonPrice && typeof jsonPrice === 'object') { - const p = jsonPrice as Record - Object.keys(jsonPrice).forEach((identifier) => { - const existingEntry = priceVariants.find( - (i) => i.identifier === identifier - ) - if (existingEntry) { - existingEntry.price = p[identifier] - } else { - priceVariants.push({ - identifier, - price: p[identifier], - }) - } - }) - } else if (typeof jsonPrice !== 'undefined') { - const defaultStock = priceVariants.find((i) => i.identifier === 'default') - if (defaultStock) { - defaultStock.price = jsonPrice - } else { - priceVariants.push({ - identifier: 'default', - price: jsonPrice, - }) - } - } - - return priceVariants.map(({ identifier, price }) => ({ - identifier, - price, - })) -} - -function handleJsonStockToStockInput({ - jsonStock, - existingProductVariantStockLocations, -}: { - jsonStock?: number | JSONProductVariantStockLocations - existingProductVariantStockLocations?: ProductStockLocation[] | null -}): undefined | StockLocationReferenceInput[] { - if (typeof jsonStock === undefined && !existingProductVariantStockLocations) { - return undefined - } - - const stockVariants: StockLocationReferenceInput[] = - existingProductVariantStockLocations || [] - - if (jsonStock && typeof jsonStock === 'object') { - const p = jsonStock as Record - Object.keys(jsonStock).forEach((identifier) => { - const existingEntry = stockVariants.find( - (i) => i.identifier === identifier - ) - if (existingEntry) { - existingEntry.stock = p[identifier] - } else { - stockVariants.push({ - identifier, - stock: p[identifier], - }) - } - }) - } else if (typeof jsonStock !== 'undefined') { - const defaultStock = stockVariants.find((i) => i.identifier === 'default') - if (defaultStock) { - defaultStock.stock = jsonStock - } else { - stockVariants.push({ - identifier: 'default', - stock: jsonStock, - }) - } - } - - return stockVariants.map(({ identifier, stock, meta }) => ({ - identifier, - stock: stock ?? 0, - meta: meta || [], - })) -} - -export async function setItems({ - spec, - onUpdate, - context, -}: Props): Promise { - if (!spec?.items) { - return - } - - const ffmpeg = await ffmpegAvailable - if (!ffmpeg) { - onUpdate({ - warning: { - code: 'FFMPEG_UNAVAILABLE', - message: - 'ffmpeg is not available. Videos will not be included. Installment instructions for ffmpeg: https://ffmpeg.org/download.html', - }, - }) - } - - const rootItemId = await getTenantRootItemId(context) - const allGrids = await getAllGrids( - context.targetLanguage || context.defaultLanguage, - context - ) - - // Get a total item count - let totalItems = 0 - function getCount(item: JSONFolder) { - totalItems++ - if (item && 'children' in item) { - item.children?.forEach(getCount) - } - } - spec.items.forEach(getCount) - - // Double the item count since we're doing add/update _and_ item relations later - totalItems *= 2 - - let finishedItems = 0 - - async function createOrUpdateItem( - item: JSONItem, - parentId: string, - treePosition: number - ): Promise { - // Create the object to store the component data in - item._componentsData = {} - - if (!item.shape) { - onUpdate({ - error: { - code: 'SHAPE_ID_MISSING', - message: `Missing shape identifier for item "${getTranslation( - item.name, - context.targetLanguage || context.defaultLanguage - )}". Got "${item.shape}"`, - }, - }) - return null - } - - // Ensure shape identifier is not too long (max 64 characters) - item.shape = validShapeIdentifier(item.shape, onUpdate) - - // Get the shape type - const shape = context.shapes?.find((s) => s.identifier === item.shape) - if (!shape) { - onUpdate({ - error: { - code: 'CANNOT_HANDLE_ITEM', - message: `Skipping "${getTranslation( - item.name, - context.targetLanguage || context.defaultLanguage - )}". Could not locate its shape (${item.shape}))`, - }, - }) - return null - } - - async function createForLanguage(language: string) { - if (!shape) { - return - } - - item._topicsData = {} - if (item.topics) { - item._topicsData = { - topicIds: await getTopicIds({ - topics: item.topics || [], - language: context.targetLanguage || context.defaultLanguage, - context, - }), - } - } - - return context.callPIM( - buildCreateItemQueryAndVariables( - { - name: getTranslation(item.name, language) || '', - shapeIdentifier: item.shape, - tenantId: context.tenantId, - ...(item.externalReference && { - externalReference: item.externalReference, - }), - tree: { - parentId, - position: treePosition, - }, - components: {}, // Do components later - ...(item.topics && item._topicsData), - ...(shape?.type === 'product' && { - ...(await createProductItemMutation(language)), - }), - }, - // @ts-expect-error this will be set. Don't worry. - shape?.type, - language - ) - ) - } - - async function updateForLanguage(language: string, itemId: string) { - if (!shape || !itemId) { - onUpdate({ - error: { - code: 'CANNOT_HANDLE_PRODUCT', - message: `Cannot update "${getTranslation( - item.name, - language - )}" for language "${language}". Missing shape or itemId`, - item, - }, - }) - return - } - - if (item._componentsData) { - item._componentsData[language] = await createComponentsInput({ - components: item.components, - componentDefinitions: shape.components, - language, - grids: allGrids, - context, - onUpdate, - }) - } - - const clearComponentsData = item._componentsData?.[language] === null - - const updates: (() => Promise)[] = [] - - /** - * If it is a product, we need to pull all the product - * variants first, and then update each field. - * We need to do this because there is (currently) not - * any product.addVariant mutation - */ - let existingProductVariants: undefined | ProductVariant[] - if (shape?.type === 'product') { - const existingProduct = await getProduct(language, itemId, context) - existingProductVariants = existingProduct.variants - - // Add vatType if it is not part of the item. - // This eases the DX, as you don't _have_ to pass vatType - // for existing products - const productItem = item as JSONProduct - if (!productItem.vatType) { - productItem.vatType = existingProduct.vatType.name - } - // Ensure that this is an array - if (!productItem.variants) { - productItem.variants = [] - } - } - - /** - * Start with the basic item information - */ - updates.push(async () => - context.callPIM( - buildUpdateItemQueryAndVariables( - itemId, - { - name: getTranslation(item.name, language) || '', - ...(item.topics && item._topicsData), - ...(shape?.type === 'product' && { - ...(await createProductItemMutation( - language, - existingProductVariants - )), - }), - ...(clearComponentsData && { - components: {}, - }), - }, - shape.type as ItemType, - language - ) - ) - ) - - /** - * Create a single update component mutation on - * each component. This will ensure that no - * component data will be lost during the update - */ - if (item._componentsData?.[language]) { - Object.keys(item._componentsData[language]).forEach( - (componentId: string) => { - const componentContent: ComponentContentInput = - item._componentsData?.[language][componentId] - - componentContent && - !hasItemRelationsComponent(componentContent) && - updates.push(() => - context.callPIM( - buildUpdateItemComponentQueryAndVariables({ - itemId, - language, - input: { - componentId, - ...componentContent, - }, - }) - ) - ) - } - ) - } - if ((item as JSONProduct).variants?.length) { - const product = item as JSONProduct - for (const variant of product.variants) { - if (!variant._componentsData) { - variant._componentsData = {} - } - variant._componentsData[language] = await createComponentsInput({ - components: variant.components, - componentDefinitions: shape.variantComponents, - language, - grids: allGrids, - context, - onUpdate, - }) - - if (variant._componentsData?.[language]) { - Object.keys(variant._componentsData?.[language]).forEach( - (componentId: string) => { - const componentContent: ComponentContentInput = - variant._componentsData?.[language][componentId] - - componentContent && - !hasItemRelationsComponent(componentContent) && - updates.push(() => - context.callPIM( - buildUpdateVariantComponentQueryAndVariables({ - productId: itemId, - sku: variant.sku, - language, - input: { - componentId, - ...componentContent, - }, - }) - ) - ) - } - ) - } - } - } - - const responses: any[] = [] - for (let i = 0; i < updates.length; i++) { - responses.push(await updates[i]()) - } - context.emit(EVENT_NAMES.ITEM_UPDATED, { - id: itemId, - name: getTranslation(item.name, language), - language, - shape: { - type: shape.type, - identifier: shape.identifier, - }, - } as ItemEventPayloadCreatedOrUpdated) - - return responses - } - - function createProductBaseInfo() { - const product = item as JSONProduct - - const vatType = context.vatTypes?.find( - (v) => v.name?.toLowerCase() === product.vatType.toLowerCase() - ) - if (!vatType) { - onUpdate({ - error: { - code: 'CANNOT_HANDLE_PRODUCT', - message: `Cannot create product "${product.name}". Vat type "${product.vatType}" does not exist`, - item, - }, - }) - return - } - - return { - vatTypeId: vatType.id || '', - } - } - - async function createProductVariant( - jsonVariant: JSONProductVariant, - language: string, - shape: Shape, - allGrids: JSONGrid[], - existingProductVariant?: ProductVariant - ): Promise { - let attributes: undefined | ProductVariantAttributeInput[] - if (jsonVariant.attributes) { - attributes = Object.keys(jsonVariant.attributes).map( - (attribute: string) => ({ - attribute, - value: jsonVariant.attributes?.[attribute] || '', - }) - ) - } - - const { - priceVariants: existingProductVariantPriceVariants, - stockLocations: existingProductVariantStockLocations, - ...restOfExistingProductVariant - } = existingProductVariant || {} - - const variant: CreateProductVariantInput = { - ...(restOfExistingProductVariant as CreateProductVariantInput), - name: getTranslation(jsonVariant.name, language), - sku: jsonVariant.sku, - isDefault: jsonVariant.isDefault || false, - stockLocations: handleJsonStockToStockInput({ - jsonStock: jsonVariant.stock, - existingProductVariantStockLocations, - }), - priceVariants: handleJsonPriceToPriceInput({ - jsonPrice: jsonVariant.price, - existingProductVariantPriceVariants, - }), - ...(attributes && { attributes }), - } - delete variant.components - if (shape.variantComponents) { - jsonVariant._componentsData = {} - if (jsonVariant.components) { - jsonVariant._componentsData[language] = await createComponentsInput({ - components: jsonVariant.components, - componentDefinitions: shape.variantComponents, - language, - grids: allGrids, - context, - onUpdate, - }) - } - if (jsonVariant._componentsData?.[language]) { - Object.keys(jsonVariant._componentsData?.[language]).forEach( - (componentId: string) => { - const componentContent: ComponentContentInput = - jsonVariant._componentsData?.[language][componentId] - variant?.components || (variant.components = []) - componentContent && - !hasItemRelationsComponent(componentContent) && - variant?.components?.push({ - componentId, - ...componentContent, - }) - } - ) - } else { - delete variant.components - } - - if (jsonVariant.subscriptionPlans) { - variant.subscriptionPlans = jsonVariant.subscriptionPlans.map( - (sP) => { - const meteredVariables = getSubscriptionPlanMeteredVariables({ - planIdentifier: sP.identifier, - context, - }) - - return { - identifier: sP.identifier, - periods: sP.periods.map((p) => { - const id = getSubscriptionPlanPeriodId({ - planIdentifier: sP.identifier, - periodName: p.name, - context, - }) - - if (!id) { - throw new Error('Plan period id is null') - } - - return { - id, - ...(p.initial && { - initial: subscriptionPlanPrincingJsonToInput( - p.initial, - meteredVariables - ), - }), - recurring: subscriptionPlanPrincingJsonToInput( - p.recurring, - meteredVariables - ), - } - }), - } - } - ) - } - - if (jsonVariant.images) { - variant.images = await createImagesInput({ - images: jsonVariant.images, - language, - context, - onUpdate, - }) - } - - // This causes an internal error at the API right now. Setting the value to an empty - // array has the same outcome as setting it to null - if (variant.images === null) { - variant.images = [] - } - } - return variant - } - - async function createProductItemMutation( - language: string, - existingProductVariants?: ProductVariant[] - ) { - const product = item as JSONProduct - - const variants: CreateProductVariantInput[] = [] - const inp = { - ...createProductBaseInfo(), - variants, - } - - // Add existing product variants - if (existingProductVariants) { - inp.variants.push( - ...existingProductVariants.map( - ({ priceVariants, stockLocations, components, ...rest }) => - ({ - ...rest, - priceVariants: priceVariants?.map((p) => ({ - identifier: p.identifier, - price: p.price, - })), - stockLocations: stockLocations?.map((p) => ({ - identifier: p.identifier, - stock: p.stock, - meta: p.meta, - })), - } as CreateProductVariantInput) - ) - ) - } - - for (let i = 0; i < product.variants.length; i++) { - const vr = product.variants[i] - let existingProductVariant = existingProductVariants?.find( - (v) => - v.sku === vr.sku || - (v.externalReference && - v.externalReference === vr.externalReference) - ) - if (!existingProductVariant && vr.externalReference) { - existingProductVariant = existingProductVariants?.find( - (v) => v.externalReference === vr.externalReference - ) - } - const variant = await createProductVariant( - vr, - language, - context.shapes?.find((s) => s.identifier === item.shape) as Shape, - allGrids, - existingProductVariant - ) - - if (existingProductVariant) { - const index = inp.variants.findIndex( - (v) => - v.sku === vr.sku || - (v.externalReference && - v.externalReference === vr.externalReference) - ) - inp.variants[index] = variant - } else { - inp.variants.push(variant) - } - } - - // Ensure that only one is set as the default - const defaultVariants = inp.variants.filter((v) => v.isDefault) - if (defaultVariants.length !== 1) { - inp.variants.forEach((v) => (v.isDefault = false)) - inp.variants[0].isDefault = true - } - - return inp - } - - let itemId = item.id - - // Get new topics - item._topicsData = { - topicIds: await getTopicIds({ - topics: item.topics || [], - language: context.targetLanguage || context.defaultLanguage, - context, - }), - } - - if (itemId) { - /** - * Pull the item version info now and store it in the - * context cace before any changes are made to it. - * The version info will be read later before a - * potential publishing of an item - */ - if (context.config.itemPublish === 'auto') { - await getItemVersionsForLanguages({ - itemId, - languages: context.languages.map((l) => l.code), - context, - }) - } - - if (item._options?.moveToRoot) { - if (item._parentId !== rootItemId) { - await context.callPIM({ - query: buildMoveItemMutation(itemId, { - parentId: rootItemId, - }), - }) - } - } else if ( - item._exists && - item._parentId !== parentId && - itemId !== parentId && - parentId !== rootItemId // Do not move items to root unless _moveToRoot is set - ) { - /** - * Move the item if it is a part of a children array, - * or if item.parentExternalReference is passed - */ - await context.callPIM({ - query: buildMoveItemMutation(itemId, { - parentId, - position: treePosition, - }), - }) - } - - // Merge in existing topic - if (item.topics && context.config.itemTopics === 'amend') { - const existingTopicIds = await getExistingTopicIdsForItem( - itemId, - context.targetLanguage || context.defaultLanguage, - context - ) - - item._topicsData.topicIds = Array.from( - new Set([...existingTopicIds, ...item._topicsData.topicIds]) - ) - } - - const responses = await updateForLanguage( - context.targetLanguage || context.defaultLanguage, - itemId - ) - if (responses?.length) { - responses.forEach((response) => { - const result = response?.data?.[shape?.type]?.update - if (result) { - const { - id, - externalReference, - tree: { path }, - } = result - context.itemCataloguePathToIDMap.set(item.cataloguePath || path, { - itemId: id, - }) - - if (externalReference) { - context.itemExternalReferenceToIDMap.set(externalReference, { - itemId: id, - }) - } - } - }) - } - } else { - // Ensure a name is set for the default language (required by the API) - if ( - !getTranslation( - item.name, - context.targetLanguage || context.defaultLanguage - ) - ) { - onUpdate({ - error: { - code: 'CANNOT_HANDLE_ITEM', - message: `Item name cannot be empty for the default language. Item id: "${item.id}"`, - }, - }) - - throw new Error(`Item name cannot be empty for the default language`) - } - - /** - * Ensure that variants are present when creating - * a product - */ - if (shape.type === 'product') { - const product = item as JSONProduct - if (!product.variants || product.variants.length === 0) { - onUpdate({ - error: { - code: 'CANNOT_HANDLE_PRODUCT', - message: `Skipping "${getTranslation( - item.name, - context.targetLanguage || context.defaultLanguage - )}". No variants defined for product`, - item, - }, - }) - return null - } - } - - const response = await createForLanguage( - context.targetLanguage || context.defaultLanguage - ) - - const result = response?.data?.[shape?.type]?.create - if (result) { - const { - id, - externalReference, - tree: { path }, - } = result - context.itemCataloguePathToIDMap.set(item.cataloguePath || path, { - itemId: id, - }) - if (externalReference) { - context.itemExternalReferenceToIDMap.set(externalReference, { - itemId: id, - }) - } - itemId = id as string - - const language = context.targetLanguage || context.defaultLanguage - context.emit(EVENT_NAMES.ITEM_CREATED, { - id, - name: getTranslation(item.name, language), - language, - shape: { - type: shape.type, - identifier: shape.identifier, - }, - } as ItemEventPayloadCreatedOrUpdated) - - // Set the component data for the item - await updateForLanguage( - context.targetLanguage || context.defaultLanguage, - itemId - ) - } - } - - if (!itemId) { - onUpdate({ - error: { - code: 'CANNOT_HANDLE_ITEM', - message: `Could not create or update item "${getTranslation( - item.name, - context.targetLanguage || context.defaultLanguage - )}"`, - }, - }) - return null - } - - // Handle remaining languages - const remainingLanguages = context.languages - .filter((l) => !l.isDefault) - .map((l) => l.code) - - for (let i = 0; i < remainingLanguages.length; i++) { - await updateForLanguage(remainingLanguages[i], itemId) - } - - return itemId - } - - async function handleItem(item: JSONItem, index: number, parentId?: string) { - if (!item) { - return - } - - const itemAndParentId = await getItemId({ - context, - externalReference: item.externalReference, - cataloguePath: item.cataloguePath, - shapeIdentifier: item.shape, - language: context.targetLanguage || context.defaultLanguage, - }) - - item.id = itemAndParentId.itemId - item._parentId = itemAndParentId.parentId - - if (item.parentExternalReference || item.parentCataloguePath) { - const parentItemAndParentId = await getItemId({ - context, - externalReference: item.parentExternalReference, - cataloguePath: item.parentCataloguePath, - shapeIdentifier: item.shape, - language: context.targetLanguage || context.defaultLanguage, - }) - parentId = parentItemAndParentId.itemId - if (!parentId) { - parentId = context.fallbackFolderId - onUpdate({ - error: { - code: 'PARENT_FOLDER_NOT_FOUND', - message: `Cannot find the specified parent folder for item`, - item, - }, - }) - } - } - - // If the item exists in Crystallize already - item._exists = Boolean(item.id) - - item.id = (await createOrUpdateItem( - item, - parentId || rootItemId, - index + 1 - )) as string - - finishedItems++ - onUpdate({ - progress: finishedItems / totalItems, - message: `Handled ${getTranslation( - item.name, - context.targetLanguage || context.defaultLanguage - )}`, - }) - if (item.id) { - /** - * Store the item id for the cataloguePath. Very useful if the generated - * cataloguePath is different than the one in the JSON spec - */ - if (item.cataloguePath) { - context.itemCataloguePathToIDMap.set(item.cataloguePath, { - itemId: item.id, - parentId: parentId || rootItemId, - }) - } - - if (item.externalReference) { - context.itemExternalReferenceToIDMap.set(item.externalReference, { - itemId: item.id, - parentId: parentId || rootItemId, - }) - } - - if (item && 'children' in item) { - const itm = item as JSONFolder - - if (itm.children) { - for (let i = 0; i < itm.children.length; i++) { - await handleItem(itm.children[i], i, itm.id) - } - } - } - } - } - - async function handleItemRelationsAndPublish(item: JSONItem) { - if (!item) { - return - } - - onUpdate({ - message: `Item relations: ${getTranslation( - item.name, - context.targetLanguage || context.defaultLanguage - )}`, - }) - - async function getItemIdsForItemRelation( - itemRelations?: JSONItemRelation[] - ): Promise { - const ids: string[] = [] - - if ( - !itemRelations || - !itemRelations.map || - typeof itemRelations.map !== 'function' - ) { - return ids - } - - await Promise.all( - itemRelations.map(async (itemRelation) => { - if (typeof itemRelation === 'object') { - const { itemId } = await getItemId({ - context, - externalReference: itemRelation.externalReference, - cataloguePath: itemRelation.cataloguePath, - language: context.targetLanguage || context.defaultLanguage, - }) - - if (itemId) { - ids.push(itemId) - } else { - onUpdate({ - error: { - code: 'CANNOT_HANDLE_ITEM_RELATION', - message: `Could not determine an ID for related item "${ - itemRelation.externalReference - ? itemRelation.externalReference - : itemRelation.cataloguePath - }`, - item, - }, - } as AreaUpdate) - } - } - }) - ) - - return ids - } - - /** - * Fetches the mutation input for a component update mutation. - * - * @param { ShapeComponent, componentContent, componentId } - * @returns Promise with mutation object or null - */ - const getComponentUpdateMutationInput = async ({ - shapeComponent, - componentContent, - componentsData, - componentId, - }: { - shapeComponent?: ShapeComponent - componentContent: JSONComponentContent - componentsData?: Record - componentId: string - }): Promise => { - if (!shapeComponent) { - return null - } - - let mutationInput = null - - switch (shapeComponent?.type) { - case 'itemRelations': { - mutationInput = { - componentId, - itemRelations: { - itemIds: await getItemIdsForItemRelation( - componentContent as JSONItemRelation[] - ), - }, - } - break - } - case 'componentChoice': - case 'contentChunk': { - const { choices } = - shapeComponent.config as ComponentChoiceComponentConfig - const { components } = - shapeComponent.config as ContentChunkComponentConfig - const itemRelationIds = (choices || components) - .filter((s: any) => s.type === 'itemRelations') - .map((s: any) => s.id) - // Get existing data for component - if (itemRelationIds.length > 0) { - const existingComponentsData = - componentsData?.[ - context.targetLanguage || context.defaultLanguage - ] - const componentData = existingComponentsData[componentId] - - if (componentData) { - if (shapeComponent.type === 'componentChoice') { - if (componentData.componentChoice?.componentId) { - const selectedDef = choices.find( - (c: any) => - c.id === componentData.componentChoice.componentId - ) - if (selectedDef?.type === 'itemRelations') { - const chosenComponentId = - componentData.componentChoice.componentId - const component = componentContent as Record< - string, - JSONItemReference[] - > - componentData.componentChoice - mutationInput = { - componentId, - componentChoice: { - componentId: chosenComponentId, - itemRelations: { - itemIds: await getItemIdsForItemRelation( - component[chosenComponentId] as JSONItemReference[] - ), - }, - }, - } - } - } - } else if (shapeComponent.type === 'contentChunk') { - mutationInput = { - componentId, - ...componentData, - } - await Promise.all( - mutationInput.contentChunk.chunks.map( - async (chunk: any, chunkIndex: number) => { - const jsonChunk = (componentContent as JSONContentChunk)[ - chunkIndex - ] - - // Get item ids within chunk - await Promise.all( - itemRelationIds.map(async (itemRelationId: string) => { - const itemRelationComponentIndex = chunk.findIndex( - (c: any) => c.componentId === itemRelationId - ) - - if (itemRelationComponentIndex !== -1) { - chunk[ - itemRelationComponentIndex - ].itemRelations.itemIds = - await getItemIdsForItemRelation( - jsonChunk[itemRelationId] as JSONItemRelation[] - ) - } - }) - ) - } - ) - ) - } - } - } - break - } - } - - return mutationInput - } - - if (item.id) { - // Pull the item info from the cache - const versionsInfo = context.itemVersions.get(item.id) - - if (item.id) { - if (item.components) { - const keys = Object.keys(item.components) - - for (let i = 0; i < keys.length; i++) { - const componentId = keys[i] - const jsonItem = item.components?.[ - componentId - ] as JSONComponentContent - - if (jsonItem) { - const shape = context.shapes?.find( - (s) => s.identifier === item.shape - ) - const def = shape?.components?.find( - (c: any) => c.id === componentId - ) - - const mutationInput = await getComponentUpdateMutationInput({ - shapeComponent: def, - componentContent: jsonItem, - componentsData: item._componentsData, - componentId, - }) - - // Update the component - if (mutationInput) { - try { - const r = await context.callPIM({ - query: gql` - mutation UPDATE_RELATIONS_COMPONENT( - $itemId: ID! - $language: String! - $input: ComponentInput! - ) { - item { - updateComponent( - itemId: $itemId - language: $language - input: $input - ) { - id - } - } - } - `, - variables: { - itemId: item.id, - language: - context.targetLanguage || context.defaultLanguage, - input: mutationInput, - }, - }) - - if (r.errors) { - throw r.errors - } - } catch (err) { - onUpdate({ - error: { - code: 'CANNOT_HANDLE_ITEM_RELATION', - message: `Unable to update relation for item id "${ - item.id - }" with input ${JSON.stringify(mutationInput)} `, - item, - }, - }) - } - } - } - } - } - - const product = item as JSONProduct - if (product.variants?.length) { - for (const variant of product.variants) { - if (!variant.components) { - continue - } - - const keys = Object.keys(variant.components) - for (let i = 0; i < keys.length; i++) { - const componentId = keys[i] - - const jsonItem = variant.components?.[ - componentId - ] as JSONComponentContent - - if (jsonItem) { - const shape = context.shapes?.find( - (s) => s.identifier === item.shape - ) - const def = shape?.variantComponents?.find( - (c: any) => c.id === componentId - ) - - const mutationInput = await getComponentUpdateMutationInput({ - shapeComponent: def, - componentContent: jsonItem, - componentsData: variant._componentsData, - componentId, - }) - - // Update the component - if (mutationInput) { - try { - const r = await context.callPIM({ - query: gql` - mutation UPDATE_VARIANT_RELATIONS_COMPONENT( - $productId: ID! - $sku: String! - $language: String! - $input: ComponentInput! - ) { - product { - updateVariantComponent( - productId: $productId - sku: $sku - language: $language - input: $input - ) { - id - } - } - } - `, - variables: { - productId: item.id, - sku: variant.sku, - language: - context.targetLanguage || context.defaultLanguage, - input: mutationInput, - }, - }) - - if (r.errors) { - throw r.errors - } - } catch (err) { - onUpdate({ - error: { - code: 'CANNOT_HANDLE_ITEM_RELATION', - message: `Unable to update relation for variant with sku "${ - variant.sku - }" with input ${JSON.stringify(mutationInput)} `, - }, - }) - } - } - } - } - } - } - } - - // Publish if needed - if (item.id) { - for (let i = 0; i < context.languages.length; i++) { - const language = context.languages[i].code - - const publish = () => { - context.emit(EVENT_NAMES.ITEM_PUBLISHED, { - id: item.id, - name: getTranslation(item.name, language), - language, - } as ItemEventPayload) - - return publishItem(language, item.id!, context) - } - - const passedPublishConfig = item._options?.publish - if (typeof passedPublishConfig === 'boolean') { - if (passedPublishConfig) { - await publish() - } - } else if ( - context.config.itemPublish === 'publish' || - !versionsInfo || - versionsInfo[context.languages[i].code] === - ItemVersionDescription.Published - ) { - await publish() - } - } - } - } - - finishedItems++ - onUpdate({ - progress: finishedItems / totalItems, - }) - - if (item && 'children' in item) { - const itm = item as JSONFolder - - if (itm.children) { - for (let i = 0; i < itm.children.length; i++) { - await handleItemRelationsAndPublish(itm.children[i]) - } - } - } - } - - for (let i = 0; i < spec.items.length; i++) { - const item = spec.items[i] - try { - await handleItem(item, i, rootItemId) - } catch (e) { - onUpdate({ - error: { - code: 'CANNOT_HANDLE_ITEM', - message: `Skipping "${getTranslation( - item.name, - context.targetLanguage || context.defaultLanguage - )}"`, - item, - }, - }) - } - } - - /** - * Item relations needs to be handled at the end, after all - * items are created - */ - onUpdate({ - message: 'Updating item relations...', - }) - - /** - * At this point we want to start using cached values - * so that we don't hit the API as much and speed things up - */ - context.useReferenceCache = true - - for (let i = 0; i < spec.items.length; i++) { - await handleItemRelationsAndPublish(spec.items[i]) - } - - // clearInterval(getFileuploaderStatusInterval) - - onUpdate({ - progress: 1, - }) -} diff --git a/src/bootstrap-tenant/bootstrapper/items/handleItem.ts b/src/bootstrap-tenant/bootstrapper/items/handleItem.ts index b4d3bf3..7ee6461 100644 --- a/src/bootstrap-tenant/bootstrapper/items/handleItem.ts +++ b/src/bootstrap-tenant/bootstrapper/items/handleItem.ts @@ -42,7 +42,7 @@ export async function handleItem( language: context.targetLanguage || context.defaultLanguage, }) parentId = parentItemAndParentId.itemId || '' - if (parentId !== '') { + if (parentId == '') { parentId = context.fallbackFolderId onUpdate({ error: {