Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

refactor(value): remove old date value components (DEV-633) #684

Merged
merged 7 commits into from Mar 18, 2022
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
18 changes: 4 additions & 14 deletions src/app/app.module.ts
Expand Up @@ -45,6 +45,7 @@ import { KnoraDatePipe } from './main/pipes/formatting/knoradate.pipe';
import { SplitPipe } from './main/pipes/split.pipe';
import { LinkifyPipe } from './main/pipes/string-transformation/linkify.pipe';
import { StringifyStringLiteralPipe } from './main/pipes/string-transformation/stringify-string-literal.pipe';
import { TitleFromCamelCasePipe } from './main/pipes/string-transformation/title-from-camel-case.pipe';
import { TruncatePipe } from './main/pipes/string-transformation/truncate.pipe';
import { SelectLanguageComponent } from './main/select-language/select-language.component';
import { DatadogRumService } from './main/services/datadog-rum.service';
Expand Down Expand Up @@ -111,10 +112,8 @@ import { ResourceComponent } from './workspace/resource/resource.component';
import { BooleanValueComponent } from './workspace/resource/values/boolean-value/boolean-value.component';
import { ColorPickerComponent } from './workspace/resource/values/color-value/color-picker/color-picker.component';
import { ColorValueComponent } from './workspace/resource/values/color-value/color-value.component';
import { CalendarHeaderComponent } from './workspace/resource/values/date-value/calendar-header/calendar-header.component';
import { DateEditComponent } from './workspace/resource/values/date-value/date-input-text/date-edit/date-edit.component';
import { DateInputTextComponent } from './workspace/resource/values/date-value/date-input-text/date-input-text.component';
import { DateInputComponent } from './workspace/resource/values/date-value/date-input/date-input.component';
import { DatePickerComponent } from './workspace/resource/values/date-value/date-picker/date-picker.component';
import { DateValueHandlerComponent } from './workspace/resource/values/date-value/date-value-handler/date-value-handler.component';
import { DateValueComponent } from './workspace/resource/values/date-value/date-value.component';
import { DecimalValueComponent } from './workspace/resource/values/decimal-value/decimal-value.component';
import { GeonameValueComponent } from './workspace/resource/values/geoname-value/geoname-value.component';
Expand All @@ -131,9 +130,6 @@ import { TextValueAsXMLComponent } from './workspace/resource/values/text-value/
import { TimeInputComponent } from './workspace/resource/values/time-value/time-input/time-input.component';
import { TimeValueComponent } from './workspace/resource/values/time-value/time-value.component';
import { UriValueComponent } from './workspace/resource/values/uri-value/uri-value.component';
import { DatePickerComponent } from './workspace/resource/values/yet-another-date-value/date-picker/date-picker.component';
import { DateValueHandlerComponent } from './workspace/resource/values/yet-another-date-value/date-value-handler/date-value-handler.component';
import { YetAnotherDateValueComponent } from './workspace/resource/values/yet-another-date-value/yet-another-date-value.component';
import { ListViewComponent } from './workspace/results/list-view/list-view.component';
import { ResourceGridComponent } from './workspace/results/list-view/resource-grid/resource-grid.component';
import { ResourceListComponent } from './workspace/results/list-view/resource-list/resource-list.component';
Expand All @@ -157,7 +153,6 @@ import { SearchSelectOntologyComponent } from './workspace/search/advanced-searc
import { ExpertSearchComponent } from './workspace/search/expert-search/expert-search.component';
import { FulltextSearchComponent } from './workspace/search/fulltext-search/fulltext-search.component';
import { SearchPanelComponent } from './workspace/search/search-panel/search-panel.component';
import { TitleFromCamelCasePipe } from './main/pipes/string-transformation/title-from-camel-case.pipe';

// translate: AoT requires an exported function for factories
export function httpLoaderFactory(httpClient: HttpClient) {
Expand All @@ -178,7 +173,6 @@ export function httpLoaderFactory(httpClient: HttpClient) {
AudioComponent,
BoardComponent,
BooleanValueComponent,
CalendarHeaderComponent,
CollaborationComponent,
CollectionListComponent,
ColorPickerComponent,
Expand All @@ -189,9 +183,6 @@ export function httpLoaderFactory(httpClient: HttpClient) {
CookiePolicyComponent,
CreateLinkResourceComponent,
DashboardComponent,
DateEditComponent,
DateInputComponent,
DateInputTextComponent,
DatePickerComponent,
DateValueComponent,
DateValueHandlerComponent,
Expand Down Expand Up @@ -294,6 +285,7 @@ export function httpLoaderFactory(httpClient: HttpClient) {
TextValueHtmlLinkDirective,
TimeInputComponent,
TimeValueComponent,
TitleFromCamelCasePipe,
TruncatePipe,
UploadComponent,
UriValueComponent,
Expand All @@ -302,8 +294,6 @@ export function httpLoaderFactory(httpClient: HttpClient) {
UserMenuComponent,
UsersComponent,
UsersListComponent,
YetAnotherDateValueComponent,
TitleFromCamelCasePipe
],
imports: [
AngularSplitModule.forRoot(),
Expand Down
Expand Up @@ -121,7 +121,7 @@ export class DefaultProperties {
{
icon: 'access_time',
label: 'Timestamp',
description: 'A single timestamp on a timeline',
description: 'A single timestamp (date and time) on a timeline',
subPropOf: Constants.HasValue,
objectType: Constants.TimeValue,
guiEle: Constants.GuiTimeStamp,
Expand Down
Expand Up @@ -3,7 +3,7 @@
<div class="form-content">
<mat-form-field class="large-field ontology-name">
<input matInput class="ontology-name" autocomplete="off" [maxlength]="nameMaxLength" [formControl]="ontologyForm.controls['name']"
[placeholder]="'Set a unique name *'">
[placeholder]="'Set a unique name'">
<mat-icon matSuffix [matTooltip]="ontologyNameInfo" matTooltipPosition="above" matTooltipClass="multi-line-tooltip">help</mat-icon>
<mat-hint *ngIf="formErrors.name" class="medium-field">
{{ formErrors.name }}
Expand All @@ -13,7 +13,7 @@

<mat-form-field class="large-field ontology-label">
<span *ngIf="!iri" matPrefix>{{project.shortname}}:&nbsp;</span>
<input matInput [formControl]="ontologyForm.controls['label']" placeholder="Label *"
<input matInput [formControl]="ontologyForm.controls['label']" placeholder="Label"
[value]="ontologyLabel">
<mat-hint *ngIf="formErrors.label">
{{ formErrors.label }}
Expand Down
Expand Up @@ -186,6 +186,7 @@ export class OntologyFormComponent implements OnInit {
label: new FormControl({
value: this.ontologyLabel, disabled: false
}, [
Validators.required,
Validators.minLength(this.nameMinLength)
]),
comment: new FormControl({
Expand Down
Expand Up @@ -41,7 +41,7 @@
<span matPrefix class="ontology-prefix-icon">
<mat-icon>fingerprint</mat-icon>&nbsp;
</span>
<mat-label>Property name *</mat-label>
<mat-label>Property name</mat-label>
<input matInput formControlName="name" >
<mat-hint class="ontology-error-with-prefix" *ngIf="formErrors.name">
{{formErrors.name}}
Expand Down
Expand Up @@ -424,6 +424,7 @@ export class PropertyFormComponent implements OnInit {

this._dspApiConnection.v2.onto.updateResourceProperty(onto4Comment).subscribe(
(classCommentResponse: ResourcePropertyDefinitionWithAllLanguages) => {
console.log(classCommentResponse);
this.lastModificationDate = classCommentResponse.lastModificationDate;

if (!this.unsupportedPropertyType) {
Expand Down Expand Up @@ -491,7 +492,7 @@ export class PropertyFormComponent implements OnInit {
const newResProp = new CreateResourceProperty();
newResProp.name = this.propertyForm.controls['name'].value;
newResProp.label = this.labels;
newResProp.comment = (this.comments.length ? this.comments : this.labels);
newResProp.comment = this.comments;
const guiAttr = this.propertyForm.controls['guiAttr'].value;
if (guiAttr) {
newResProp.guiAttributes = this.setGuiAttribute(guiAttr);
Expand Down
Expand Up @@ -10,7 +10,7 @@
<span matPrefix class="ontology-prefix-icon">
<mat-icon>fingerprint</mat-icon>&nbsp;
</span>
<mat-label>Class name *</mat-label>
<mat-label>Class name</mat-label>
<input matInput formControlName="name" >
<mat-hint class="ontology-error-with-prefix" *ngIf="formErrors.name">
{{formErrors.name}}
Expand All @@ -34,15 +34,12 @@
<!-- description -->
<div class="large-field string-literal-container more-space-top">
<app-string-literal-input
[placeholder]="'Comment *'"
[placeholder]="'Comment'"
[value]="resourceClassComments"
[textarea]="true"
(touched)="resourceClassCommentsTouched = $event"
(dataChanged)="handleData($event, 'comment')">
</app-string-literal-input>
<mat-hint class="string-literal-error" *ngIf="formErrors.comment">
{{ formErrors.comment }}
</mat-hint>
</div>
</div>

Expand All @@ -56,12 +53,12 @@
<span class="fill-remaining-space"></span>
<span>
<button *ngIf="!edit" mat-raised-button type="button" color="primary"
[disabled]="!resourceClassLabels.length || !resourceClassComments.length"
[disabled]="!resourceClassLabels.length || (resourceClassForm.enabled && !resourceClassForm.valid)"
(click)="submitData()">
Submit
</button>
<button *ngIf="edit" mat-raised-button type="button" color="primary"
[disabled]="!resourceClassLabels.length || !resourceClassComments.length"
[disabled]="!resourceClassLabels.length || (resourceClassForm.enabled && !resourceClassForm.valid)"
(click)="submitData()">
<app-progress-indicator [color]="'white'" [status]="0" *ngIf="loading" class="submit-progress">
</app-progress-indicator>
Expand Down
Expand Up @@ -103,8 +103,7 @@ export class ResourceClassFormComponent implements OnInit, AfterViewChecked {
// form errors on the following fields:
formErrors = {
'name': '',
'label': '',
'comment': ''
'label': ''
};

// in case of form error: show message
Expand All @@ -116,9 +115,6 @@ export class ResourceClassFormComponent implements OnInit, AfterViewChecked {
},
'label': {
'required': 'Label is required.'
},
'comment': {
'required': 'Comment is required.'
}
};

Expand Down Expand Up @@ -216,9 +212,7 @@ export class ResourceClassFormComponent implements OnInit, AfterViewChecked {
]),
comment: new FormControl({
value: this.resourceClassComments, disabled: false
}, [
Validators.required
])
})
});

this.resourceClassForm.valueChanges.subscribe(data => this.onValueChanged(data));
Expand Down Expand Up @@ -253,6 +247,7 @@ export class ResourceClassFormComponent implements OnInit, AfterViewChecked {
switch (type) {
case 'label':
this.resourceClassLabels = data;
this.resourceClassForm.controls['label'].setValue(data);
this.handleError(this.resourceClassLabelsTouched, type);
break;

Expand Down Expand Up @@ -307,7 +302,7 @@ export class ResourceClassFormComponent implements OnInit, AfterViewChecked {

const updateComment = new UpdateResourceClassComment();
updateComment.id = this.iri;
updateComment.comments = this.resourceClassComments;
updateComment.comments = (this.resourceClassComments.length ? this.resourceClassComments : this.resourceClassLabels);
onto4Comment.entity = updateComment;

this._dspApiConnection.v2.onto.updateResourceClass(onto4Label).subscribe(
Expand Down Expand Up @@ -351,6 +346,7 @@ export class ResourceClassFormComponent implements OnInit, AfterViewChecked {
newResClass.subClassOf = [this.iri];

onto.entity = newResClass;

this._dspApiConnection.v2.onto.createResourceClass(onto).subscribe(
(classResponse: ResourceClassDefinitionWithAllLanguages) => {
// need lmd from classResponse
Expand Down
6 changes: 3 additions & 3 deletions src/app/project/project-form/project-form.component.html
Expand Up @@ -6,7 +6,7 @@

<!-- short and long name -->
<mat-form-field class="small-field">
<input matInput placeholder="{{'appLabels.form.project.general.shortcode' | translate}} * "
<input matInput placeholder="{{'appLabels.form.project.general.shortcode' | translate}}"
[maxlength]="shortcodeMaxLength" [formControl]="form.controls['shortcode']">
<mat-hint *ngIf="formErrors.shortcode">
{{ formErrors.shortcode }}
Expand All @@ -16,7 +16,7 @@
<!-- short name -->
<mat-form-field class="medium-field">
<input matInput [maxlength]="shortnameMaxLength"
placeholder="{{'appLabels.form.project.general.shortname' | translate}} * "
placeholder="{{'appLabels.form.project.general.shortname' | translate}}"
[formControl]="form.controls['shortname']">
<mat-hint *ngIf="formErrors.shortname">
{{ formErrors.shortname }}
Expand All @@ -25,7 +25,7 @@

<!-- long name -->
<mat-form-field class="large-field">
<input matInput placeholder="{{'appLabels.form.project.general.longname' | translate}} *"
<input matInput placeholder="{{'appLabels.form.project.general.longname' | translate}}"
[formControl]="form.controls['longname']">
<mat-hint *ngIf="formErrors.longname">
{{ formErrors.longname }}
Expand Down
Expand Up @@ -17,7 +17,7 @@ <h4 class="mat-subheading-1">As system admin, please enter your password before
<mat-form-field class="large-field">
<input matInput [type]="showRequesterPassword ? 'text' : 'password'"
[formControl]="confirmForm.controls['requesterPassword']" autocomplete="old-password"
placeholder="{{ 'appLabels.form.user.general.requesterPassword' | translate}} *">
placeholder="{{ 'appLabels.form.user.general.requesterPassword' | translate}}">

<button mat-button type="button" matSuffix mat-icon-button class="input-icon"
attr.aria-label="{{showRequesterPassword ? 'Hide password' : 'Show password'}}"
Expand Down Expand Up @@ -57,7 +57,7 @@ <h4 class="mat-subheading-1">As system admin, please enter your password before

<input matInput [type]="showRequesterPassword ? 'text' : 'password'"
[formControl]="form.controls['requesterPassword']" autocomplete="current-password"
placeholder="{{ 'appLabels.form.user.general.oldPassword' | translate}} *">
placeholder="{{ 'appLabels.form.user.general.oldPassword' | translate}}">

<button mat-button type="button" matSuffix mat-icon-button class="input-icon"
attr.aria-label="{{showRequesterPassword ? 'Hide password' : 'Show password'}}"
Expand All @@ -76,7 +76,7 @@ <h4 class="mat-subheading-1">As system admin, please enter your password before
<!-- password -->
<mat-form-field class="large-field">
<input matInput [type]="showPassword ? 'text' : 'password'" [formControl]="form.controls['password']"
autocomplete="new-password" placeholder="{{ 'appLabels.form.user.general.password' | translate}} *">
autocomplete="new-password" placeholder="{{ 'appLabels.form.user.general.password' | translate}}">

<button mat-button type="button" matSuffix mat-icon-button class="input-icon"
attr.aria-label="{{showPassword ? 'Hide password' : 'Show password'}}"
Expand All @@ -97,7 +97,7 @@ <h4 class="mat-subheading-1">As system admin, please enter your password before
<input matInput [type]="showConfirmPassword ? 'text' : 'password'"
[class.error]="formErrors.confirmPassword || (!matchingPasswords)"
[formControl]="form.controls['confirmPassword']" autocomplete="confirm-password"
placeholder="{{ 'appLabels.form.user.general.confirmPassword' | translate}} *">
placeholder="{{ 'appLabels.form.user.general.confirmPassword' | translate}}">

<button mat-button type="button" matSuffix mat-icon-button class="input-icon"
attr.aria-label="{{showConfirmPassword ? 'Hide password' : 'Show password'}}"
Expand Down
8 changes: 4 additions & 4 deletions src/app/user/user-form/user-form.component.html
Expand Up @@ -12,7 +12,7 @@
<!-- username -->
<mat-form-field [class.large-field]="!username" [class.small-field]="username">
<input matInput type="text" [formControl]="form.controls['username']"
placeholder="{{ 'appLabels.form.user.general.username' | translate }} *" autocomplete="username">
placeholder="{{ 'appLabels.form.user.general.username' | translate }}" autocomplete="username">
<mat-hint *ngIf="formErrors.username">
{{ formErrors.username }}
</mat-hint>
Expand All @@ -21,7 +21,7 @@
<!-- email -->
<mat-form-field [class.large-field]="!username" [class.medium-field]="username">
<input matInput type="email" [formControl]="form.controls['email']"
placeholder="{{ 'appLabels.form.user.general.email' | translate }} *" autocomplete="email">
placeholder="{{ 'appLabels.form.user.general.email' | translate }}" autocomplete="email">
<mat-hint *ngIf="formErrors.email">
{{ formErrors.email }}
</mat-hint>
Expand All @@ -31,7 +31,7 @@
<!-- given name -->
<mat-form-field class="small-field">
<input matInput [formControl]="form.controls['givenName']"
placeholder="{{ 'appLabels.form.user.general.givenName' | translate }} *" autocomplete="givenname">
placeholder="{{ 'appLabels.form.user.general.givenName' | translate }}" autocomplete="givenname">
<mat-hint *ngIf="formErrors.givenName">
{{ formErrors.givenName }}
</mat-hint>
Expand All @@ -40,7 +40,7 @@
<!-- family name -->
<mat-form-field class="medium-field">
<input matInput [formControl]="form.controls['familyName']"
placeholder="{{ 'appLabels.form.user.general.familyName' | translate }} *" autocomplete="name">
placeholder="{{ 'appLabels.form.user.general.familyName' | translate }}" autocomplete="name">
<mat-hint *ngIf="formErrors.familyName">
{{ formErrors.familyName }}
</mat-hint>
Expand Down
Expand Up @@ -14,7 +14,7 @@
<app-geoname-value #createVal *ngSwitchCase="constants.GeonameValue" [mode]="mode"></app-geoname-value>
<app-link-value #createVal *ngSwitchCase="constants.LinkValue" [mode]="mode"
[parentResource]="parentResource" [propIri]="resourcePropertyDefinition.id"></app-link-value>
<app-yet-another-date-value #createVal *ngSwitchCase="constants.DateValue" [mode]="mode"></app-yet-another-date-value>
<app-date-value #createVal *ngSwitchCase="constants.DateValue" [mode]="mode"></app-date-value>
<app-list-value #createVal *ngSwitchCase="constants.ListValue" [mode]="mode"
[propertyDef]="resourcePropertyDefinition"></app-list-value>
</span>
Expand Down