58789: Feedback 2019-01-24

resources/i18n/en.json

@@ -181,6 +181,7 @@
         "title": "DSpace Angular :: Metadata Schema Registry",
         "head": "Metadata Schema",
         "description": "This is the metadata schema for \"{{namespace}}\".",
+        "return": "Return",
         "form": {
           "create": "Create metadata field",
           "edit": "Edit metadata field",

src/app/+admin/admin-registries/metadata-registry/metadata-registry.component.html

@@ -48,12 +48,12 @@
 
             </ds-pagination>
 
-            <button type="submit" class="btn btn-primary" (click)="deleteSchemas()">{{'admin.registries.metadata.schemas.table.delete' | translate}}
+            <div *ngIf="(metadataSchemas | async)?.payload?.totalElements == 0" class="alert alert-info w-100 mb-2" role="alert">
-            </button>
+              {{'admin.registries.metadata.schemas.no-items' | translate}}
+            </div>
 
-            <div *ngIf="(metadataSchemas | async)?.payload?.totalElements == 0" class="alert alert-info"
+            <div>
-                 role="alert">
+              <button *ngIf="(metadataSchemas | async)?.payload?.page?.length > 0" type="submit" class="btn btn-danger float-right" (click)="deleteSchemas()">{{'admin.registries.metadata.schemas.table.delete' | translate}}</button>
-                {{'admin.registries.metadata.schemas.no-items' | translate}}
             </div>
 
         </div>

src/app/+admin/admin-registries/metadata-registry/metadata-registry.component.ts

@@ -34,7 +34,7 @@ export class MetadataRegistryComponent {
    */
   config: PaginationComponentOptions = Object.assign(new PaginationComponentOptions(), {
     id: 'registry-metadataschemas-pagination',
-    pageSize: 2
+    pageSize: 25
   });
 
   constructor(private registryService: RegistryService,
@@ -143,6 +143,7 @@ export class MetadataRegistryComponent {
             this.showNotification(false, failedResponses.length);
           }
           this.registryService.deselectAllMetadataSchema();
+          this.registryService.cancelEditMetadataSchema();
           this.forceUpdateSchemas();
         });
       }

src/app/+admin/admin-registries/metadata-registry/metadata-schema-form/metadata-schema-form.component.ts

@@ -1,4 +1,4 @@
-import {Component, EventEmitter, OnInit, Output} from '@angular/core';
+import { Component, EventEmitter, OnDestroy, OnInit, Output } from '@angular/core';
 import {
   DynamicFormControlModel,
   DynamicFormGroupModel,
@@ -20,7 +20,7 @@ import { combineLatest } from 'rxjs/internal/observable/combineLatest';
 /**
  * A form used for creating and editing metadata schemas
  */
-export class MetadataSchemaFormComponent implements OnInit {
+export class MetadataSchemaFormComponent implements OnInit, OnDestroy {
 
   /**
    * A unique id used for ds-form
@@ -147,7 +147,25 @@ export class MetadataSchemaFormComponent implements OnInit {
             this.submitForm.emit(updatedSchema);
           });
         }
+        this.clearFields();
       }
     );
   }
+
+  /**
+   * Reset all input-fields to be empty
+   */
+  clearFields() {
+    this.formGroup.patchValue({
+      prefix: '',
+      namespace: ''
+    });
+  }
+
+  /**
+   * Cancel the current edit when component is destroyed
+   */
+  ngOnDestroy(): void {
+    this.onCancel();
+  }
 }

src/app/+admin/admin-registries/metadata-schema/metadata-field-form/metadata-field-form.component.ts

@@ -1,4 +1,4 @@
-import { Component, EventEmitter, Input, OnInit, Output } from '@angular/core';
+import { Component, EventEmitter, Input, OnDestroy, OnInit, Output } from '@angular/core';
 import { MetadataSchema } from '../../../../core/metadata/metadataschema.model';
 import {
   DynamicFormControlModel,
@@ -22,7 +22,7 @@ import { combineLatest } from 'rxjs/internal/observable/combineLatest';
 /**
  * A form used for creating and editing metadata fields
  */
-export class MetadataFieldFormComponent implements OnInit {
+export class MetadataFieldFormComponent implements OnInit, OnDestroy {
 
   /**
    * A unique id used for ds-form
@@ -175,7 +175,26 @@ export class MetadataFieldFormComponent implements OnInit {
             this.submitForm.emit(updatedField);
           });
         }
+        this.clearFields();
       }
     );
   }
+
+  /**
+   * Reset all input-fields to be empty
+   */
+  clearFields() {
+    this.formGroup.patchValue({
+      element: '',
+      qualifier: '',
+      scopeNote: ''
+    });
+  }
+
+  /**
+   * Cancel the current edit when component is destroyed
+   */
+  ngOnDestroy(): void {
+    this.onCancel();
+  }
 }

src/app/+admin/admin-registries/metadata-schema/metadata-schema.component.html

@@ -47,13 +47,15 @@
         </div>
       </ds-pagination>
 
-      <button type="submit" class="btn btn-primary" (click)="deleteFields()">{{'admin.registries.schema.fields.table.delete' | translate}}
+      <div *ngIf="(metadataFields | async)?.payload?.totalElements == 0" class="alert alert-info w-100 mb-2" role="alert">
-      </button>
-
-      <div *ngIf="(metadataFields | async)?.payload?.totalElements == 0" class="alert alert-info" role="alert">
         {{'admin.registries.schema.fields.no-items' | translate}}
       </div>
 
+      <div>
+        <button [routerLink]="['/admin/registries/metadata']" class="btn btn-primary">{{'admin.registries.schema.return' | translate}}</button>
+        <button *ngIf="(metadataFields | async)?.payload?.page?.length > 0" type="submit" class="btn btn-danger float-right" (click)="deleteFields()">{{'admin.registries.schema.fields.table.delete' | translate}}</button>
+      </div>
+
     </div>
   </div>
 </div>

src/app/+admin/admin-registries/metadata-schema/metadata-schema.component.ts

@@ -175,6 +175,7 @@ export class MetadataSchemaComponent implements OnInit {
             this.showNotification(false, failedResponses.length);
           }
           this.registryService.deselectAllMetadataField();
+          this.registryService.cancelEditMetadataField();
           this.forceUpdateFields();
         });
       }