fix: if no spam protector set, fail sliently

.gitignore

@@ -1 +1,5 @@
-.DS_Store
+.DS_Store
+/vendor
+/composer.lock
+/public
+.phpunit.result.cache

README.md

@@ -9,14 +9,6 @@
 composer require silverstripe/spamprotection
 ```
 
-## Maintainer Contact
-
- * Saophalkun Ponlu
-   <phalkunz (at) silverstripe (dot) com>
-
- * Will Rossiter
-   <will (at) fullscreen (dot) io>
-
 ## Documentation
 
 This module provides a generic, consistent API for adding spam protection to
@@ -40,7 +32,7 @@ need to rebuild your database through `dev/build` and set the default protector
 via SilverStripe's config system. This will update any Form instances that have
 spam protection hooks with that protector.
 
-*mysite/_config/spamprotection.yml*
+*app/_config/spamprotection.yml*
 
 ```yaml
 ---
@@ -67,10 +59,10 @@ implementation client side or server side.
 `enableSpamProtection` takes a hash of optional configuration values.
 
 ```php
-$form->enableSpamProtection(array(
+$form->enableSpamProtection([
     'protector' => MathSpamProtector::class,
     'name' => 'Captcha'
-));
+]);
 ```
 
 Options to configure are:
@@ -121,6 +113,7 @@ class CustomSpamProtector implements SpamProtector
 }
 ```
 
+
 ## Using Spam Protection with User Forms
 
 This module provides an `EditableSpamProtectionField` wrapper which you can add

composer.json

@@ -6,16 +6,6 @@
         "silverstripe",
         "spamprotection"
     ],
-    "authors": [
-        {
-            "name": "Saophalkun Ponlu",
-            "email": "[email protected]"
-        },
-        {
-            "name": "Will Rossiter",
-            "email": "[email protected]"
-        }
-    ],
     "require": {
         "php": "^8.1",
         "silverstripe/framework": "^5"
@@ -33,11 +23,17 @@
     },
     "autoload": {
         "psr-4": {
-            "SilverStripe\\SpamProtection\\": "code/",
+            "SilverStripe\\SpamProtection\\": "src/",
             "SilverStripe\\SpamProtection\\Tests\\": "tests/"
         }
     },
     "license": "BSD-3-Clause",
     "minimum-stability": "dev",
-    "prefer-stable": true
+    "prefer-stable": true,
+    "config": {
+        "allow-plugins": {
+            "composer/installers": true,
+            "silverstripe/vendor-plugin": true
+        }
+    }
 }

phpcs.xml.dist

@@ -2,7 +2,7 @@
 <ruleset name="SilverStripe">
     <description>CodeSniffer ruleset for SilverStripe coding conventions.</description>
 
-    <file>code</file>
+    <file>src</file>
     <file>tests</file>
 
     <rule ref="PSR2" >

phpunit.xml.dist

@@ -8,7 +8,7 @@
 
 	<filter>
 		<whitelist addUncoveredFilesFromWhitelist="true">
-			<directory suffix=".php">code/</directory>
+			<directory suffix=".php">src/</directory>
 			<exclude>
 				<directory suffix=".php">tests/</directory>
 			</exclude>

code/EditableSpamProtectionField.php → src/EditableSpamProtectionField.php

@@ -3,7 +3,6 @@
 namespace SilverStripe\SpamProtection;
 
 use SilverStripe\Core\ClassInfo;
-use SilverStripe\Core\Convert;
 use SilverStripe\Core\Manifest\ModuleLoader;
 use SilverStripe\Forms\DropdownField;
 use SilverStripe\Forms\FieldGroup;
@@ -41,15 +40,15 @@ class EditableSpamProtectionField extends EditableFormField
      * @var array
      * @config
      */
-    private static $check_fields = array(
+    private static $check_fields = [
         EditableEmailField::class,
         EditableTextField::class,
         EditableNumericField::class
-    );
+    ];
 
-    private static $db = array(
+    private static $db = [
         'SpamFieldSettings' => 'Text'
-    );
+    ];
 
     /**
      * @var FormField
@@ -69,12 +68,14 @@ public function getFormField()
         }
 
         // Extract saved field mappings and update this field.
-        $fieldMapping = array();
+        $fieldMapping = [];
+
         foreach ($this->getCandidateFields() as $otherField) {
             $mapSetting = "Map-{$otherField->Name}";
             $spamField = $this->spamMapValue($mapSetting);
             $fieldMapping[$otherField->Name] = $spamField;
         }
+
         $protector->setFieldMapping($fieldMapping);
 
         // Generate field
@@ -106,7 +107,8 @@ protected function getCandidateFields()
 
         // Get list of all configured classes available for spam detection
         $types = $this->config()->get('check_fields');
-        $typesInherit = array();
+        $typesInherit = [];
+
         foreach ($types as $type) {
             $subTypes = ClassInfo::subclassesFor($type);
             $typesInherit = array_merge($typesInherit, $subTypes);
@@ -131,8 +133,9 @@ protected function getCandidateFields()
     public function onBeforeWrite()
     {
         $fieldMap = json_decode($this->SpamFieldSettings ?? '', true);
+
         if (empty($fieldMap)) {
-            $fieldMap = array();
+            $fieldMap = [];
         }
 
         foreach ($this->record as $key => $value) {
@@ -202,7 +205,7 @@ public function spamMapValue($mapSetting)
     {
         $map = json_decode($this->SpamFieldSettings ?? '', true);
         if (empty($map)) {
-            $map = array();
+            $map = [];
         }
 
         if (array_key_exists($mapSetting, $map ?? [])) {
@@ -255,16 +258,19 @@ public function validateField($data, $form)
         }
     }
 
+
     public function getFieldValidationOptions()
     {
         return FieldList::create();
     }
 
+
     public function getRequired()
     {
         return false;
     }
 
+
     public function getIcon()
     {
         $resource = ModuleLoader::getModule('silverstripe/spamprotection')

code/Extension/CommentSpamProtection.php → src/Extension/CommentSpamProtection.php

@@ -14,19 +14,19 @@ class CommentSpamProtection extends Extension
 {
     public function alterCommentForm(&$form)
     {
-        $form->enableSpamProtection(array(
+        $form->enableSpamProtection([
             'name' => 'IsSpam',
-            'mapping' => array(
+            'mapping' => [
                 'Name' => 'authorName',
                 'Email' => 'authorEmail',
                 'URL' => 'authorUrl',
                 'Comment' => 'body',
                 'ReturnURL' => 'contextUrl'
-            ),
-            'checks' => array(
+            ],
+            'checks' => [
                 'spam',
                 'profanity'
-            )
-        ));
+            ]
+        ]);
     }
 }

code/Extension/FormSpamProtectionExtension.php → src/Extension/FormSpamProtectionExtension.php

@@ -6,6 +6,7 @@
 use SilverStripe\Core\Config\Configurable;
 use SilverStripe\Core\Extension;
 use SilverStripe\Core\Injector\Injector;
+use SilverStripe\View\Requirements;
 
 /**
  * An extension to the {@link Form} class which provides the method
@@ -37,7 +38,7 @@ class FormSpamProtectionExtension extends Extension
      *
      * @var array $mappable_fields
      */
-    private static $mappable_fields =  array(
+    private static $mappable_fields =  [
         'id',
         'title',
         'body',
@@ -48,7 +49,7 @@ class FormSpamProtectionExtension extends Extension
         'authorUrl',
         'authorIp',
         'authorId'
-    );
+    ];
 
     /**
      * @config
@@ -88,7 +89,7 @@ public static function get_protector($options = null)
      * @throws LogicException when get_protector method returns NULL.
      * @return Object
      */
-    public function enableSpamProtection($options = array())
+    public function enableSpamProtection($options = [])
     {
 
         // captcha form field name (must be unique)
@@ -109,7 +110,8 @@ public function enableSpamProtection($options = array())
         $protector = self::get_protector($options);
 
         if ($protector === null) {
-            throw new LogicException('No spam protector has been set. Null is not valid value.');
+            Requirements::customScript('console.error("No spam protector has been set on this form.")');
+            return $this->owner;
         }
 
         if ($protector && isset($options['mapping'])) {