Changeset 62310

Timestamp:

05/06/2026 09:44:53 PM (5 months ago)

Author:

jorgefilipecosta

Message:

Connectors: Register Akismet Anti-Spam as a connector.

Akismet is bundled with WordPress core, but it wasn't appearing on the Connectors screen even when present on the file system.

Partially reverts [62210], bringing back the main part of [62193].
Follow-up to [62210], [62193].
Developed in ​https://github.com/WordPress/wordpress-develop/pull/11688.

Props audrasjb, bluefuton, davidbaumwald, desrosj, eclev91, gziolo, johnbillion, jorbin, jorgefilipecosta, lukecarbis, matt, matveb, peterwilsoncc, westonruter.
See #65012.

Location:

trunk

Files:

• src/wp-includes/connectors.php (modified) (1 diff)
• tests/phpunit/tests/connectors/wpConnectorsGetConnectorSettings.php (modified) (3 diffs)

trunk/src/wp-includes/connectors.php

@@ -210 +210 @@
                 _wp_connectors_register_default_ai_providers( $registry );
         }
+
+        // Non-AI default connectors.
+        $registry->register(
+                'akismet',
+                array(
+                        'name'           => __( 'Akismet Anti-spam' ),
+                        'description'    => __( 'Protect your site from spam.' ),
+                        'type'           => 'spam_filtering',
+                        'plugin'         => array(
+                                'file'      => 'akismet/akismet.php',
+                                'is_active' => static function () {
+                                        return defined( 'AKISMET_VERSION' );
+                                },
+                        ),
+                        'authentication' => array(
+                                'method'          => 'api_key',
+                                'credentials_url' => 'https://akismet.com/get/',
+                                'setting_name'    => 'wordpress_api_key',
+                                'constant_name'   => 'WPCOM_API_KEY',
+                        ),
+                )
+        );
 
         /**

trunk/tests/phpunit/tests/connectors/wpConnectorsGetConnectorSettings.php

@@ -38 +38 @@
                 $this->assertArrayHasKey( 'openai', $connectors );
                 $this->assertArrayHasKey( 'anthropic', $connectors );
+                $this->assertArrayHasKey( 'akismet', $connectors );
                 $this->assertArrayHasKey( 'mock-connectors-test', $connectors );
-                $this->assertCount( 4, $connectors );
+                $this->assertCount( 5, $connectors );
         }
 
@@ -57 +58 @@
                         $this->assertIsString( $connector_data['description'], "Connector '{$connector_id}' description should be a string." );
                         $this->assertArrayHasKey( 'type', $connector_data, "Connector '{$connector_id}' is missing 'type'." );
-                        $this->assertContains( $connector_data['type'], array( 'ai_provider' ), "Connector '{$connector_id}' has unexpected type '{$connector_data['type']}'." );
+                        $this->assertContains( $connector_data['type'], array( 'ai_provider', 'spam_filtering' ), "Connector '{$connector_id}' has unexpected type '{$connector_data['type']}'." );
                         $this->assertArrayHasKey( 'authentication', $connector_data, "Connector '{$connector_id}' is missing 'authentication'." );
                         $this->assertIsArray( $connector_data['authentication'], "Connector '{$connector_id}' authentication should be an array." );
@@ -80 +81 @@
 
                         $this->assertArrayHasKey( 'setting_name', $connector_data['authentication'], "Connector '{$connector_id}' authentication is missing 'setting_name'." );
+                        $this->assertNotEmpty( $connector_data['authentication']['setting_name'], "Connector '{$connector_id}' setting_name should not be empty." );
+
+                        if ( 'ai_provider' !== $connector_data['type'] ) {
+                                continue;
+                        }
+
                         $this->assertSame(
                                 'connectors_ai_' . str_replace( '-', '_', $connector_id ) . '_api_key',
-                                $connector_data['authentication']['setting_name'] ?? null,
-                                "Connector '{$connector_id}' setting_name does not match expected format."
+                                $connector_data['authentication']['setting_name'],
+                                "Connector '{$connector_id}' setting_name does not match expected AI provider format."
                         );
                 }