wip

Author: osbre

.editorconfig

@@ -7,3 +7,4 @@ insert_final_newline = true
 indent_style = space
 indent_size = 4
 trim_trailing_whitespace = true
+max_line_length = 80

src/Plugin.php

@@ -9,25 +9,29 @@
 use SimpleAnalytics\Scripts\AutomatedEventsScript;
 use SimpleAnalytics\Scripts\InactiveScript;
 
-class Plugin
+final class Plugin
 {
     use PluginLifecycle;
 
+    #[\Override]
     protected function onBoot(): void
     {
         if (is_admin()) $this->defineAdminPage();
     }
 
+    #[\Override]
     public function onActivation(): void
     {
-
+        $this->registerOptions();
     }
 
-    public function onDeactivation(): void
+    #[\Override]
+    public function onUninstall(): void
     {
-
+        $this->deleteOptions();
     }
 
+    #[\Override]
     public function onInit(): void
     {
         $shouldCollect = (new TrackingPolicy)->shouldCollectAnalytics();
@@ -43,6 +47,25 @@ public function onInit(): void
         }
     }
 
+    /**
+     * Register all options with autoload=true
+     * to preload them on each page and avoid
+     * extra database queries.
+     */
+    protected function registerOptions(): void
+    {
+        foreach (SettingName::cases() as $name) {
+            add_option($name, null, null, true);
+        }
+    }
+
+    protected function deleteOptions(): void
+    {
+        foreach (SettingName::cases() as $name) {
+            delete_option($name);
+        }
+    }
+
     protected function addScripts(bool $collect): void
     {
         $scripts = new ScriptManager;

src/PluginLifecycle.php

@@ -18,5 +18,5 @@ abstract public function onInit(): void;
 
     abstract public function onActivation(): void;
 
-    abstract public function onDeactivation(): void;
+    abstract public function onUninstall(): void;
 }

src/SettingName.php

@@ -27,4 +27,9 @@ class SettingName
     const string EVENT_USE_TITLE = 'simpleanalytics_event_use_title';
     const string EVENT_FULL_URLS = 'simpleanalytics_event_full_urls';
     const string EVENT_SA_GLOBAL = 'simpleanalytics_event_sa_global';
+
+    static function cases(): array
+    {
+        return (new \ReflectionClass(__CLASS__))->getConstants();
+    }
 }

src/Settings/Blocks/CalloutBlock.php

@@ -12,6 +12,7 @@ public function __construct(
     ) {
     }
 
+    #[\Override]
     public function render(): void
     {
         ?>