@truedat/dd 7.0.2 → 7.0.4

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@truedat/dd",
-  "version": "7.0.2",
+  "version": "7.0.4",
   "description": "Truedat Web Data Dictionary",
   "sideEffects": false,
   "jsnext:main": "src/index.js",
@@ -34,7 +34,7 @@
     "@testing-library/jest-dom": "^5.16.5",
     "@testing-library/react": "^12.0.0",
     "@testing-library/user-event": "^13.2.1",
-    "@truedat/test": "7.0.2",
+    "@truedat/test": "7.0.4",
     "babel-jest": "^28.1.0",
     "babel-plugin-dynamic-import-node": "^2.3.3",
     "babel-plugin-lodash": "^3.3.4",
@@ -88,9 +88,9 @@
   },
   "dependencies": {
     "@apollo/client": "^3.7.1",
-    "@truedat/auth": "7.0.2",
-    "@truedat/core": "7.0.2",
-    "@truedat/df": "7.0.2",
+    "@truedat/auth": "7.0.4",
+    "@truedat/core": "7.0.4",
+    "@truedat/df": "7.0.4",
     "lodash": "^4.17.21",
     "moment": "^2.29.4",
     "path-to-regexp": "^1.7.0",
@@ -115,5 +115,5 @@
     "react-dom": ">= 16.8.6 < 17",
     "semantic-ui-react": ">= 2.0.3 < 2.2"
   },
-  "gitHead": "09343887ddeed9a87dffd993a089458eb9d06363"
+  "gitHead": "b2c0938fa6ea458e3009c82e4e91b7bc79c8ffed"
 }

package/src/components/GrantRequest.js

@@ -4,7 +4,7 @@ import PropTypes from "prop-types";
 import { connect } from "react-redux";
 import { Link } from "react-router-dom";
 import { linkTo } from "@truedat/core/routes";
-import { FormattedMessage, useIntl } from "react-intl";
+import { useIntl } from "react-intl";
 import { Grid, Segment, List, Label } from "semantic-ui-react";
 import GenericCrumbs from "@truedat/core/components/GenericCrumbs";
 import { getGrantRequestTemplate } from "../selectors";

package/src/components/StructureSelector.js

@@ -1,7 +1,7 @@
 import _ from "lodash/fp";
 import React, { useState, useEffect } from "react";
 import PropTypes from "prop-types";
-import { connect } from "react-redux";
+import { connect, useSelector } from "react-redux";
 import { Segment } from "semantic-ui-react";
 import SearchWidget from "@truedat/core/search/SearchWidget";
 import {
@@ -13,6 +13,8 @@ import {
   useDataStructureFilters,
   useDataStructureSearch,
 } from "../hooks/useStructures";
+import { defaultColumnsForStructureSelector } from "../selectors";
+
 import StructuresSearchResults from "./StructuresSearchResults";
 
 function StructureSelectorContent({
@@ -70,6 +72,13 @@ export const StructureSelector = (props) => {
   const pageSize = _.prop("pageSize")(props);
   const withGrantRequests = _.prop("withGrantRequests")(props);
   const withDataFields = _.prop("withDataFields")(props);
+  const defaultOverwriteColumns = useSelector(
+    defaultColumnsForStructureSelector
+  );
+  const overwriteColumns = _.propOr(
+    defaultOverwriteColumns,
+    "overwriteColumns"
+  )(props);
 
   const enrichSearchPayload = {
     my_grant_requests: !!withGrantRequests,
@@ -90,10 +99,12 @@ export const StructureSelector = (props) => {
     enrichSearchPayload,
   };
 
+  const selectorProps = { ...props, overwriteColumns };
+
   return (
     <>
       <SearchContextProvider {...searchProps}>
-        <StructureSelectorContent {...props} />
+        <StructureSelectorContent {...selectorProps} />
       </SearchContextProvider>
     </>
   );

package/src/components/StructureTabPane.js

@@ -98,9 +98,12 @@ StructureTabPane.propTypes = {
   tabVisibility: PropTypes.object,
 };
 
-const mapStateToProps = (state, { match: { path } }) => {
+const mapStateToProps = (
+  { structureTabsOrder, ...state },
+  { match: { path } }
+) => {
   const tabVisibility = getTabVisibility(state);
-  const activeTab = getActiveTab(tabVisibility, path);
+  const activeTab = getActiveTab(tabVisibility, path, structureTabsOrder);
   return {
     tabVisibility,
     activeTab,

package/src/components/StructureTabPaneRoutes.js

@@ -3,6 +3,8 @@ import { Route, Switch } from "react-router-dom";
 import {
   STRUCTURE,
   STRUCTURE_CHILDREN,
+  STRUCTURE_FIELDS,
+  STRUCTURE_VERSION_FIELDS,
   STRUCTURE_EVENTS,
   STRUCTURE_GRANTS,
   STRUCTURE_IMPACT,
@@ -24,6 +26,7 @@ import StructureTabPane from "./StructureTabPane";
 export const StructureTabPaneRoutes = ({}) => (
   <Switch>
     <Route path={STRUCTURE_VERSION_VERSIONS} component={StructureTabPane} />
+    <Route path={STRUCTURE_VERSION_FIELDS} component={StructureTabPane} />
     <Route path={STRUCTURE_VERSION} component={StructureTabPane} />
     <Route path={STRUCTURE_PROFILE} component={StructureTabPane} />
     <Route path={STRUCTURE_STRUCTURE_LINKS} component={StructureTabPane} />
@@ -36,6 +39,7 @@ export const StructureTabPaneRoutes = ({}) => (
     <Route path={STRUCTURE_MEMBERS} component={StructureTabPane} />
     <Route path={STRUCTURE_RULES} component={StructureTabPane} />
     <Route path={STRUCTURE_CHILDREN} component={StructureTabPane} />
+    <Route path={STRUCTURE_FIELDS} component={StructureTabPane} />
     <Route path={STRUCTURE_PARENTS} component={StructureTabPane} />
     <Route path={STRUCTURE_LINEAGE} component={StructureTabPane} />
     <Route path={STRUCTURE_IMPACT} component={StructureTabPane} />

package/src/components/StructureTabRoutes.js

@@ -3,6 +3,8 @@ import { Route, Switch } from "react-router-dom";
 import {
   STRUCTURE,
   STRUCTURE_CHILDREN,
+  STRUCTURE_FIELDS,
+  STRUCTURE_VERSION_FIELDS,
   STRUCTURE_EVENTS,
   STRUCTURE_GRANTS,
   STRUCTURE_IMPACT,
@@ -24,6 +26,7 @@ import StructureTabs from "./StructureTabs";
 export const StructureTabRoutes = () => (
   <Switch>
     <Route path={STRUCTURE_VERSION_VERSIONS} component={StructureTabs} />
+    <Route path={STRUCTURE_VERSION_FIELDS} component={StructureTabs} />
     <Route path={STRUCTURE_VERSION} component={StructureTabs} />
     <Route path={STRUCTURE_PROFILE} component={StructureTabs} />
     <Route path={STRUCTURE_LINKS} component={StructureTabs} />
@@ -36,6 +39,7 @@ export const StructureTabRoutes = () => (
     <Route path={STRUCTURE_RULES} component={StructureTabs} />
     <Route path={STRUCTURE_LINEAGE} component={StructureTabs} />
     <Route path={STRUCTURE_CHILDREN} component={StructureTabs} />
+    <Route path={STRUCTURE_FIELDS} component={StructureTabs} />
     <Route path={STRUCTURE_PARENTS} component={StructureTabs} />
     <Route path={STRUCTURE_IMPACT} component={StructureTabs} />
     <Route path={STRUCTURE} component={StructureTabs} exact />

package/src/components/StructureTabs.js

@@ -10,10 +10,11 @@ import { linkTo } from "@truedat/core/routes";
 import { getActiveTab } from "../selectors/getActiveTab";
 import { getTabVisibility } from "../selectors/getTabVisibility";
 
-const structureLink = ({ id }, version) =>
-  _.isNil(version)
-    ? linkTo.STRUCTURE({ id })
-    : linkTo.STRUCTURE_VERSION({ id, version });
+const structureFields = ({ id }, version) => {
+  return _.isNil(version)
+    ? linkTo.STRUCTURE_FIELDS({ id })
+    : linkTo.STRUCTURE_VERSION_FIELDS({ id, version });
+};
 
 const versionsLink = ({ id }, version) =>
   _.isNil(version)
@@ -40,6 +41,7 @@ export const StructureTabs = ({
     versions,
   } = {},
   structure,
+  structureTabsOrder,
   version,
 }) => {
   const { formatMessage } = useIntl();
@@ -48,7 +50,7 @@ export const StructureTabs = ({
         {
           active: activeTab === "fields",
           as: Link,
-          to: structureLink(structure, version),
+          to: structureFields(structure, version),
           content: fields ? formatMessage({ id: "tabs.dd.fields" }) : false,
           key: "fields",
         },
@@ -154,6 +156,24 @@ export const StructureTabs = ({
         },
       ])
     : null;
+
+  structureTabsOrder
+    ? items.sort((a, b) => {
+        const aIndex = _.indexOf(a.key)(structureTabsOrder);
+        const bIndex = _.indexOf(b.key)(structureTabsOrder);
+        if (aIndex !== -1 && bIndex !== -1) {
+          return aIndex - bIndex;
+        }
+        if (aIndex !== -1) {
+          return -1;
+        }
+        if (bIndex !== -1) {
+          return 1;
+        }
+        return 0;
+      })
+    : items;
+
   return (
     <>
       <Menu
@@ -171,22 +191,24 @@ export const StructureTabs = ({
 
 StructureTabs.propTypes = {
   structure: PropTypes.object,
+  structureTabsOrder: PropTypes.array,
   version: PropTypes.string,
   activeTab: PropTypes.string,
   tabVisibility: PropTypes.object,
 };
 
-const mapStateToProps = (state, ownProps) => {
+const mapStateToProps = ({ structureTabsOrder, ...state }, ownProps) => {
   const {
     match: { path },
   } = ownProps;
   const { structureVersion: version, structure } = state;
   const tabVisibility = getTabVisibility(state, path);
-  const activeTab = getActiveTab(tabVisibility, path);
+  const activeTab = getActiveTab(tabVisibility, path, structureTabsOrder);
 
   return {
     activeTab,
     structure,
+    structureTabsOrder,
     tabVisibility,
     version,
   };

package/src/components/__tests__/__snapshots__/StructureSelector.spec.js.snap

@@ -142,12 +142,7 @@ exports[`<StructureSelector /> matches the latest snapshot 1`] = `
             <td
               class="structure-cell-overflow"
             >
-              <a
-                href="/structures/1"
-                title="1"
-              >
-                1
-              </a>
+              1
             </td>
             <td
               class=""
@@ -179,12 +174,7 @@ exports[`<StructureSelector /> matches the latest snapshot 1`] = `
             <td
               class="structure-cell-overflow"
             >
-              <a
-                href="/structures/2"
-                title="2"
-              >
-                2
-              </a>
+              2
             </td>
             <td
               class=""

package/src/components/__tests__/__snapshots__/StructureTabRoutes.spec.js.snap

@@ -6,6 +6,10 @@ exports[`<StructureTabRoutes /> matches the latest snapshot 1`] = `
     component={[Function]}
     path="/structures/:id/versions/:version/versions"
   />
+  <Route
+    component={[Function]}
+    path="/structures/:id/versions/:version/fields"
+  />
   <Route
     component={[Function]}
     path="/structures/:id/versions/:version"
@@ -54,6 +58,10 @@ exports[`<StructureTabRoutes /> matches the latest snapshot 1`] = `
     component={[Function]}
     path="/structures/:id/children"
   />
+  <Route
+    component={[Function]}
+    path="/structures/:id/fields"
+  />
   <Route
     component={[Function]}
     path="/structures/:id/parents"

package/src/components/__tests__/__snapshots__/StructureTabs.spec.js.snap

@@ -7,7 +7,7 @@ exports[`<StructureTabs /> matches the latest snapshot: metadata 1`] = `
   >
     <a
       class="item"
-      href="/structures/123/versions/1"
+      href="/structures/123/versions/1/fields"
     >
       Fields
     </a>

package/src/selectors/__tests__/getActiveTab.spec.js

@@ -32,7 +32,7 @@ describe("selectors: defaultTab", () => {
   const versions = truthy;
   const grants = truthy;
 
-  it("should determine value according to precedence", () => {
+  it("should determine value according to precedence without custom order", () => {
     expect(defaultTab({ children, fields })).toBe("fields");
     expect(defaultTab({ children, notes })).toBe("children");
     expect(defaultTab({ notes, parents })).toBe("notes");
@@ -45,6 +45,34 @@ describe("selectors: defaultTab", () => {
     expect(defaultTab({ versions })).toBe("versions");
     expect(defaultTab({})).toBe(undefined);
   });
+
+  it("should determine value according to precedence with custom order", () => {
+    const tabsOrder = [
+      "versions",
+      "links",
+      "roles",
+      "rules",
+      "profile",
+      "parents",
+      "metadata",
+      "grants",
+      "notes",
+      "children",
+      "fields",
+    ];
+    expect(defaultTab({ children, fields }, tabsOrder)).toBe("children");
+    expect(defaultTab({ children, notes }, tabsOrder)).toBe("notes");
+    expect(defaultTab({ notes, parents }, tabsOrder)).toBe("parents");
+    expect(defaultTab({ grants, parents }, tabsOrder)).toBe("parents");
+    expect(defaultTab({ parents, profile }, tabsOrder)).toBe("profile");
+    expect(defaultTab({ profile, rules }, tabsOrder)).toBe("rules");
+    expect(defaultTab({ links, rules }, tabsOrder)).toBe("links");
+    expect(defaultTab({ links, roles }, tabsOrder)).toBe("links");
+    expect(defaultTab({ links, versions }, tabsOrder)).toBe("versions");
+    expect(defaultTab({ versions }, tabsOrder)).toBe("versions");
+    expect(defaultTab({ fields }, tabsOrder)).toBe("fields");
+    expect(defaultTab({}, tabsOrder)).toBe(undefined);
+  });
 });
 
 describe("selectors: getActiveTab", () => {

package/src/selectors/__tests__/structureColumnsSelector.spec.js

@@ -1,4 +1,7 @@
-import { structureColumnsSelector } from "..";
+import {
+  defaultColumnsForStructureSelector,
+  structureColumnsSelector,
+} from "..";
 
 describe("selectors: structureColumnsSelector", () => {
   it("should return default structuresColumns when present", () => {
@@ -28,3 +31,28 @@ describe("selectors: structureColumnsSelector", () => {
     expect(res).toEqual([{ name: "foo" }, { name: "bar" }]);
   });
 });
+
+describe("selectors: defaultColumnsForStructureSelector", () => {
+  it("should replace name from object with decorator ", () => {
+    const structuresColumns = [
+      {
+        name: "name",
+        sort: { name: "name.sort" },
+        fieldDecorator: { foo: "bar" },
+        width: 2,
+      },
+      { name: "group", width: 1 },
+    ];
+    const res = defaultColumnsForStructureSelector({ structuresColumns }, {});
+
+    expect(res).toHaveLength(2);
+    expect(res).toEqual([
+      {
+        name: "name",
+        sort: { name: "name.sort" },
+        width: 2,
+      },
+      { name: "group", width: 1 },
+    ]);
+  });
+});

package/src/selectors/getActiveTab.js

@@ -3,6 +3,8 @@ import {
   STRUCTURE,
   STRUCTURE_VERSION,
   STRUCTURE_CHILDREN,
+  STRUCTURE_FIELDS,
+  STRUCTURE_VERSION_FIELDS,
   STRUCTURE_METADATA,
   STRUCTURE_NOTES,
   STRUCTURE_NOTES_EDIT,
@@ -21,24 +23,30 @@ import {
   STRUCTURE_IMPACT,
 } from "@truedat/core/routes";
 
-export const defaultTab = _.cond([
-  [_.conformsTo({ fields: _.identity }), _.constant("fields")],
-  [_.conformsTo({ children: _.identity }), _.constant("children")],
-  [_.conformsTo({ notes: _.identity }), _.constant("notes")],
-  [_.conformsTo({ grants: _.identity }), _.constant("grants")],
-  [_.conformsTo({ metadata: _.identity }), _.constant("metadata")],
-  [_.conformsTo({ parents: _.identity }), _.constant("parents")],
-  [_.conformsTo({ profile: _.identity }), _.constant("profile")],
-  [_.conformsTo({ rules: _.identity }), _.constant("rules")],
-  [_.conformsTo({ roles: _.identity }), _.constant("roles")],
-  [_.conformsTo({ links: _.identity }), _.constant("links")],
-  [_.conformsTo({ versions: _.identity }), _.constant("versions")],
-]);
+export const defaultTab = (tabVisibility, tabsOrder = []) => {
+  const defaultOrder = [
+    "fields",
+    "children",
+    "notes",
+    "grants",
+    "metadata",
+    "parents",
+    "profile",
+    "rules",
+    "roles",
+    "links",
+    "versions",
+  ];
 
-export const getActiveTab = (tabVisibility, path) => {
+  return _.find((tab) => tabVisibility[tab])([...tabsOrder, ...defaultOrder]);
+};
+
+export const getActiveTab = (tabVisibility, path, tabsOrder) => {
   return _.cond([
-    [_.eq(STRUCTURE), _.constant(defaultTab(tabVisibility))],
-    [_.eq(STRUCTURE_VERSION), _.constant(defaultTab(tabVisibility))],
+    [_.eq(STRUCTURE), _.constant(defaultTab(tabVisibility, tabsOrder))],
+    [_.eq(STRUCTURE_VERSION), _.constant(defaultTab(tabVisibility, tabsOrder))],
+    [_.eq(STRUCTURE_VERSION_FIELDS), _.constant("fields")],
+    [_.eq(STRUCTURE_FIELDS), _.constant("fields")],
     [_.eq(STRUCTURE_CHILDREN), _.constant("children")],
     [_.eq(STRUCTURE_NOTES), _.constant("notes")],
     [_.eq(STRUCTURE_NOTES_EDIT), _.constant("notes")],

package/src/selectors/index.js

@@ -1,6 +1,7 @@
 export {
   defaultStructureColumns,
   structureColumnsSelector,
+  defaultColumnsForStructureSelector,
 } from "./structureColumnsSelector";
 export {
   defaultBusinessConceptsLinksToStructuresColumns,

package/src/selectors/structureColumnsSelector.js

@@ -81,3 +81,14 @@ export const structureColumnsSelector = (state, defaultFilters, ownColumns) =>
         )(columns);
     return res;
   })(state);
+
+const rawName = {
+  name: "name",
+  sort: { name: "name.sort" },
+  width: 2,
+};
+export const defaultColumnsForStructureSelector = (state) =>
+  createSelector(
+    (state) => structureColumnsSelector(state, null, null),
+    _.map((item) => (item?.name === "name" ? rawName : item))
+  )(state);