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;
+		}
 	}
 	
 	/**