clearbold/audit-pro

Audit Pro is the missing activity tracker for ExpressionEngine. It uses any available Control Panel hooks to log creating, editing or deleting entries as well as other activity.

Clearbold, LLC

audit-pro's Issues

set_variable depreciated in 2.6.0 causes error in function manage_notifications

Fixed by adding ee_version check to construct

    //normal
    if (defined('APP_VER'))
    {
        $this->ee_version = APP_VER;
    }
    //install wizard
    else if (isset(ee()->version))
    {
        $this->ee_version = ee()->version;
    }

And in manage_notifications function

    //set_variable depracated in 2.6.0
    if (version_compare($this->ee_version, '2.6.0', '>='))
    {
        $this->EE->view->cp_page_title = 'Manage Notifications';
    }
    else
    {
        $this->EE->cp->set_variable('cp_page_title', 'Manage Notifications');
    }

Multiple emails...

Would it be possible for you to add some functionality to this?

We would like to use something like https://github.com/vector/VMG-Chosen-Member#single-variable-tag that would add a custom field to entries, this field outputs a pipe delimited list of members IDs.

So when an entry is added or updated Audit pro would ONLY email the members from the custom field on the entry that was updated.

Manage Notifications: Fatal error [EE 2.8.1]

Fatal error: Call to undefined method Cp::set_variable() in /system/expressionengine/third_party/audit_pro/mcp.audit_pro.php on line 194

Fatal error: Call to undefined method Cp::set_variable() in mcp.aidut_pro.php

Fatal error: Call to undefined method Cp::set_variable() in C:\inetpub\wwwroot\WEBSITE\system\expressionengine\third_party\audit_pro\mcp.audit_pro.php on line 26

Recently updated to EE 2.8.0. Had to run updates for almost every installed plugin, would appear that Audit Pro needs some love too.

Can you add an explicit license to this?

(ideally MIT or BSD 3-clause.) I'm VERY hesitant to fork, extend, or even USE code that doesn't have an explicit license!

Pagination

In the current version of AP, we're having trouble paging through entries on the main module page (the pagination links don't work). Also, the "manage notification" button the module entry throws the following message.

Fatal error: Call to undefined method Cp::set_variable() in /var/www/sjsys/expressionengine/third_party/audit_pro/mcp.audit_pro.php on line 194

List of member usernames can throw a js error

Where you echo out the list of usernames in mcp_manage_notifications.php (line 57) it's possible for a username to contain a double quote or line break, which will break js.

Maybe do something like this?

<option value=\"<?php echo $member['member_id'] ?>\" ><?php echo addslashes(trim($member['username'])) ?></option>\

Or, rather than injecting directly into the js/markup, use a js template with Mustache or Handlebars and pass it JSON instead.

set_human_time depreciated in 2.6 causes error

Causes error on ext.audit_pro.php on line 534

EE3 support

Will it come?

Best,
Rein

Error Number: 1366 Incorrect integer value: ‘’ for column ‘item_id’ at row 1

I am getting an error when trying to add a notification error as follows:
Any pointers?
thanks.
Error Number: 1366
Incorrect integer value: ‘’ for column ‘item_id’ at row 1
INSERT INTO exp_audit_notifications (email_address, is_sms, member_id, group_id, item_type, channel_id, item_id) VALUES (‘[email protected]’, 0, 0, 6, ‘entry_update’, ‘0’, ‘’)
Filename: third_party/audit_pro/mcp.audit_pro.php
Line Number: 295

Audit changed fields

Hi!

Are there any plans to audit the changed fields (old and new values) as well? It would be great to see the changed fields including there values in the mail.

Regards