search

xiidea / EasyAuditBundle

A Symfony Bundle To Log Selective Events
http://xiidea.github.io/EasyAuditBundle/
MIT License
89 stars 22 forks source link

AuditLog::setUser()- proper way use User object instead of property #20

Closed DangerLifter closed 6 years ago

ronisaha commented 6 years ago

No, That will not be a proper way for various reasons.

  1. AuditLog Should not have foreignKey reference with any entity
  2. AuditLog is not only for use with ORM
  3. The user property is configurable through a configuration option to allow user the choice to store username/fullName/id whichever suits the needs

Thanks