Skip to content

Commit

Permalink
form-edit: migrate from setItems to using .items
Browse files Browse the repository at this point in the history
Signed-off-by: Patrizio Bekerle <[email protected]>
  • Loading branch information
pbek committed Feb 12, 2025
1 parent 392cb87 commit c46dec7
Show file tree
Hide file tree
Showing 7 changed files with 10 additions and 69 deletions.
2 changes: 1 addition & 1 deletion package-lock.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -58,7 +58,7 @@
"@dbp-toolkit/auth": "^0.3.0",
"@dbp-toolkit/common": "^0.3.0",
"@dbp-toolkit/file-handling": "^0.3.0",
"@dbp-toolkit/form-elements": "^0.2.0",
"@dbp-toolkit/form-elements": "^0.2.6",
"@dbp-toolkit/language-select": "^0.3.0",
"@dbp-toolkit/notification": "^0.3.3",
"@dbp-toolkit/pdf-viewer": "^0.1.0",
Expand Down
21 changes: 3 additions & 18 deletions src/baseObject.js
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,6 @@ import {classMap} from 'lit/directives/class-map.js';
import {getSelectorFixCSS} from './styles.js';
import {getIconSVGURL} from './utils.js';
import {gatherFormDataFromElement, validateRequiredFields} from '@dbp-toolkit/form-elements/src/utils.js';
import {createRef, ref} from 'lit/directives/ref.js';

export class BaseObject {
name = 'baseObject';
Expand Down Expand Up @@ -105,9 +104,6 @@ export class BaseFormElement extends ScopedElementsMixin(DBPLitElement) {
this.entryPointUrl = '';
this.auth = {};
this.saveButtonEnabled = true;
this.studyFieldRef = createRef();
this.semesterRef = createRef();
this.isPartOfRef = createRef();
}

enableSaveButton() {
Expand All @@ -128,17 +124,6 @@ export class BaseFormElement extends ScopedElementsMixin(DBPLitElement) {
};
}

connectedCallback() {
super.connectedCallback();

this.updateComplete.then(() => {
// Set the items for the enum components
this.studyFieldRef.value.setItems(this.getStudyFields());
this.semesterRef.value.setItems(this.getSemesters());
this.isPartOfRef.value.setItems(BaseFormElement.getIsPartOfItems());
});
}

getSemesters = () => {
let currentDate = new Date();
let currentYear = currentDate.getFullYear();
Expand Down Expand Up @@ -196,28 +181,28 @@ export class BaseFormElement extends ScopedElementsMixin(DBPLitElement) {
</dbp-form-string-element>
<dbp-form-enum-element
${ref(this.studyFieldRef)}
subscribe="lang"
name="studyField"
label="Study field"
.items=${this.getStudyFields()}
.value=${baseData.studyField || ''}
required>
</dbp-form-enum-element>
<dbp-form-enum-element
${ref(this.semesterRef)}
subscribe="lang"
name="semester"
label="Semester"
.items=${this.getSemesters()}
.value=${defaultSemester}
required>
</dbp-form-enum-element>
<dbp-form-enum-element
${ref(this.isPartOfRef)}
subscribe="lang"
name="isPartOf"
label="Speicherzweck-Löschfristen"
.items=${BaseFormElement.getIsPartOfItems()}
.value=${baseData.isPartOf || ''}
multiple
required>
Expand Down
18 changes: 2 additions & 16 deletions src/objectTypes/fileAdmissionNotice.js
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ import {css, html} from 'lit';
import {BaseObject, BaseFormElement, BaseHitElement, BaseViewElement, getCommonStyles} from '../baseObject.js';
import * as viewElements from './viewElements.js';
import { PersonHit } from './person.js';
import {createRef, ref} from 'lit/directives/ref.js';

export default class extends BaseObject {
name = 'file-cabinet-admissionNotice';

Expand All @@ -27,20 +27,6 @@ export default class extends BaseObject {
}

class CabinetFormElement extends BaseFormElement {
constructor() {
super();
this.decisionRef = createRef();
}

connectedCallback() {
super.connectedCallback();

this.updateComplete.then(() => {
// Set the items for the enum component
this.decisionRef.value.setItems(CabinetFormElement.getDecisions());
});
}

static getAdditionalTypes = () => {
return {
'AdmissionNotice': 'Admission Notice',
Expand Down Expand Up @@ -82,10 +68,10 @@ class CabinetFormElement extends BaseFormElement {
</dbp-form-string-element>
<dbp-form-enum-element
${ref(this.decisionRef)}
subscribe="lang"
name="decision"
label="Decision"
.items=${CabinetFormElement.getDecisions()}
.value=${data.decision || ''}>
</dbp-form-enum-element>
Expand Down
17 changes: 1 addition & 16 deletions src/objectTypes/fileCitizenshipCertificate.js
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,6 @@ import {BaseFormElement, BaseHitElement, BaseObject, BaseViewElement, getCommonS
import * as formElements from './formElements.js';
import * as viewElements from './viewElements.js';
import { PersonHit } from './person.js';
import {createRef, ref} from 'lit/directives/ref.js';

export default class extends BaseObject {
name = 'file-cabinet-citizenshipCertificate';
Expand All @@ -29,20 +28,6 @@ export default class extends BaseObject {
}

class CabinetFormElement extends BaseFormElement {
constructor() {
super();
this.nationalityRef = createRef();
}

connectedCallback() {
super.connectedCallback();

this.updateComplete.then(() => {
// Set the items for the enum component
this.nationalityRef.value.setItems(formElements.getNationalityItems());
});
}

static getAdditionalTypes = () => {
return {
'CitizenshipCertificate': 'Citizenship Certificate',
Expand All @@ -60,10 +45,10 @@ class CabinetFormElement extends BaseFormElement {
return html`
<form>
<dbp-form-enum-element
${ref(this.nationalityRef)}
subscribe="lang"
name="nationality"
label="Nationality"
.items=${formElements.getNationalityItems()}
.value=${data.nationality || ''}>
</dbp-form-enum-element>
Expand Down
17 changes: 1 addition & 16 deletions src/objectTypes/fileIdentityDocument.js
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,6 @@ import {BaseObject, BaseFormElement, BaseHitElement, BaseViewElement, getCommonS
import * as formElements from './formElements.js';
import * as viewElements from './viewElements.js';
import { PersonHit } from './person.js';
import {createRef, ref} from 'lit/directives/ref.js';

export default class extends BaseObject {
name = 'file-cabinet-identityDocument';
Expand All @@ -29,20 +28,6 @@ export default class extends BaseObject {
}

class CabinetFormElement extends BaseFormElement {
constructor() {
super();
this.nationalityRef = createRef();
}

connectedCallback() {
super.connectedCallback();

this.updateComplete.then(() => {
// Set the items for the enum component
this.nationalityRef.value.setItems(formElements.getNationalityItems());
});
}

static getAdditionalTypes() {
return {
'DriversLicence': 'Drivers Licence',
Expand Down Expand Up @@ -71,10 +56,10 @@ class CabinetFormElement extends BaseFormElement {
</dbp-form-string-element>
<dbp-form-enum-element
${ref(this.nationalityRef)}
subscribe="lang"
name="nationality"
label="Nationality"
.items=${formElements.getNationalityItems()}
.value=${data.nationality || ''}
required>
</dbp-form-enum-element>
Expand Down

0 comments on commit c46dec7

Please sign in to comment.