Merge branch "release/v1.0.1"

Author: rhukster

CHANGELOG.md

@@ -1,3 +1,11 @@
+# v1.0.1
+## 04/29/2026
+
+1. [](#bugfix)
+    * Fixed PHP 8.1+ deprecation notice — explicit string casts where `null` was being passed to string-typed function arguments.
+    * Fixed `Cannot use bool as array` fatal on PHP 8.x when the `slack_members` cache entry hadn't been seeded yet. `Cache::fetch()` returns `false` on a miss, but the plugin was destructuring the return value directly. Now guards on `is_array()` before unpacking.
+    * Skip all Slack API work when `slack_token` is empty. Previously the plugin would still hit Slack on every frontend request even without a token, paying ~200ms of network round-trip just to receive a 401 — slowing down every page on a freshly-installed-but-unconfigured site.
+
 # v1.0.0
 ## 02/04/2017
 

blueprints.yaml

@@ -1,5 +1,5 @@
 name: Slack Invite
-version: 1.0.0
+version: 1.0.1
 description: Easily invite users to Slack
 icon: slack
 author:
@@ -12,6 +12,9 @@ bugs: https://github.com/trilbymedia/grav-plugin-slack-invite/issues
 docs: https://github.com/trilbymedia/grav-plugin-slack-invite/blob/develop/README.md
 license: MIT
 
+compatibility:
+  grav: ['1.7', '2.0']
+
 dependencies:
     - { name: form, version: '~2.0' }
 
@@ -42,3 +45,11 @@ form:
       size: x-large
       label: Slack Token
       help: You must generate a token from the Slack API site
+
+    blacklist:
+        type: selectize
+        size: large
+        label: Blacklist
+        classes: fancy
+        validate:
+            type: commalist

slack-invite.php

@@ -47,18 +47,31 @@ public function onPluginsInitialized()
             return;
         }
 
+        // No token configured — skip everything. Without this guard the plugin
+        // would still hit Slack on every frontend request and pay the network
+        // round-trip just to get back a 401, blocking page render for ~200ms.
+        // Also rejects the shipped placeholder ('XXX-XXXX...') that survives
+        // a fresh install when the site owner hasn't yet swapped in their key.
+        $token = $this->config['plugins.slack-invite.slack_token'] ?? '';
+        if (!$token || str_starts_with((string) $token, 'XXX-')) {
+            return;
+        }
+
         // invite the user
         require_once __DIR__ . '/vendor/autoload.php';
 
         $interactor = new CurlInteractor();
         $interactor->setResponseFactory(new SlackResponseFactory);
 
         /** @var Commander $commander */
-        $this->slack = new Commander($this->config['plugins.slack-invite.slack_token'], $interactor);
+        $this->slack = new Commander($token, $interactor);
 
         $cache = $this->grav['cache'];
 
-        list($this->total_members, $this->active_members) = $cache->fetch('slack_members');
+        $cached = $cache->fetch('slack_members');
+        if (is_array($cached) && count($cached) === 2) {
+            [$this->total_members, $this->active_members] = $cached;
+        }
 
         if (!$this->total_members) {
 
@@ -127,7 +140,7 @@ public function onFormProcessed(Event $event)
             case 'slack-invite':
                 // make sure we have the email
                 $email = $form->value('email');
-                if ($email) {
+                if ($email && $this->isValidUser($email)) {
 
                     /** @var SlackResponse $response */
                     $response = $this->slack->execute('users.admin.invite', [
@@ -158,4 +171,17 @@ public function onFormProcessed(Event $event)
                 break;
         }
     }
+
+    protected function isValidUser($email)
+    {
+        $blacklist = (array) $this->config['plugins.slack-invite.blacklist'];
+
+        foreach($blacklist as $var) {
+            if (strpos((string) $email, (string) $var) !== false) {
+                return false;
+            }
+        }
+
+        return true;
+    }
 }

slack-invite.yaml

@@ -1,4 +1,5 @@
 enabled: true
 cache_timeout: 3600
 slack_token: XXX-XXXXXXXXXXXXX-XXXXXXXXXXXXX-XXXXXXXXXXXXX
+blacklist: