Skip to content
Snippets Groups Projects
Commit ea264386 authored by Vink, S.A. (Sjoerd)'s avatar Vink, S.A. (Sjoerd)
Browse files

feat(map_nodelink): fixed initialization issue

parent dd65283a
No related branches found
No related tags found
1 merge request!151feat(map_nodelink)
Pipeline #137617 passed
This commit is part of merge request !151. Comments created here will be created in the context of that merge request.
......@@ -58,7 +58,7 @@ export class NodeLinkLayer extends CompositeLayer<LayerProps> {
new LineLayer({
id: layerId,
data: edgeData,
visible: !config[label].hidden,
visible: !config[label]?.hidden,
pickable: true,
getWidth: (d: any) => config[label].width,
getSourcePosition: (d: any) => getNodeLocation(d.from),
......
......@@ -72,10 +72,10 @@ export default function NodeLinkOptions({ settings, graphMetadata, updateSetting
updateValue={(val: number[]) => updateSettings({ [nodeType]: { ...settings?.[nodeType], color: val } })}
/>
<Button
iconComponent={settings?.[nodeType].hidden ? <VisibilityOff /> : <Visibility />}
iconComponent={settings?.[nodeType]?.hidden ? <VisibilityOff /> : <Visibility />}
variant="ghost"
onClick={() => {
updateSettings({ [nodeType]: { ...settings?.[nodeType], hidden: !settings?.[nodeType].hidden as boolean } });
updateSettings({ [nodeType]: { ...settings?.[nodeType], hidden: !settings?.[nodeType]?.hidden as boolean } });
}}
/>
</div>
......@@ -137,10 +137,10 @@ export default function NodeLinkOptions({ settings, graphMetadata, updateSetting
updateValue={(val: number[]) => updateSettings({ [edgeType]: { ...settings?.[edgeType], color: val } })}
/>
<Button
iconComponent={settings?.[edgeType].hidden ? <VisibilityOff /> : <Visibility />}
iconComponent={settings?.[edgeType]?.hidden ? <VisibilityOff /> : <Visibility />}
variant="ghost"
onClick={() => {
updateSettings({ [edgeType]: { ...settings?.[edgeType], hidden: !settings?.[edgeType].hidden as boolean } });
updateSettings({ [edgeType]: { ...settings?.[edgeType], hidden: !settings?.[edgeType]?.hidden as boolean } });
}}
/>
</div>
......
......@@ -67,10 +67,8 @@ export const MapVis = ({ data, settings, handleSelect, graphMetadata }: Visualiz
);
useEffect(() => {
const layerType = settings.layer ? layerTypes?.[settings.layer] : layerTypes.node;
setLayer({
type: layerType,
type: settings.layer ? layerTypes?.[settings.layer] : layerTypes.nodelink,
config: settings,
});
}, [settings.layer]);
......@@ -80,10 +78,10 @@ export const MapVis = ({ data, settings, handleSelect, graphMetadata }: Visualiz
const coordinateLookup: { [id: string]: Coordinate } = data.nodes.reduce(
(acc, node) => {
const latitude = settings[node.label].lat ? (node?.attributes?.[settings[node.label].lat] as string) : undefined;
const longitude = settings[node.label].lon ? (node?.attributes?.[settings[node.label].lon] as string) : undefined;
const latitude = settings?.[node.label]?.lat ? (node?.attributes?.[settings[node.label].lat] as string) : undefined;
const longitude = settings?.[node.label]?.lon ? (node?.attributes?.[settings[node.label].lon] as string) : undefined;
if (!!latitude && !!longitude) {
if (latitude !== undefined && longitude !== undefined) {
acc[node._id] = [parseFloat(longitude), parseFloat(latitude)];
}
......@@ -98,9 +96,7 @@ export const MapVis = ({ data, settings, handleSelect, graphMetadata }: Visualiz
config: settings,
selected: selected,
hoverObject: hoverObject,
getNodeLocation: (d: string) => {
return coordinateLookup[d];
},
getNodeLocation: (d: string) => coordinateLookup[d],
flyToBoundingBox: flyToBoundingBox,
setLayerIds: (val: string[]) => setLayerIds(val),
});
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment