Skip to content
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

not able to change maintenance_type #52

Open
urskaufmann opened this issue Dec 11, 2023 · 9 comments
Open

not able to change maintenance_type #52

urskaufmann opened this issue Dec 11, 2023 · 9 comments
Labels
enhancement New feature or request

Comments

@urskaufmann
Copy link
Contributor

I get these error when editing data from vsa_version 2015:
WARNING Fehlerhafte Abfrage: UPDATE "tww_app"."vw_tww_maintenance_event" SET "maintenance_event_type"='examination' WHERE "obj_id"::text='ch080qwzEH001128' ergab 7 [ERROR: Type change not allowed for maintenance_event
HINT: You cannot switch from unknown to examination
CONTEXT: PL/pgSQL function tww_app.ft_vw_tww_maintenance_event_update() line 26 at RAISE

I agree it should not be possible (without further UI) to change the type, after it is set. But I think it should be possible to change once from "unknown" to one of the other types. "Unknown" is not a "type" (not a subclass of maintenance_event).

@3nids
Copy link
Contributor

3nids commented Dec 12, 2023

Is it allowed to have a maintenance_event without a type ? (i.e. is it abstract or not)

@sjib
Copy link
Contributor

sjib commented Dec 12, 2023

There was a change from 2015 to 2020: maintenance_event is in release 2020 ABSTRACT, before it was not.

@urskaufmann
Copy link
Contributor Author

urskaufmann commented Dec 12, 2023

if I import the VSA-Musterdatensatz, I get type=unkown and I'm not able to correct this...
What does the VSA-Checker convert2020 with old maintenance_event records?

@3nids
Copy link
Contributor

3nids commented Dec 12, 2023

So, this is not a problem of the data model but:

  • the data
  • pirogue (the tool which creates the view): if no type is set, it writes "unknown", but then we are not allowing type change

We can discuss if we want to fix/adapt pirogue for this.

@sjib sjib added the enhancement New feature or request label Feb 2, 2024
@cymed
Copy link
Contributor

cymed commented Aug 5, 2024

It also does not allow manually inserting a row, as it is set to "unknown" as a default and then does not allow changing that value

@sjib
Copy link
Contributor

sjib commented Aug 6, 2024

May be we should change the default to maintenance instead and take out the option unknown, as unknown just seems to create problems.

@sjib
Copy link
Contributor

sjib commented Aug 6, 2024

@urskaufmann with my last import I did not have problems.
20240806_tww_app vw_maintenance_event

Maybe we first re-check before we further go with this.

@sjib
Copy link
Contributor

sjib commented Aug 6, 2024

We still are missing the data in the tww_od.re_maintenance_event_wastewater_structure table after import.
see #352

@cymed
Copy link
Contributor

cymed commented Jan 15, 2025

Wil be fixed in #524

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

4 participants