Hi Ivan,
I updated the plugin with your useful comments. You have a version 0.4
available.
Regarding the schema tables names, I have filled a pull request here
<https://github.com/2ndQuadrant/audit-trigger/pull/2>https://github.com/2ndQuadrant/audit-trigger/pull/2/files
You can use my branch if you like:
https://github.com/3nids/audit-trigger/blob/schemasafe/audit.sql
I have an issue tracker on github:
https://github.com/3nids/postgres91plusauditor/issues
Thanks a lot for your comments and for your time at testing, it's really
appreciated! I am waiting for these bugs ;)
Greetings,
Denis
On 05/22/2013 01:53 PM, Ivan Mincik wrote:
Hi Denis,
I have found a problem. The problem was that menu item 'Define logged
action layer' was confusing for me and I was choosing my PostGIS
geometry layer there. I recommend to rename it to something like
'Choose audit history table' (avoid using 'layer') with some
description that the table should be loaded to QGIS. Now 'search
history' windows opens correctly.
Also I recommend to exclude at least any geometry tables from 'logged
action layer chooser' dialog and also exclude 'history table' from
choosing itself from selecbox in 'search history' window.
One more question, how do you activate audit if table exists in some
schema outside of default search_path ? Using
SELECT audit.audit_table('<schema>.<table>');
is not working for me.
I have found some other bugs which will be better to submit in bug
tracker. Do you have some ?
I have hesitated with creating something like your approach, but I
decided to use something which is "official". As, the trigger is
available on postgres website, I hope it will last a bit.
I see one big disadvantage with this Audit trigger implementation
which is performance, if history of all tables is recorded to one huge
table. If you have many changes in one big table it will affect all
other. Moreover I am missing tests (which of course can be added).
Thanks,
Ivan
_______________________________________________
Qgis-developer mailing list
[email protected]
http://lists.osgeo.org/mailman/listinfo/qgis-developer
