From 6c272c62f0e295f0bd00aa8ee7ece4e105fffdf0 Mon Sep 17 00:00:00 2001
From: David Jernej <david.jernej@tugraz.at>
Date: Wed, 11 Dec 2019 11:46:59 +0100
Subject: [PATCH] Fix person select fetch problem due to jsonLD inits twice
with presets
---
packages/person-select/src/vpu-person-select.js | 7 ++++---
1 file changed, 4 insertions(+), 3 deletions(-)
diff --git a/packages/person-select/src/vpu-person-select.js b/packages/person-select/src/vpu-person-select.js
index aa22297e..3d779fd4 100644
--- a/packages/person-select/src/vpu-person-select.js
+++ b/packages/person-select/src/vpu-person-select.js
@@ -79,7 +79,7 @@ class PersonSelect extends VPULitElementJQuery {
});
}
- initJSONLD() {
+ initJSONLD(ignorePreset = false) {
const that = this;
JSONLD.initialize(this.entryPointUrl, function (jsonld) {
@@ -88,7 +88,7 @@ class PersonSelect extends VPULitElementJQuery {
// we need to poll because maybe the user interface isn't loaded yet
// Note: we need to call initSelect2() in a different function so we can access "this" inside initSelect2()
- commonUtils.pollFunc(() => { return that.initSelect2(); }, 10000, 100);
+ commonUtils.pollFunc(() => { return that.initSelect2(ignorePreset); }, 10000, 100);
}, {}, this.lang);
}
@@ -264,7 +264,8 @@ class PersonSelect extends VPULitElementJQuery {
this.ignoreValueUpdate = false;
break;
case "entryPointUrl":
- this.initJSONLD();
+ // we don't need to preset the selector if the entry point url changes
+ this.initJSONLD(true);
break;
}
});
--
GitLab