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

audit trigger: problem with schema #2

Open
3nids opened this issue Feb 21, 2013 · 2 comments
Open

audit trigger: problem with schema #2

3nids opened this issue Feb 21, 2013 · 2 comments

Comments

@3nids
Copy link

3nids commented Feb 21, 2013

The quote_ident calls are useless and prevent the use of the function with schema defined regclass (i.e. schema.table).
So, to make the trigger available on other schemas than public, I had to remove the quote_ident.

@ringerc
Copy link
Owner

ringerc commented Mar 7, 2013

It looks like you're talking about audit.sql. If so, the quote_ident calls are moderately useful (since table names can include SQL metachars) but it should instead really use format() and the %I specifier, since that's properly schema-aware. If I get the chance I'll patch this; otherwise it's pretty easy for you to convert by replacing string concatenation with format using %L and %I.

@3nids
Copy link
Author

3nids commented Apr 11, 2013

Hi, do you have by any chance some time to fix this?
Thanks a lot,
Denis

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

No branches or pull requests

2 participants