Problem/Motivation

This patch should improve the approach to saving the module's data, also it is designed to improve a flexibility with logs.

Why should we use entities?

Entity API is the fundamental ideology of Drupal. I'm researched with some of these questions before creating a patch:

1. Separate Storage Backends into submodules - This issue has not been active for the last 8 months, but entity approch allows to override an entity store interface at least (although this issue will require additional development).

2. Provide views integration - Good implementation, but there is to much custom code, Drupal already has a great views implementation and we shouldn't clone to much custom code.

User interface changes

User will be able to create a views with audit log (see. Remaining tasks)

Remaining tasks

Provide a user interface (in progress)

API changes

We will need to add version 2.x because the patch is uncompatible with earlier versions at the moment.

Сonclusion

Our team is now actively developing this module and we are open to suggestions on github:
https://github.com/propeopleua/audit_log

Comments

spheresh created an issue. See original summary.

spheresh’s picture

StatusFileSize
new46.33 KB
spheresh’s picture

Issue summary: View changes
spheresh’s picture

Status: Active » Needs review
StatusFileSize
new43.32 KB

Code review & bug fixing.

Status: Needs review » Needs work

The last submitted patch, 4: 2936158-entity-integration.patch, failed testing. View results

spheresh’s picture

StatusFileSize
new37.87 KB
new9.02 KB

Refactoring & small fixes:
- composer package name was changed to correct
- .circleci config was removed as non-required for contrib module
- Error when accessing "/admin/structure/audit_log/settings" fix

  • spheresh committed 9479523 on 8.x-2.x
    Issue #2936158 by spheresh: Audit log entity storage implementation
    
spheresh’s picture

Version: 8.x-1.x-dev » 8.x-2.x-dev
Status: Needs work » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.