@raketa-cloud/admin 1.10.10 → 1.10.12

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@raketa-cloud/admin",
-  "version": "1.10.10",
+  "version": "1.10.12",
   "main": "src/index.js",
   "engines": {
     "node": ">=22"

package/src/cloud/ResourceChanges.js

@@ -18,7 +18,7 @@ export default class ResourceChanges extends RaketaClient {
           resourceType,
           pagination: {
             ...(vars.input.pagination || {}),
-            page_size: 50,
+            pageSize: 50,
           },
         },
       },

package/src/handlers/resources/TranslationsHandler.js

@@ -1,3 +1,21 @@
+function matchResourceIdAndLocale(resources) {
+  const idLocaleMap = new Map();
+
+  resources.forEach((r) => {
+    Object.keys(r.translations).forEach((locale) => {
+      const translation = r.translations[locale];
+
+      if (!translation.id) {
+        throw new Error("Missing id in translation!");
+      }
+
+      idLocaleMap.set(translation.id.toString(), locale);
+    });
+  });
+
+  return idLocaleMap;
+}
+
 const index = async (ctx) => {
   const { parent_id } = ctx.params;
   const { translations } = ctx.query;
@@ -222,12 +240,23 @@ const create = async (ctx) => {
   }
 
   const revisionInputs = [];
+  const resourceIdLocaleMap =
+    resources.length === selectedLocales.length
+      ? null
+      : matchResourceIdAndLocale(resources);
 
   for (let i = 0; i < resources.length; i++) {
     const translated = resources[i];
-    const locale = selectedLocales[i];
 
     if (translated) {
+      const locale = resourceIdLocaleMap
+        ? resourceIdLocaleMap.get(translated.id.toString())
+        : selectedLocales[i];
+
+      if (!selectedLocales.includes(locale)) {
+        continue;
+      }
+
       blueprint.translations = {
         ...blueprint.translations,
         [locale]: {

package/src/tests/cloud/ResourceChanges.test.js

@@ -0,0 +1,25 @@
+import ResourceChanges from "../../cloud/ResourceChanges";
+import { ResourceHistoryDocument } from "../../cloud/documents/resourcesDocuments";
+
+describe("ResourceChanges", () => {
+  describe("list", () => {
+    test("uses camelCase pageSize when requesting resource history", async () => {
+      const client = new ResourceChanges(() => ({}), "token");
+      const query = jest
+        .spyOn(client, "query")
+        .mockResolvedValue({ resourceHistory: { list: [] } });
+
+      await client.list({ page: "2" }, "resource-1", "en", "page");
+
+      expect(query).toHaveBeenCalledWith(ResourceHistoryDocument, {
+        input: {
+          id: "resource-1",
+          locale: "en",
+          resourceType: "page",
+          filters: {},
+          pagination: { page: 2, pageSize: 50 },
+        },
+      });
+    });
+  });
+});

package/src/tests/handlers/resources/TranslationsHandler.test/create.test.js

@@ -324,6 +324,38 @@ describe("Resource Translations Handler", () => {
     });
   });
 
+  test("creates revisions and redirect pairs from returned locales when the backend omits a requested locale", async () => {
+    const resource = Factory.build("resource", {
+      translations: {},
+      categories: [{ id: "cat1" }],
+    });
+    const ctx = buildCtx({ languages: ["de", "en-us", "fr"] }, resource);
+
+    jest.spyOn(ctx.client.resources, "find").mockImplementation(() => resource);
+    jest
+      .spyOn(ctx.client.resources, "multiTranslate")
+      .mockImplementation(() => ({
+        resources: [
+          { id: "de-1", translations: { fr: { id: "fr-1" } } },
+          { id: "fr-1", translations: { de: { id: "de-1" } } },
+        ],
+        errors: null,
+      }));
+    const createBatch = jest
+      .spyOn(ctx.client.resourceRevisions, "createBatch")
+      .mockImplementation(() => [{ errors: null }, { errors: null }]);
+
+    const result = await TranslationsHandler.create(ctx);
+
+    expect(createBatch).toHaveBeenCalledWith([
+      { resourceId: "de-1", resourceType: "article", locale: "de" },
+      { resourceId: "fr-1", resourceType: "article", locale: "fr" },
+    ]);
+    expect(result).toEqual({
+      redirect: `/en/articles/${resource.id}/translations?translations=de:de-1;fr:fr-1`,
+    });
+  });
+
   test("create translation fails with revision errors", async () => {
     const resource = Factory.build("resource", {
       translations: {},

package/src/tests/handlers/resources/TranslationsHandler.test/edge_cases.test.js

@@ -156,7 +156,10 @@ describe("Resource Translations Handler create", () => {
     jest
       .spyOn(ctx.client.resources, "multiTranslate")
       .mockImplementation(() => ({
-        resources: [{ id: "fr-1" }],
+        resources: [
+          { id: "en-1", translations: { fr: { id: "fr-1" } } },
+          { id: "fr-1", translations: { en: { id: "en-1" } } },
+        ],
         errors: null,
       }));
     const mockRevisions = jest.spyOn(