-
Notifications
You must be signed in to change notification settings - Fork 23
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
OerebLex Prebublinks #1617
Comments
Hi @michmuel
|
Here is a sample prepub-Link: |
Yes. There are two issues: First, the xml provided by the prepublink is not valid against the xml schema definition used by the xml format check routine of the geolink_formatter. This causes the error indicated in your post. Second, the document element in the prepublink does not provide all the attributes expected by the OerebLex source of pyramid_oereb. E.G. there is no attribute "enactment_date" but an attribute "status_start_date". Another question is if the attribute "status" should be used for generating the document title. I think pyramid oereb should be able to process prepublinks in a general way. More elaborated cantonal rules would require to write and use individual functions. |
Changes in oereblex source implemented. Handling of prepublinks is possible now. |
We use OerebLex in BL and will use the OerebLex-"Prepub-Module" for managing the "Laufende Aenderungen". The "prepublinks"-xml differs a bit from the "geolinks"-xml by the attributes of the element "document" (if attribute doctype="prepublication": additional attributes "status", "status_start_date", "status_end_date" but without attributes "municipality", "category", "enactment_date", "abrogation_date", "decree_date").
The mapping of the lexlink information to the document record has to be refined (
pyramid_oereb/pyramid_oereb/contrib/data_sources/oereblex/sources/document.py
Line 141 in 85f816b
I can provide these changes but have to know if someone else is using the OerebLex-Prepub-Module for the "Laufende Aenderungen" to see what are the requirements for a standard mapping.
The text was updated successfully, but these errors were encountered: