Merge pull request #75 from creative-commoners/pulls/4/hook

ENH Use extension hook instead of proxy-db
silverstripe · Apr 29, 2024 · b115bc7 · b115bc7
2 parents 047d9ad + 6913409
commit b115bc7
Show file tree

Hide file tree

Showing 5 changed files with 19 additions and 37 deletions.
diff --git a/_config/config.yml b/_config/config.yml
@@ -18,6 +18,10 @@ SilverStripe\Security\Member:
   extensions:
     - SilverStripe\Auditor\AuditHook
 
+SilverStripe\ORM\Connect\Database:
+  extensions:
+    - SilverStripe\Auditor\Extensions\DatabaseExtension
+
 ---
 Name: auditorcms
 Only:

diff --git a/_config/dbproxy.yml b/_config/dbproxy.yml
diff --git a/code/Extensions/DatabaseExtension.php b/code/Extensions/DatabaseExtension.php
@@ -0,0 +1,14 @@
+<?php
+
+namespace SilverStripe\Auditor\Extensions;
+
+use SilverStripe\Core\Extension;
+use SilverStripe\Auditor\AuditHook;
+
+class DatabaseExtension extends Extension
+{
+    public function onBeforeManipulate(array &$manipulation)
+    {
+        AuditHook::handle_manipulation($manipulation);
+    }
+}
diff --git a/code/Extensions/ProxyDBExtension.php b/code/Extensions/ProxyDBExtension.php
diff --git a/composer.json b/composer.json
@@ -11,8 +11,7 @@
         "php": "^8.1",
         "silverstripe/framework": "^6",
         "monolog/monolog": "^3.2",
-        "psr/log": "^3",
-        "silverstripe/silverstripe-proxy-db": "^3"
+        "psr/log": "^3"
     },
     "require-dev": {
         "phpunit/phpunit": "^9.6",