[FEATURE] #13 Exclude configured admin users from logging

Author: therouv

src/app/code/community/FireGento/AdminMonitoring/Helper/Data.php

@@ -42,14 +42,30 @@ class FireGento_AdminMonitoring_Helper_Data extends Mage_Core_Helper_Abstract
     const ACTION_INSERT = 1;
     const ACTION_UPDATE = 2;
     const ACTION_DELETE = 3;
-    const ACTION_LOGIN  = 4;
+    const ACTION_LOGIN = 4;
 
     /**
      * STATUS
      */
     const STATUS_SUCCESS = 1;
     const STATUS_FAILURE = 2;
 
+    /**
+     * Checks if the given admin user id is excluded
+     *
+     * @param  int $userId User ID
+     * @return bool
+     */
+    public function isAdminUserIdExcluded($userId)
+    {
+        $excludedUserIds = explode(',', Mage::getStoreConfig('admin/firegento_adminmonitoring/exclude_admin_users'));
+        if (in_array($userId, $excludedUserIds)) {
+            return true;
+        }
+
+        return false;
+    }
+
     /**
      * Retrieve the attribute type by provided class name
      *
@@ -113,7 +129,7 @@ public function getRowUrl(FireGento_AdminMonitoring_Model_History $row)
      * @param  string $separator Separator
      * @return string
      */
-    public function getFullActionName($separator='_')
+    public function getFullActionName($separator = '_')
     {
         $request = array(
             Mage::app()->getRequest()->getModuleName(),

src/app/code/community/FireGento/AdminMonitoring/Model/History.php

@@ -36,6 +36,11 @@
  */
 class FireGento_AdminMonitoring_Model_History extends Mage_Core_Model_Abstract
 {
+    /**
+     * @var bool
+     */
+    protected $_forcedLogging = false;
+
     /**
      * Inits the resource model and resource collection model
      */
@@ -44,6 +49,35 @@ protected function _construct()
         $this->_init('firegento_adminmonitoring/history');
     }
 
+    /**
+     * Processing object before save data
+     *
+     * @return FireGento_AdminMonitoring_Model_History
+     */
+    protected function _beforeSave()
+    {
+        if (Mage::helper('firegento_adminmonitoring')->isAdminUserIdExcluded($this->getData('user_id'))
+            && !$this->_forcedLogging
+        ) {
+            $this->_dataSaveAllowed = false;
+        }
+
+        return parent::_beforeSave();
+    }
+
+    /**
+     * Set the forced logging value
+     *
+     * @param  bool $flag Flag
+     * @return FireGento_AdminMonitoring_Model_History
+     */
+    public function setForcedLogging($flag)
+    {
+        $this->_forcedLogging = $flag;
+
+        return $this;
+    }
+
     /**
      * Retrieve the original model
      *

src/app/code/community/FireGento/AdminMonitoring/etc/system.xml

@@ -69,6 +69,15 @@
                             <show_in_website>0</show_in_website>
                             <show_in_store>0</show_in_store>
                         </product_mass_update_logging>
+                        <exclude_admin_users translate="label">
+                            <label>Exclude admin users</label>
+                            <frontend_type>multiselect</frontend_type>
+                            <source_model>firegento_adminmonitoring/system_config_source_admin_user</source_model>
+                            <sort_order>50</sort_order>
+                            <show_in_default>1</show_in_default>
+                            <show_in_website>0</show_in_website>
+                            <show_in_store>0</show_in_store>
+                        </exclude_admin_users>
                     </fields>
                 </firegento_adminmonitoring>
             </groups>