[MAGNOLIA-4778] Audit log is (partially) not working since 4.5 Created: 17/Jan/13 Updated: 24/Oct/13 Resolved: 09/May/13 |
|
| Status: | Closed |
| Project: | Magnolia |
| Component/s: | core |
| Affects Version/s: | None |
| Fix Version/s: | 4.5.9 |
| Type: | Bug | Priority: | Critical |
| Reporter: | Magnolia International | Assignee: | Milan Divilek |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||||||||||||||||||||||||||||||||||
| Template: |
|
||||||||||||||||||||||||||||||||||||||||
| Acceptance criteria: |
Empty
|
||||||||||||||||||||||||||||||||||||||||
| Task DoD: |
[ ]*
Doc/release notes changes? Comment present?
[ ]*
Downstream builds green?
[ ]*
Solution information and context easily available?
[ ]*
Tests
[ ]*
FixVersion filled and not yet released
[ ] 
Architecture Decision Record (ADR)
|
||||||||||||||||||||||||||||||||||||||||
| Bug DoR: |
[ ]*
Steps to reproduce, expected, and actual results filled
[ ]*
Affected version filled
|
||||||||||||||||||||||||||||||||||||||||
| Description |
|
The audit log classes are only used used when using the Content API. When using the JCR APIs, it seems they're just never used. This could be reimplemented by using the wrappers. I understand that all our session objects are already wrapped, so it shouldn't be too big a deal. Also linking further improvement to the audit log API - to take into account, not necessarily implement as a fix to this issue. Concept page: http://wiki.magnolia-cms.com/display/DEV/Concept+-+Move+AuditLog+from+Content+API+to+JCR+API |
| Comments |
| Comment by Magnolia International [ 03/May/13 ] |
|
Also consider some of the issues linked to |
| Comment by Magnolia International [ 08/May/13 ] |
|
This is breaking a bunch of other builds/tests, among which DAM and Imaging. if ("imaging".equals(logicalWorkspaceName) || "Expressions".equals(logicalWorkspaceName) || "Store".equals(logicalWorkspaceName)) { // don't wrap imaging, expression and store into audit logging session return new MgnlVersioningSession(session); } Stuff should be in info.magnolia.audit, not .logging |