Fix visibility dependencies for table view. Add missing question titles if a field is table.

dmp-frontend/src/app/core/pipes/field-value.pipe.ts

@@ -14,8 +14,6 @@ export class FieldValuePipe implements PipeTransform {
 		let value = controlValue.value;
 		let renderStyle = controlValue.viewStyle?.renderStyle;
 		if (renderStyle && value) {
-			console.log(renderStyle);
-			console.log(value);
 			switch (renderStyle) {
 				case DatasetProfileFieldViewStyle.Currency:
 					if (value) {

dmp-frontend/src/app/ui/admin/dataset-profile/editor/components/composite-field/dataset-profile-editor-composite-field.component.html

@@ -204,7 +204,7 @@
 			</div>
 			<div  [id]="'preview_container'+ form.get('id').value" class="w-100" style="margin-right: -15px; margin-left: -15px;" >
 				<div *ngIf="previewForm && showPreview && firstField?.get('viewStyle').get('renderStyle').value" [@fade-in-fast]>
-					<app-form-section-inner  [form]="previewForm" [tableView]="form.value.multiplicity.tableView">
+					<app-form-section-inner  [form]="previewForm" [tableView]="form.getRawValue().multiplicity?.tableView">
 					</app-form-section-inner>
 				</div>
 			</div>

dmp-frontend/src/app/ui/dataset/dataset-wizard/dataset-wizard.component.ts

@@ -532,7 +532,6 @@ export class DatasetWizardComponent extends CheckDeactivateBaseComponent impleme
 		if (profiledId && profiledId.length > 0) {
 			this.formGroup.removeControl('datasetProfileDefinition');
 			this.getDefinition(profiledId);
-			console.log(this.table0fContents.tocentries.length + 1);
 		}
 	}
 

dmp-frontend/src/app/ui/misc/dataset-description-form/components/form-composite-field-dialog/form-composite-field-dialog.component.html

@@ -5,7 +5,7 @@
 		</div>
 	</div>
 	<div *ngIf="data.formGroup" mat-dialog-content class="row">
-		<app-form-composite-field class="align-self-center col" [form]="data.formGroup" [datasetProfileId]="data.datasetProfileId"
+		<app-form-composite-field class="align-self-center col" [form]="data.formGroup" [datasetProfileId]="data.datasetProfileId" [altVisibilityRulesService]="visibilityRulesService"
 								  [isChild]="false" [showDelete]="false" [showTitle]="false" [placeholderTitle]="true"></app-form-composite-field>
 	</div>
 	<div mat-dialog-actions class="row">

dmp-frontend/src/app/ui/misc/dataset-description-form/components/form-composite-field-dialog/form-composite-field-dialog.component.ts

@@ -8,12 +8,13 @@ import {VisibilityRulesService} from "@app/ui/misc/dataset-description-form/visi
 })
 export class FormCompositeFieldDialogComponent {
 
+	public visibilityRulesService: VisibilityRulesService;
+
 	constructor(
-		private visibilityRulesService: VisibilityRulesService,
 		private dialogRef: MatDialogRef<FormCompositeFieldDialogComponent>,
 		@Inject(MAT_DIALOG_DATA) public data: any
 	) {
-		this.visibilityRulesService.buildVisibilityRules([], this.data.formGroup);
+		this.visibilityRulesService = data.visibilityRulesService;
 	}
 
 	cancel() {

dmp-frontend/src/app/ui/misc/dataset-description-form/components/form-composite-field/form-composite-field.component.html

@@ -37,7 +37,7 @@
 							</a>
 						</div> -->
 					<div class="row">
-						<h5 *ngIf="placeholderTitle" class="col-auto font-weight-bold">{{this.fieldFormGroup.get('data').value.label}}</h5>
+						<h5 *ngIf="placeholderTitle && this.visibilityRulesService.checkElementVisibility(this.fieldFormGroup.get('id').value)" class="col-auto font-weight-bold">{{this.fieldFormGroup.get('data').value.label}}</h5>
 					</div>
 					<app-form-field *ngIf="this.visibilityRulesService.checkElementVisibility(this.fieldFormGroup.get('id').value)" [form]="fieldFormGroup" class="col-12 compositeField" [datasetProfileId]="datasetProfileId" [isChild]="true"></app-form-field>
 					<!-- <div *ngIf="fieldFormGroup" class="col-12">
@@ -60,7 +60,9 @@
 	</div>
 </div>
 <ng-container *ngIf="form && this.visibilityRulesService.checkElementVisibility(this.form.get('id').value) && tableRow">
-	<td *ngFor="let fieldFormGroup of form.get('fields')['controls'];" class="text-wrap">{{fieldFormGroup.getRawValue() | fieldValue | translate}}</td>
+	<ng-container  *ngFor="let fieldFormGroup of form.get('fields')['controls'];">
+		<td *ngIf="this.visibilityRulesService.checkElementVisibility(this.fieldFormGroup.get('id').value)" class="text-wrap">{{fieldFormGroup.getRawValue() | fieldValue | translate}}</td>
+	</ng-container>
 	<td class="actions">
 		<button mat-icon-button type="button" class="deleteBtn btn-sm" (click)="editCompositeFieldInDialog()">
 			<mat-icon>edit</mat-icon>

dmp-frontend/src/app/ui/misc/dataset-description-form/components/form-composite-field/form-composite-field.component.ts

@@ -26,21 +26,23 @@ export class FormCompositeFieldComponent extends BaseComponent {
 	@Input() tableRow: boolean = false;
 	@Input() showTitle: boolean = true;
 	@Input() placeholderTitle: boolean = false;
+	@Input() altVisibilityRulesService: VisibilityRulesService;
 
 	constructor(
 		private dialog: MatDialog,
 		public visibilityRulesService: VisibilityRulesService,
 		private changeDetector: ChangeDetectorRef
-		//private markForConsiderationService: MarkForConsiderationService,
 	) {
 		super();
-		this.visibilityRulesService.getElementVisibilityMapObservable().pipe(takeUntil(this._destroyed)).subscribe(x => {
-			this.changeDetector.markForCheck();
-		});
 	}
 
 	ngOnInit() {
-
+		if(this.altVisibilityRulesService) {
+			this.visibilityRulesService = this.altVisibilityRulesService;
+		}
+		this.visibilityRulesService.getElementVisibilityMapObservable().pipe(takeUntil(this._destroyed)).subscribe(x => {
+			this.changeDetector.markForCheck();
+		});
 		if (this.tocentry) {
 			this.form = this.tocentry.form as FormGroup;
 		}
@@ -62,7 +64,8 @@ export class FormCompositeFieldComponent extends BaseComponent {
 			disableClose: true,
 			data: {
 				formGroup: cloneAbstractControl(this.form),
-				datasetProfileId: this.datasetProfileId
+				datasetProfileId: this.datasetProfileId,
+				visibilityRulesService: this.visibilityRulesService
 			}
 		});
 		dialogRef.afterClosed().pipe(takeUntil(this._destroyed)).subscribe(data => {

dmp-frontend/src/app/ui/misc/dataset-description-form/components/form-section/form-section-inner/form-section-inner.component.html

@@ -8,10 +8,17 @@
 					<app-form-composite-field class="align-self-center col" [form]="compositeFieldFormGroup" [datasetProfileId]="datasetProfileId"
 						[isChild]="false" [showDelete]="(compositeFieldFormGroup.get('multiplicityItems').length) > 0"></app-form-composite-field>
 				</div>
-				<table *ngIf="tableView" class="table table-bordered" style="table-layout: fixed">
+				<ng-container *ngIf="tableView">
+					<div class="row">
+						<div class="col-12">
+							<app-form-composite-title class="row" [form]="compositeFieldFormGroup"></app-form-composite-title>
+						</div>
+					</div>
+					<table class="table table-bordered" style="table-layout: fixed">
 						<tr>
-						<th *ngFor="let fieldFormGroup of compositeFieldFormGroup.get('fields')['controls']; let i = index;"
+							<ng-container *ngFor="let fieldFormGroup of compositeFieldFormGroup.get('fields')['controls']; let i = index;">
-							class="text-truncate" [matTooltip]="fieldFormGroup.get('data').value.label" >{{fieldFormGroup.get('data').value.label}}</th>
+								<th *ngIf="this.visibilityRulesService.checkElementVisibility(fieldFormGroup.get('id').value)" class="text-truncate" [matTooltip]="fieldFormGroup.get('data').value.label" >{{fieldFormGroup.get('data').value.label}}</th>
+							</ng-container>
 							<th class="actions"></th>
 						</tr>
 						<tr app-form-composite-field [form]="compositeFieldFormGroup" [datasetProfileId]="datasetProfileId"
@@ -22,7 +29,7 @@
 								[isChild]="true" [showDelete]="true" [tableRow]="true"></tr>
 						</ng-container>
 						<tr *ngIf="(compositeFieldFormGroup.get('multiplicity').value.max - 1) > (compositeFieldFormGroup.get('multiplicityItems').length)">
-						<td [colSpan]="compositeFieldFormGroup.get('fields')['controls'].length + 1" class="text-center">
+							<td [colSpan]="visibleControls(compositeFieldFormGroup.get('fields')['controls']).length + 1" class="text-center">
 							<span class="pointer d-inline-flex align-items-center" (click)="addMultipleField(i)" >
 								<button mat-icon-button color="primary" [disabled]="compositeFieldFormGroup.disabled">
 									<mat-icon>add_circle</mat-icon>
@@ -33,6 +40,7 @@
 							</td>
 						</tr>
 					</table>
+				</ng-container>
 			</div>
 
 			<div *ngIf="compositeFieldFormGroup && !tableView" class="col-12">

dmp-frontend/src/app/ui/misc/dataset-description-form/components/form-section/form-section-inner/form-section-inner.component.ts

@@ -100,4 +100,7 @@ export class FormSectionInnerComponent extends BaseComponent implements OnInit,
 	// next(compositeField: CompositeField) {
 	// 	this.formFocusService.focusNext(compositeField);
 	// }
+	visibleControls(controls: any[]) {
+		return controls.filter(control => this.visibilityRulesService.checkElementVisibility(control.get('id').value));
+	}
 }

dmp-frontend/src/app/ui/misc/dataset-description-form/components/form-section/form-section.component.html

@@ -93,10 +93,18 @@
 							<app-form-composite-field [tocentry]="fieldsetEntry" class="align-self-center col" [form]="fieldsetEntry.form" [datasetProfileId]="datasetProfileId"
 								[isChild]="false" [showDelete]="(fieldsetEntry.form.get('multiplicityItems').length) > 0"></app-form-composite-field>
 						</div>
-						<table *ngIf="fieldsetEntry.form.get('multiplicity').value.tableView" class="table table-bordered" style="table-layout: fixed">
+						<ng-container *ngIf="fieldsetEntry.form.get('multiplicity').value.tableView">
+							<div class="row">
+								<div class="col-12">
+									<app-form-composite-title [tocentry]="fieldsetEntry" class="row" [form]="form"></app-form-composite-title>
+								</div>
+							</div>
+							<table class="table table-bordered" style="table-layout: fixed">
 								<tr>
-								<th *ngFor="let fieldFormGroup of fieldsetEntry.form.get('fields')['controls']; let i = index;"
+									<ng-container *ngFor="let fieldFormGroup of fieldsetEntry.form.get('fields')['controls']; let i = index;">
+										<th *ngIf="this.visibilityRulesService.checkElementVisibility(fieldFormGroup.get('id').value)"
 											class="text-wrap">{{fieldFormGroup.get('data').value.label}}</th>
+									</ng-container>
 									<th class="actions"></th>
 								</tr>
 								<tr app-form-composite-field [form]="fieldsetEntry.form" [datasetProfileId]="datasetProfileId"
@@ -107,8 +115,8 @@
 											[isChild]="true" [showDelete]="true" [tableRow]="true"></tr>
 								</ng-container>
 								<tr *ngIf="(fieldsetEntry.form.get('multiplicity').value.max - 1) > (fieldsetEntry.form.get('multiplicityItems').length)">
-								<td [colSpan]="fieldsetEntry.form.get('fields')['controls'].length + 1" class="text-center">
+									<td [colSpan]="visibleControls(fieldsetEntry.form.get('fields')['controls']).length + 1" class="text-center">
-							<span class="pointer d-inline-flex align-items-center" (click)="addMultipleField(i)" >
+										<span class="pointer d-inline-flex align-items-center" (click)="addMultipleField(i)">
 											<button mat-icon-button color="primary" [disabled]="fieldsetEntry.form.disabled">
 												<mat-icon>add_circle</mat-icon>
 											</button>
@@ -118,6 +126,7 @@
 									</td>
 								</tr>
 							</table>
+						</ng-container>
 					</div>
 
 					<div *ngIf="fieldsetEntry.form && !fieldsetEntry.form.get('multiplicity').value.tableView" class="col-12">

dmp-frontend/src/app/ui/misc/dataset-description-form/components/form-section/form-section.component.ts

@@ -269,4 +269,8 @@ export class FormSectionComponent extends BaseComponent implements OnInit, OnCha
 		this.panelExpanded = true;
 		this.askedToScroll.emit(id);
 	}
+
+	visibleControls(controls: any[]) {
+		return controls.filter(control => this.visibilityRulesService.checkElementVisibility(control.get('id').value));
+	}
 }

dmp-frontend/src/app/ui/misc/dataset-description-form/visibility-rules/visibility-rules.service.ts

@@ -346,7 +346,6 @@ export class VisibilityRulesService {
 	public addNewRule(rule: Rule, currentVisibility = this.DEFAULTVISIBILITY): void {
 		const targetId = rule.targetField;
 		const sourceId = rule.sourceField;
-		console.log(rule);
 		this.visibilityRuleContext.addToVisibilityRulesContext(rule);