Merge pull request #50602 from nextcloud/fix/template-field-title

fix: template field title

apps/files/src/components/TemplateFiller.vue

@@ -4,13 +4,18 @@
 -->
 
 <template>
-	<NcModal>
+	<NcModal label-id="template-field-modal__label">
 		<div class="template-field-modal__content">
 			<form>
-				<h3>{{ t('files', 'Fill template fields') }}</h3>
+				<h3 id="template-field-modal__label">
+					{{ t('files', 'Fill template fields') }}
+				</h3>
 
 				<div v-for="field in fields" :key="field.index">
-					<component :is="getFieldComponent(field.type)" :field="field" @input="trackInput" />
+					<component :is="getFieldComponent(field.type)"
+						v-if="fieldHasLabel(field)"
+						:field="field"
+						@input="trackInput" />
 				</div>
 			</form>
 		</div>
@@ -80,6 +85,9 @@ export default defineComponent({
 
 			return `Template${fieldComponentType}Field`
 		},
+		fieldHasLabel(field) {
+			return field.name || field.alias
+		},
 		async submit() {
 			this.loading = true
 

apps/files/src/components/TemplateFiller/TemplateCheckboxField.vue

@@ -40,7 +40,7 @@ export default defineComponent({
 
 	computed: {
 		fieldLabel() {
-			const label = this.field.name ?? this.field.alias ?? 'Unknown field'
+			const label = this.field.name || this.field.alias
 
 			return label.charAt(0).toUpperCase() + label.slice(1)
 		},

apps/files/src/components/TemplateFiller/TemplateRichTextField.vue

@@ -45,7 +45,7 @@ export default defineComponent({
 
 	computed: {
 		fieldLabel() {
-			const label = this.field.name ?? this.field.alias ?? 'Unknown field'
+			const label = this.field.name || this.field.alias
 
 			return (label.charAt(0).toUpperCase() + label.slice(1))
 		},

dist/4341-4341.js.map.license

@@ -0,0 +1 @@
+4341-4341.js.license

dist/465-465.js.map.license

@@ -1 +0,0 @@
-465-465.js.license