Marc Würth

Results 183 comments of Marc Würth

On your "P.S:" 1. Should be fixed now. 2. Since merging #102 there are actual models with a relationship set between them: https://github.com/robwilkerson/CakePHP-Audit-Log-Plugin/blob/cc1862c4217a97d69962699361261c7ec7ea7051/Model/Audit.php#L14 https://github.com/robwilkerson/CakePHP-Audit-Log-Plugin/blob/cc1862c4217a97d69962699361261c7ec7ea7051/Model/AuditDelta.php#L14

I'm sorry, I don't see the problem. The code in 2.1.0 works in my project and the behavior should make sure the models are bound here: https://github.com/robwilkerson/CakePHP-Audit-Log-Plugin/blob/2.1.0/Model/Behavior/AuditableBehavior.php#L171-L174 Could you try...

This should be taken into consideration when implementing #103.

A test would be nice to showcase the regression and its fix.

@xhs345 I guess we can close this due to inactivity and more, e.g. @robwilkerson closed the (partly?) related PR #15 due to inactivity. It looks, that @cdvrooman has long moved...

I agree, but without having looked at the code, I'm wondering why it does not do that already. :confused: That should certainly be possible. Could you track it down in...

@fr0z3nfyr it's not "my" repo, but I think the plugin should make it possible to save the additional data in HABTM associations. Personally I often use such a setup, too....

@fr0z3nfyr thanks. Can you reformat the whole data with four backticks and proper indentation? It's barely readable like this. 😼 How about: - false = no additional fields (default, current...

@fr0z3nfyr if data in such a post isn't readable, then it's losing its point 😼 Thanks for reformatting and the link. Now I see the problem with the amount of...

@xhs345 you are welcome! @fr0z3nfyr ``afterAuditProperty()`` is an event callback. Unfortunately they are not documented yet, as explained [here](https://github.com/robwilkerson/CakePHP-Audit-Log-Plugin/issues/82). Personally haven't used them. Those event callbacks seem to date back...