Closed qgau closed 1 week ago
During feature ingestion process, resto computes automatic classifications based on the feature properties. For instance, it computes geographical classification on the feature location (i.e. "Europe", "France", "Haute-Garonne", etc.).
All these classifications are provided as catalogs and thus this line is here to reflect this behavior. Since these catalogs are "internals" catalogs, there is no ownership check when they are created. So this is transparent to the user and so you point 3 is not true i.e. it does not require user to have the rights to create catalog.
The point 2 is an issue - the /catalogs/collections should returns a 404 The point 4 is an issue (see #430)
Concerning point 1. There is however, a way for a user to create catalog during feature ingestion by setting a "rest:catalogs" property within the feature to be inserted. This is a very useful behavior that should be kept in resto. See the code here (https://github.com/jjrom/resto/blob/e45793ee7021d67fe0526116e5486adeef53d610/app/resto/core/addons/Cataloger.php#L438-L479)
Since this is not a documented behavior, I don't see any issue here. The probability that a user set a "resto:catalogs" property in input feature is quite low unless the user understand it in the code and thus do it on purpose (which means that this functionality is useful to him ;) )
Hi!
Thank for the answer, it is clearer to me now.
About point 3, you're probably right, this is not something that I experienced, but assumed because of this line. But I didn't test it.
So point 2 is a bug, I will try to fix it, but I wont' be able to during the next three weeks, so don't be suprised if you have no news.
Hello,
This line in FeaturesFunctions.php store/create catalogs when a user insert a feature in an existing collections.
It has at least four consequences that could be undesirable:
catalogMinMatch
, impacts /catalogs/collections but not /collections.Could you highlight why are these catalogs created on-the-fly needed? Could it makes sense to keep these hidden to the users? If they are really needed, then is there any sense to have different rights for insertfeatures and createcatalogs? As the second require necessarily the first in case collections are managed by others users than the ones insterting features.