strapi-plugin-navigation 2.0.5 → 2.0.8

package/admin/src/pages/View/components/NavigationItemForm/index.js

@@ -2,6 +2,7 @@ import React, { useEffect, useMemo, useState, useCallback } from 'react';
 import { debounce, find, get, isEmpty, isEqual, isNil, isString } from 'lodash';
 import PropTypes from 'prop-types';
 import { Formik } from 'formik'
+import slugify from 'slugify';
 
 // Design System
 import { ModalBody } from '@strapi/design-system/ModalLayout';
@@ -10,10 +11,7 @@ import { Grid, GridItem } from '@strapi/design-system/Grid';
 import { Form, GenericInput } from '@strapi/helper-plugin';
 
 import { NavigationItemPopupFooter } from '../NavigationItemPopup/NavigationItemPopupFooter';
-
-
 import { navigationItemAdditionalFields, navigationItemType } from '../../utils/enums';
-import slugify from 'slugify';
 import { extractRelatedItemLabel } from '../../utils/parsers';
 import { form as formDefinition } from './utils/form';
 import { checkFormValidity } from '../../utils/form';
@@ -76,20 +74,19 @@ const NavigationItemForm = ({
   };
 
   const sanitizePayload = (payload = {}) => {
-    const { onItemClick, onItemLevelAddClick, related, relatedType, menuAttached, ...purePayload } = payload;
-    const sanitizedType = purePayload.type || navigationItemType.INTERNAL;
+    const { onItemClick, onItemLevelAddClick, related, relatedType, menuAttached, type, ...purePayload } = payload;
     const relatedId = related
     const relatedCollectionType = relatedType;
     const title = payload.title || relatedSelectOptions.find(v => v.key == relatedId)?.label
     return {
       ...purePayload,
       title,
+      type,
       menuAttached: isNil(menuAttached) ? false : menuAttached,
-      type: sanitizedType,
-      path: sanitizedType === navigationItemType.INTERNAL ? purePayload.path : undefined,
-      externalPath: sanitizedType === navigationItemType.EXTERNAL ? purePayload.externalPath : undefined,
-      related: relatedId,
-      relatedType: relatedCollectionType,
+      path: type !== navigationItemType.EXTERNAL ? purePayload.path : undefined,
+      externalPath: type === navigationItemType.EXTERNAL ? purePayload.externalPath : undefined,
+      related: type === navigationItemType.INTERNAL ? relatedId : undefined,
+      relatedType: type === navigationItemType.INTERNAL ? relatedCollectionType : undefined,
       isSingle: isSingleSelected,
       uiRouterKey: generateUiRouterKey(purePayload.title, relatedId, relatedCollectionType),
     };
@@ -109,11 +106,8 @@ const NavigationItemForm = ({
     }
   };
 
-  const onTypeChange = ({ target: { name, value } }) =>
-    onChange({ target: { name, value: value ? navigationItemType.INTERNAL : navigationItemType.EXTERNAL } });
-
   const onAudienceChange = (value) => {
-    onChange({target: {name: `${inputsPrefix}audience`, value}});
+    onChange({ target: { name: `${inputsPrefix}audience`, value } });
   }
 
   const onChange = ({ target: { name, value } }) => {
@@ -148,6 +142,20 @@ const NavigationItemForm = ({
     [relatedTypeSelectValue, contentTypes],
   );
 
+  const navigationItemTypeOptions = Object.keys(navigationItemType).map(key => {
+    const value = navigationItemType[key].toLowerCase();
+    return {
+      key,
+      value: navigationItemType[key],
+      metadatas: {
+        intlLabel: {
+          id: getTradId(`popup.item.form.type.${value}.label`),
+          defaultMessage: getTradId(`popup.item.form.type.${value}.label`),
+        }
+      }
+    }
+  });
+
   const relatedSelectOptions = contentTypeEntities
     .filter((item) => {
       const usedContentTypeEntitiesOfSameType = usedContentTypeEntities
@@ -178,7 +186,9 @@ const NavigationItemForm = ({
   const isExternal = form.type === navigationItemType.EXTERNAL;
   const pathSourceName = isExternal ? 'externalPath' : 'path';
 
-  const submitDisabled = (form.type !== navigationItemType.EXTERNAL) && isNil(form.related);
+  const submitDisabled =
+    (form.type === navigationItemType.INTERNAL && !isSingleSelected && isNil(get(form, `${inputsPrefix}related`))) ||
+    (form.type === navigationItemType.WRAPPER && isNil(get(form, `${inputsPrefix}title`)));
 
   const debouncedSearch = useCallback(
     debounce(nextValue => setContentTypeSearchQuery(nextValue), 500),
@@ -285,7 +295,21 @@ const NavigationItemForm = ({
                   value={get(form, `${inputsPrefix}title`, '')}
                 />
               </GridItem>
-              <GridItem key={`${inputsPrefix}menuAttached`} col={6} lg={12}>
+              <GridItem key={`${inputsPrefix}type`} col={4} lg={12}>
+                <GenericInput
+                  intlLabel={{
+                    id: getTradId('popup.item.form.type.label'),
+                    defaultMessage: 'Internal link',
+                  }}
+                  name={`${inputsPrefix}type`}
+                  options={navigationItemTypeOptions}
+                  type='select'
+                  error={get(formErrors, `${inputsPrefix}type.id`)}
+                  onChange={onChange}
+                  value={get(form, `${inputsPrefix}type`, '')}
+                />
+              </GridItem>
+              <GridItem key={`${inputsPrefix}menuAttached`} col={4} lg={12}>
                 <GenericInput
                   intlLabel={{
                     id: getTradId('popup.item.form.menuAttached.label'),
@@ -299,19 +323,6 @@ const NavigationItemForm = ({
                   disabled={!(data.isMenuAllowedLevel && data.parentAttachedToMenu)}
                 />
               </GridItem>
-              <GridItem key={`${inputsPrefix}type`} col={6} lg={12}>
-                <GenericInput
-                  intlLabel={{
-                    id: getTradId('popup.item.form.type.label'),
-                    defaultMessage: 'Internal link',
-                  }}
-                  name={`${inputsPrefix}type`}
-                  type='bool'
-                  error={get(formErrors, `${inputsPrefix}type.id`)}
-                  onChange={onTypeChange}
-                  value={get(form, `${inputsPrefix}type`, '') === navigationItemType.INTERNAL}
-                />
-              </GridItem>
               <GridItem key={`${inputsPrefix}path`} col={12}>
                 <GenericInput
                   intlLabel={{
@@ -330,7 +341,7 @@ const NavigationItemForm = ({
                   description={generatePreviewPath()}
                 />
               </GridItem>
-              {!isExternal && (
+              {get(form, `${inputsPrefix}type`) === navigationItemType.INTERNAL && (
                 <>
                   <GridItem col={6} lg={12}>
                     <GenericInput
@@ -402,8 +413,14 @@ const NavigationItemForm = ({
                     label={getMessage('popup.item.form.audience.label')}
                     onChange={onAudienceChange}
                     value={audience}
+                    hint={
+                      !isLoading && isEmpty(audienceOptions)
+                        ? getMessage('popup.item.form.audience.empty', 'There are no more audiences')
+                        : undefined
+                    }
                     multi
                     withTags
+                    disabled={isEmpty(audienceOptions)}
                   >
                     {audienceOptions.map(({ value, label }) => <Option key={value} value={value}>{label}</Option>)}
                   </Select>

package/admin/src/pages/View/components/NavigationItemPopup/NavigationItemPopupHeader.js

@@ -1,16 +1,16 @@
 
 
 import React from 'react';
-import { ButtonText } from '@strapi/design-system/Text';
+import { Typography } from '@strapi/design-system/Typography';
 import { ModalHeader } from '@strapi/design-system/ModalLayout';
 import { getMessage } from '../../../../utils';
 
 export const NavigationItemPopupHeader = ({isNewItem}) => {
 	return (
 		<ModalHeader>
-			<ButtonText textColor="neutral800" as="h2" id="asset-dialog-title">
+			<Typography variant="omega" fontWeight="bold" textColor="neutral800" as="h2" id="asset-dialog-title">
 				{getMessage(`popup.item.header.${isNewItem ? 'new' : 'edit'}`)}
-			</ButtonText>
+			</Typography>
 		</ModalHeader>
 	);
 };

package/admin/src/pages/View/index.js

@@ -120,6 +120,37 @@ const View = () => {
   }, []);
   const filteredList = !isSearchEmpty ? filteredListFactory(changedActiveNavigation.items, (item) => item?.title.includes(searchValue)) : [];
 
+  const changeCollapseItemDeep = (item, collapse) => {
+    if (item.collapsed !== collapse) {
+      return {
+        ...item,
+        collapsed: collapse,
+        updated: true,
+        items: item.items?.map(el => changeCollapseItemDeep(el, collapse))
+      }
+    }
+    return {
+      ...item,
+      items: item.items?.map(el => changeCollapseItemDeep(el, collapse))
+    }
+  }
+
+  const handleCollapseAll = () => {
+    handleChangeNavigationData({
+      ...changedActiveNavigation,
+      items: changedActiveNavigation.items.map(item => changeCollapseItemDeep(item, true))
+    }, true);
+    setStructureChanged(true);
+  }
+
+  const handleExpandAll = () => {
+    handleChangeNavigationData({
+      ...changedActiveNavigation,
+      items: changedActiveNavigation.items.map(item => changeCollapseItemDeep(item, false))
+    }, true);
+    setStructureChanged(true);
+  }
+
   const handleItemReOrder = (item, newOrder) => {
     handleSubmitNavigationItem({
       ...item,
@@ -141,6 +172,14 @@ const View = () => {
     });
   };
 
+  const handleItemToggleCollapse = (item) => {
+    handleSubmitNavigationItem({
+      ...item,
+      collapsed: !item.collapsed,
+      updated: true,
+    });
+  }
+
   const handleItemEdit = (
     item,
     levelPath = '',
@@ -164,6 +203,33 @@ const View = () => {
     setSearchValue('');
   }
 
+  const endActions = [
+    {
+      onClick: handleExpandAll,
+      disabled: isLoadingForSubmit,
+      type: "submit",
+      variant: 'tertiary',
+      tradId: 'header.action.expandAll',
+      margin: '8px',
+    },
+    {
+      onClick: handleCollapseAll,
+      disabled: isLoadingForSubmit,
+      type: "submit",
+      variant: 'tertiary',
+      tradId: 'header.action.collapseAll',
+      margin: '8px',
+    },
+    {
+      onClick: addNewNavigationItem,
+      startIcon: <PlusIcon />,
+      disabled: isLoadingForSubmit,
+      type: "submit",
+      tradId: 'header.action.newItem',
+      margin: '16px',
+    },
+  ]
+
   return (
     <Main labelledBy="title" aria-busy={isLoadingForSubmit}>
       <NavigationHeader
@@ -180,18 +246,15 @@ const View = () => {
           <>
             <NavigationContentHeader
               startActions={<Search value={searchValue} setValue={setSearchValue} />}
-              endActions={<Button
-                onClick={addNewNavigationItem}
-                startIcon={<PlusIcon />}
-                disabled={isLoadingForSubmit}
-                type="submit"
-              >
-                {formatMessage(getTrad('header.action.newItem'))}
-              </Button>}
+              endActions={endActions.map(({ tradId, margin, ...item }, i) =>
+                <Box marginLeft={margin} key={i}>
+                  <Button {...item}> {formatMessage(getTrad(tradId))} </Button>
+                </Box>
+              )}
             />
             {isEmpty(changedActiveNavigation.items || []) && (
               <Flex direction="column" minHeight="400px" justifyContent="center">
-                <Icon as={EmptyDocumentsIcon} width="160px" height="88px" color=""/>
+                <Icon as={EmptyDocumentsIcon} width="160px" height="88px" color="" />
                 <Box padding={4}>
                   <Typography variant="beta" textColor="neutral600">{formatMessage(getTrad('empty'))}</Typography>
                 </Box>
@@ -214,6 +277,7 @@ const View = () => {
                 onItemEdit={handleItemEdit}
                 onItemRestore={handleItemRestore}
                 onItemReOrder={handleItemReOrder}
+                onItemToggleCollapse={handleItemToggleCollapse}
                 displayFlat={!isSearchEmpty}
                 root
                 error={error}

package/admin/src/pages/View/utils/enums.js

@@ -1,6 +1,7 @@
 export const navigationItemType = {
   INTERNAL: "INTERNAL",
   EXTERNAL: "EXTERNAL",
+  WRAPPER: "WRAPPER",
 };
 
 export const navigationItemAdditionalFields = {

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

@@ -24,12 +24,14 @@ export const transformItemToRESTPayload = (
     order,
     audience = [],
     items = [],
+    collapsed,
   } = item;
   const isExternal = type === navigationItemType.EXTERNAL;
+  const isWrapper = type === navigationItemType.WRAPPER;
   const { contentTypes = [] } = config;
 
   const parsedRelated = Number(related);
-  const relatedId = isExternal || isNaN(parsedRelated) ? related?.value || related : parsedRelated;
+  const relatedId = isExternal || isWrapper || isNaN(parsedRelated) ? related?.value || related : parsedRelated;
 
   const relatedContentType = relatedType ?
     find(contentTypes,
@@ -46,13 +48,14 @@ export const transformItemToRESTPayload = (
     removed,
     order,
     uiRouterKey,
+    collapsed,
     menuAttached: itemAttachedToMenu,
     audience: audience.map((audienceItem) =>
       isObject(audienceItem) ? audienceItem.value : audienceItem,
     ),
     path: isExternal ? undefined : path,
     externalPath: isExternal ? externalPath : undefined,
-    related: isExternal
+    related: isExternal || isWrapper
       ? undefined
       : [
         {
@@ -274,7 +277,7 @@ export const usedContentTypes = (items = []) => items.flatMap(
 
 export const isRelationCorrect = ({ related, type }) => {
   const isRelationDefined = !isNil(related);
-  return type === navigationItemType.EXTERNAL || (type === navigationItemType.INTERNAL && isRelationDefined);
+  return type !== navigationItemType.INTERNAL || (type === navigationItemType.INTERNAL && isRelationDefined);
 };
 
 export const isRelationPublished = ({ relatedRef, relatedType = {}, type, isCollection }) => {

package/admin/src/translations/en.json

@@ -3,6 +3,8 @@
 	"header.title": "Navigation",
 	"header.description": "Define your portal navigation",
 	"header.action.newItem": "New Item",
+	"header.action.collapseAll": "Collapse All",
+	"header.action.expandAll": "Expand All",
 	"submit.cta.cancel": "Cancel",
 	"submit.cta.save": "Save",
 	"empty": "Your navigation is empty",
@@ -20,12 +22,14 @@
 	"popup.item.form.externalPath.placeholder": "Link to the external source",
 	"popup.item.form.externalPath.validation.type": "This value is not a proper url.",
 	"popup.item.form.menuAttached.label": "Attach to menu",
-	"popup.item.form.type.label": "Internal link",
+	"popup.item.form.type.label": "Navigation item type",
 	"popup.item.form.type.internal.label": "Internal source",
 	"popup.item.form.type.external.label": "External source",
+	"popup.item.form.type.wrapper.label": "Wrapper element",
 	"popup.item.form.type.external.description": "Output path: {value}",
 	"popup.item.form.audience.label": "Audience",
 	"popup.item.form.audience.placeholder": "Select audience...",
+	"popup.item.form.audience.empty": "There are no more audiences",
 	"popup.item.form.relatedSection.label": "Relation to",
 	"popup.item.form.relatedType.label": "Content Type",
 	"popup.item.form.relatedType.placeholder": "Select content type...",
@@ -81,12 +85,17 @@
 	"pages.settings.form.nameField.label": "Name fields",
 	"pages.settings.form.nameField.placeholder": "Select at least one or leave empty to apply defaults",
 	"pages.settings.form.nameField.hint": "If left empty name field is going to take following ordered fields: \"title\", \"subject\" and \"name\"",
+	"pages.settings.form.nameField.empty": "This content type doesn't have any string attributes",
+	"pages.settings.form.populate.label": "Fields to populate",
+	"pages.settings.form.populate.placeholder": "Select at least one or leave empty to disable populating relation fields",
+	"pages.settings.form.populate.hint": "Selected relation fields will be populated inside API responses",
+	"pages.settings.form.populate.empty": "This content type doesn't have any relation fields",
 	"pages.settings.form.contentTypesSettings.label": "Content types",
 	"pages.settings.form.contentTypesSettings.tooltip": "Custom configuration per content type",
 	"components.navigationItem.action.newItem": "Add nested item",
 	"components.navigationItem.badge.removed": "Removed",
-	"components.navigationItem.badge.draft": "{type}: Draft",
-	"components.navigationItem.badge.published": "{type}: Published",
+	"components.navigationItem.badge.draft": "Draft",
+	"components.navigationItem.badge.published": "Published",
 	"components.confirmation.dialog.button.cancel": "Cancel",
 	"components.confirmation.dialog.button.confirm": "Confirm",
 	"components.confirmation.dialog.description": "Do you want to continue?",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "strapi-plugin-navigation",
-  "version": "2.0.5",
+  "version": "2.0.8",
   "description": "Strapi - Navigation plugin",
   "strapi": {
     "name": "navigation",
@@ -17,9 +17,8 @@
     "test:unit": "jest --verbose --coverage"
   },
   "dependencies": {
-    "@strapi/utils": "^4.1.0",
+    "@strapi/utils": "^4.1.5",
     "uuid": "^8.3.0",
-    "bad-words": "^3.0.3",
     "lodash": "^4.17.11",
     "react": "^16.9.0",
     "react-dom": "^16.9.0",

package/server/bootstrap.js

@@ -42,30 +42,11 @@ module.exports = async ({ strapi }) => {
   }
   
   // Initialize configuration
-  const pluginStore = strapi.store({
-    environment: '',
-    type: 'plugin',
-    name: 'navigation',
-  });
-
-  const config = await pluginStore.get({ key: 'config' });
-  const pluginDefaultConfig = await strapi.plugin('navigation').config
-  const defaultConfigValue = {
-    additionalFields: pluginDefaultConfig('additionalFields'),
-    contentTypes: pluginDefaultConfig('contentTypes'),
-    contentTypesNameFields: pluginDefaultConfig('contentTypesNameFields'),
-    allowedLevels: pluginDefaultConfig('allowedLevels'),
-    gql: pluginDefaultConfig('gql'),
-  }
-  
-  if (!config) {
-    pluginStore.set({
-      key: 'config', value: defaultConfigValue
-    });
-  }
+  const config = await strapi.plugin('navigation').service('navigation').setDefaultConfig()
 
+  // Initialize graphql configuration
   if (strapi.plugin('graphql')) {
     const graphqlConfiguration = require('./graphql')
-    await graphqlConfiguration({ strapi, config: config || defaultConfigValue });
+    await graphqlConfiguration({ strapi, config });
   }
 };

package/server/config/index.js

@@ -3,6 +3,7 @@ module.exports = {
 		additionalFields: [],
 		contentTypes: [],
 		contentTypesNameFields: {},
+		contentTypesPopulate: {},
 		allowedLevels: 2
 	}
 }

package/server/config.js

@@ -3,6 +3,7 @@ module.exports = {
 		additionalFields: [],
 		contentTypes: [],
 		contentTypesNameFields: {},
+		contentTypesPopulate: {},
 		allowedLevels: 2
 	}
 }

package/server/content-types/navigation-item/lifecycle.js

@@ -10,6 +10,7 @@ module.exports = {
   type: {
     INTERNAL: 'INTERNAL',
     EXTERNAL: 'EXTERNAL',
+    WRAPPER: 'WRAPPER',
   },
   additionalFields: {
     AUDIENCE: 'audience',

package/server/content-types/navigation-item/schema.json

@@ -37,7 +37,8 @@
       "type": "enumeration",
       "enum": [
         "INTERNAL",
-        "EXTERNAL"
+        "EXTERNAL",
+        "WRAPPER"
       ],
       "default": "INTERNAL",
       "configurable": false
@@ -65,6 +66,11 @@
       "default": 0,
       "configurable": false
     },
+    "collapsed": {
+      "type": "boolean",
+      "default": false,
+      "configurable": false
+    },
     "related": {
       "type": "relation",
       "relation": "oneToOne",

package/server/graphql/types/content-types-name-fields.js

@@ -1,8 +1,8 @@
-module.exports = ({ nexus }) => nexus.objectType({
+module.exports = ({ nexus, strapi }) => nexus.objectType({
   name: "ContentTypesNameFields",
   async definition(t) {
 		t.nonNull.list.nonNull.string("default")
-		const pluginStore = strapi.store({ type: 'plugin', name: 'navigation' });
+		const pluginStore = await strapi.plugin('navigation').service('navigation').getPluginStore();
 		const config = await pluginStore.get({ key: 'config' });
 		const contentTypesNameFields = config.contentTypesNameFields;
 		Object.keys(contentTypesNameFields || {}).forEach(key => t.nonNull.list.string(key))

package/server/services/navigation.js

@@ -72,10 +72,11 @@ module.exports = ({ strapi }) => {
     // Get plugin config
     async config(viaSettingsPage = false) {
       const { audienceModel, service } = utilsFunctions.extractMeta(strapi.plugins);
-      const pluginStore = await strapi.store({ type: 'plugin', name: 'navigation' });
+      const pluginStore = await strapi.plugin('navigation').service('navigation').getPluginStore()
       const config = await pluginStore.get({ key: 'config' });
       const additionalFields = config.additionalFields;
       const contentTypesNameFields = config.contentTypesNameFields;
+      const contentTypesPopulate = config.contentTypesPopulate;
       const allowedLevels = config.allowedLevels;
       const isGQLPluginEnabled = !isNil(strapi.plugin('graphql'));
 
@@ -86,6 +87,9 @@ module.exports = ({ strapi }) => {
           default: contentTypesNameFieldsDefaults,
           ...(isObject(contentTypesNameFields) ? contentTypesNameFields : {}),
         },
+        contentTypesPopulate: {
+          ...(isObject(contentTypesPopulate) ? contentTypesPopulate : {}),
+        },
         allowedLevels,
         additionalFields,
         isGQLPluginEnabled: viaSettingsPage ? isGQLPluginEnabled : undefined,
@@ -111,28 +115,42 @@ module.exports = ({ strapi }) => {
     },
 
     async updateConfig(newConfig) {
-      const pluginStore = await strapi.store({ type: 'plugin', name: 'navigation' });
+      const pluginStore = await strapi.plugin('navigation').service('navigation').getPluginStore()
       await pluginStore.set({ key: 'config', value: newConfig });
     },
 
+    async getPluginStore() {
+      return await strapi.store({ type: 'plugin', name: 'navigation' });
+    },
+
+    async setDefaultConfig() {
+      const pluginStore = await strapi.plugin('navigation').service('navigation').getPluginStore()
+      const config = await pluginStore.get({ key: 'config' });
+      const pluginDefaultConfig = await strapi.plugin('navigation').config
+
+      // If new value gets introduced to the config it either is read from plugin store or from default plugin config
+      // This is fix for backwards compatibility and migration of config to newer version of the plugin
+      const defaultConfigValue = {
+        additionalFields: get(config, 'additionalFields', pluginDefaultConfig('additionalFields')),
+        contentTypes: get(config, 'contentTypes', pluginDefaultConfig('contentTypes')),
+        contentTypesNameFields: get(config, 'contentTypesNameFields',  pluginDefaultConfig('contentTypesNameFields')),
+        contentTypesPopulate: get(config, 'contentTypesPopulate',  pluginDefaultConfig('contentTypesPopulate')),
+        allowedLevels: get(config, 'allowedLevels',  pluginDefaultConfig('allowedLevels')),
+        gql: get(config, 'gql',  pluginDefaultConfig('gql')),
+      }
+      pluginStore.set({ key: 'config', value: defaultConfigValue });
+
+      return defaultConfigValue;
+    },
+
     async restoreConfig() {
-      const pluginStore = await strapi.store({ type: 'plugin', name: 'navigation' });
-      const defaultConfig = await strapi.plugin('navigation').config
-
-      await pluginStore.delete({ key: 'config' })
-      await pluginStore.set({
-        key: 'config', value: {
-          additionalFields: defaultConfig('additionalFields'),
-          contentTypes: defaultConfig('contentTypes'),
-          contentTypesNameFields: defaultConfig('contentTypesNameFields'),
-          allowedLevels: defaultConfig('allowedLevels'),
-          gql: defaultConfig('gql'),
-        }
-      });
+      const pluginStore = await strapi.plugin('navigation').service('navigation').getPluginStore()
+      await pluginStore.delete({ key: 'config' });
+      await strapi.plugin('navigation').service('navigation').setDefaultConfig();
     },
 
     async configContentTypes() {
-      const pluginStore = strapi.store({ type: 'plugin', name: 'navigation' });
+      const pluginStore = await strapi.plugin('navigation').service('navigation').getPluginStore()
       const config = await pluginStore.get({ key: 'config' });
       const eligibleContentTypes =
         await Promise.all(
@@ -212,6 +230,8 @@ module.exports = ({ strapi }) => {
     },
 
     async getRelatedItems(entityItems) {
+      const pluginStore = await strapi.plugin('navigation').service('navigation').getPluginStore()
+      const config = await pluginStore.get({ key: 'config' });
       const relatedTypes = new Set(entityItems.flatMap((item) => get(item.related, 'related_type')));
       const groupedItems = Array.from(relatedTypes).filter((relatedType) => relatedType).reduce(
         (acc, relatedType) => Object.assign(acc, {
@@ -233,8 +253,9 @@ module.exports = ({ strapi }) => {
                   .query(model)
                   .findMany({
                     where: {
-                      id: { $in: map(related, 'related_id') }
-                    }
+                      id: { $in: map(related, 'related_id') },
+                    },
+                    populate: config.contentTypesPopulate[model] || []
                   });
                 return relationData
                   .flatMap(_ =>
@@ -264,8 +285,12 @@ module.exports = ({ strapi }) => {
     },
 
     async getContentTypeItems(model) {
+      const pluginStore = await strapi.plugin('navigation').service('navigation').getPluginStore()
+      const config = await pluginStore.get({ key: 'config' });
       try {
-        const contentTypeItems = await strapi.query(model).findMany()
+        const contentTypeItems = await strapi.query(model).findMany({
+          populate: config.contentTypesPopulate[model] || []
+        })
         return contentTypeItems;
       } catch (err) {
         return [];

package/server/services/utils/functions.js

@@ -7,7 +7,8 @@ const {
   isString,
   get,
   isNil,
-  isArray
+  isArray,
+  first,
 } = require('lodash');
 
 const { type: itemType } = require('../../content-types/navigation-item/lifecycle');
@@ -213,7 +214,7 @@ module.exports = ({ strapi }) => {
           false;
         return item.type === itemType.INTERNAL ? isRelatedDefinedAndPublished : true;
       }
-      return (item.type === itemType.EXTERNAL) || relatedItem;
+      return (item.type !== itemType.INTERNAL) || relatedItem;
     },
   };
 }