@unchainedshop/platform 2.0.0-beta8 → 2.0.0-beta9

package/src/bulk-importer/handlers/product/update.ts

@@ -1,11 +1,11 @@
-import { Context } from '@unchainedshop/types/api';
+import { UnchainedCore } from '@unchainedshop/types/core';
 import upsertVariations from './upsertVariations';
 import upsertMedia from './upsertMedia';
 import upsertProductContent from './upsertProductContent';
 import transformSpecificationToProductStructure from './transformSpecificationToProductStructure';
 
-export default async function createProduct(payload: any, { logger, authorId }, unchainedAPI: Context) {
-  const { modules, userId } = unchainedAPI;
+export default async function createProduct(payload: any, { logger }, unchainedAPI: UnchainedCore) {
+  const { modules } = unchainedAPI;
   const { specification, media, variations, _id } = payload;
 
   if (!(await modules.products.productExists({ productId: _id }))) {
@@ -15,14 +15,9 @@ export default async function createProduct(payload: any, { logger, authorId },
   if (specification) {
     const productData = transformSpecificationToProductStructure(specification);
     logger.debug('update product object', productData);
-    await modules.products.update(
-      _id,
-      {
-        ...productData,
-        authorId,
-      },
-      userId,
-    );
+    await modules.products.update(_id, {
+      ...productData,
+    });
 
     if (specification.content) {
       logger.debug('replace localized content for product', specification.content);
@@ -30,7 +25,6 @@ export default async function createProduct(payload: any, { logger, authorId },
         {
           content: specification.content,
           productId: _id,
-          authorId,
         },
         unchainedAPI,
       );
@@ -43,7 +37,6 @@ export default async function createProduct(payload: any, { logger, authorId },
       {
         variations: variations || [],
         productId: _id,
-        authorId,
       },
       unchainedAPI,
     );
@@ -51,7 +44,7 @@ export default async function createProduct(payload: any, { logger, authorId },
 
   if (media) {
     logger.debug('replace product media', media);
-    await upsertMedia({ media, productId: _id, authorId }, unchainedAPI);
+    await upsertMedia({ media, productId: _id }, unchainedAPI);
   }
 
   return {

package/src/bulk-importer/handlers/product/upsertMedia.ts

@@ -1,11 +1,11 @@
-import { Context } from '@unchainedshop/types/api';
 import { ProductMedia, ProductMediaText } from '@unchainedshop/types/products.media';
+import { UnchainedCore } from '@unchainedshop/types/core';
 import convertTagsToLowerCase from '../utils/convertTagsToLowerCase';
 import upsertAsset from '../../upsertAsset';
 
-const upsertProductMedia = async (productMedia: ProductMedia, { modules, userId }: Context) => {
+const upsertProductMedia = async (productMedia: ProductMedia, { modules }: UnchainedCore) => {
   try {
-    const productMediaObj = await modules.products.media.create(productMedia, userId);
+    const productMediaObj = await modules.products.media.create(productMedia);
     return productMediaObj;
   } catch (e) {
     const { _id, ...productMediaData } = productMedia;
@@ -14,8 +14,8 @@ const upsertProductMedia = async (productMedia: ProductMedia, { modules, userId
   }
 };
 
-export default async function upsertMedia({ media, authorId, productId }, unchainedAPI: Context) {
-  const { modules, userId } = unchainedAPI;
+export default async function upsertMedia({ media, productId }, unchainedAPI: UnchainedCore) {
+  const { modules } = unchainedAPI;
 
   const productMediaObjects = await Promise.all(
     media.map(async ({ asset, content, ...mediaData }) => {
@@ -29,7 +29,6 @@ export default async function upsertMedia({ media, authorId, productId }, unchai
       const fileId = file._id;
       const productMedia = await upsertProductMedia(
         {
-          authorId,
           ...mediaData,
           tags,
           productId,
@@ -41,16 +40,13 @@ export default async function upsertMedia({ media, authorId, productId }, unchai
 
       if (content) {
         await Promise.all(
-          Object.entries(content).map(
-            async ([locale, { authorId: tAuthorId, ...localizedData }]: [string, ProductMediaText]) => {
-              return modules.products.media.texts.upsertLocalizedText(
-                productMedia._id,
-                locale,
-                localizedData,
-                tAuthorId || authorId || userId,
-              );
-            },
-          ),
+          Object.entries(content).map(async ([locale, localizedData]: [string, ProductMediaText]) => {
+            return modules.products.media.texts.upsertLocalizedText(
+              productMedia._id,
+              locale,
+              localizedData,
+            );
+          }),
         );
       }
       return productMedia;

package/src/bulk-importer/handlers/product/upsertProductContent.ts

@@ -1,20 +1,10 @@
-import { Context } from '@unchainedshop/types/api';
+import { UnchainedCore } from '@unchainedshop/types/core';
 import { ProductText } from '@unchainedshop/types/products';
 
-export default async function upsertProductContent(
-  { productId, content, authorId },
-  { modules, userId }: Context,
-) {
+export default async function upsertProductContent({ productId, content }, { modules }: UnchainedCore) {
   await Promise.all(
-    Object.entries(content).map(
-      async ([locale, { authorId: tAuthorId, ...localizedData }]: [string, ProductText]) => {
-        return modules.products.texts.upsertLocalizedText(
-          productId,
-          locale,
-          localizedData,
-          tAuthorId || authorId || userId,
-        );
-      },
-    ),
+    Object.entries(content).map(async ([locale, localizedData]: [string, ProductText]) => {
+      return modules.products.texts.upsertLocalizedText(productId, locale, localizedData);
+    }),
   );
 }

package/src/bulk-importer/handlers/product/upsertVariations.ts

@@ -1,27 +1,23 @@
-import { Context } from '@unchainedshop/types/api';
+import { UnchainedCore } from '@unchainedshop/types/core';
 import { ProductVariation, ProductVariationText } from '@unchainedshop/types/products.variations';
 
-const upsert = async (productVariation: ProductVariation, unchainedAPI: Context) => {
-  const { modules, userId } = unchainedAPI;
+const upsert = async (productVariation: ProductVariation, unchainedAPI: UnchainedCore) => {
+  const { modules } = unchainedAPI;
   try {
-    const newVariation = await modules.products.variations.create(productVariation, userId);
+    const newVariation = await modules.products.variations.create(productVariation);
     return newVariation;
   } catch (e) {
     return modules.products.variations.update(productVariation._id, productVariation);
   }
 };
 
-export default async function upsertVariations(
-  { variations, authorId, productId },
-  unchainedAPI: Context,
-) {
-  const { modules, userId } = unchainedAPI;
+export default async function upsertVariations({ variations, productId }, unchainedAPI: UnchainedCore) {
+  const { modules } = unchainedAPI;
 
-  const productVariationObjects = await Promise.all(
+  const upsertedProductVariationIds = await Promise.all(
     variations.map(async ({ content, options, ...variationsRest }) => {
       const variation = await upsert(
         {
-          authorId,
           ...variationsRest,
           options: options.map((option) => option.value),
           productId,
@@ -33,10 +29,7 @@ export default async function upsertVariations(
         options.map(async ({ content: optionContent, value: optionValue }) => {
           await Promise.all(
             Object.entries(optionContent).map(
-              async ([locale, { authorId: tAuthorId, ...localizedData }]: [
-                string,
-                ProductVariationText,
-              ]) => {
+              async ([locale, localizedData]: [string, ProductVariationText]) => {
                 return modules.products.variations.texts.upsertLocalizedText(
                   {
                     productVariationId: variation._id,
@@ -44,7 +37,6 @@ export default async function upsertVariations(
                   },
                   locale,
                   localizedData,
-                  tAuthorId || authorId || userId,
                 );
               },
             ),
@@ -52,27 +44,27 @@ export default async function upsertVariations(
         }),
       );
       await Promise.all(
-        Object.entries(content).map(
-          async ([locale, { authorId: tAuthorId, ...localizedData }]: [
-            string,
-            ProductVariationText,
-          ]) => {
-            return modules.products.variations.texts.upsertLocalizedText(
-              {
-                productVariationId: variation._id,
-              },
-              locale,
-              localizedData,
-              tAuthorId || authorId || userId,
-            );
-          },
-        ),
+        Object.entries(content).map(async ([locale, localizedData]: [string, ProductVariationText]) => {
+          return modules.products.variations.texts.upsertLocalizedText(
+            {
+              productVariationId: variation._id,
+            },
+            locale,
+            localizedData,
+          );
+        }),
       );
-      return variation;
+      return variation._id;
     }),
   );
-  await modules.products.variations.deleteVariations({
+  const allVariations = await modules.products.variations.findProductVariations({
     productId,
-    excludedProductVariationIds: productVariationObjects.map((obj) => obj._id),
   });
+  await Promise.all(
+    allVariations.map(async (variation) => {
+      if (!upsertedProductVariationIds.includes(variation._id)) {
+        await modules.products.variations.delete(variation._id);
+      }
+    }),
+  );
 }

package/src/bulk-importer/upsertAsset.ts

@@ -1,12 +1,12 @@
-import { Context } from '@unchainedshop/types/api';
+import { UnchainedCore } from '@unchainedshop/types/core';
 import { File } from '@unchainedshop/types/files';
 
 const upsertAsset = async (
   directoryName: string,
   asset: File & { fileName: string; headers?: Record<string, unknown> },
-  unchainedAPI: Context,
+  unchainedAPI: UnchainedCore,
 ) => {
-  const { modules, services, userId } = unchainedAPI;
+  const { modules, services } = unchainedAPI;
   const { _id: fileId, fileName, url, meta, headers, ...assetData } = asset;
 
   try {
@@ -16,7 +16,7 @@ const upsertAsset = async (
       if (JSON.stringify(newMeta) === JSON.stringify(existingFile.meta)) {
         return existingFile;
       }
-      await modules.files.update(fileId, { meta: newMeta, ...assetData }, userId);
+      await modules.files.update(fileId, { meta: newMeta, ...assetData });
       const updatedFile = await modules.files.findFile({ fileId });
       return updatedFile;
     }
@@ -30,7 +30,6 @@ const upsertAsset = async (
           headers,
         },
         meta: { ...meta, fileId },
-        userId,
       },
       unchainedAPI,
     );

package/src/setup/setupAccounts.ts

@@ -29,14 +29,11 @@ export const setupAccounts = (unchainedAPI: UnchainedCore) => {
     if (!input) return true;
     if (input.skipMessaging) return true;
 
-    return unchainedAPI.modules.worker.addWork(
-      {
-        type: 'MESSAGE',
-        retries: 0,
-        input,
-      },
-      undefined,
-    );
+    return unchainedAPI.modules.worker.addWork({
+      type: 'MESSAGE',
+      retries: 0,
+      input,
+    });
   };
 
   accountsServer.services.guest = {
@@ -76,6 +73,8 @@ export const setupAccounts = (unchainedAPI: UnchainedCore) => {
 
   accountsServer.on('LoginTokenCreated', async (props) => {
     const { userId, connection = {} } = props;
+
+    // TODO: Doubt that there is countryContext etc. here?
     const { userIdBeforeLogin, countryContext, remoteAddress, remotePort, userAgent, normalizedLocale } =
       connection;
 

package/src/setup/setupCarts.ts

@@ -17,6 +17,7 @@ export const setupCarts = async (unchainedAPI: UnchainedCore, options: SetupCart
     await Promise.all(
       users.map((user) => {
         const locale = unchainedAPI.modules.users.userLocale(user);
+
         return unchainedAPI.modules.orders.ensureCartForUser(
           { user, countryCode: locale.country },
           unchainedAPI,

package/src/templates/utils/getOrderAttachmentsData.ts

@@ -1,10 +1,10 @@
-import { Context } from '@unchainedshop/types/api';
+import { UnchainedCore } from '@unchainedshop/types/core';
 import { Order } from '@unchainedshop/types/orders';
 
 export const getOrderAttachmentsData = async (
   order: Order,
   params: { fileType: string },
-  { modules }: Context,
+  { modules }: UnchainedCore,
 ) => {
   const attachments = [];
 

package/src/templates/utils/getOrderPositionsData.ts

@@ -1,11 +1,11 @@
-import { Context } from '@unchainedshop/types/api';
 import { Locale } from '@unchainedshop/types/common';
+import { UnchainedCore } from '@unchainedshop/types/core';
 import { Order } from '@unchainedshop/types/orders';
 
 export const getOrderPositionsData = async (
   order: Order,
   params: { locale?: Locale },
-  context: Context,
+  context: UnchainedCore,
 ) => {
   const { modules } = context;
   const orderPositions = await modules.orders.positions.findOrderPositions({

package/tests/platform-index.test.ts

@@ -0,0 +1,5 @@
+describe('Platform', () => {
+  it('Init', async () => {
+    expect(true).toBeTruthy()
+  });
+});