18402105232

Committed 10 Oct 2025 09:15AM UTC coverage: 87.04% (-0.08%) from 87.119%

Build # 18402105232

Build Type

Pull #2597

github

Committed by

web-flow

Commit Message

Merge 0eaa3f10f into a969ced58

Pull Request Pull Request #2597: Dynamic extruded polygon style update

Run Details

2836 of 3777 branches covered (75.09%)

Branch coverage included in aggregate %.

81 of 121 new or added lines in 3 files covered. (66.94%)

1 existing line in 1 file now uncovered.

26494 of 29920 relevant lines covered (88.55%)

1083.72 hits per line

Source File
Press 'n' to go to next uncovered line, 'b' for previous

62.96

/packages/Main/src/Process/FeatureProcessing.js

import LayerUpdateState from 'Layer/LayerUpdateState';
import ObjectRemovalHelper from 'Process/ObjectRemovalHelper';
import handlingError from 'Process/handlerNodeError';
import { Coordinates } from '@itowns/geographic';
import { geoidLayerIsVisible } from 'Layer/GeoidLayer';
import Feature2Mesh from 'Converter/Feature2Mesh';

const coord = new Coordinates('EPSG:4326', 0, 0, 0);

export default {
    update(context, layer, node) {
        if (!node.parent && node.children.length) {
            // if node has been removed dispose three.js resource
            ObjectRemovalHelper.removeChildrenAndCleanupRecursively(layer, node);
            return;
        }
        if (!node.visible) {
            return;
        }

        if (node.layerUpdateState[layer.id] === undefined) {
            node.layerUpdateState[layer.id] = new LayerUpdateState();
        } else if (!node.layerUpdateState[layer.id].canTryUpdate()) {
            // toggle visibility features
            node.link[layer.id]?.forEach((f) => {
                f.layer.object3d.add(f);
                f.meshes.position.z = geoidLayerIsVisible(layer.parent) ? node.geoidHeight : 0;
                f.meshes.updateMatrixWorld();

                // update existing features
                for (const featureMesh of f.meshes.children) {
                    Feature2Mesh.updateFillStyle(featureMesh, { style: layer.style });
                }
            });
            return;
        }

        const extentsDestination = node.getExtentsByProjection(layer.source.crs) || [node.extent];

        const zoomDest = extentsDestination[0].zoom;

        // check if it's tile level is equal to display level layer.
        // TO DO updata at all level asked
        // if ((zoomDest < layer.zoom.min && zoomDest > layer.zoom.max) ||
        if (zoomDest != layer.zoom.min ||
        // check if there's data in extent tile.
            !this.source.extentInsideLimit(node.extent, zoomDest) ||
        // In FileSource case, check if the feature center is in extent tile.
            (layer.source.isFileSource && !node.extent.isPointInside(layer.source.extent.center(coord)))) {
        // if not, there's not data to add at this tile.
            node.layerUpdateState[layer.id].noMoreUpdatePossible();
            return;
        }

        node.layerUpdateState[layer.id].newTry();

        const command = {
            layer,
            extentsSource: extentsDestination,
            view: context.view,
            requester: node,
        };

        return context.scheduler.execute(command).then((featureMeshes) => {
            node.layerUpdateState[layer.id].noMoreUpdatePossible();

            featureMeshes.forEach((featureMesh) => {
                if (featureMesh) {
                    node.link[layer.id] = node.link[layer.id] || [];
                    featureMesh.as(context.view.referenceCrs);
                    featureMesh.meshes.position.z = geoidLayerIsVisible(layer.parent) ? node.geoidHeight : 0;
                    featureMesh.updateMatrixWorld();

                    if (layer.onMeshCreated) {
                        layer.onMeshCreated(featureMesh, context);
                    }

                    if (!node.parent) {
                        // TODO: Clean cache needs a refactory, because it isn't really efficient and used
                        ObjectRemovalHelper.removeChildrenAndCleanupRecursively(layer, featureMesh);
                    } else {
                        layer.object3d.add(featureMesh);
                        node.link[layer.id].push(featureMesh);
                    }
                    featureMesh.layer = layer;
                } else {
                    // TODO: verify if it's possible the featureMesh is undefined.
                    node.layerUpdateState[layer.id].failure(1, true);
                }
            });
        },
        err => handlingError(err, node, layer, node.level, context.view));
    },
};

1	import LayerUpdateState from 'Layer/LayerUpdateState';	1✔
2	import ObjectRemovalHelper from 'Process/ObjectRemovalHelper';	1✔
3	import handlingError from 'Process/handlerNodeError';	1✔
4	import { Coordinates } from '@itowns/geographic';	1✔
5	import { geoidLayerIsVisible } from 'Layer/GeoidLayer';	1✔
6	import Feature2Mesh from 'Converter/Feature2Mesh';	1✔
7		1✔
8	const coord = new Coordinates('EPSG:4326', 0, 0, 0);	1✔
9		1✔
10	export default {	1✔
11	update(context, layer, node) {	1✔
12	if (!node.parent && node.children.length) {	10!
13	// if node has been removed dispose three.js resource	×
14	ObjectRemovalHelper.removeChildrenAndCleanupRecursively(layer, node);	×
15	return;	×
16	}	×
17	if (!node.visible) {	10!
18	return;	×
19	}	×
20		10✔
21	if (node.layerUpdateState[layer.id] === undefined) {	10✔
22	node.layerUpdateState[layer.id] = new LayerUpdateState();	10✔
23	} else if (!node.layerUpdateState[layer.id].canTryUpdate()) {	10!
24	// toggle visibility features	×
25	node.link[layer.id]?.forEach((f) => {	×
26	f.layer.object3d.add(f);	×
27	f.meshes.position.z = geoidLayerIsVisible(layer.parent) ? node.geoidHeight : 0;	×
28	f.meshes.updateMatrixWorld();	×
NEW 29		×
NEW 30	// update existing features	×
NEW 31	for (const featureMesh of f.meshes.children) {	×
NEW 32	Feature2Mesh.updateFillStyle(featureMesh, { style: layer.style });	×
NEW 33	}	×
34	});	×
35	return;	×
36	}	×
37		10✔
38	const extentsDestination = node.getExtentsByProjection(layer.source.crs) \|\| [node.extent];	10!
39		10✔
40	const zoomDest = extentsDestination[0].zoom;	10✔
41		10✔
42	// check if it's tile level is equal to display level layer.	10✔
43	// TO DO updata at all level asked	10✔
44	// if ((zoomDest < layer.zoom.min && zoomDest > layer.zoom.max) \|\|	10✔
45	if (zoomDest != layer.zoom.min \|\|	10✔
46	// check if there's data in extent tile.	10✔
47	!this.source.extentInsideLimit(node.extent, zoomDest) \|\|	10✔
48	// In FileSource case, check if the feature center is in extent tile.	10✔
49	(layer.source.isFileSource && !node.extent.isPointInside(layer.source.extent.center(coord)))) {	10!
50	// if not, there's not data to add at this tile.	×
51	node.layerUpdateState[layer.id].noMoreUpdatePossible();	×
52	return;	×
53	}	×
54		10✔
55	node.layerUpdateState[layer.id].newTry();	10✔
56		10✔
57	const command = {	10✔
58	layer,	10✔
59	extentsSource: extentsDestination,	10✔
60	view: context.view,	10✔
61	requester: node,	10✔
62	};	10✔
63		10✔
64	return context.scheduler.execute(command).then((featureMeshes) => {	10✔
65	node.layerUpdateState[layer.id].noMoreUpdatePossible();	8✔
66		8✔
67	featureMeshes.forEach((featureMesh) => {	8✔
68	if (featureMesh) {	8✔
69	node.link[layer.id] = node.link[layer.id] \|\| [];	8✔
70	featureMesh.as(context.view.referenceCrs);	8✔
71	featureMesh.meshes.position.z = geoidLayerIsVisible(layer.parent) ? node.geoidHeight : 0;	8!
72	featureMesh.updateMatrixWorld();	8✔
73		8✔
74	if (layer.onMeshCreated) {	8!
75	layer.onMeshCreated(featureMesh, context);	×
76	}	×
77		8✔
78	if (!node.parent) {	8!
79	// TODO: Clean cache needs a refactory, because it isn't really efficient and used	×
80	ObjectRemovalHelper.removeChildrenAndCleanupRecursively(layer, featureMesh);	×
81	} else {	8✔
82	layer.object3d.add(featureMesh);	8✔
83	node.link[layer.id].push(featureMesh);	8✔
84	}	8✔
85	featureMesh.layer = layer;	8✔
86	} else {	8!
87	// TODO: verify if it's possible the featureMesh is undefined.	×
88	node.layerUpdateState[layer.id].failure(1, true);	×
89	}	×
90	});	8✔
91	},	10✔
92	err => handlingError(err, node, layer, node.level, context.view));	10✔
93	},	10✔
94	};	1✔

iTowns / itowns / 18402105232

Source File Press 'n' to go to next uncovered line, 'b' for previous

Source File
Press 'n' to go to next uncovered line, 'b' for previous