silverstripe / silverstripe-akismet

code/AkismetField.php

Braces +24 added lines, -8 removed lines

@@ -27,7 +27,9 @@ 
 	protected function confirmationField() {
 		// Check if confirmation is required
 		$requireConfirmation = Config::inst()->get('AkismetSpamProtector', 'require_confirmation');
-		if(empty($requireConfirmation)) return null;
+		if(empty($requireConfirmation)) {
+			return null;
+		}
 		
 		// If confirmation is required then return a checkbox
 		return CheckboxField::create(
@@ -41,19 +43,25 @@ 
 	
 	public function Field($properties = array()) {
 		$checkbox = $this->confirmationField();
-		if($checkbox) return $checkbox->Field($properties);
+		if($checkbox) {
+			return $checkbox->Field($properties);
+		}
 	}
 	
 	function FieldHolder($properties = array()) {
 		$checkbox = $this->confirmationField();
-		if($checkbox) return $checkbox->FieldHolder($properties);
+		if($checkbox) {
+			return $checkbox->FieldHolder($properties);
+		}
 	}
 	
 	/**
 	 * @return array
 	 */
 	public function getSpamMappedData() {
-		if(empty($this->fieldMapping)) return null;
+		if(empty($this->fieldMapping)) {
+			return null;
+		}
 		
 		$result = array();
 		$data = $this->form->getData();
@@ -93,7 +101,9 @@ 
 		
 		// Check result
 		$isSpam = $this->getIsSpam();
-		if(!$isSpam) return true;
+		if(!$isSpam) {
+			return true;
+		}
 
 		// Save error message
 		$errorMessage = _t(
@@ -127,15 +137,21 @@ 
 	 */
 	public function getIsSpam() {
 		// Prevent multiple API calls
-		if($this->isSpam !== null) return $this->isSpam;
+		if($this->isSpam !== null) {
+			return $this->isSpam;
+		}
 
 		// Check bypass permission
 		$permission = Config::inst()->get('AkismetSpamProtector', 'bypass_permission');
-		if($permission && Permission::check($permission)) return false;
+		if($permission && Permission::check($permission)) {
+			return false;
+		}
 
 		// if the user has logged and there's no force check on member
 		$bypassMember = Config::inst()->get('AkismetSpamProtector', 'bypass_members');
-		if($bypassMember && Member::currentUser()) return false;
+		if($bypassMember && Member::currentUser()) {
+			return false;
+		}
 
 		// Map input fields to spam fields
 		$mappedData = $this->getSpamMappedData();

code/AkismetSpamProtector.php

Braces +9 added lines, -3 removed lines

@@ -79,14 +79,20 @@
 	 * @return string
 	 */
 	protected static function get_api_key() {
-		if(self::$_api_key) return self::$_api_key;
+		if(self::$_api_key) {
+			return self::$_api_key;
+		}
 		
 		// Check config
 		$key = Config::inst()->get('AkismetSpamProtector', 'api_key');
-		if(!empty($key)) return $key;
+		if(!empty($key)) {
+			return $key;
+		}
 		
 		// Check environment
-		if(defined('SS_AKISMET_API_KEY')) return SS_AKISMET_API_KEY;
+		if(defined('SS_AKISMET_API_KEY')) {
+			return SS_AKISMET_API_KEY;
+		}
 	}
 	
 	/**

code/service/AkismetService.php

Indentation +30 added lines, -30 removed lines

@@ -6,34 +6,34 @@
 interface AkismetService {
 	
 	/**
-     * Check if the comment is spam or not
-     * This is basically the core of everything. This call takes a number of
-     * arguments and characteristics about the submitted content and then
-     * returns a thumbs up or thumbs down.
-     * Almost everything is optional, but performance can drop dramatically if
-     * you exclude certain elements.
-     * REMARK: If you are having trouble triggering you can send
-     * "viagra-test-123" as the author and it will trigger a true response,
-     * always.
-     *
-     * @param string[optional] $content   The content that was submitted.
-     * @param string[optional] $author    The name.
-     * @param string[optional] $email     The email address.
-     * @param string[optional] $url       The URL.
-     * @param string[optional] $permalink The permanent location of the entry
-     *                                    the comment was submitted to.
-     * @param string[optional] $type The type, can be blank, comment,
-     *                                    trackback, pingback, or a made up
-     *                                    value like "registration".
-     * @return bool If the comment is spam true will be
-     *                                    returned, otherwise false.
-     */
-    public function isSpam(
-        $content,
-        $author = null,
-        $email = null,
-        $url = null,
-        $permalink = null,
-        $type = null
-    );
+	 * Check if the comment is spam or not
+	 * This is basically the core of everything. This call takes a number of
+	 * arguments and characteristics about the submitted content and then
+	 * returns a thumbs up or thumbs down.
+	 * Almost everything is optional, but performance can drop dramatically if
+	 * you exclude certain elements.
+	 * REMARK: If you are having trouble triggering you can send
+	 * "viagra-test-123" as the author and it will trigger a true response,
+	 * always.
+	 *
+	 * @param string[optional] $content   The content that was submitted.
+	 * @param string[optional] $author    The name.
+	 * @param string[optional] $email     The email address.
+	 * @param string[optional] $url       The URL.
+	 * @param string[optional] $permalink The permanent location of the entry
+	 *                                    the comment was submitted to.
+	 * @param string[optional] $type The type, can be blank, comment,
+	 *                                    trackback, pingback, or a made up
+	 *                                    value like "registration".
+	 * @return bool If the comment is spam true will be
+	 *                                    returned, otherwise false.
+	 */
+	public function isSpam(
+		$content,
+		$author = null,
+		$email = null,
+		$url = null,
+		$permalink = null,
+		$type = null
+	);
 }