BetterBrief / silverstripe-googlemapfield

code/GoogleMapField.php

Doc Comments +3 added lines

@@ -98,6 +98,9 @@
 		return $this->data->$fieldNames['lng'];
 	}
 
+	/**
+	 * @param string $name
+	 */
 	public function getOption($name) {
 		// Quicker execution path for "."-free names
 		if (strpos($name, '.') === false) {

Spacing +9 added lines, -9 removed lines

@@ -48,10 +48,10 @@ 
 	}
 
 	public function Field($properties = array()) {
-		$key = $this->options['apikey'] ? "&key=".$this->options['apikey'] : "";
-		Requirements::javascript(GOOGLEMAPFIELD_BASE .'/javascript/GoogleMapField.js');
-		Requirements::javascript("//maps.googleapis.com/maps/api/js?callback=googlemapfieldInit".$key);
-		Requirements::css(GOOGLEMAPFIELD_BASE .'/css/GoogleMapField.css');
+		$key = $this->options['apikey'] ? "&key=" . $this->options['apikey'] : "";
+		Requirements::javascript(GOOGLEMAPFIELD_BASE . '/javascript/GoogleMapField.js');
+		Requirements::javascript("//maps.googleapis.com/maps/api/js?callback=googlemapfieldInit" . $key);
+		Requirements::css(GOOGLEMAPFIELD_BASE . '/css/GoogleMapField.css');
 		$jsOptions = array(
 			'coords' => array($this->getLatData(), $this->getLngData()),
 			'map' => array(
@@ -59,7 +59,7 @@ 
 				'mapTypeId' => 'ROADMAP',
 			),
 		);
-		if(!$this->options['showSearchBox']){
+		if (!$this->options['showSearchBox']) {
 			$this->children->removeByName("Search");
 		}
 		$jsOptions = array_replace_recursive($jsOptions, $this->options);
@@ -113,8 +113,8 @@ 
 
 			$var = $this->options;
 
-			foreach($names as $n) {
-				if(!isset($var[$n])) {
+			foreach ($names as $n) {
+				if (!isset($var[$n])) {
 					return null;
 				}
 				$var = $var[$n];
@@ -126,7 +126,7 @@ 
 
 	public function setOption($name, $val) {
 		// Quicker execution path for "."-free names
-		if(strpos($name,'.') === false) {
+		if (strpos($name, '.') === false) {
 			$this->options[$name] = $val;
 		} else {
 			$names = explode('.', $name);
@@ -134,7 +134,7 @@ 
 			// We still want to do this even if we have strict path checking for legacy code
 			$var = &$this->options;
 
-			foreach($names as $n) {
+			foreach ($names as $n) {
 				$var = &$var[$n];
 			}
 

Braces +4 added lines, -2 removed lines

@@ -59,7 +59,7 @@ 
 				'mapTypeId' => 'ROADMAP',
 			),
 		);
-		if(!$this->options['showSearchBox']){
+		if(!$this->options['showSearchBox']) {
 			$this->children->removeByName("Search");
 		}
 		$jsOptions = array_replace_recursive($jsOptions, $this->options);
@@ -107,7 +107,9 @@ 
 	public function getOption($name) {
 		// Quicker execution path for "."-free names
 		if (strpos($name, '.') === false) {
-			if (isset($this->options[$name])) return $this->options[$name];
+			if (isset($this->options[$name])) {
+				return $this->options[$name];
+			}
 		} else {
 			$names = explode('.', $name);