diff --git a/src/core/js/caosdb.js b/src/core/js/caosdb.js
index 5a30ce554e265bf1f27814e54573daf8b5a0c4a3..39109d444ff4bc8bbd0f2b2811004967565a3361 100644
--- a/src/core/js/caosdb.js
+++ b/src/core/js/caosdb.js
@@ -5,8 +5,8 @@
* Copyright (C) 2018-2020 Alexander Schlemmer
* Copyright (C) 2018 Research Group Biomedical Physics,
* Max-Planck-Institute for Dynamics and Self-Organization Göttingen
- * Copyright (C) 2019-2020 IndiScale GmbH (info@indiscale.com)
- * Copyright (C) 2019-2020 Timm Fitschen (t.fitschen@indiscale.com)
+ * Copyright (C) 2019-2022 IndiScale GmbH (info@indiscale.com)
+ * Copyright (C) 2019-2022 Timm Fitschen (t.fitschen@indiscale.com)
*
* This program is free software: you can redistribute it and/or modify
* it under the terms of the GNU Affero General Public License as
@@ -765,6 +765,7 @@ var _constructXpaths = function (selectors) {
* (or an array of values, when the entity has list properties).
*/
var getPropertyValues = function (entities, selectors) {
+ // @review Florian Spreckelsen 2022-05-06
const entity_iter = entities.evaluate("/Response/Record", entities);
const table = [];
diff --git a/src/core/js/ext_map.js b/src/core/js/ext_map.js
index b4e4444645639de80cab07a89ead269d500f8d24..39e6d510f9bbdf8e18426dc9b56577a235850a8d 100644
--- a/src/core/js/ext_map.js
+++ b/src/core/js/ext_map.js
@@ -543,6 +543,7 @@ var caosdb_map = new function () {
* recordtype)
*/
this._set_subprops_at_top = function (entities, depth, datamodel) {
+ // @review Florian Spreckelsen 2022-05-06
var latlong = caosdb_map._get_leaf_prop(entities, depth, datamodel);
for (let rec_id in latlong) {
@@ -550,13 +551,13 @@ var caosdb_map = new function () {
const is_list_lng = Array.isArray(latlong[rec_id][0]);
var lat, lng;
if (is_list_lat) {
- var lat_val = `<Value>${latlong[rec_id][0].join("</Value><Value>")}</Value>`;
- lat = `<Property name="${datamodel.lat}" datatype="LIST<lat>">${lat_val}</Property>`;
- var lng_val = `<Value>${latlong[rec_id][1].join("</Value><Value>")}</Value>`;
- lng = `<Property name="${datamodel.lng}" datatype="LIST<lng>">${lng_val}</Property>`;
+ var lat_val = `<Value>${latlong[rec_id][0].join("</Value><Value>")}</Value>`;
+ lat = `<Property name="${datamodel.lat}" datatype="LIST<lat>">${lat_val}</Property>`;
+ var lng_val = `<Value>${latlong[rec_id][1].join("</Value><Value>")}</Value>`;
+ lng = `<Property name="${datamodel.lng}" datatype="LIST<lng>">${lng_val}</Property>`;
} else {
- lat = `<Property name="${datamodel.lat}">${latlong[rec_id][0]}</Property>`;
- lng = `<Property name="${datamodel.lng}">${latlong[rec_id][1]}</Property>`;
+ lat = `<Property name="${datamodel.lat}">${latlong[rec_id][0]}</Property>`;
+ lng = `<Property name="${datamodel.lng}">${latlong[rec_id][1]}</Property>`;
}
let tmp_rec = caosdb_map._get_toplvl_rec_with_id(entities, rec_id);
tmp_rec.append(str2xml(lat).firstElementChild);
@@ -605,6 +606,7 @@ var caosdb_map = new function () {
* @returns {HTMLElement} a popup element.
*/
this._make_map_popup = function (entity, datamodel, lat, lng) {
+ // @review Florian Spreckelsen 2022-05-06
const role_label = $(entity).find(
".label.caosdb-f-entity-role").first().clone();
const parent_list = caosdb_map.make_parent_labels(entity);
@@ -871,8 +873,8 @@ var caosdb_map = new function () {
let panel = this.create_map_panel();
- if(!show) {
- $(panel).hide();
+ if (!show) {
+ $(panel).hide();
}
$('nav').first().after(panel);
@@ -1007,8 +1009,8 @@ var caosdb_map = new function () {
}
this.config = config;
var show_map = config.show;
- if(sessionStorage["caosdb_map.show"]) {
- show_map = JSON.parse(sessionStorage["caosdb_map.show"]);
+ if (sessionStorage["caosdb_map.show"]) {
+ show_map = JSON.parse(sessionStorage["caosdb_map.show"]);
}
this.init_select_handler();
this.init_view_change_handler();
@@ -1108,8 +1110,8 @@ var caosdb_map = new function () {
// indicate that the map is ready: map button is present and
// map is hidden or shown but initialized in either case.
- this._map.whenReady(()=>{
- document.body.dispatchEvent(caosdb_map.map_ready);
+ this._map.whenReady(() => {
+ document.body.dispatchEvent(caosdb_map.map_ready);
});
} catch (err) {
logger.error("Could not initialize the map.",
@@ -1544,8 +1546,9 @@ var caosdb_map = new function () {
* @param {mapEntityPopupGenerator} [make_popup] - creates popup content.
* @returns {L.Marker} the marker for the map.
*/
- var _create_single_entity_marker = function(map_entity, datamodel, lat,
- lng, icon_options, zIndexOffset, make_popup) {
+ var _create_single_entity_marker = function (map_entity, datamodel, lat,
+ lng, icon_options, zIndexOffset, make_popup) {
+ // @review Florian Spreckelsen 2022-05-06
var marker = L.marker([lat, lng], {
icon: L.divIcon(icon_options)
});
@@ -1572,9 +1575,10 @@ var caosdb_map = new function () {
* @returns {L.Marker[]} an array of markers for the map.
*/
this.create_entity_markers = function (entities, datamodel, make_popup,
- zIndexOffset, icon_options) {
+ zIndexOffset, icon_options) {
+ // @review Florian Spreckelsen 2022-05-06
logger.trace("enter create_entity_markers", entities, datamodel,
- zIndexOffset, icon_options);
+ zIndexOffset, icon_options);
var ret = []
for (const map_entity of entities) {
@@ -1601,11 +1605,11 @@ var caosdb_map = new function () {
}
// both array's length must match
- if (is_list_lng !== is_list_lat
- || (is_list_lat && is_list_lng
- && lat_vals.length !== lng_vals.length)) {
- logger.error("Cannot show this entity on the map. "
- + "Its lat/long properties have different lenghts: ",
+ if (is_list_lng !== is_list_lat ||
+ (is_list_lat && is_list_lng &&
+ lat_vals.length !== lng_vals.length)) {
+ logger.error("Cannot show this entity on the map. " +
+ "Its lat/long properties have different lenghts: ",
map_entity);
continue;
}
@@ -1616,15 +1620,15 @@ var caosdb_map = new function () {
// [lng1, lng2, ... lngN]
// into one
// [[lat1,lng1],[lat2,lng2],... [latN,lngN]]
- var latlngs = lat_vals.map(function(e, i) {
+ var latlngs = lat_vals.map(function (e, i) {
return [e, lng_vals[i]];
});
logger.debug(`create point marker(s) at ${latlngs} for`,
- map_entity);
+ map_entity);
for (let latlng of latlngs) {
var marker = _create_single_entity_marker(map_entity,
- datamodel, latlng[0], latlng[1],
- icon_options, zIndexOffset, make_popup);
+ datamodel, latlng[0], latlng[1],
+ icon_options, zIndexOffset, make_popup);
ret.push(marker);
}