Merge pull request #5 from timkung/pulls/validate-fix

fixing validate method to return false and making PSR-2 compliant
camspiers · Oct 21, 2015 · 297dee3 · 297dee3
2 parents 3e3a28b + 3b7fd21
commit 297dee3
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/code/HoneyPotField.php b/code/HoneyPotField.php
@@ -9,10 +9,10 @@ public function validate($validator)
             $validator->validationError(
                 $this->name,
                 "This field should not be completed",
-                "validation",
-                false
+                "validation"
             );
 
+            return false;
         }
 
         return true;
@@ -23,4 +23,4 @@ public function FieldHolder($properties = array())
         return $this->renderWith('HoneyPotField_holder');
     }
 
-}
+}
diff --git a/composer.json b/composer.json
@@ -2,7 +2,7 @@
     "name": "camspiers/honeypot",
     "type": "silverstripe-module",
     "require": {
-        "silverstripe/spamprotection": "~1.2.0",
+        "silverstripe/spamprotection": "~2.0.0",
         "composer/installers": "~1.0"
     }
 }