Merge pull request #23 from KelvinTegelaar/main

[pull] main from KelvinTegelaar:main

Author: pull[bot]

generate-placeholders.js

@@ -43,7 +43,7 @@ const pages = [
   { title: "BPA Report Builder", path: "/tenant/tools/bpa-report-builder" },
   { title: "Standards", path: "/tenant/standards" },
   { title: "Edit Standards", path: "/tenant/standards/list-applied-standards" },
-  { title: "List Standards", path: "/tenant/standards/list-standards" },
+  { title: "List Standards", path: "/tenant/standards" },
   { title: "Best Practice Analyser", path: "/tenant/standards/bpa-report" },
   { title: "Domains Analyser", path: "/tenant/standards/domains-analyser" },
   { title: "Conditional Access", path: "/tenant/administration" },

package.json

@@ -1,6 +1,6 @@
 {
   "name": "cipp",
-  "version": "8.8.1",
+  "version": "8.8.2",
   "author": "CIPP Contributors",
   "homepage": "https://cipp.app/",
   "bugs": {

public/version.json

@@ -1,3 +1,3 @@
 {
-  "version": "8.8.1"
+  "version": "8.8.2"
 }

src/components/CippComponents/CippBreadcrumbNav.jsx

@@ -16,7 +16,7 @@ async function loadTabOptions() {
     "/email/administration/exchange-retention",
     "/cipp/custom-data",
     "/cipp/super-admin",
-    "/tenant/standards/list-standards",
+    "/tenant/standards",
     "/tenant/manage",
     "/tenant/administration/applications",
     "/tenant/administration/tenants",
@@ -499,6 +499,46 @@ export const CippBreadcrumbNav = () => {
     return result;
   };
 
+  // Check if a path is valid and return its title from navigation or tabs
+  const getPathInfo = (path) => {
+    if (!path) return { isValid: false, title: null };
+
+    const normalizedPath = path.replace(/\/$/, "");
+
+    // Helper function to recursively search menu items
+    const findInMenu = (items) => {
+      for (const item of items) {
+        if (item.path) {
+          const normalizedItemPath = item.path.replace(/\/$/, "");
+          if (normalizedItemPath === normalizedPath) {
+            return { isValid: true, title: item.title };
+          }
+        }
+        if (item.items && item.items.length > 0) {
+          const found = findInMenu(item.items);
+          if (found.isValid) {
+            return found;
+          }
+        }
+      }
+      return { isValid: false, title: null };
+    };
+
+    // Check if path exists in navigation
+    const menuResult = findInMenu(nativeMenuItems);
+    if (menuResult.isValid) {
+      return menuResult;
+    }
+
+    // Check if path exists in tab options
+    const matchingTab = tabOptions.find((tab) => tab.path.replace(/\/$/, "") === normalizedPath);
+    if (matchingTab) {
+      return { isValid: true, title: matchingTab.title };
+    }
+
+    return { isValid: false, title: null };
+  };
+
   // Handle click for hierarchical breadcrumbs
   const handleHierarchicalClick = (path, query) => {
     if (path) {
@@ -580,6 +620,9 @@ export const CippBreadcrumbNav = () => {
         >
           {breadcrumbs.map((crumb, index) => {
             const isLast = index === breadcrumbs.length - 1;
+            const pathInfo = getPathInfo(crumb.path);
+            // Use title from nav/tabs if available, otherwise use the crumb's title
+            const displayTitle = pathInfo.title || crumb.title;
 
             // Items without paths (headers/groups) - show as text
             if (!crumb.path) {
@@ -590,31 +633,46 @@ export const CippBreadcrumbNav = () => {
                   variant="subtitle2"
                   sx={{ fontWeight: isLast ? 500 : 400 }}
                 >
-                  {crumb.title}
+                  {displayTitle}
                 </Typography>
               );
             }
 
-            // All items with paths are clickable, including the last one
-            return (
-              <Link
-                key={index}
-                component="button"
-                variant="subtitle2"
-                onClick={() => handleHierarchicalClick(crumb.path, crumb.query)}
-                sx={{
-                  textDecoration: "none",
-                  color: isLast ? "text.primary" : "text.secondary",
-                  fontWeight: isLast ? 500 : 400,
-                  "&:hover": {
-                    textDecoration: "underline",
-                    color: "primary.main",
-                  },
-                }}
-              >
-                {crumb.title}
-              </Link>
-            );
+            // Items with valid paths are clickable
+            // Items with invalid paths (fallback) are shown as plain text
+            if (pathInfo.isValid) {
+              return (
+                <Link
+                  key={index}
+                  component="button"
+                  variant="subtitle2"
+                  onClick={() => handleHierarchicalClick(crumb.path, crumb.query)}
+                  sx={{
+                    textDecoration: "none",
+                    color: isLast ? "text.primary" : "text.secondary",
+                    fontWeight: isLast ? 500 : 400,
+                    "&:hover": {
+                      textDecoration: "underline",
+                      color: "primary.main",
+                    },
+                  }}
+                >
+                  {displayTitle}
+                </Link>
+              );
+            } else {
+              // Invalid path - show as text only
+              return (
+                <Typography
+                  key={index}
+                  color={isLast ? "text.primary" : "text.secondary"}
+                  variant="subtitle2"
+                  sx={{ fontWeight: isLast ? 500 : 400 }}
+                >
+                  {displayTitle}
+                </Typography>
+              );
+            }
           })}
         </Breadcrumbs>
       </Box>

src/components/CippComponents/CippCADeployDrawer.jsx

@@ -9,6 +9,7 @@ import CippJsonView from "../CippFormPages/CippJSONView";
 import { CippApiResults } from "./CippApiResults";
 import { useSettings } from "../../hooks/use-settings";
 import { CippFormTenantSelector } from "./CippFormTenantSelector";
+import { CippFormCondition } from "./CippFormCondition";
 
 export const CippCADeployDrawer = ({
   buttonText = "Deploy CA Policy",
@@ -24,6 +25,10 @@ export const CippCADeployDrawer = ({
   const CATemplates = ApiGetCall({ url: "/api/ListCATemplates", queryKey: "CATemplates" });
   const [JSONData, setJSONData] = useState();
   const watcher = useWatch({ control: formControl.control, name: "TemplateList" });
+  const selectedReplaceMode = useWatch({
+    control: formControl.control,
+    name: "replacename",
+  });
 
   // Use external open state if provided, otherwise use internal state
   const drawerVisible = open !== null ? open : internalDrawerVisible;
@@ -199,13 +204,25 @@ export const CippCADeployDrawer = ({
             label="Disable Security Defaults if enabled when creating policy"
             formControl={formControl}
           />
-
-          <CippFormComponent
-            type="switch"
-            name="CreateGroups"
-            label="Create groups if they do not exist"
+          <CippFormCondition
             formControl={formControl}
-          />
+            field="replacename"
+            compareType="is"
+            compareValue="displayName"
+            action="disable"
+          >
+            <CippFormComponent
+              type="switch"
+              name="CreateGroups"
+              label="Create groups if they do not exist"
+              formControl={formControl}
+              helperText={
+                selectedReplaceMode !== "displayName"
+                  ? "Select 'Replace by display name' to create groups specified in the template."
+                  : "Enable this option to create groups that do not exist in the tenant."
+              }
+            />
+          </CippFormCondition>
         </Stack>
       </CippOffCanvas>
     </>

src/components/CippComponents/CippCentralSearch.jsx

@@ -46,7 +46,7 @@ async function loadTabOptions() {
     "/email/administration/exchange-retention",
     "/cipp/custom-data",
     "/cipp/super-admin",
-    "/tenant/standards/list-standards",
+    "/tenant/standards",
     "/tenant/manage",
     "/tenant/administration/applications",
     "/tenant/administration/tenants",

src/components/CippComponents/CippRestoreBackupDrawer.jsx

@@ -13,6 +13,7 @@ import { ApiPostCall } from "../../api/ApiCall";
 export const CippRestoreBackupDrawer = ({
   buttonText = "Restore Backup",
   backupName = null,
+  backupData = null,
   requiredPermissions = [],
   PermissionButton = Button,
   ...props
@@ -85,7 +86,12 @@ export const CippRestoreBackupDrawer = ({
     const values = formControl.getValues();
     const startDate = new Date();
     const unixTime = Math.floor(startDate.getTime() / 1000) - 45;
-    const tenantFilterValue = tenantFilter;
+
+    // If in AllTenants context, use the tenant from the backup data
+    let tenantFilterValue = tenantFilter;
+    if (tenantFilter === "AllTenants" && backupData?.tenantSource) {
+      tenantFilterValue = backupData.tenantSource;
+    }
 
     const shippedValues = {
       TenantFilter: tenantFilterValue,
@@ -202,7 +208,12 @@ export const CippRestoreBackupDrawer = ({
                   queryKey: `BackupList-${tenantFilter}-autocomplete`,
                   labelField: (option) => {
                     const match = option.BackupName.match(/.*_(\d{4}-\d{2}-\d{2})-(\d{2})(\d{2})/);
-                    return match ? `${match[1]} @ ${match[2]}:${match[3]}` : option.BackupName;
+                    const dateTime = match
+                      ? `${match[1]} @ ${match[2]}:${match[3]}`
+                      : option.BackupName;
+                    const tenantDisplay =
+                      tenantFilter === "AllTenants" ? ` (${option.TenantFilter})` : "";
+                    return `${dateTime}${tenantDisplay}`;
                   },
                   valueField: "BackupName",
                   data: {

src/hooks/use-securescore.js

@@ -68,7 +68,7 @@ export function useSecureScore({ waiting = true } = {}) {
           complianceInformation: translation?.complianceInformation,
           actionUrl: remediation
             ? //this needs to be updated to be a direct url to apply this standard.
-              "/tenant/standards/list-standards"
+              "/tenant/standards"
             : translation?.actionUrl,
           remediation: remediation
             ? `1. Enable the CIPP Standard: ${remediation.label}`

src/layouts/HeaderedTabbedLayout.jsx

@@ -115,7 +115,7 @@ export const HeaderedTabbedLayout = (props) => {
               !mdDown && {
                 flexGrow: 1,
                 overflow: "auto",
-                height: "calc(100vh - 30px)",
+                height: "calc(100vh - 350px)",
               }
             }
           >

src/pages/tenant/manage/applied-standards.js

@@ -1027,7 +1027,7 @@ const Page = () => {
       tabOptions={tabOptions}
       title={title}
       subtitle={subtitle}
-      backUrl="/tenant/standards/list-standards"
+      backUrl="/tenant/standards"
       actions={actions}
       actionsData={{}}
       isFetching={comparisonApi.isFetching || templateDetails.isFetching}