strapi-plugin-navigation 1.0.4 → 1.1.3

package/README.md

@@ -23,6 +23,11 @@ A plugin for [Strapi Headless CMS](https://github.com/strapi/strapi) that provid
 
 <img src="public/assets/preview.png" alt="UI preview" />
 
+### Versions
+
+- **Stable** - [v1.1.2](https://github.com/VirtusLab-Open-Source/strapi-plugin-navigation)
+- **Beta** - v4 support - [v2.0.0-beta.x](https://github.com/VirtusLab-Open-Source/strapi-plugin-navigation/tree/feat/strapi-v4-support)
+
 ### ⏳ Installation
 
 (Use **yarn** to install this plugin within your Strapi project (recommended). [Install yarn with these docs](https://yarnpkg.com/lang/en/docs/install/).)
@@ -73,40 +78,22 @@ Complete installation requirements are exact same as for Strapi itself and can b
 
 ## Content Type model relation to Navigation Item
 
-To enable Content Type to work with Navigation Item, you've to add following field to your model `*.settings.json`:
-
-```
-    "navigation": {
-      "model": "navigationitem",
-      "plugin": "navigation",
-      "via": "related",
-      "configurable": false,
-      "hidden": true
-    }
-```
-
-inside the `attributes` section like in example below:
-
-```
-    "attributes": {
-        ...,
-        "navigation": {
-            "model": "navigationitem",
-            "plugin": "navigation",
-            "via": "related",
-            "configurable": false,
-            "hidden": true
-        },
-        ...
-    },
+We can define in `config/plugins.js`
+```js
+  navigation: {
+    ...
+    relatedContentTypes: [
+      'application::pages.pages'
+    ],
+    ...
+  },
 ```
 
 ## Configuration
-To setup the plugin properly we recommend to put following snippet as part of `config/custom.js` or `config/<env>/custom.js` file. If you've got already configurations for other plugins stores by this way, use just the `navigation` part within exising `plugins` item.
+To setup the plugin properly we recommend to put following snippet as part of `config/plugins.js` or `config/<env>/plugins.js` file. If you've got already configurations for other plugins stores by this way, use just the `navigation` part within exising `plugins` item.
 
 ```js
     ...
-    plugins: {
       navigation: {
         additionalFields: ['audience'],
         allowedLevels: 2,
@@ -116,7 +103,6 @@ To setup the plugin properly we recommend to put following snippet as part of `c
         },
         gql: { ... }
       },
-    },
     ...
 ```
 

package/__mocks__/helpers/strapi.js

@@ -1,3 +1,4 @@
+const {get} = require('lodash');
 function setupStrapi() {
     Object.defineProperty(global, 'strapi', {
         value: {
@@ -14,6 +15,9 @@ function setupStrapi() {
                       },
                   },
               },
+            get(path, defaultValue) {
+              return get(strapi, path, defaultValue);
+            },
           },
           api: {
               'home-page': {
@@ -40,13 +44,13 @@ function setupStrapi() {
                 modelName: 'pages',
                 associations: [{ model: 'navigationitem' }],
             },
-            'blog-post': {
+            'application::blog-post.blog-post': {
                 ...require('./blog-post.settings.json'),
                 apiName: 'blog-posts',
                 modelName: 'blog-posts',
                 associations: [{ model: 'navigationitem' }],
             },
-            'my-homepage': {
+            'application::my-homepages.my-homepage': {
                 ...require('./my-homepage.settings.json'),
                 apiName: 'my-homepage',
                 modelName: 'my-homepage',
@@ -58,12 +62,12 @@ function setupStrapi() {
                 modelName: 'home-page',
                 associations: [{ model: 'navigationitem' }],
             },
-            'plugin-page': {
+            'plugins::another-plugin.pages': {
                 ...require('./another-plugin/pages.settings.json'),
                 modelName: 'plugin-pages',
                 associations: [{ model: 'navigationitem' }],
             },
-            'plugin-blog-post': {
+            'plugins::another-plugin.blog-post': {
                 ...require('./another-plugin/blog-post.settings.json'),
                 modelName: 'plugin-blog-posts',
                 associations: [{ model: 'navigationitem' }],
@@ -73,7 +77,15 @@ function setupStrapi() {
               navigation: {
                   services: {
                       navigation: jest.fn().mockImplementation(),
-                  }
+                  },
+                relatedContentTypes: [
+                  'application::pages.pages',
+                  'application::blog-post.blog-post',
+                  'application::my-homepages.my-homepage',
+                  'application::page-homes.home-page',
+                  'plugins::another-plugin.pages',
+                  'plugins::another-plugin.blog-post'
+                ]
               },
               anotherPlugin: {
                   models: {

package/admin/src/containers/View/components/NavigationItemForm/utils/form.js

@@ -29,7 +29,7 @@ export const form = {
           is: val => val === navigationItemType.EXTERNAL,
           then: yup.string()
             .required(translatedErrors.required)
-            .matches(/(#.*)|(https?:\/\/(?:www\.|(?!www))[a-zA-Z0-9][a-zA-Z0-9-]+[a-zA-Z0-9]\.[^\s]{2,}|www\.[a-zA-Z0-9][a-zA-Z0-9-]+[a-zA-Z0-9]\.[^\s]{2,}|https?:\/\/(?:www\.|(?!www))[a-zA-Z0-9]+\.[^\s]{2,}|www\.[a-zA-Z0-9]+\.[^\s]{2,})/, { 
+            .matches(/(#.*)|(https?:\/\/(?:www\.|(?!www))[a-zA-Z0-9][a-zA-Z0-9-]+[a-zA-Z0-9]\.[^\s]{2,}|www\.[a-zA-Z0-9][a-zA-Z0-9-]+[a-zA-Z0-9]\.[^\s]{2,}|https?:\/\/(?:www\.|(?!www))[a-zA-Z0-9]+\.[^\s]{2,}|www\.[a-zA-Z0-9]+\.[^\s]{2,}|mailto:.+@(([a-zA-Z0-9]|[a-zA-Z0-9][a-zA-Z0-9-]*[a-zA-Z0-9])\.)+[^.\s]{2,})/, { 
               excludeEmptyString: true,
               message: `${pluginId}.popup.item.form.externalPath.validation.type`,
             }),

package/admin/src/containers/View/utils/parsers.js

@@ -1,5 +1,5 @@
 import { isUuid, uuid } from 'uuidv4';
-import { find, get, isArray, isEmpty, isNil, isNumber, isObject, isString, kebabCase, last, omit, orderBy } from 'lodash';
+import { find, get, isArray, isEmpty, isNil, isNumber, isObject, isString, last, omit, orderBy } from 'lodash';
 import { navigationItemType } from './enums';
 
 export const transformItemToRESTPayload = (
@@ -46,8 +46,7 @@ export const transformItemToRESTPayload = (
     order,
     uiRouterKey,
     menuAttached,
-    audience: audience.map((audienceItem) =>
-      isObject(audienceItem) ? audienceItem.value : audienceItem,
+    audience: audience.map((audienceItem) => isObject(audienceItem) ? audienceItem.value || audienceItem.id : audienceItem,
     ),
     path: isExternal ? undefined : path,
     externalPath: isExternal ? externalPath : undefined,
@@ -65,7 +64,7 @@ export const transformItemToRESTPayload = (
 };
 
 export const transformToRESTPayload = (payload, config = {}) => {
-  const { id, name, visible, items } = payload;
+	const { id, name, visible, items } = payload;
   return {
     id,
     name,
@@ -120,7 +119,7 @@ const linkRelations = (item, config) => {
   const shouldBuildRelated = !relatedRef || (relatedRef && (relatedRef.id !== relatedId));
   if (shouldBuildRelated && !shouldFindRelated) {
     const relatedContentType = find(contentTypes,
-      ct => kebabCase(ct.contentTypeName) === kebabCase(relatedItem.__contentType), {});
+      ct => ct.uid === relatedItem.__contentType, {});
     const { uid, labelSingular, isSingle } = relatedContentType;
     relation = {
       related: relatedItem.id,
@@ -262,7 +261,7 @@ export const usedContentTypes = (items = []) => items.flatMap(
     if (item.relatedRef) {
       return [item.relatedRef, ...used];
     }
-    return used; 
+    return used;
   },
 );
 
@@ -284,11 +283,11 @@ export const isRelationPublished = ({ relatedRef, relatedType = {}, type, isColl
   return true;
 };
 
-export const validateNavigationStructure = (items = []) => 
-  items.map(item => 
-    (item.removed || isRelationCorrect({ 
-      related: item.related, 
+export const validateNavigationStructure = (items = []) =>
+  items.map(item =>
+    (item.removed || isRelationCorrect({
+      related: item.related,
       type: item.type,
-    })) && 
+    })) &&
     validateNavigationStructure(item.items)
-  ).filter(item => !item).length === 0;
+  ).filter(item => !item).length === 0;

package/config/functions/bootstrap.js

@@ -1,26 +1,112 @@
-const { isEmpty } = require("lodash");
+const { isEmpty, get, last } = require('lodash');
+
+const saveJSONParse = (value) => {
+  try {
+    return JSON.parse(value).map((_) => ({ ..._, id: _._id }));
+  } catch (e) {
+    return null;
+  }
+};
+
+const getDefaultConnectionName = (strapi) => strapi.config.get('database.defaultConnection');
+
+const isMongo = (strapi) => {
+  const connectionName = getDefaultConnectionName(strapi);
+  return strapi.config.get(`database.connections.${connectionName}.connector`).includes('mongo');
+};
+
+const getNavigationMorphData = (strapi) => {
+  const connectionName = getDefaultConnectionName(strapi);
+  const { [connectionName]: knex } = strapi.connections;
+  return knex.schema.hasTable('navigations_items_morph').then((exist)=> exist ?  knex('navigations_items_morph').select('*') : []);
+};
+
+const getNavigationItemsModel = (strapi) => strapi.query('navigationitem', 'navigation');
+
+const getRelatedModel = (strapi) => strapi.query('navigations_items_related', 'navigation');
+
+const createRelatedData = (relatedModel, navigationItemsModel, items) => ({
+  field,
+  order,
+  related_id,
+  related_type,
+  navigations_items_id,
+}) => {
+  const item = items.find(item => item.id === navigations_items_id);
+  const modelUID = get(strapi.query(related_type), 'model.uid');
+  if (item && modelUID) {
+    const relatedData = {
+      field,
+      order,
+      related_id,
+      related_type: modelUID,
+      master: get(item.master, 'id', item.master),
+    };
+    return relatedModel.create(relatedData)
+      .then(
+        ({ id }) => navigationItemsModel.update({ id: navigations_items_id }, { related: id }),
+      );
+  }
+  return Promise.resolve();
+};
+
+const migrateNavigationItemsSQL = async (strapi) => {
+  const morphData = await getNavigationMorphData(strapi);
+  if (morphData.length) {
+    const relatedModel = getRelatedModel(strapi);
+    const navigationItemsModel = getNavigationItemsModel(strapi);
+    const items = await navigationItemsModel.find({});
+    await Promise.all(morphData.map(createRelatedData(relatedModel, navigationItemsModel, items)));
+  }
+};
+
+const migrateNavigationItemsMongo = async (strapi) => {
+  const navigationItemsModel = getNavigationItemsModel(strapi);
+  const connectionName = getDefaultConnectionName(strapi);
+  const models = strapi.connections[connectionName].models;
+  const items = (await models.NavigationNavigationitem.find({}))
+    // workaround to change type from object to int
+    .map(_ => ({ ..._.toObject(), related: last(saveJSONParse(get(_.errors, 'related.properties.value', null))) }))
+    .filter(_ => _.related);
+
+  if (items.length) {
+    await Promise.all(items.map(item => {
+      const data = {
+        related_id: item.related.ref,
+        related_type: models[item.related.kind].uid,
+        field: item.related.field,
+        order: 1,
+        master: item.master,
+      };
+      return getRelatedModel(strapi)
+        .create(data)
+        .then(result => navigationItemsModel.update({ id: item.id }, { related: [result.id] }));
+    }));
+
+  }
+};
 
 module.exports = async () => {
   // Check if the plugin users-permissions is installed because the navigation needs it
-  if (Object.keys(strapi.plugins).indexOf("users-permissions") === -1) {
+  if (Object.keys(strapi.plugins).indexOf('users-permissions') === -1) {
     throw new Error(
-      "In order to make the navigation plugin work the users-permissions plugin is required",
+      'In order to make the navigation plugin work the users-permissions plugin is required',
     );
   }
 
   // Add permissions
   const actions = [
     {
-      section: "plugins",
-      displayName: "Access the Navigation",
-      uid: "read",
-      pluginName: "navigation",
+      section: 'plugins',
+      displayName: 'Access the Navigation',
+      uid: 'read',
+      pluginName: 'navigation',
     },
     {
-      section: "plugins",
-      displayName: "Ability to change the Navigation",
-      uid: "update",
-      pluginName: "navigation",
+      section: 'plugins',
+      displayName: 'Ability to change the Navigation',
+      uid: 'update',
+      pluginName: 'navigation',
     },
   ];
 
@@ -36,6 +122,16 @@ module.exports = async () => {
         visible: true,
       });
   }
+  const relatedModel = getRelatedModel(global.strapi);
+  const isMigrated = !!(await relatedModel.count({}));
+  if (!isMigrated) {
+    const isMongoDB = isMongo(global.strapi);
+    if (isMongoDB) {
+      await migrateNavigationItemsMongo(global.strapi);
+    } else {
+      await migrateNavigationItemsSQL(global.strapi);
+    }
+  }
 
   const { actionProvider } = strapi.admin.services.permission;
   await actionProvider.registerMany(actions);

package/config/schema.graphql.js

@@ -1,4 +1,3 @@
-const {get} = require('lodash');
 const NAVIGATION_DATE = `
 # SQL
   created_at: String
@@ -25,12 +24,12 @@ const NAVIGATION = `
 `;
 
 const getContentTypesNamesFields = () => {
-  const contentTypesNameFields = strapi.config.get('custom.plugins.navigation.contentTypesNameFields');
+  const contentTypesNameFields = strapi.config.get('plugins.navigation.contentTypesNameFields');
   return Object.keys(contentTypesNameFields || {}).map(key => `${key}: [String]!`).join('\n');
 };
 
 const getNavigationRelated = () => {
-  const related = strapi.config.get('custom.plugins.navigation.gql.navigationItemRelated');
+  const related = strapi.config.get('plugins.navigation.gql.navigationItemRelated');
   if (related) {
     return related;
   }
@@ -67,7 +66,7 @@ module.exports = {
     parent: Int
     master: Int
     items: [NavigationItem]
-    related: NavigationRelated
+    related: [NavigationRelated]
     audience: [String]
     ${NAVIGATION_DATE}
     ${NAVIGATION_USER}
@@ -149,6 +148,11 @@ module.exports = {
     navigationUpdate(id: String!, navigation: CreateNavigation!): Navigation!
   `,
   resolver: {
+    NavigationRelated: {
+      __resolveType: (data) => {
+        return strapi.contentTypes[data.__contentType]?.globalId
+      }
+    },
     Query: {
       renderNavigation: {
         resolverOf: 'plugins::navigation.navigation.render',

package/controllers/navigation.js

@@ -23,35 +23,33 @@ const errorHandler = (ctx) => (error) => {
   }
   throw error;
 };
+const getService = () => strapi.plugins.navigation.services.navigation;
 
 module.exports = {
-  getService() {
-    return strapi.plugins.navigation.services.navigation;
-  },
   /**
    * Default action.
    *
    * @return {Object}
    */
   async config() {
-    return this.getService().config();
+    return getService().config();
   },
 
   async get() {
-    return this.getService().get();
+    return getService().get();
   },
 
   async getById(ctx) {
     const { params } = ctx;
     const { id } = parseParams(params);
-    return this.getService().getById(id);
+    return getService().getById(id);
   },
 
   async render(ctx) {
     const { params, query = {} } = ctx;
     const { type, menu: menuOnly } = query;
     const { idOrSlug } = parseParams(params);
-    return this.getService().render(
+    return getService().render(
       idOrSlug,
       type,
       menuOnly,
@@ -61,7 +59,7 @@ module.exports = {
     const { params, query = {} } = ctx;
     const { type, menu: menuOnly } = query;
     const { idOrSlug, childUIKey } = parseParams(params);
-    return this.getService().renderChildren(
+    return getService().renderChildren(
       idOrSlug,
       childUIKey,
       type,
@@ -72,14 +70,14 @@ module.exports = {
   post(ctx) {
     const { auditLog } = ctx;
     const { body = {} } = ctx.request;
-    return this.getService().post(body, auditLog);
+    return getService().post(body, auditLog);
   },
 
   put(ctx) {
     const { params, auditLog } = ctx;
     const { id } = parseParams(params);
     const { body = {} } = ctx.request;
-    return this.getService().put(id, body, auditLog)
+    return getService().put(id, body, auditLog)
       .catch(errorHandler(ctx));
   },
 };

package/models/navigation.settings.json

@@ -10,7 +10,7 @@
   },
   "pluginOptions": {
     "content-manager": {
-      "visible": false
+      "visible": true
     },
     "content-type-builder": {
       "visible": false

package/models/navigationItem.js

@@ -1,4 +1,5 @@
-"use strict";
+'use strict';
+const { camelCase } = require('lodash');
 
 /**
  * Read the documentation (https://strapi.io/documentation/3.0.0-beta.x/concepts/models.html#life-cycle-callbacks)
@@ -7,10 +8,27 @@
 
 module.exports = {
   type: {
-    INTERNAL: "INTERNAL",
-    EXTERNAL: "EXTERNAL",
+    INTERNAL: 'INTERNAL',
+    EXTERNAL: 'EXTERNAL',
   },
   additionalFields: {
     AUDIENCE: 'audience',
   },
+  lifecycles: {
+    afterFind(results) {
+      results.forEach(_ => {
+        _?.related.forEach(entity => {
+          for (const [key, value] of Object.entries(entity)) {
+            const newKey = camelCase(key);
+            if (value) {
+              entity[newKey] = value;
+            }
+            if (newKey !== key) {
+              delete entity[key];
+            }
+          }
+        });
+      });
+    },
+  },
 };

package/models/navigationItem.settings.json

@@ -15,17 +15,28 @@
     },
     "content-type-builder": {
       "visible": false
+    },
+    "i18n": {
+      "localized": false
     }
   },
   "attributes": {
     "title": {
       "type": "text",
       "configurable": false,
-      "required": true
+      "required": true,
+      "pluginOptions": {
+        "i18n": {
+          "localized": false
+        }
+      }
     },
     "type": {
       "type": "enumeration",
-      "enum": ["INTERNAL", "EXTERNAL"],
+      "enum": [
+        "INTERNAL",
+        "EXTERNAL"
+      ],
       "default": "INTERNAL",
       "configurable": false
     },
@@ -53,8 +64,8 @@
       "configurable": false
     },
     "related": {
-      "collection": "*",
-      "filter": "field",
+      "collection": "navigations_items_related",
+      "plugin": "navigation",
       "configurable": false
     },
     "parent": {

package/models/navigations_items_related.js

@@ -0,0 +1,19 @@
+const { camelCase } = require('lodash');
+module.exports = {
+  lifecycles: {
+    afterFind(results) {
+      results
+        .forEach(entity => {
+          for (const [key, value] of Object.entries(entity)) {
+            const newKey = camelCase(key);
+            if (value) {
+              entity[newKey] = value;
+            }
+            if (newKey !== key) {
+              delete entity[key];
+            }
+          }
+        });
+    },
+  },
+};

package/models/navigations_items_related.settings.json

@@ -0,0 +1,45 @@
+{
+  "collectionName": "navigations_items_related",
+  "info": {
+    "name": "navigations_items_related",
+    "description": ""
+  },
+  "options": {
+    "increments": true,
+    "timestamps": false,
+    "populateCreatorFields": false
+  },
+  "pluginOptions": {
+    "content-manager": {
+      "visible": false
+    },
+    "content-type-builder": {
+      "visible": false
+    },
+    "i18n": {
+      "localized": false
+    }
+  },
+  "attributes": {
+    "related_id": {
+      "type": "string",
+      "required": true
+    },
+    "related_type": {
+      "type": "string",
+      "required": true
+    },
+    "field": {
+      "type": "string",
+      "required": true
+    },
+    "order": {
+      "type": "integer",
+      "required": true
+    },
+    "master": {
+      "type": "string",
+      "required": true
+    }
+  }
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "strapi-plugin-navigation",
-  "version": "1.0.4",
+  "version": "1.1.3",
   "description": "Strapi - Navigation plugin",
   "strapi": {
     "name": "Navigation",