dan-da / cdc_audit Goto Github PK

I'm having an issue with the FOR EACH ROW BEGIN loop wrapping the insert statement. If I remove the BEGIN and END, creating the trigger works just fine.

Using mysql 5.5.

``
#1064 - You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '' at line 3

``

Working example,

DELIMITER @@ CREATE TRIGGERrate_types_after_insertAFTER INSERT ONrate_typesFOR EACH ROW insert intorate_types_audit (id, type, audit_event, audit_timestamp) values(NEW.id, NEW.type, 'insert', CURRENT_TIMESTAMP); @@

Have you thought about differentiating between the before update and after update column changes? I'm thinking the audit would be more valuable if you knew what the column values were changed from...

Overall though, amazing project! Love the ease of generating audit tables on the fly.

Hi @dan-da
I've got an update that adds support for a separate audit schema by extending the namespace capabilities. Additionally, the update adds a "snapshot" query to the end of every file generated, this query can be used to take a snapshot of the current state of the source table. This is useful for starting off a new audit table, or catching up a table thats falling out of sync.

It looks like I can't send in my pull request with the current security settings on the repo, if you would like the update please let me know that you've added me.

Thank you so much for putting this together, this tool helped me a lot.

Warning this is my first PHP code, hopefully the update will meet your quality standards, but if not let me know how to improve.

Unfortunately, PHP7 now uses PDO.
Did you update your scripts for that version of PHP ?