[ninsbl]

This PR makes sure the temporal framework does not write file metadata to the source mapset of map from another mapset and that it does not write "ghost" metadata for that map in the current mapset (the C-tools should write to current anyway).

The readonly property may become useful in other contexts as well (see e.g.: #2448).

Fixes: #3394
Replaces: #7211

[petrasovaa]

Suggested change

	name, layer = ident.split(":", 1)
	name, layer = name.split(":", 1)

[ninsbl]

Good catch. I feel I read this pattern (or variations of it) a bit too often. In #7670 I added a private function in open_stds.py to parse components of a STDS ID string. I wonder if such a function could / should go into e.g. an utils module or core.py to not repeat this splitting many places...

[ninsbl]

Removing semantic_label or timestamp should probably be guarded too...

[echoix]

What’s missing here to be done?

[ninsbl]

What’s missing here to be done?

I was considering having something like this:

grass/python/grass/temporal/open_stds.py

Line 35 in ddf66e9

def _parse_id(

as a public function e.g. in temporal.utils because this code seems repeated several places: https://github.com/search?q=repo%3AOSGeo%2Fgrass+find%28%22%40%22%29&type=code ... what du you think, better suited in a separate PR?

[ninsbl]

A helper function for STDS id parsing is better suited for a separate PR. This should be good to go...