robwilkerson / CakePHP-Audit-Log-Plugin

Records changes made to an object during CRUD operations.
104 stars 78 forks source link

Make description field more flexible #84

Closed ravage84 closed 8 years ago

ravage84 commented 8 years ago

The description field seems to be filled by the currentUser() method:

https://github.com/robwilkerson/CakePHP-Audit-Log-Plugin/blob/c69e83c2929d2f4896f34e9d12743f0e701de07f/Model/Behavior/AuditableBehavior.php#L165

This functionality is not dcumented anywhere and I doubt most people have a column description in their users table, thus this field stays empty in the audits table all the time.

It should be configurable, falling back to the current field name description for backwards compatibilty.

xhs345 commented 8 years ago

I updated the documentation and added a note about the field.

I agree that having something more configurable would be good

ravage84 commented 8 years ago

I guess the documentation fix does suffice. :bow: