Merge branch 'main' into bug/14620-sist-endret-field-is-misleading-fo…

…r-resource-not-checked-in-to-gitea

backend/src/Designer/Infrastructure/GitRepository/AltinnAppGitRepository.cs

@@ -54,10 +54,11 @@ public class AltinnAppGitRepository : AltinnGitRepository
         private static string ProcessDefinitionFilePath => Path.Combine(ProcessDefinitionFolderPath, ProcessDefinitionFilename);
 
         private const string LayoutSettingsSchemaUrl = "https://altinncdn.no/schemas/json/layout/layoutSettings.schema.v1.json";
-
         private const string LayoutSchemaUrl = "https://altinncdn.no/schemas/json/layout/layout.schema.v1.json";
 
         private const string TextResourceFileNamePattern = "resource.??.json";
+        private const string SchemaFilePatternJson = "*.schema.json";
+        private const string SchemaFilePatternXsd = "*.xsd";
 
         public static readonly string InitialLayoutFileName = "Side1.json";
 
@@ -976,6 +977,41 @@ public List<string> GetAllImageFileNames()
             return allFilePaths;
         }
 
+        /// <summary>
+        /// Finds all schema files in App/models directory.
+        /// </summary>
+        public IList<AltinnCoreFile> GetSchemaFiles(bool xsd = false)
+        {
+            string schemaFilesPattern = xsd ? SchemaFilePatternXsd : SchemaFilePatternJson;
+            string schemaFilesPath = Path.Combine(ModelFolderPath, schemaFilesPattern);
+            IEnumerable<string> schemaFiles;
+
+            try
+            {
+                schemaFiles = FindFiles(new[] { schemaFilesPath });
+            }
+            catch (DirectoryNotFoundException)
+            {
+                schemaFiles = new List<string>();
+            }
+
+            var altinnCoreSchemaFiles = MapFilesToAltinnCoreFiles(schemaFiles);
+
+            return altinnCoreSchemaFiles;
+        }
+
+        private List<AltinnCoreFile> MapFilesToAltinnCoreFiles(IEnumerable<string> schemaFiles)
+        {
+            List<AltinnCoreFile> altinnCoreSchemaFiles = new();
+
+            foreach (string file in schemaFiles)
+            {
+                altinnCoreSchemaFiles.Add(AltinnCoreFile.CreateFromPath(file, RepositoryDirectory));
+            }
+
+            return altinnCoreSchemaFiles;
+        }
+
         /// <summary>
         /// Gets the relative path to a json schema model.
         /// </summary>

backend/src/Designer/Infrastructure/GitRepository/AltinnGitRepository.cs

@@ -1,4 +1,3 @@
-using System.Collections.Generic;
 using System.IO;
 using System.Text.Json;
 using System.Text.Json.Serialization;
@@ -23,10 +22,7 @@ namespace Altinn.Studio.Designer.Infrastructure.GitRepository
     /// and not any methods that are specific to App or Datamodels repositories.</remarks>
     public class AltinnGitRepository : GitRepository, IAltinnGitRepository
     {
-        private const string SCHEMA_FILES_PATTERN_JSON = "*.schema.json";
-        private const string SCHEMA_FILES_PATTERN_XSD = "*.xsd";
         private const string STUDIO_SETTINGS_FILEPATH = ".altinnstudio/settings.json";
-        private const string TEXT_FILES_PATTERN_JSON = "*.texts.json";
 
         private AltinnStudioSettings _altinnStudioSettings;
 
@@ -80,19 +76,6 @@ public async Task SaveAltinnStudioSettings(AltinnStudioSettings altinnStudioSett
             await WriteObjectByRelativePathAsync(STUDIO_SETTINGS_FILEPATH, altinnStudioSettings, true);
         }
 
-        /// <summary>
-        /// Finds all schema files regardless of location in repository.
-        /// </summary>
-        public IList<AltinnCoreFile> GetSchemaFiles(bool xsd = false)
-        {
-            string schemaFilesPattern = xsd ? SCHEMA_FILES_PATTERN_XSD : SCHEMA_FILES_PATTERN_JSON;
-            var schemaFiles = FindFiles(new[] { schemaFilesPattern });
-
-            var altinnCoreSchemaFiles = MapFilesToAltinnCoreFiles(schemaFiles);
-
-            return altinnCoreSchemaFiles;
-        }
-
         /// <summary>
         /// Parses the filename and extracts the logical schema name.
         /// </summary>
@@ -242,17 +225,5 @@ private bool IsDatamodelsRepo()
         {
             return Repository.Contains("-datamodels");
         }
-
-        private List<AltinnCoreFile> MapFilesToAltinnCoreFiles(IEnumerable<string> schemaFiles)
-        {
-            List<AltinnCoreFile> altinnCoreSchemaFiles = new();
-
-            foreach (string file in schemaFiles)
-            {
-                altinnCoreSchemaFiles.Add(AltinnCoreFile.CreateFromPath(file, RepositoryDirectory));
-            }
-
-            return altinnCoreSchemaFiles;
-        }
     }
 }

backend/src/Designer/Services/Implementation/SchemaModelService.cs

@@ -82,9 +82,9 @@ public SchemaModelService(
         /// <inheritdoc/>
         public IList<AltinnCoreFile> GetSchemaFiles(AltinnRepoEditingContext altinnRepoEditingContext, bool xsd = false)
         {
-            var altinnGitRepository = _altinnGitRepositoryFactory.GetAltinnGitRepository(altinnRepoEditingContext.Org, altinnRepoEditingContext.Repo, altinnRepoEditingContext.Developer);
+            var altinnAppGitRepository = _altinnGitRepositoryFactory.GetAltinnAppGitRepository(altinnRepoEditingContext.Org, altinnRepoEditingContext.Repo, altinnRepoEditingContext.Developer);
 
-            return altinnGitRepository.GetSchemaFiles(xsd);
+            return altinnAppGitRepository.GetSchemaFiles(xsd);
         }
 
         /// <inheritdoc/>

backend/tests/Designer.Tests/Infrastructure/GitRepository/AltinnAppGitRepositoryTests.cs

@@ -385,6 +385,54 @@ public async Task CreateOrOverwriteOptions_WithAppThatHasOptionLists_ShouldOverw
             Assert.Equal(newOptionsListString, savedOptionsList);
         }
 
+        [Theory]
+        [InlineData("ttd", "apps-test", "testUser", 0)]
+        [InlineData("ttd", "ttd-datamodels", "testUser", 0)]
+        [InlineData("ttd", "hvem-er-hvem", "testUser", 7)]
+        public async Task GetSchemaFiles_FilesExist_ShouldReturnFiles(string org, string repository, string developer, int expectedSchemaFiles)
+        {
+            string targetRepository = TestDataHelper.GenerateTestRepoName();
+            await TestDataHelper.CopyRepositoryForTest(org, repository, developer, targetRepository);
+            AltinnAppGitRepository altinnAppGitRepository = PrepareRepositoryForTest(org, targetRepository, developer);
+
+            var files = altinnAppGitRepository.GetSchemaFiles();
+
+            Assert.Equal(expectedSchemaFiles, files.Count);
+        }
+
+        [Fact]
+        public async Task GetSchemaFiles_FilesExist_ShouldReturnFilesWithCorrectProperties()
+        {
+            string org = "ttd";
+            string repository = "hvem-er-hvem";
+            string developer = "testUser";
+            string targetRepository = TestDataHelper.GenerateTestRepoName();
+
+            await TestDataHelper.CopyRepositoryForTest(org, repository, developer, targetRepository);
+            AltinnAppGitRepository altinnAppGitRepository = PrepareRepositoryForTest(org, targetRepository, developer);
+
+            var file = altinnAppGitRepository.GetSchemaFiles().First(f => f.FileName == "HvemErHvem_ExternalTypes.schema.json");
+
+            Assert.Equal(".json", file.FileType);
+            Assert.Equal(@"/App/models/HvemErHvem_ExternalTypes.schema.json", file.RepositoryRelativeUrl);
+        }
+
+        [Fact]
+        public async Task GetSchemaFiles_FilesExistOutsideModelsFolder_ShouldNotReturnFiles()
+        {
+            string org = "ttd";
+            string repository = "app-with-misplaced-datamodels";
+            string developer = "testUser";
+            string targetRepository = TestDataHelper.GenerateTestRepoName();
+
+            await TestDataHelper.CopyRepositoryForTest(org, repository, developer, targetRepository);
+            AltinnAppGitRepository altinnAppGitRepository = PrepareRepositoryForTest(org, targetRepository, developer);
+
+            var files = altinnAppGitRepository.GetSchemaFiles();
+
+            Assert.Empty(files);
+        }
+
         private static AltinnAppGitRepository PrepareRepositoryForTest(string org, string repository, string developer)
         {
 

backend/tests/Designer.Tests/Infrastructure/GitRepository/AltinnGitRepositoryTests.cs

@@ -69,31 +69,6 @@ public void Constructor_InvalidPathParameters_ShouldThrowException()
             Assert.Throws<ArgumentException>(() => new AltinnGitRepository("ttd", "hvem-er-hvem", "testUser", repositoriesRootDirectory, repositoryDirectory));
         }
 
-        [Theory]
-        [InlineData("ttd", "apps-test", "testUser", 0)]
-        [InlineData("ttd", "ttd-datamodels", "testUser", 0)]
-        [InlineData("ttd", "hvem-er-hvem", "testUser", 7)]
-        public void GetSchemaFiles_FilesExist_ShouldReturnFiles(string org, string repository, string developer, int expectedSchemaFiles)
-        {
-            var repositoriesRootDirectory = TestDataHelper.GetTestDataRepositoriesRootDirectory();
-            var altinnGitRepositoryFactory = new AltinnGitRepositoryFactory(repositoriesRootDirectory);
-
-            var altinnGitRepository = altinnGitRepositoryFactory.GetAltinnGitRepository(org, repository, developer);
-            var files = altinnGitRepository.GetSchemaFiles();
-
-            Assert.Equal(expectedSchemaFiles, files.Count);
-        }
-
-        [Fact]
-        public void GetSchemaFiles_FilesExist_ShouldReturnFilesWithCorrectProperties()
-        {
-            var altinnGitRepository = GetTestRepository("ttd", "hvem-er-hvem", "testUser");
-            var file = altinnGitRepository.GetSchemaFiles().First(f => f.FileName == "HvemErHvem_ExternalTypes.schema.json");
-
-            Assert.Equal(".json", file.FileType);
-            Assert.Equal(@"/App/models/HvemErHvem_ExternalTypes.schema.json", file.RepositoryRelativeUrl);
-        }
-
         [Fact]
         public async Task RepositoryType_SettingsExists_ShouldUseThat()
         {

backend/tests/Designer.Tests/Services/SchemaModelServiceTests.cs

@@ -115,7 +115,7 @@ public async Task DeleteSchema_ModelsRepo_ShouldDelete()
         {
             // Arrange
             var org = "ttd";
-            var sourceRepository = "xyz-datamodels";
+            var sourceRepository = "hvem-er-hvem";
             var developer = "testUser";
             var targetRepository = TestDataHelper.GenerateTestRepoName();
             var editingContext = AltinnRepoEditingContext.FromOrgRepoDeveloper(org, targetRepository, developer);
@@ -125,15 +125,15 @@ public async Task DeleteSchema_ModelsRepo_ShouldDelete()
             {
 
                 var schemaFiles = _schemaModelService.GetSchemaFiles(editingContext);
-                Assert.Equal(6, schemaFiles.Count);
+                Assert.Equal(7, schemaFiles.Count);
 
                 // Act
                 var schemaToDelete = schemaFiles.First(s => s.FileName == "Kursdomene_HvemErHvem_M_2021-04-08_5742_34627_SERES.schema.json");
                 await _schemaModelService.DeleteSchema(editingContext, schemaToDelete.RepositoryRelativeUrl);
 
                 // Assert
                 schemaFiles = _schemaModelService.GetSchemaFiles(editingContext);
-                Assert.Equal(5, schemaFiles.Count);
+                Assert.Equal(6, schemaFiles.Count);
             }
             finally
             {

backend/tests/Designer.Tests/_TestData/Repositories/testUser/ttd/app-with-misplaced-datamodels/.altinnstudio/settings.json

@@ -0,0 +1,4 @@
+{
+  "repotype": "datamodels",
+  "datamodelling.preference": "jsonSchema"
+}

backend/tests/Designer.Tests/_TestData/Repositories/testUser/ttd/app-with-misplaced-datamodels/App/model.schema.json

@@ -0,0 +1,35 @@
+{
+    "$schema": "https://json-schema.org/draft/2020-12/schema",
+    "$id": "http://altinn-repositories:3000/bjosttveit/v4-data/App/models/model.schema.json",
+    "info": {
+        "rootNode": ""
+    },
+    "@xsdNamespaces": {
+        "xsd": "http://www.w3.org/2001/XMLSchema",
+        "xsi": "http://www.w3.org/2001/XMLSchema-instance",
+        "seres": "http://seres.no/xsd/forvaltningsdata"
+    },
+    "@xsdSchemaAttributes": {
+        "AttributeFormDefault": "Unqualified",
+        "ElementFormDefault": "Qualified",
+        "BlockDefault": "None",
+        "FinalDefault": "None"
+    },
+    "@xsdRootElement": "model",
+    "type": "object",
+    "required": [
+        "property1",
+        "property2"
+    ],
+    "properties": {
+        "property1": {
+            "type": "string"
+        },
+        "property2": {
+            "type": "string"
+        },
+        "property3": {
+            "type": "string"
+        }
+    }
+}

backend/tests/Designer.Tests/_TestData/Repositories/testUser/ttd/app-with-misplaced-datamodels/App/model.xsd

@@ -0,0 +1,17 @@
+<?xml version="1.0" encoding="utf-8"?>
+<xsd:schema xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xmlns:seres="http://seres.no/xsd/forvaltningsdata" xmlns:xs="http://www.w3.org/2001/XMLSchema" attributeFormDefault="unqualified" elementFormDefault="qualified" xmlns:xsd="http://www.w3.org/2001/XMLSchema">
+    <xs:annotation>
+        <xs:documentation>
+            <xsd:attribute name="rootNode" fixed="" />
+        </xs:documentation>
+    </xs:annotation>
+    <xs:element name="model">
+        <xs:complexType>
+            <xs:sequence>
+                <xs:element name="property1" type="xs:string" />
+                <xs:element name="property2" type="xs:string" />
+                <xs:element minOccurs="0" name="property3" type="xs:string" />
+            </xs:sequence>
+        </xs:complexType>
+    </xs:element>
+</xsd:schema>

frontend/language/src/nb.json

@@ -729,7 +729,7 @@
   "process_editor.configuration_panel_actions_action_card_custom": "Lag egendefinert handling",
   "process_editor.configuration_panel_actions_action_card_custom_label": "Skriv inn navnet på handlingen du vil lage",
   "process_editor.configuration_panel_actions_action_card_title": "Handling {{actionIndex}}",
-  "process_editor.configuration_panel_actions_action_label": "Handling {{actionIndex}}:",
+  "process_editor.configuration_panel_actions_action_label": "Handling {{actionIndex}}",
   "process_editor.configuration_panel_actions_action_selector_label": "Velg en handling fra listen",
   "process_editor.configuration_panel_actions_action_tab_predefined": "Velg standard handling",
   "process_editor.configuration_panel_actions_action_type_help_text": "Hjelpetekst for å velge handlingstype",
@@ -777,15 +777,15 @@
   "process_editor.configuration_panel_layout_set_name": "Navn på sidegruppe: ",
   "process_editor.configuration_panel_layout_set_name_label": "Navn på sidegruppe",
   "process_editor.configuration_panel_missing_task": "Oppgave",
-  "process_editor.configuration_panel_name_label": "Navn: ",
+  "process_editor.configuration_panel_name_label": "Navn",
   "process_editor.configuration_panel_no_data_model_to_select": "Du må ha tilgjengelige datamodeller du kan knytte til dette steget.",
   "process_editor.configuration_panel_no_data_types_to_sign_to_select": "Du må ha tilgjengelige datatyper som kan signeres.",
   "process_editor.configuration_panel_no_task_message": "Velg en oppgave i diagrammet til venstre for å se detaljer om valgt oppgave.",
   "process_editor.configuration_panel_no_task_title": "Du har ikke valgt noen oppgave",
   "process_editor.configuration_panel_payment_task": "Oppgave: Betaling",
   "process_editor.configuration_panel_policy_title": "Tilganger",
   "process_editor.configuration_panel_set_data_model": "Datamodell: {{dataModelName}}",
-  "process_editor.configuration_panel_set_data_model_label": "Datamodell:",
+  "process_editor.configuration_panel_set_data_model_label": "Datamodell",
   "process_editor.configuration_panel_set_data_model_link": "Legg til datamodell",
   "process_editor.configuration_panel_set_data_types_to_sign": "Datatyper som skal signeres:",
   "process_editor.configuration_panel_set_unique_from_signatures_in_data_types": "Samme person kan ikke signere både denne oppgaven og:",
@@ -1420,11 +1420,13 @@
   "ux_editor.component_properties.optionalIndicator": "Vis valgfri-indikator på ledetekst",
   "ux_editor.component_properties.options": "Alternativer",
   "ux_editor.component_properties.optionsId": "Kodeliste",
+  "ux_editor.component_properties.page": "Side",
   "ux_editor.component_properties.pageBreak": "PDF-innstillinger (pageBreak)",
   "ux_editor.component_properties.pageRef": "Navnet til siden det gjelder (pageRef)",
   "ux_editor.component_properties.pagination": "Sidenummerering",
   "ux_editor.component_properties.position": "Plassering av valuta",
   "ux_editor.component_properties.preselectedOptionIndex": "Angi det valget som skal være forhåndsvalgt.",
+  "ux_editor.component_properties.preselectedOptionIndex_button": "Plassering av forhåndsvalgt verdi (indeks)",
   "ux_editor.component_properties.queryParameters": "Parametere i spørringen",
   "ux_editor.component_properties.readOnly": "Feltet kan kun leses",
   "ux_editor.component_properties.receiver": "Den som mottar skjemaet",
@@ -1452,7 +1454,7 @@
   "ux_editor.component_properties.showIcon": "Vis ikon",
   "ux_editor.component_properties.showLabelsInTable": "Alternativene skal alltid vises i tabeller",
   "ux_editor.component_properties.showPageInAccordion": "Vis side i trekkspilliste",
-  "ux_editor.component_properties.showValidations": "Vis valideringer",
+  "ux_editor.component_properties.showValidations": "Vis valideringstyper",
   "ux_editor.component_properties.simplified": "Forenklet visning",
   "ux_editor.component_properties.size": "Størrelse",
   "ux_editor.component_properties.sortOrder": "Sorteringsrekkefølge",

frontend/libs/studio-components/src/components/StudioTextResourceInput/StudioTextResourceInput.tsx

@@ -27,7 +27,7 @@ type TextResourceInputPropsBase = {
   currentIdClass?: string;
   inputClass?: string;
   onChangeCurrentId: (id: string | null) => void;
-  onChangeTextResource: (textResource: TextResource) => void;
+  onChangeTextResource?: (textResource: TextResource) => void;
   textResources: TextResource[];
   texts: TextResourceInputTexts;
   toggleClass?: string;
@@ -62,7 +62,7 @@ export const StudioTextResourceInput = forwardRef<HTMLInputElement, StudioTextRe
     const handleTextResourceChange = (newTextResource: TextResource): void => {
       const newList = changeTextResourceInList(textResources, newTextResource);
       setTextResources(newList);
-      onChangeTextResource(newTextResource);
+      onChangeTextResource?.(newTextResource);
     };
 
     const rootClass = cn(givenClass, classes.container);

frontend/libs/studio-components/src/components/StudioToggleableTextfield/StudioToggleableTextfield.tsx

@@ -1,7 +1,7 @@
 import React, { forwardRef, useEffect, useState } from 'react';
 import { StudioIconTextfield, type StudioIconTextfieldProps } from '../StudioIconTextfield';
 import { StudioProperty } from '../StudioProperty';
-import { KeyVerticalIcon } from '@studio/icons';
+import { KeyHorizontalIcon } from '@studio/icons';
 import classes from './StudioToggleableTextfield.module.css';
 import type { Override } from '../../types/Override';
 
@@ -18,7 +18,7 @@ export const StudioToggleableTextfield = forwardRef<HTMLDivElement, StudioToggle
     {
       error,
       customValidation,
-      icon = <KeyVerticalIcon />,
+      icon = <KeyHorizontalIcon />,
       label,
       onBlur,
       onChange,