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

Interlis Import only works when the user owns all triggers #87

Closed
cymed opened this issue Jan 17, 2024 · 4 comments
Closed

Interlis Import only works when the user owns all triggers #87

cymed opened this issue Jan 17, 2024 · 4 comments
Labels
INTERLIS About INTERLIS exchange format (import / export) plugin Concerns the wastewater plugin

Comments

@cymed
Copy link
Contributor

cymed commented Jan 17, 2024

Describe the bug
Interlis Import only works when the user owns all triggers, otherwise drop_symbology_functions runs into an error

To Reproduce
Exact steps to reproduce the behavior:

  1. Alter Ownership of one trigger to another user (i.e. on_reach_point_update)
  2. Try Import
  3. See error

Expected behavior
Import working

Screenshots / data
sqlalchemy.exc.ProgrammingError: (psycopg2.errors.InsufficientPrivilege) must be owner of relation reach_point CONTEXT: SQL statement "DROP TRIGGER IF EXISTS on_reach_point_update ON tww_od.reach_point" PL/pgSQL function tww_sys.drop_symbology_triggers() line 3 at SQL statement [SQL: SELECT tww_sys.drop_symbology_triggers();] (Background on this error at: https://sqlalche.me/e/14/f405)

Desktop (please complete the following information):

  • TEKSI plugin version: dev
  • TEKSI datamodel version: dev
  • TEKSI project file version: 2024_01_15
  • QGIS Version - see the About menu: 3.28.11
  • Postgres: 12.17
  • Postgis: POSTGIS="3.0.0 r17983" [EXTENSION] PGSQL="120" GEOS="3.8.0-CAPI-1.13.1 " PROJ="6.3.1" LIBXML="2.9.4" LIBJSON="0.13.1" LIBPROTOBUF="1.3.3" WAGYU="0.4.3 (Internal)"
  • OS: Windows

I deleted the statement timeout option temporarily (see #76)

Additional context

@cymed
Copy link
Contributor Author

cymed commented Jan 17, 2024

Why do we drop the symbology triggers instead of disabling and re-enabling them anyway?

@cymed
Copy link
Contributor Author

cymed commented Jan 17, 2024

To be solved by #88. Changes tested.

@cymed cymed closed this as completed Jan 17, 2024
@sjib
Copy link
Contributor

sjib commented Jan 17, 2024

Reopened - as not yet merged

@sjib sjib reopened this Jan 17, 2024
@3nids 3nids added plugin Concerns the wastewater plugin INTERLIS About INTERLIS exchange format (import / export) labels Jan 19, 2024
@3nids
Copy link
Contributor

3nids commented Jan 23, 2024

should be fixed in last plugin demo, please re-open if needed

@3nids 3nids closed this as completed Jan 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
INTERLIS About INTERLIS exchange format (import / export) plugin Concerns the wastewater plugin
Projects
None yet
Development

No branches or pull requests

3 participants