Merge branch 'develop' into feature/html-syntax-highligths

apps/sensenet/cypress/e2e/search/search.cy.ts

@@ -70,4 +70,52 @@ describe('Search', () => {
         .then((fullname) => expect(fullname).to.eq('Business Cat'))
     })
   })
+
+  context('more contentTypes filter', () => {
+    const term = 'admin*'
+    before(() => {
+      cy.login()
+      cy.visit(pathWithQueryParams({ path: '/', newParams: { repoUrl: Cypress.env('repoUrl') } }))
+    })
+    beforeEach(() => {
+      cy.get('[data-test="sensenet-logo"]').click()
+      cy.get('[data-test="search-button"]')
+        .click()
+        .get('[data-test="command-box"] input')
+        .type(term, { delay: 250 })
+        .get('[data-test="search-suggestion-list"] ul')
+        .children()
+        .as('search')
+        .first()
+        .click()
+        .location()
+        .should((loc) => {
+          expect(loc.pathname).to.eq(PATHS.search.appPath)
+          expect(loc.search).to.eq(`?term=${term}`)
+        })
+    })
+
+    it('result contains users and groups', () => {
+      cy.get('[data-test="table-cell-admin"]').should('exist')
+      cy.get('[data-test="table-cell-administrators"]').should('exist')
+      cy.get('[data-test="table-cell-captain-picard"]').should('not.exist')
+    })
+
+    it('the "more" menu contains more than 20 items including "group" and "user"', () => {
+      cy.get('[data-test="more-type-filter-button"]').click()
+      cy.get('[id="more-type-filter"]').contains('li', 'User')
+      cy.get('[id="more-type-filter"]').contains('li', 'Group')
+      cy.get('[data-test="more-menu-item-user"]').parent().children().its('length').should('be.gt', 20)
+      // press <ESC> to hide the menu to unblock the UI
+      cy.get('body').trigger('keydown', { keyCode: 27 })
+    })
+
+    it('result contains one user after  use the filter', () => {
+      cy.get('[data-test="more-type-filter-button"]').click().get('[data-test="more-menu-item-user"]').click()
+      // "MORE" changed to "USER"
+      cy.get('[data-test="more-type-filter-button"]').contains('User')
+      // "Administrators" disappeared
+      cy.get('[data-test="table-cell-administrators"]').should('not.exist')
+    })
+  })
 })

apps/sensenet/cypress/e2e/setup/setup.cy.ts

@@ -42,16 +42,16 @@ describe('Setup', () => {
   it('should open a binary editor with the content of the "settings item" if Edit button is clicked', () => {
     cy.get('[data-test="content-card-documentpreview.settings"]')
       .within(() => {
-        cy.get('[data-test="documentpreview.settings-edit-button"]').click()
+        cy.get('[data-test="documentpreview-edit-button"]').click()
       })
       .get('[data-test="editor-title"]')
       .should('have.text', 'DocumentPreview.settings')
   })
 
   it('should open the document of the selected "settings item" if "Learn more" button is clicked', () => {
     cy.get('[data-test="content-card-documentpreview.settings"]').within(() => {
-      cy.get('[data-test="content-card-learnmore-button"]')
-        .get('a[href="https://docs.sensenet.com/concepts/basics/07-settings#documentpreview-settings"]')
+      cy.get('[data-test="documentpreview-learnmore-button"]')
+        .get('a[href="https://docs.sensenet.com/guides/settings/setup#documentpreview-settings"]')
         .should('have.attr', 'target', '_blank')
     })
   })
@@ -86,6 +86,23 @@ describe('Setup', () => {
       })
   })
 
+  it('check white- and blacklisted buttons', () => {
+    // custom settings can be deleted and never hasn't documentation
+    cy.get(`[data-test="testsettings-delete-button"]`).should('exist')
+    cy.get(`[data-test="testsettings-edit-button"]`).should('exist')
+    cy.get(`[data-test="testsettings-learnmore-button"]`).should('not.exist')
+
+    // indexing is system and documented
+    cy.get(`[data-test="indexing-delete-button"]`).should('not.exist')
+    cy.get(`[data-test="indexing-edit-button"]`).should('exist')
+    cy.get(`[data-test="indexing-learnmore-button"]`).should('exist')
+
+    // logging is system and not documented
+    cy.get(`[data-test="logging-delete-button"]`).should('not.exist')
+    cy.get(`[data-test="logging-edit-button"]`).should('exist')
+    cy.get(`[data-test="logging-learnmore-button"]`).should('not.exist')
+  })
+
   it('should delete a setup file', () => {
     cy.get('[data-test="settings-container"]').then((grid) => {
       cy.scrollToItem({

apps/sensenet/src/components/content-list/contenttype-list.tsx

@@ -1,12 +1,21 @@
 import React, { lazy, useState } from 'react'
 import { PATHS } from '../../application-paths'
 import { Switch } from '@sensenet/controls-react'
+import { useRepository } from '@sensenet/hooks-react'
 
 const ContentComponent = lazy(() => import(/* webpackChunkName: "content" */ '../content'))
 
 const ContentTypeList: React.FC = () => {
+  const repository = useRepository()
   const [showHiddenTypes, setShowHiddenTypes] = useState(false)
+  const categoryField = repository.schemas.getFieldTypeByName('Categories')
+  const isCategoryFieldAvailable = categoryField !== undefined
+
   const renderBeforeGrid = () => {
+    if (!isCategoryFieldAvailable) {
+      return <></>
+    }
+
     return (
       <div style={{ marginTop: '20px', marginBottom: '20px' }}>
         <label htmlFor="showHiddenTypes" style={{ marginRight: '10px' }}>
@@ -23,7 +32,9 @@ const ContentTypeList: React.FC = () => {
   }
 
   const contentTypeQuery =
-    "+TypeIs:'ContentType'" + (!showHiddenTypes ? ' -Categories:*HideByDefault*' : '') + ' .AUTOFILTERS:OFF'
+    "+TypeIs:'ContentType'" +
+    (isCategoryFieldAvailable && !showHiddenTypes ? ' -Categories:*HideByDefault*' : '') +
+    ' .AUTOFILTERS:OFF'
 
   return (
     <ContentComponent

apps/sensenet/src/components/search/filters/type-filter.tsx

@@ -5,7 +5,10 @@ import Image from '@material-ui/icons/Image'
 import InsertDriveFile from '@material-ui/icons/InsertDriveFile'
 import Person from '@material-ui/icons/Person'
 import Search from '@material-ui/icons/Search'
-import React, { useState } from 'react'
+import { ConstantContent } from '@sensenet/client-core'
+import { GenericContent } from '@sensenet/default-content-types'
+import { useRepository } from '@sensenet/hooks-react'
+import React, { useEffect, useState } from 'react'
 import { useSearch } from '../../../context/search'
 import { useLocalization } from '../../../hooks'
 
@@ -75,14 +78,54 @@ const useStyles = makeStyles(() => {
   })
 })
 
+type moreOptionsItem = {
+  name: string
+  type: string
+}
+
 export const TypeFilter = () => {
+  const repo = useRepository()
   const classes = useStyles()
   const localization = useLocalization().search.filters.type
   const [anchorEl, setAnchorEl] = useState<HTMLButtonElement | null>(null)
-
   const searchState = useSearch()
+  const [otherContentTypes, setOtherContentTypes] = useState<moreOptionsItem[]>([])
+
+  useEffect(() => {
+    const ac = new AbortController()
+    const categoryField = repo.schemas.getFieldTypeByName('Categories')
+    const fetchData = async () => {
+      try {
+        if (categoryField) {
+          const response = await repo.loadCollection<GenericContent>({
+            path: ConstantContent.PORTAL_ROOT.Path,
+            oDataOptions: {
+              query: "-Categories:'*HideByDefault*' +TypeIs:'ContentType' .AUTOFILTERS:OFF",
+              select: ['Type', 'DisplayName'],
+              orderby: 'Name',
+            },
+            requestInit: { signal: ac.signal },
+          })
+          const items: moreOptionsItem[] = response.d.results.map((item) => ({
+            name: item.DisplayName ?? item.Name,
+            type: item.Name,
+          }))
+          setOtherContentTypes(items)
+        } else {
+          setOtherContentTypes(moreOptions)
+        }
+      } catch (error) {
+        console.error('Error fetching data:', error)
+      }
+    }
+    fetchData()
+
+    return () => {
+      ac.abort()
+    }
+  }, [repo])
 
-  const [[activeFromMore], othersFromMore] = moreOptions.reduce(
+  const [[activeFromMore], othersFromMore] = otherContentTypes.reduce(
     ([pass, fail], filter) => {
       return filter.name === searchState.filters.type.name ? [[...pass, filter], fail] : [pass, [...fail, filter]]
     },
@@ -108,6 +151,7 @@ export const TypeFilter = () => {
       ))}
 
       <Button
+        data-test="more-type-filter-button"
         aria-controls="more-type-filter"
         aria-haspopup="true"
         variant="outlined"
@@ -116,7 +160,7 @@ export const TypeFilter = () => {
         onClick={(event) => {
           setAnchorEl(event.currentTarget)
         }}>
-        {activeFromMore ? localization[activeFromMore.name as keyof typeof localization] : localization.more}
+        {activeFromMore ? activeFromMore.name : localization.more}
       </Button>
       <Menu
         id="more-type-filter"
@@ -133,18 +177,21 @@ export const TypeFilter = () => {
           vertical: 'top',
           horizontal: 'right',
         }}>
-        {(othersFromMore as Filter[]).map((filter) => (
-          <MenuItem
-            key={filter.name}
-            onClick={() => {
-              setAnchorEl(null)
-              searchState.setFilters((filters) =>
-                filters.type.name === filter.name ? filters : { ...filters, type: filter },
-              )
-            }}>
-            {localization[filter.name as keyof typeof localization]}
-          </MenuItem>
-        ))}
+        {(othersFromMore as Filter[])
+          .sort((a, b) => a.name.localeCompare(b.name))
+          .map((filter) => (
+            <MenuItem
+              key={filter.name}
+              data-test={`more-menu-item-${filter.name.toLowerCase()}`}
+              onClick={() => {
+                setAnchorEl(null)
+                searchState.setFilters((filters) =>
+                  filters.type.name === filter.name ? filters : { ...filters, type: filter },
+                )
+              }}>
+              {filter.name}
+            </MenuItem>
+          ))}
       </Menu>
     </div>
   )

apps/sensenet/src/components/search/search-bar.tsx

@@ -59,15 +59,13 @@ export const SearchBar = () => {
                     className={classes.inputButton}
                     aria-label={localization.clearTerm}
                     title={localization.clearTerm}
-                    onClick={() => null}>
-                    <Cancel
-                      onClick={() => {
-                        if (searchInputRef.current) {
-                          searchInputRef.current.value = ''
-                        }
-                        searchState.setTerm('')
-                      }}
-                    />
+                    onClick={() => {
+                      if (searchInputRef.current) {
+                        searchInputRef.current.value = ''
+                      }
+                      searchState.setTerm('')
+                    }}>
+                    <Cancel />
                   </IconButton>
                 )}
                 <IconButton