INTERLIS-Export: dataowner and provider are not implemented

[urskaufmann]

Describe the bug
Datenherr and Datenlieferant are exported always as unknown, also when there are values in this fields.

This is not a bug (it's documentated in the code), but the correct values are not implemented.

[sjib]

See #36 for more background and considerations

[sjib]

Needs adaption here:
datenherr=getattr(row.fk_dataowner__REL, "name", "unknown"), # TODO : is unknown ok ?
datenlieferant=getattr(row.fk_provider__REL, "name", "unknown"), # TODO : is unknown ok ?

See

qgepqwat2ili/qgepqwat2ili/qgep/export.py

Line 96 in df62333

def create_metaattributes(row):

Suggested Change Options:
a) Change "name" to "identifier" as there is no column "name" in qgep datamodel in class organisation to get the identifier in case there is no fk_ set
b) It should only set 'unknown' if there is no fk_provider set. Just add 'unknown' if there is no fk_ set (becaus if NULL then we get an error in the INTERLIS file) and else leave the foreignkey value

[sjib]

With identifier instead of name:
datenherr=getattr(row.fk_dataowner__REL, "identifier", "unknown"), # TODO : is unknown ok ?
datenlieferant=getattr(row.fk_provider__REL, "identifier", "unknown"), # TODO : is unknown ok ?

[sjib]

With obj_id instead of name:
datenherr=getattr(row.fk_dataowner__REL, "obj_id", "unknown"), # TODO : is unknown ok ?
datenlieferant=getattr(row.fk_provider__REL, "obj_id", "unknown"), # TODO : is unknown ok ?

[sjib]

Please vote with thumb which of the two option should be implemented.

[sjib]

Should be correct now with obj:

qgepqwat2ili/qgepqwat2ili/qgep/export.py

Line 123 in 2455a6a

datenherr=getattr(row.fk_dataowner__REL, "obj_id", "unknown"), # TODO : is unknown ok ?

qgepqwat2ili/qgepqwat2ili/qgepdss/export.py

Line 128 in 2455a6a

datenherr=getattr(row.fk_dataowner__REL, "obj_id", "unknown"), # TODO : is unknown ok ?

qgepqwat2ili/qgepqwat2ili/qgepsia405/export.py

Line 121 in 2455a6a

datenherr=getattr(row.fk_dataowner__REL, "obj_id", "unknown"), # TODO : is unknown ok ?

@urskaufmann Please check and confirm, then this can be closed.