@truedat/bg 7.5.10 → 7.5.12

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@truedat/bg",
-  "version": "7.5.10",
+  "version": "7.5.12",
   "description": "Truedat Web Business Glossary",
   "sideEffects": false,
   "module": "src/index.js",
@@ -46,34 +46,34 @@
   "devDependencies": {
     "@testing-library/dom": "^10.4.0",
     "@testing-library/jest-dom": "^6.6.3",
-    "@testing-library/react": "^16.2.0",
+    "@testing-library/react": "^16.3.0",
     "@testing-library/user-event": "^14.6.1",
-    "@truedat/test": "7.5.10",
+    "@truedat/test": "7.5.12",
     "identity-obj-proxy": "^3.0.0",
     "jest": "^29.7.0",
     "redux-saga-test-plan": "^4.0.6"
   },
   "dependencies": {
-    "@apollo/client": "^3.13.5",
-    "axios": "^1.8.4",
+    "@apollo/client": "^3.13.8",
+    "axios": "^1.9.0",
     "file-saver": "^2.0.5",
-    "graphql": "^16.10.0",
+    "graphql": "^16.11.0",
     "is-hotkey": "^0.2.0",
     "is-url": "^1.2.4",
     "lodash": "^4.17.21",
     "moment": "^2.30.1",
     "path-to-regexp": "^8.2.0",
     "prop-types": "^15.8.1",
-    "query-string": "^7.1.2",
-    "react": "^19.0.0",
+    "query-string": "^7.1.3",
+    "react": "^19.1.0",
     "react-csv": "^2.2.2",
-    "react-dom": "^19.0.0",
+    "react-dom": "^19.1.0",
     "react-dropzone": "^14.3.8",
-    "react-hook-form": "^7.54.2",
-    "react-intl": "^7.1.10",
+    "react-hook-form": "^7.56.4",
+    "react-intl": "^7.1.11",
     "react-moment": "^1.1.3",
     "react-redux": "^9.2.0",
-    "react-router": "^7.4.0",
+    "react-router": "^7.6.0",
     "redux": "^5.0.1",
     "redux-saga": "^1.3.0",
     "redux-saga-routines": "^3.2.3",
@@ -82,5 +82,5 @@
     "semantic-ui-react": "^3.0.0-beta.2",
     "swr": "^2.3.3"
   },
-  "gitHead": "8a8c49e9d5d6bb4e5f2b503b063bfd6dd20a503d"
+  "gitHead": "1476061ceb9ed15e0bacec71e20d695a0cd57986"
 }

package/src/concepts/components/ConceptActions.js

@@ -2,10 +2,10 @@ import _ from "lodash/fp";
 import PropTypes from "prop-types";
 import { connect } from "react-redux";
 import { Button, Icon } from "semantic-ui-react";
-import { Link } from "react-router";
+import { Link, useNavigate } from "react-router";
 import { FormattedMessage } from "react-intl";
 import { ConfirmModal, GroupActions } from "@truedat/core/components";
-import { linkTo } from "@truedat/core/routes";
+import { CONCEPTS, linkTo } from "@truedat/core/routes";
 import { conceptAction } from "../routines";
 import { mapActionProps } from "../constants/mappings";
 import ConceptDeleteModalBody from "./ConceptDeleteModalBody";
@@ -45,37 +45,43 @@ const ModalDelete = ({
   deleteAction,
   conceptVersion,
   concept,
-}) => (
-  <ConfirmModal
-    icon="trash"
-    trigger={
-      <Button
-        icon={<Icon name="trash alternate outline" color="red" />}
-        content={<FormattedMessage id="actions.delete" />}
-      />
-    }
-    header={
-      <FormattedMessage id="concepts.actions.delete.confirmation.header" />
-    }
-    content={
-      <>
-        {conceptVersion === 1 ? (
-          <ConceptDeleteModalBody
-            conceptName={_.prop("name")(concept)}
-            conceptDomain={_.prop("domain.name")(concept)}
-            linkCount={_.prop("link_count")(concept)}
-            ruleCount={_.prop("rule_count")(concept)}
-            conceptCount={_.prop("concept_count")(concept)}
-            conceptVersion={_.prop("version")(concept)}
-          />
-        ) : (
-          <FormattedMessage id="concepts.actions.delete.confirmation.content" />
-        )}
-      </>
-    }
-    onConfirm={() => conceptAction({ action: "delete", ...deleteAction })}
-  />
-);
+}) => {
+  const navigate = useNavigate();
+  return (
+    <ConfirmModal
+      icon="trash"
+      trigger={
+        <Button
+          icon={<Icon name="trash alternate outline" color="red" />}
+          content={<FormattedMessage id="actions.delete" />}
+        />
+      }
+      header={
+        <FormattedMessage id="concepts.actions.delete.confirmation.header" />
+      }
+      content={
+        <>
+          {conceptVersion === 1 ? (
+            <ConceptDeleteModalBody
+              conceptName={_.prop("name")(concept)}
+              conceptDomain={_.prop("domain.name")(concept)}
+              linkCount={_.prop("link_count")(concept)}
+              ruleCount={_.prop("rule_count")(concept)}
+              conceptCount={_.prop("concept_count")(concept)}
+              conceptVersion={_.prop("version")(concept)}
+            />
+          ) : (
+            <FormattedMessage id="concepts.actions.delete.confirmation.content" />
+          )}
+        </>
+      }
+      onConfirm={() => {
+        conceptAction({ action: "delete", ...deleteAction });
+        navigate(CONCEPTS);
+      }}
+    />
+  );
+};
 
 ModalDelete.propTypes = {
   concept: PropTypes.object,

package/src/concepts/components/ConceptEdit.js

@@ -46,7 +46,7 @@ export const ConceptEdit = ({
 
     const business_concept_version = {
       name: defaultContent.name,
-      domain_id: domain.id,
+      domain_id: domain?.id,
       type: template.name,
       content: newContent,
       i18n_content: i18nSendConcept,
@@ -63,7 +63,7 @@ export const ConceptEdit = ({
     <>
       <ConceptForm
         actionKey={actionKey}
-        domainId={domain.id}
+        domainId={domain?.id}
         template={template}
         i18nConcept={i18nConcept}
         loading={templateLoading || localesLoading}
@@ -94,7 +94,7 @@ export const ConceptEditLoader = (props) => {
   const { locales } = useLanguage();
   const { data, loading: templateLoading } = useTemplate({
     name: templateName,
-    domainIds: [domain.id],
+    domainIds: [domain?.id],
   });
 
   const template = _.prop("template")(data);

package/src/concepts/reducers/conceptRedirect.js

@@ -26,11 +26,9 @@ export const conceptRedirect = (
       return initialState;
     case conceptAction.SUCCESS:
       const data = _.prop("data")(payload);
-      return _.propEq("method", "DELETE")(meta)
-        ? CONCEPTS
-        : _.includes(_.prop("action")(meta))(["version", "update", "create"])
-          ? conceptUrl(_.pick(["id", "business_concept_id"])(data))
-          : state;
+      return _.includes(_.prop("action")(meta))(["version", "update", "create"])
+        ? conceptUrl(_.pick(["id", "business_concept_id"])(data))
+        : state;
     case linkConcept.SUCCESS: {
       const { redirectUrl } = meta;
       return redirectUrl;

package/src/concepts/relations/components/ConceptRelations.js

@@ -211,7 +211,8 @@ export const ConceptRelations = ({
                 maxDepth={maxDepth}
               />
               <RelationGraph
-                navigateToConcept={navigateToConcept}
+                key={`graph-${depth}-${_.size(limitedRelationGraph?.nodes)}`}
+                navigate={navigateToConcept}
                 currentId={currentId}
                 relationsGraph={limitedRelationGraph}
               />

package/src/taxonomy/components/__tests__/Domain.spec.js

@@ -79,7 +79,7 @@ describe("<Domain />", () => {
   });
 
   it("renders null when domain is not available", async () => {
-    useDomain.mockImplementationOnce(() => ({
+    useDomain.mockImplementation(() => ({
       data: null,
     }));
 

package/src/taxonomy/components/__tests__/DomainForm.spec.js

@@ -6,82 +6,81 @@ import { useDomains } from "../../../hooks/useDomains";
 
 jest.mock("../../../hooks/useDomains", () => ({
   ...jest.requireActual("../../../hooks/useDomains"),
-  useDomains: jest.fn(() => ({
-    data: [
-      {
-        description: "No tiene padre",
-        domain_group: null,
-        external_id: "1",
-        id: 1,
-        name: "domain_1",
-        parent_id: null,
-        parents: null,
-        type: null,
-      },
-      {
-        description: "No tiene padre",
-        domain_group: null,
-        external_id: "2",
-        id: 2,
-        name: "domain_2",
-        parent_id: null,
-        parents: null,
-        type: null,
-      },
-      {
-        description: "el padre es domain_2",
-        domain_group: null,
-        external_id: "3",
-        id: 3,
-        name: "domain_3",
-        parent_id: 2,
-        parents: null,
-        type: null,
-      },
-      {
-        description: "el padre es domain_2",
-        domain_group: null,
-        external_id: "4",
-        id: 4,
-        name: "domain_4",
-        parent_id: 2,
-        parents: null,
-        type: null,
-      },
-      {
-        description: "No tiene padre",
-        domain_group: null,
-        external_id: "5",
-        id: 5,
-        name: "domain_5",
-        parent_id: null,
-        parents: null,
-        type: null,
-      },
-      {
-        description: "el padre es domain_5",
-        domain_group: null,
-        external_id: "6",
-        id: 6,
-        name: "domain_6",
-        parent_id: 5,
-        parents: null,
-        type: null,
-      },
-      {
-        description: "el padre es domain_6",
-        domain_group: null,
-        external_id: "7",
-        id: 7,
-        name: "domain_7",
-        parent_id: 6,
-        parents: null,
-        type: null,
-      },
-    ],
-  })),
+  useDomains: jest.fn(() => {}),
 }));
 
+const data = [
+  {
+    description: "No tiene padre",
+    domain_group: null,
+    external_id: "1",
+    id: 1,
+    name: "domain_1",
+    parent_id: null,
+    parents: null,
+    type: null,
+  },
+  {
+    description: "No tiene padre",
+    domain_group: null,
+    external_id: "2",
+    id: 2,
+    name: "domain_2",
+    parent_id: null,
+    parents: null,
+    type: null,
+  },
+  {
+    description: "el padre es domain_2",
+    domain_group: null,
+    external_id: "3",
+    id: 3,
+    name: "domain_3",
+    parent_id: 2,
+    parents: null,
+    type: null,
+  },
+  {
+    description: "el padre es domain_2",
+    domain_group: null,
+    external_id: "4",
+    id: 4,
+    name: "domain_4",
+    parent_id: 2,
+    parents: null,
+    type: null,
+  },
+  {
+    description: "No tiene padre",
+    domain_group: null,
+    external_id: "5",
+    id: 5,
+    name: "domain_5",
+    parent_id: null,
+    parents: null,
+    type: null,
+  },
+  {
+    description: "el padre es domain_5",
+    domain_group: null,
+    external_id: "6",
+    id: 6,
+    name: "domain_6",
+    parent_id: 5,
+    parents: null,
+    type: null,
+  },
+  {
+    description: "el padre es domain_6",
+    domain_group: null,
+    external_id: "7",
+    id: 7,
+    name: "domain_7",
+    parent_id: 6,
+    parents: null,
+    type: null,
+  },
+];
 describe("<DomainForm />", () => {
   const onSubmit = jest.fn();
   const domain = { id: 1, name: "nn", external_id: "foo", description: "dd" };
@@ -89,12 +88,14 @@ describe("<DomainForm />", () => {
   const props = { domain, onSubmit, isSubmitting };
 
   it("matches the latest snapshot", async () => {
+    useDomains.mockImplementation(() => ({ data }));
     const rendered = render(<DomainForm {...props} />);
     await waitForLoad(rendered);
     expect(rendered.container).toMatchSnapshot();
   });
 
   it("enables submit when required fields have values", async () => {
+    useDomains.mockImplementation(() => ({ data }));
     const props = { domain: {}, onSubmit: jest.fn() };
     const rendered = render(<DomainForm {...props} />);
     await waitForLoad(rendered);
@@ -121,7 +122,7 @@ describe("<DomainForm />", () => {
   });
 
   it("should generate unique domain group options", async () => {
-    useDomains.mockImplementationOnce(() => ({
+    useDomains.mockImplementation(() => ({
       data: [
         { domain_group: { name: "DomainGroup1" } },
         { domain_group: { name: "DomainGroup2" } },
@@ -149,7 +150,7 @@ describe("<DomainForm />", () => {
   });
 
   it("should generate unique type options", async () => {
-    useDomains.mockImplementationOnce(() => ({
+    useDomains.mockImplementation(() => ({
       data: [
         { type: "Type1" },
         { type: "Type2" },
@@ -172,6 +173,7 @@ describe("<DomainForm />", () => {
   });
 
   it("should generate domain parent options", async () => {
+    useDomains.mockImplementation(() => ({ data }));
     const domain = { id: 2, parentable_ids: [1, 5, 6, 7] };
 
     const rendered = render(

package/src/taxonomy/components/__tests__/Domains.spec.js

@@ -162,8 +162,8 @@ describe("<Domains />", () => {
 
   it("redirects to domains list if domainId is invalid", async () => {
     const mockHistoryPush = jest.fn();
-    useNavigate.mockImplementationOnce(() => mockHistoryPush);
-    useParams.mockImplementationOnce(() => ({ id: 999 }));
+    useNavigate.mockImplementation(() => mockHistoryPush);
+    useParams.mockImplementation(() => ({ id: 999 }));
 
     const rendered = render(<Domains />, renderOpts);
     await waitForLoad(rendered);
@@ -172,14 +172,14 @@ describe("<Domains />", () => {
   });
 
   it("renders EmptyImage when no domains available", async () => {
-    useParams.mockImplementationOnce(() => ({ id: undefined }));
-    useDomains.mockImplementationOnce(() => ({
+    useParams.mockImplementation(() => ({ id: undefined }));
+    useDomains.mockImplementation(() => ({
       data: [],
       actions: {},
       loading: false,
     }));
     const mockHistoryPush = jest.fn();
-    useNavigate.mockImplementationOnce(() => mockHistoryPush);
+    useNavigate.mockImplementation(() => mockHistoryPush);
 
     const rendered = render(<Domains />, renderOpts);
     await waitForLoad(rendered);