insideout10 / wordlift-plugin

src/shortcodes/wordlift_shortcode_geomap.php

Doc Comments -1 removed lines

@@ -160,7 +160,6 @@
  * @uses wl_shortcode_geomap_get_places() in order to retrieve places
  * @uses wl_shortcode_geomap_prepare_map() in order to encode retireved places in a Leaflet friendly format
  *
- * @param array $places An array of place posts.
  *
  * @return array An array of place posts.
  */

Spacing +36 added lines, -36 removed lines

@@ -16,35 +16,35 @@ 
  *
  * @return array An array of place posts.
  */
-function wl_shortcode_geomap_get_places( $post_id = null ) {
+function wl_shortcode_geomap_get_places($post_id = null) {
 
 	// If $post_id is null or is not numeric it means this is a global geomap
-	$is_global = is_null( $post_id ) || ! is_numeric( $post_id );
+	$is_global = is_null($post_id) || ! is_numeric($post_id);
 
 	// If the current one is not a global geomap, retrieve related entities ids
-	if ( $is_global ) {
+	if ($is_global) {
 		$related_ids = array();
 	} else {
-		$related_ids = wl_core_get_related_entity_ids( $post_id, array(
+		$related_ids = wl_core_get_related_entity_ids($post_id, array(
 			'status' => 'publish',
-		) );
+		));
 
 		// Also include current entity
-		if ( Wordlift_Entity_Service::get_instance()->is_entity( $post_id ) ) {
+		if (Wordlift_Entity_Service::get_instance()->is_entity($post_id)) {
 			$related_ids[] = $post_id;
 		}
 	}
 
 	// If is not a global geomap, an empty $related_ids means that no entities are related to the post
 	// An empty array can be returned in this case
-	if ( ! $is_global && empty( $related_ids ) ) {
+	if ( ! $is_global && empty($related_ids)) {
 		return array();
 	}
 
 	// Retrieve all 'published' places with geo coordinates defined
 	// If $related_ids is not empty, it's used to limit query results to the current post related places
 	// Please note that when $place_ids is an empty array, the 'post__in' parameter is not considered in the query
-	return get_posts( array(
+	return get_posts(array(
 		'post__in'    => $related_ids,
 		'post_type'   => Wordlift_Entity_Service::valid_entity_post_types(),
 		'nopaging'    => true,
@@ -69,7 +69,7 @@ 
 				'terms'    => 'place',
 			),
 		),
-	) );
+	));
 }
 
 /**
@@ -82,7 +82,7 @@ 
  *
  * @return array An array of markers and boundaries for Leaflet.
  */
-function wl_shortcode_geomap_prepare_map( $places ) {
+function wl_shortcode_geomap_prepare_map($places) {
 
 	// Prepare for min/max lat/long in case we need to define a view boundary for the client JavaScript.
 	$min_latitude  = PHP_INT_MAX;
@@ -97,13 +97,13 @@ 
 	$boundaries = array();
 
 	// Add a POI for each entity that has coordinates.
-	foreach ( $places as $entity ) {
+	foreach ($places as $entity) {
 
 		// Get the coordinates.
-		$coordinates = wl_get_coordinates( $entity->ID );
+		$coordinates = wl_get_coordinates($entity->ID);
 
 		// Don't show the widget if the coordinates aren't set.
-		if ( $coordinates['latitude'] == 0 || $coordinates['longitude'] == 0 ) {
+		if ($coordinates['latitude'] == 0 || $coordinates['longitude'] == 0) {
 			continue;
 		}
 
@@ -111,38 +111,38 @@ 
 		// This function should be focused on returning pure data instead
 
 		// Get the title, URL and thumb of the entity.
-		$title = esc_attr( $entity->post_title );
-		$link  = esc_attr( get_permalink( $entity->ID ) );
-		if ( '' !== ( $thumbnail_id = get_post_thumbnail_id( $entity->ID ) ) &&
-		     false !== ( $attachment = wp_get_attachment_image_src( $thumbnail_id ) )
+		$title = esc_attr($entity->post_title);
+		$link  = esc_attr(get_permalink($entity->ID));
+		if ('' !== ($thumbnail_id = get_post_thumbnail_id($entity->ID)) &&
+		     false !== ($attachment = wp_get_attachment_image_src($thumbnail_id))
 		) {
-			$img_src = esc_attr( $attachment[0] );
+			$img_src = esc_attr($attachment[0]);
 		}
 
 		// Build HTML popup. TODO: move thumb width in css
 		$content = "<a href=$link><h6>$title</h6>";
-		if ( isset( $img_src ) ) {
-			$content = $content . "<img src=$img_src style='width:100%'/>";
+		if (isset($img_src)) {
+			$content = $content."<img src=$img_src style='width:100%'/>";
 		}
-		$content = $content . "</a><ul>";
+		$content = $content."</a><ul>";
 		// Get the related posts (published) and print them in the popup.
-		$related_posts = wl_core_get_related_post_ids( $entity->ID, array(
+		$related_posts = wl_core_get_related_post_ids($entity->ID, array(
 			'status' => 'publish',
-		) );
-		foreach ( $related_posts as $rp_id ) {
+		));
+		foreach ($related_posts as $rp_id) {
 
-			$rp      = get_post( $rp_id );
-			$title   = esc_attr( $rp->post_title );
-			$link    = esc_attr( get_permalink( $rp->ID ) );
-			$content = $content . "<li><a href=$link>$title</a></li>";
+			$rp      = get_post($rp_id);
+			$title   = esc_attr($rp->post_title);
+			$link    = esc_attr(get_permalink($rp->ID));
+			$content = $content."<li><a href=$link>$title</a></li>";
 		}
-		$content = $content . "</ul>";
+		$content = $content."</ul>";
 
 		// Formatting POI in geoJSON.
 		// http://leafletjs.com/examples/geojson.html
 		$poi = array(
 			'type'       => 'Feature',
-			'properties' => array( 'popupContent' => $content ),
+			'properties' => array('popupContent' => $content),
 			'geometry'   => array(
 				'type'        => 'Point',
 				'coordinates' => array(
@@ -183,16 +183,16 @@ 
  */
 function wl_shortcode_geomap_ajax() {
 	// Get the post Id.
-	$post_id = ( isset( $_REQUEST['post_id'] ) ? $_REQUEST['post_id'] : null );
+	$post_id = (isset($_REQUEST['post_id']) ? $_REQUEST['post_id'] : null);
 
-	$places   = wl_shortcode_geomap_get_places( $post_id );
-	$map_data = wl_shortcode_geomap_prepare_map( $places );
+	$places   = wl_shortcode_geomap_get_places($post_id);
+	$map_data = wl_shortcode_geomap_prepare_map($places);
 
-	wl_core_send_json( $map_data );
+	wl_core_send_json($map_data);
 }
 
-add_action( 'wp_ajax_wl_geomap', 'wl_shortcode_geomap_ajax' );
-add_action( 'wp_ajax_nopriv_wl_geomap', 'wl_shortcode_geomap_ajax' );
+add_action('wp_ajax_wl_geomap', 'wl_shortcode_geomap_ajax');
+add_action('wp_ajax_nopriv_wl_geomap', 'wl_shortcode_geomap_ajax');
 
 ///**
 // * Print geomap shortcode

Indentation +124 added lines, -124 removed lines

@@ -18,58 +18,58 @@ 
  */
 function wl_shortcode_geomap_get_places( $post_id = null ) {
 
-	// If $post_id is null or is not numeric it means this is a global geomap
-	$is_global = is_null( $post_id ) || ! is_numeric( $post_id );
+    // If $post_id is null or is not numeric it means this is a global geomap
+    $is_global = is_null( $post_id ) || ! is_numeric( $post_id );
 
-	// If the current one is not a global geomap, retrieve related entities ids
-	if ( $is_global ) {
-		$related_ids = array();
-	} else {
-		$related_ids = wl_core_get_related_entity_ids( $post_id, array(
-			'status' => 'publish',
-		) );
+    // If the current one is not a global geomap, retrieve related entities ids
+    if ( $is_global ) {
+        $related_ids = array();
+    } else {
+        $related_ids = wl_core_get_related_entity_ids( $post_id, array(
+            'status' => 'publish',
+        ) );
 
-		// Also include current entity
-		if ( Wordlift_Entity_Service::get_instance()->is_entity( $post_id ) ) {
-			$related_ids[] = $post_id;
-		}
-	}
+        // Also include current entity
+        if ( Wordlift_Entity_Service::get_instance()->is_entity( $post_id ) ) {
+            $related_ids[] = $post_id;
+        }
+    }
 
-	// If is not a global geomap, an empty $related_ids means that no entities are related to the post
-	// An empty array can be returned in this case
-	if ( ! $is_global && empty( $related_ids ) ) {
-		return array();
-	}
+    // If is not a global geomap, an empty $related_ids means that no entities are related to the post
+    // An empty array can be returned in this case
+    if ( ! $is_global && empty( $related_ids ) ) {
+        return array();
+    }
 
-	// Retrieve all 'published' places with geo coordinates defined
-	// If $related_ids is not empty, it's used to limit query results to the current post related places
-	// Please note that when $place_ids is an empty array, the 'post__in' parameter is not considered in the query
-	return get_posts( array(
-		'post__in'    => $related_ids,
-		'post_type'   => Wordlift_Entity_Service::valid_entity_post_types(),
-		'nopaging'    => true,
-		'post_status' => 'publish',
-		'meta_query'  => array(
-			'relation' => 'AND',
-			array(
-				'key'     => Wordlift_Schema_Service::FIELD_GEO_LATITUDE,
-				'value'   => null,
-				'compare' => '!=',
-			),
-			array(
-				'key'     => Wordlift_Schema_Service::FIELD_GEO_LONGITUDE,
-				'value'   => null,
-				'compare' => '!=',
-			),
-		),
-		'tax_query'   => array(
-			array(
-				'taxonomy' => Wordlift_Entity_Type_Taxonomy_Service::TAXONOMY_NAME,
-				'field'    => 'slug',
-				'terms'    => 'place',
-			),
-		),
-	) );
+    // Retrieve all 'published' places with geo coordinates defined
+    // If $related_ids is not empty, it's used to limit query results to the current post related places
+    // Please note that when $place_ids is an empty array, the 'post__in' parameter is not considered in the query
+    return get_posts( array(
+        'post__in'    => $related_ids,
+        'post_type'   => Wordlift_Entity_Service::valid_entity_post_types(),
+        'nopaging'    => true,
+        'post_status' => 'publish',
+        'meta_query'  => array(
+            'relation' => 'AND',
+            array(
+                'key'     => Wordlift_Schema_Service::FIELD_GEO_LATITUDE,
+                'value'   => null,
+                'compare' => '!=',
+            ),
+            array(
+                'key'     => Wordlift_Schema_Service::FIELD_GEO_LONGITUDE,
+                'value'   => null,
+                'compare' => '!=',
+            ),
+        ),
+        'tax_query'   => array(
+            array(
+                'taxonomy' => Wordlift_Entity_Type_Taxonomy_Service::TAXONOMY_NAME,
+                'field'    => 'slug',
+                'terms'    => 'place',
+            ),
+        ),
+    ) );
 }
 
 /**
@@ -84,91 +84,91 @@ 
  */
 function wl_shortcode_geomap_prepare_map( $places ) {
 
-	// Prepare for min/max lat/long in case we need to define a view boundary for the client JavaScript.
-	$min_latitude  = PHP_INT_MAX;
-	$min_longitude = PHP_INT_MAX;
-	$max_latitude  = ~PHP_INT_MAX;
-	$max_longitude = ~PHP_INT_MAX;
+    // Prepare for min/max lat/long in case we need to define a view boundary for the client JavaScript.
+    $min_latitude  = PHP_INT_MAX;
+    $min_longitude = PHP_INT_MAX;
+    $max_latitude  = ~PHP_INT_MAX;
+    $max_longitude = ~PHP_INT_MAX;
 
-	// Prepare an empty array of POIs in geoJSON format.
-	$pois = array();
-	// And store list of points to allow Leaflet compute the optimal bounding box.
-	// The main reason for this is that geoJSON has swapped coordinates (lon. lat)
-	$boundaries = array();
+    // Prepare an empty array of POIs in geoJSON format.
+    $pois = array();
+    // And store list of points to allow Leaflet compute the optimal bounding box.
+    // The main reason for this is that geoJSON has swapped coordinates (lon. lat)
+    $boundaries = array();
 
-	// Add a POI for each entity that has coordinates.
-	foreach ( $places as $entity ) {
+    // Add a POI for each entity that has coordinates.
+    foreach ( $places as $entity ) {
 
-		// Get the coordinates.
-		$coordinates = wl_get_coordinates( $entity->ID );
+        // Get the coordinates.
+        $coordinates = wl_get_coordinates( $entity->ID );
 
-		// Don't show the widget if the coordinates aren't set.
-		if ( $coordinates['latitude'] == 0 || $coordinates['longitude'] == 0 ) {
-			continue;
-		}
+        // Don't show the widget if the coordinates aren't set.
+        if ( $coordinates['latitude'] == 0 || $coordinates['longitude'] == 0 ) {
+            continue;
+        }
 
-		// TODO Map html rendering should be delegated to the wordlift js ui layer
-		// This function should be focused on returning pure data instead
+        // TODO Map html rendering should be delegated to the wordlift js ui layer
+        // This function should be focused on returning pure data instead
 
-		// Get the title, URL and thumb of the entity.
-		$title = esc_attr( $entity->post_title );
-		$link  = esc_attr( get_permalink( $entity->ID ) );
-		if ( '' !== ( $thumbnail_id = get_post_thumbnail_id( $entity->ID ) ) &&
-		     false !== ( $attachment = wp_get_attachment_image_src( $thumbnail_id ) )
-		) {
-			$img_src = esc_attr( $attachment[0] );
-		}
+        // Get the title, URL and thumb of the entity.
+        $title = esc_attr( $entity->post_title );
+        $link  = esc_attr( get_permalink( $entity->ID ) );
+        if ( '' !== ( $thumbnail_id = get_post_thumbnail_id( $entity->ID ) ) &&
+             false !== ( $attachment = wp_get_attachment_image_src( $thumbnail_id ) )
+        ) {
+            $img_src = esc_attr( $attachment[0] );
+        }
 
-		// Build HTML popup. TODO: move thumb width in css
-		$content = "<a href=$link><h6>$title</h6>";
-		if ( isset( $img_src ) ) {
-			$content = $content . "<img src=$img_src style='width:100%'/>";
-		}
-		$content = $content . "</a><ul>";
-		// Get the related posts (published) and print them in the popup.
-		$related_posts = wl_core_get_related_post_ids( $entity->ID, array(
-			'status' => 'publish',
-		) );
-		foreach ( $related_posts as $rp_id ) {
+        // Build HTML popup. TODO: move thumb width in css
+        $content = "<a href=$link><h6>$title</h6>";
+        if ( isset( $img_src ) ) {
+            $content = $content . "<img src=$img_src style='width:100%'/>";
+        }
+        $content = $content . "</a><ul>";
+        // Get the related posts (published) and print them in the popup.
+        $related_posts = wl_core_get_related_post_ids( $entity->ID, array(
+            'status' => 'publish',
+        ) );
+        foreach ( $related_posts as $rp_id ) {
 
-			$rp      = get_post( $rp_id );
-			$title   = esc_attr( $rp->post_title );
-			$link    = esc_attr( get_permalink( $rp->ID ) );
-			$content = $content . "<li><a href=$link>$title</a></li>";
-		}
-		$content = $content . "</ul>";
+            $rp      = get_post( $rp_id );
+            $title   = esc_attr( $rp->post_title );
+            $link    = esc_attr( get_permalink( $rp->ID ) );
+            $content = $content . "<li><a href=$link>$title</a></li>";
+        }
+        $content = $content . "</ul>";
 
-		// Formatting POI in geoJSON.
-		// http://leafletjs.com/examples/geojson.html
-		$poi = array(
-			'type'       => 'Feature',
-			'properties' => array( 'popupContent' => $content ),
-			'geometry'   => array(
-				'type'        => 'Point',
-				'coordinates' => array(
-					// Leaflet geoJSON wants them swapped
-					$coordinates['longitude'],
-					$coordinates['latitude'],
-				),
-			),
-		);
+        // Formatting POI in geoJSON.
+        // http://leafletjs.com/examples/geojson.html
+        $poi = array(
+            'type'       => 'Feature',
+            'properties' => array( 'popupContent' => $content ),
+            'geometry'   => array(
+                'type'        => 'Point',
+                'coordinates' => array(
+                    // Leaflet geoJSON wants them swapped
+                    $coordinates['longitude'],
+                    $coordinates['latitude'],
+                ),
+            ),
+        );
 
-		$pois[] = $poi;
+        $pois[] = $poi;
 
-		// Formatting boundaries in a Leaflet-like format (see LatLngBounds).
-		// http://leafletjs.com/reference.html#latlngbounds
-		$boundaries[] = array(
-			$coordinates['latitude'],
-			$coordinates['longitude'],
-		);
+        // Formatting boundaries in a Leaflet-like format (see LatLngBounds).
+        // http://leafletjs.com/reference.html#latlngbounds
+        $boundaries[] = array(
+            $coordinates['latitude'],
+            $coordinates['longitude'],
+        );
 
-	}
+    }
 
-	$map_data               = array();
-	$map_data['features']   = $pois;
-	$map_data['boundaries'] = $boundaries;
+    $map_data               = array();
+    $map_data['features']   = $pois;
+    $map_data['boundaries'] = $boundaries;
 
-	return $map_data;
+    return $map_data;
 }
 
 /**
@@ -182,13 +182,13 @@ 
  * @return array An array of place posts.
  */
 function wl_shortcode_geomap_ajax() {
-	// Get the post Id.
-	$post_id = ( isset( $_REQUEST['post_id'] ) ? $_REQUEST['post_id'] : null );
+    // Get the post Id.
+    $post_id = ( isset( $_REQUEST['post_id'] ) ? $_REQUEST['post_id'] : null );
 
-	$places   = wl_shortcode_geomap_get_places( $post_id );
-	$map_data = wl_shortcode_geomap_prepare_map( $places );
+    $places   = wl_shortcode_geomap_get_places( $post_id );
+    $map_data = wl_shortcode_geomap_prepare_map( $places );
 
-	wl_core_send_json( $map_data );
+    wl_core_send_json( $map_data );
 }
 
 add_action( 'wp_ajax_wl_geomap', 'wl_shortcode_geomap_ajax' );

src/admin/WL_Metabox/WL_Metabox_Field_coordinates.php

Spacing +18 added lines, -18 removed lines

@@ -3,7 +3,7 @@ 
 
 class WL_Metabox_Field_coordinates extends WL_Metabox_Field {
 
-	public function __construct( $args ) {
+	public function __construct($args) {
 
 		// Just set up the necessary info without calling the parent constructor.
 		// TODO: write a parent class for grouped properties
@@ -15,7 +15,7 @@ 
 
 	public function get_data() {
 		$entity_id  = get_the_ID();
-		$this->data = wl_get_coordinates( $entity_id );
+		$this->data = wl_get_coordinates($entity_id);
 	}
 
 	public function html() {
@@ -33,20 +33,20 @@ 
 		$data        = $this->data;
 		// TODO: We temporary use here 0,0 as default coordinates for the marker, but if no coordinates are given we
 		// want to use the current user location for the marker.
-		$coordinates = ( ! empty( $data['latitude'] ) && ! empty( $data['longitude'] ) ? sprintf( '[%f,%f]', $data['latitude'], $data['longitude'] ) : '[0,0]' );
+		$coordinates = ( ! empty($data['latitude']) && ! empty($data['longitude']) ? sprintf('[%f,%f]', $data['latitude'], $data['longitude']) : '[0,0]');
 		$map_init    = '[0,0]' === $coordinates
 			? 'locate( {setView: true, maxZoom: 16} )'
-			: sprintf( "setView( [%f,%f], 9 )", $data['latitude'], $data['longitude'] );
+			: sprintf("setView( [%f,%f], 9 )", $data['latitude'], $data['longitude']);
 
 		// Print input fields
-		$html .= '<label for="wl_place_lat">' . __( 'Latitude', 'wordlift' ) . '</label>';
-		$html .= '<input type="text" id="wl_place_lat" name="wl_metaboxes[coordinates][]" value="' . $data['latitude'] . '" style="width:100%" />';
+		$html .= '<label for="wl_place_lat">'.__('Latitude', 'wordlift').'</label>';
+		$html .= '<input type="text" id="wl_place_lat" name="wl_metaboxes[coordinates][]" value="'.$data['latitude'].'" style="width:100%" />';
 
-		$html .= '<label for="wl_place_lon">' . __( 'Longitude', 'wordlift' ) . '</label>';
-		$html .= '<input type="text" id="wl_place_lon" name="wl_metaboxes[coordinates][]" value="' . $data['longitude'] . '" style="width:100%" />';
+		$html .= '<label for="wl_place_lon">'.__('Longitude', 'wordlift').'</label>';
+		$html .= '<input type="text" id="wl_place_lon" name="wl_metaboxes[coordinates][]" value="'.$data['longitude'].'" style="width:100%" />';
 
 		// Show Leaflet map to pick coordinates
-		$element_id = uniqid( 'wl-gep-map-' );
+		$element_id = uniqid('wl-gep-map-');
 		$html .= <<<EOF
 
 <div id="$element_id"></div>
@@ -84,23 +84,23 @@ 
 		return $html;
 	}
 
-	public function save_data( $coords ) {
+	public function save_data($coords) {
 
-		$this->sanitize_data( $coords );
+		$this->sanitize_data($coords);
 
 		$entity_id = get_the_ID();
 
 		// Take away old values
-		delete_post_meta( $entity_id, Wordlift_Schema_Service::FIELD_GEO_LATITUDE );
-		delete_post_meta( $entity_id, Wordlift_Schema_Service::FIELD_GEO_LONGITUDE );
+		delete_post_meta($entity_id, Wordlift_Schema_Service::FIELD_GEO_LATITUDE);
+		delete_post_meta($entity_id, Wordlift_Schema_Service::FIELD_GEO_LONGITUDE);
 
 		$latitude  = $this->data[0];
 		$longitude = $this->data[1];
 
 		// insert new coordinate values
-		if ( ! empty( $latitude ) && ! empty( $longitude ) ) {
-			add_post_meta( $entity_id, Wordlift_Schema_Service::FIELD_GEO_LATITUDE, $latitude, true );
-			add_post_meta( $entity_id, Wordlift_Schema_Service::FIELD_GEO_LONGITUDE, $longitude, true );
+		if ( ! empty($latitude) && ! empty($longitude)) {
+			add_post_meta($entity_id, Wordlift_Schema_Service::FIELD_GEO_LATITUDE, $latitude, true);
+			add_post_meta($entity_id, Wordlift_Schema_Service::FIELD_GEO_LONGITUDE, $longitude, true);
 		}
 
 	}
@@ -108,11 +108,11 @@ 
 	/**
 	 * Only accept float numbers
 	 */
-	public function sanitize_data_filter( $value ) {
+	public function sanitize_data_filter($value) {
 
 		// DO NOT set latitude/longitude to 0/0 as default values. It's a specific place on the globe:
 		// "The zero/zero point of this system is located in the Gulf of Guinea about 625 km (390 mi) south of Tema, Ghana."
-		if ( ! is_numeric( $value ) ) {
+		if ( ! is_numeric($value)) {
 			return '';
 		}
 

Indentation +62 added lines, -62 removed lines

@@ -3,51 +3,51 @@ 
 
 class WL_Metabox_Field_coordinates extends WL_Metabox_Field {
 
-	public function __construct( $args ) {
+    public function __construct( $args ) {
 
-		// Just set up the necessary info without calling the parent constructor.
-		// TODO: write a parent class for grouped properties
+        // Just set up the necessary info without calling the parent constructor.
+        // TODO: write a parent class for grouped properties
 
-		// we use 'coordinates' to namespace the Field in $_POST data.
-		// In  the DB the correct meta names will be used.
-		$this->meta_name = 'coordinates';
-	}
+        // we use 'coordinates' to namespace the Field in $_POST data.
+        // In  the DB the correct meta names will be used.
+        $this->meta_name = 'coordinates';
+    }
 
-	public function get_data() {
-		$entity_id  = get_the_ID();
-		$this->data = wl_get_coordinates( $entity_id );
-	}
+    public function get_data() {
+        $entity_id  = get_the_ID();
+        $this->data = wl_get_coordinates( $entity_id );
+    }
 
-	public function html() {
+    public function html() {
 
-		// Open main <div> for the Field
-		$html = $this->html_wrapper_open();
+        // Open main <div> for the Field
+        $html = $this->html_wrapper_open();
 
-		// Label
-		$html .= '<h3>coordinates</h3>';
+        // Label
+        $html .= '<h3>coordinates</h3>';
 
-		// print nonce
-		$html .= $this->html_nonce();
+        // print nonce
+        $html .= $this->html_nonce();
 
-		// Get coordinates
-		$data        = $this->data;
-		// TODO: We temporary use here 0,0 as default coordinates for the marker, but if no coordinates are given we
-		// want to use the current user location for the marker.
-		$coordinates = ( ! empty( $data['latitude'] ) && ! empty( $data['longitude'] ) ? sprintf( '[%f,%f]', $data['latitude'], $data['longitude'] ) : '[0,0]' );
-		$map_init    = '[0,0]' === $coordinates
-			? 'locate( {setView: true, maxZoom: 16} )'
-			: sprintf( "setView( [%f,%f], 9 )", $data['latitude'], $data['longitude'] );
+        // Get coordinates
+        $data        = $this->data;
+        // TODO: We temporary use here 0,0 as default coordinates for the marker, but if no coordinates are given we
+        // want to use the current user location for the marker.
+        $coordinates = ( ! empty( $data['latitude'] ) && ! empty( $data['longitude'] ) ? sprintf( '[%f,%f]', $data['latitude'], $data['longitude'] ) : '[0,0]' );
+        $map_init    = '[0,0]' === $coordinates
+            ? 'locate( {setView: true, maxZoom: 16} )'
+            : sprintf( "setView( [%f,%f], 9 )", $data['latitude'], $data['longitude'] );
 
-		// Print input fields
-		$html .= '<label for="wl_place_lat">' . __( 'Latitude', 'wordlift' ) . '</label>';
-		$html .= '<input type="text" id="wl_place_lat" name="wl_metaboxes[coordinates][]" value="' . $data['latitude'] . '" style="width:100%" />';
+        // Print input fields
+        $html .= '<label for="wl_place_lat">' . __( 'Latitude', 'wordlift' ) . '</label>';
+        $html .= '<input type="text" id="wl_place_lat" name="wl_metaboxes[coordinates][]" value="' . $data['latitude'] . '" style="width:100%" />';
 
-		$html .= '<label for="wl_place_lon">' . __( 'Longitude', 'wordlift' ) . '</label>';
-		$html .= '<input type="text" id="wl_place_lon" name="wl_metaboxes[coordinates][]" value="' . $data['longitude'] . '" style="width:100%" />';
+        $html .= '<label for="wl_place_lon">' . __( 'Longitude', 'wordlift' ) . '</label>';
+        $html .= '<input type="text" id="wl_place_lon" name="wl_metaboxes[coordinates][]" value="' . $data['longitude'] . '" style="width:100%" />';
 
-		// Show Leaflet map to pick coordinates
-		$element_id = uniqid( 'wl-gep-map-' );
-		$html .= <<<EOF
+        // Show Leaflet map to pick coordinates
+        $element_id = uniqid( 'wl-gep-map-' );
+        $html .= <<<EOF
 
 <div id="$element_id"></div>
 
@@ -79,44 +79,44 @@ 
 EOF;
 
 
-		$html .= $this->html_wrapper_close();
+        $html .= $this->html_wrapper_close();
 
-		return $html;
-	}
+        return $html;
+    }
 
-	public function save_data( $coords ) {
+    public function save_data( $coords ) {
 
-		$this->sanitize_data( $coords );
+        $this->sanitize_data( $coords );
 
-		$entity_id = get_the_ID();
+        $entity_id = get_the_ID();
 
-		// Take away old values
-		delete_post_meta( $entity_id, Wordlift_Schema_Service::FIELD_GEO_LATITUDE );
-		delete_post_meta( $entity_id, Wordlift_Schema_Service::FIELD_GEO_LONGITUDE );
+        // Take away old values
+        delete_post_meta( $entity_id, Wordlift_Schema_Service::FIELD_GEO_LATITUDE );
+        delete_post_meta( $entity_id, Wordlift_Schema_Service::FIELD_GEO_LONGITUDE );
 
-		$latitude  = $this->data[0];
-		$longitude = $this->data[1];
+        $latitude  = $this->data[0];
+        $longitude = $this->data[1];
 
-		// insert new coordinate values
-		if ( ! empty( $latitude ) && ! empty( $longitude ) ) {
-			add_post_meta( $entity_id, Wordlift_Schema_Service::FIELD_GEO_LATITUDE, $latitude, true );
-			add_post_meta( $entity_id, Wordlift_Schema_Service::FIELD_GEO_LONGITUDE, $longitude, true );
-		}
+        // insert new coordinate values
+        if ( ! empty( $latitude ) && ! empty( $longitude ) ) {
+            add_post_meta( $entity_id, Wordlift_Schema_Service::FIELD_GEO_LATITUDE, $latitude, true );
+            add_post_meta( $entity_id, Wordlift_Schema_Service::FIELD_GEO_LONGITUDE, $longitude, true );
+        }
 
-	}
+    }
 
-	/**
-	 * Only accept float numbers
-	 */
-	public function sanitize_data_filter( $value ) {
+    /**
+     * Only accept float numbers
+     */
+    public function sanitize_data_filter( $value ) {
 
-		// DO NOT set latitude/longitude to 0/0 as default values. It's a specific place on the globe:
-		// "The zero/zero point of this system is located in the Gulf of Guinea about 625 km (390 mi) south of Tema, Ghana."
-		if ( ! is_numeric( $value ) ) {
-			return '';
-		}
+        // DO NOT set latitude/longitude to 0/0 as default values. It's a specific place on the globe:
+        // "The zero/zero point of this system is located in the Gulf of Guinea about 625 km (390 mi) south of Tema, Ghana."
+        if ( ! is_numeric( $value ) ) {
+            return '';
+        }
 
-		return $value;
-	}
+        return $value;
+    }
 }
 

src/includes/class-wordlift-i18n.php

Indentation +33 added lines, -33 removed lines

@@ -26,38 +26,38 @@
  */
 class Wordlift_i18n {
 
-	/**
-	 * The domain specified for this plugin.
-	 *
-	 * @since    1.0.0
-	 * @access   private
-	 * @var      string    $domain    The domain identifier for this plugin.
-	 */
-	private $domain;
-
-	/**
-	 * Load the plugin text domain for translation.
-	 *
-	 * @since    1.0.0
-	 */
-	public function load_plugin_textdomain() {
-
-		load_plugin_textdomain(
-			$this->domain,
-			false,
-			dirname( dirname( plugin_basename( __FILE__ ) ) ) . '/languages/'
-		);
-
-	}
-
-	/**
-	 * Set the domain equal to that of the specified domain.
-	 *
-	 * @since    1.0.0
-	 * @param    string    $domain    The domain that represents the locale of this plugin.
-	 */
-	public function set_domain( $domain ) {
-		$this->domain = $domain;
-	}
+    /**
+     * The domain specified for this plugin.
+     *
+     * @since    1.0.0
+     * @access   private
+     * @var      string    $domain    The domain identifier for this plugin.
+     */
+    private $domain;
+
+    /**
+     * Load the plugin text domain for translation.
+     *
+     * @since    1.0.0
+     */
+    public function load_plugin_textdomain() {
+
+        load_plugin_textdomain(
+            $this->domain,
+            false,
+            dirname( dirname( plugin_basename( __FILE__ ) ) ) . '/languages/'
+        );
+
+    }
+
+    /**
+     * Set the domain equal to that of the specified domain.
+     *
+     * @since    1.0.0
+     * @param    string    $domain    The domain that represents the locale of this plugin.
+     */
+    public function set_domain( $domain ) {
+        $this->domain = $domain;
+    }
 
 }

Spacing +2 added lines, -2 removed lines

@@ -45,7 +45,7 @@ 
 		load_plugin_textdomain(
 			$this->domain,
 			false,
-			dirname( dirname( plugin_basename( __FILE__ ) ) ) . '/languages/'
+			dirname(dirname(plugin_basename(__FILE__))).'/languages/'
 		);
 
 	}
@@ -56,7 +56,7 @@ 
 	 * @since    1.0.0
 	 * @param    string    $domain    The domain that represents the locale of this plugin.
 	 */
-	public function set_domain( $domain ) {
+	public function set_domain($domain) {
 		$this->domain = $domain;
 	}
 

src/includes/class-wordlift-loader.php

Indentation +102 added lines, -102 removed lines

@@ -23,107 +23,107 @@
  */
 class Wordlift_Loader {
 
-	/**
-	 * The array of actions registered with WordPress.
-	 *
-	 * @since    1.0.0
-	 * @access   protected
-	 * @var      array    $actions    The actions registered with WordPress to fire when the plugin loads.
-	 */
-	protected $actions;
-
-	/**
-	 * The array of filters registered with WordPress.
-	 *
-	 * @since    1.0.0
-	 * @access   protected
-	 * @var      array    $filters    The filters registered with WordPress to fire when the plugin loads.
-	 */
-	protected $filters;
-
-	/**
-	 * Initialize the collections used to maintain the actions and filters.
-	 *
-	 * @since    1.0.0
-	 */
-	public function __construct() {
-
-		$this->actions = array();
-		$this->filters = array();
-
-	}
-
-	/**
-	 * Add a new action to the collection to be registered with WordPress.
-	 *
-	 * @since    1.0.0
-	 * @param    string               $hook             The name of the WordPress action that is being registered.
-	 * @param    object               $component        A reference to the instance of the object on which the action is defined.
-	 * @param    string               $callback         The name of the function definition on the $component.
-	 * @param    int                  $priority         Optional. he priority at which the function should be fired. Default is 10.
-	 * @param    int                  $accepted_args    Optional. The number of arguments that should be passed to the $callback. Default is 1.
-	 */
-	public function add_action( $hook, $component, $callback, $priority = 10, $accepted_args = 1 ) {
-		$this->actions = $this->add( $this->actions, $hook, $component, $callback, $priority, $accepted_args );
-	}
-
-	/**
-	 * Add a new filter to the collection to be registered with WordPress.
-	 *
-	 * @since    1.0.0
-	 * @param    string               $hook             The name of the WordPress filter that is being registered.
-	 * @param    object               $component        A reference to the instance of the object on which the filter is defined.
-	 * @param    string               $callback         The name of the function definition on the $component.
-	 * @param    int                  $priority         Optional. he priority at which the function should be fired. Default is 10.
-	 * @param    int                  $accepted_args    Optional. The number of arguments that should be passed to the $callback. Default is 1
-	 */
-	public function add_filter( $hook, $component, $callback, $priority = 10, $accepted_args = 1 ) {
-		$this->filters = $this->add( $this->filters, $hook, $component, $callback, $priority, $accepted_args );
-	}
-
-	/**
-	 * A utility function that is used to register the actions and hooks into a single
-	 * collection.
-	 *
-	 * @since    1.0.0
-	 * @access   private
-	 * @param    array                $hooks            The collection of hooks that is being registered (that is, actions or filters).
-	 * @param    string               $hook             The name of the WordPress filter that is being registered.
-	 * @param    object               $component        A reference to the instance of the object on which the filter is defined.
-	 * @param    string               $callback         The name of the function definition on the $component.
-	 * @param    int                  $priority         The priority at which the function should be fired.
-	 * @param    int                  $accepted_args    The number of arguments that should be passed to the $callback.
-	 * @return   array                                  The collection of actions and filters registered with WordPress.
-	 */
-	private function add( $hooks, $hook, $component, $callback, $priority, $accepted_args ) {
-
-		$hooks[] = array(
-			'hook'          => $hook,
-			'component'     => $component,
-			'callback'      => $callback,
-			'priority'      => $priority,
-			'accepted_args' => $accepted_args
-		);
-
-		return $hooks;
-
-	}
-
-	/**
-	 * Register the filters and actions with WordPress.
-	 *
-	 * @since    1.0.0
-	 */
-	public function run() {
-
-		foreach ( $this->filters as $hook ) {
-			add_filter( $hook['hook'], array( $hook['component'], $hook['callback'] ), $hook['priority'], $hook['accepted_args'] );
-		}
-
-		foreach ( $this->actions as $hook ) {
-			add_action( $hook['hook'], array( $hook['component'], $hook['callback'] ), $hook['priority'], $hook['accepted_args'] );
-		}
-
-	}
+    /**
+     * The array of actions registered with WordPress.
+     *
+     * @since    1.0.0
+     * @access   protected
+     * @var      array    $actions    The actions registered with WordPress to fire when the plugin loads.
+     */
+    protected $actions;
+
+    /**
+     * The array of filters registered with WordPress.
+     *
+     * @since    1.0.0
+     * @access   protected
+     * @var      array    $filters    The filters registered with WordPress to fire when the plugin loads.
+     */
+    protected $filters;
+
+    /**
+     * Initialize the collections used to maintain the actions and filters.
+     *
+     * @since    1.0.0
+     */
+    public function __construct() {
+
+        $this->actions = array();
+        $this->filters = array();
+
+    }
+
+    /**
+     * Add a new action to the collection to be registered with WordPress.
+     *
+     * @since    1.0.0
+     * @param    string               $hook             The name of the WordPress action that is being registered.
+     * @param    object               $component        A reference to the instance of the object on which the action is defined.
+     * @param    string               $callback         The name of the function definition on the $component.
+     * @param    int                  $priority         Optional. he priority at which the function should be fired. Default is 10.
+     * @param    int                  $accepted_args    Optional. The number of arguments that should be passed to the $callback. Default is 1.
+     */
+    public function add_action( $hook, $component, $callback, $priority = 10, $accepted_args = 1 ) {
+        $this->actions = $this->add( $this->actions, $hook, $component, $callback, $priority, $accepted_args );
+    }
+
+    /**
+     * Add a new filter to the collection to be registered with WordPress.
+     *
+     * @since    1.0.0
+     * @param    string               $hook             The name of the WordPress filter that is being registered.
+     * @param    object               $component        A reference to the instance of the object on which the filter is defined.
+     * @param    string               $callback         The name of the function definition on the $component.
+     * @param    int                  $priority         Optional. he priority at which the function should be fired. Default is 10.
+     * @param    int                  $accepted_args    Optional. The number of arguments that should be passed to the $callback. Default is 1
+     */
+    public function add_filter( $hook, $component, $callback, $priority = 10, $accepted_args = 1 ) {
+        $this->filters = $this->add( $this->filters, $hook, $component, $callback, $priority, $accepted_args );
+    }
+
+    /**
+     * A utility function that is used to register the actions and hooks into a single
+     * collection.
+     *
+     * @since    1.0.0
+     * @access   private
+     * @param    array                $hooks            The collection of hooks that is being registered (that is, actions or filters).
+     * @param    string               $hook             The name of the WordPress filter that is being registered.
+     * @param    object               $component        A reference to the instance of the object on which the filter is defined.
+     * @param    string               $callback         The name of the function definition on the $component.
+     * @param    int                  $priority         The priority at which the function should be fired.
+     * @param    int                  $accepted_args    The number of arguments that should be passed to the $callback.
+     * @return   array                                  The collection of actions and filters registered with WordPress.
+     */
+    private function add( $hooks, $hook, $component, $callback, $priority, $accepted_args ) {
+
+        $hooks[] = array(
+            'hook'          => $hook,
+            'component'     => $component,
+            'callback'      => $callback,
+            'priority'      => $priority,
+            'accepted_args' => $accepted_args
+        );
+
+        return $hooks;
+
+    }
+
+    /**
+     * Register the filters and actions with WordPress.
+     *
+     * @since    1.0.0
+     */
+    public function run() {
+
+        foreach ( $this->filters as $hook ) {
+            add_filter( $hook['hook'], array( $hook['component'], $hook['callback'] ), $hook['priority'], $hook['accepted_args'] );
+        }
+
+        foreach ( $this->actions as $hook ) {
+            add_action( $hook['hook'], array( $hook['component'], $hook['callback'] ), $hook['priority'], $hook['accepted_args'] );
+        }
+
+    }
 
 }

Spacing +9 added lines, -9 removed lines

@@ -63,8 +63,8 @@ 
 	 * @param    int                  $priority         Optional. he priority at which the function should be fired. Default is 10.
 	 * @param    int                  $accepted_args    Optional. The number of arguments that should be passed to the $callback. Default is 1.
 	 */
-	public function add_action( $hook, $component, $callback, $priority = 10, $accepted_args = 1 ) {
-		$this->actions = $this->add( $this->actions, $hook, $component, $callback, $priority, $accepted_args );
+	public function add_action($hook, $component, $callback, $priority = 10, $accepted_args = 1) {
+		$this->actions = $this->add($this->actions, $hook, $component, $callback, $priority, $accepted_args);
 	}
 
 	/**
@@ -77,8 +77,8 @@ 
 	 * @param    int                  $priority         Optional. he priority at which the function should be fired. Default is 10.
 	 * @param    int                  $accepted_args    Optional. The number of arguments that should be passed to the $callback. Default is 1
 	 */
-	public function add_filter( $hook, $component, $callback, $priority = 10, $accepted_args = 1 ) {
-		$this->filters = $this->add( $this->filters, $hook, $component, $callback, $priority, $accepted_args );
+	public function add_filter($hook, $component, $callback, $priority = 10, $accepted_args = 1) {
+		$this->filters = $this->add($this->filters, $hook, $component, $callback, $priority, $accepted_args);
 	}
 
 	/**
@@ -95,7 +95,7 @@ 
 	 * @param    int                  $accepted_args    The number of arguments that should be passed to the $callback.
 	 * @return   array                                  The collection of actions and filters registered with WordPress.
 	 */
-	private function add( $hooks, $hook, $component, $callback, $priority, $accepted_args ) {
+	private function add($hooks, $hook, $component, $callback, $priority, $accepted_args) {
 
 		$hooks[] = array(
 			'hook'          => $hook,
@@ -116,12 +116,12 @@ 
 	 */
 	public function run() {
 
-		foreach ( $this->filters as $hook ) {
-			add_filter( $hook['hook'], array( $hook['component'], $hook['callback'] ), $hook['priority'], $hook['accepted_args'] );
+		foreach ($this->filters as $hook) {
+			add_filter($hook['hook'], array($hook['component'], $hook['callback']), $hook['priority'], $hook['accepted_args']);
 		}
 
-		foreach ( $this->actions as $hook ) {
-			add_action( $hook['hook'], array( $hook['component'], $hook['callback'] ), $hook['priority'], $hook['accepted_args'] );
+		foreach ($this->actions as $hook) {
+			add_action($hook['hook'], array($hook['component'], $hook['callback']), $hook['priority'], $hook['accepted_args']);
 		}
 
 	}

src/shortcodes/wordlift_shortcode_field.php

Spacing +9 added lines, -9 removed lines

@@ -9,28 +9,28 @@ 
  * @param array $atts An array of shortcode attributes.
  * @return string A dom element with requested property value(s).
  */
-function wl_shortcode_field( $atts ) {
+function wl_shortcode_field($atts) {
 	
     // Extract attributes and set default values.
-    $field_atts = shortcode_atts( array(
+    $field_atts = shortcode_atts(array(
         'id'    => null,
         'name'  => null
-    ), $atts );
+    ), $atts);
     
     // Get id of the post
     $entity_id = $field_atts['id'];
-    if( is_null( $field_atts['id'] ) || !is_numeric( $field_atts['id'] ) ) {
+    if (is_null($field_atts['id']) || ! is_numeric($field_atts['id'])) {
         $entity_id = get_the_ID();
     }
 
     $property_name = $field_atts['name'];
-    if( !is_null( $property_name ) ) {
-        $values = wl_schema_get_value( $entity_id, $property_name );
+    if ( ! is_null($property_name)) {
+        $values = wl_schema_get_value($entity_id, $property_name);
     }
     
     // Return
-    if( is_array( $values ) ) {
-        return implode( ', ', $values );
+    if (is_array($values)) {
+        return implode(', ', $values);
     } else {
         return null;
     }
@@ -40,5 +40,5 @@ 
     add_shortcode('wl_field', 'wl_shortcode_field');
 }
 
-add_action( 'init', 'wl_register_shortcode_field');
+add_action('init', 'wl_register_shortcode_field');
 

src/widgets/wordlift_widget_geo.php

Spacing +4 added lines, -4 removed lines

@@ -27,15 +27,15 @@ 
      * @param array $args
      * @param array $instance
      */
-    public function widget( $args, $instance )
+    public function widget($args, $instance)
     {
         // Get the widget's title.
         $title = apply_filters('widget_title', $instance['title']);
 
         // Print the HTML output.
         echo $args['before_widget'];
-        if (!empty($title)) {
-            echo $args['before_title'] . $title . $args['after_title'];
+        if ( ! empty($title)) {
+            echo $args['before_title'].$title.$args['after_title'];
         }
 
         // Print the geomap shortcode
@@ -81,7 +81,7 @@ 
     {
 
         $instance = array();
-        $instance['title'] = (!empty($new_instance['title'])) ? strip_tags($new_instance['title']) : '';
+        $instance['title'] = ( ! empty($new_instance['title'])) ? strip_tags($new_instance['title']) : '';
 
         return $instance;
     }

src/includes/class-wordlift-ui-service.php

Indentation +64 added lines, -64 removed lines

@@ -7,80 +7,80 @@
  */
 class Wordlift_UI_Service {
 
-	/**
-	 * The button element HTML code.
-	 *
-	 * @since 3.2.0
-	 */
-	const BUTTON_HTML = '<a id="%s" class="button wl-button">%s</a>';
+    /**
+     * The button element HTML code.
+     *
+     * @since 3.2.0
+     */
+    const BUTTON_HTML = '<a id="%s" class="button wl-button">%s</a>';
 
-	/**
-	 * The template HTML code.
-	 *
-	 * @since 3.2.0
-	 */
-	const TEMPLATE_HTML = '<script id="%s" type="text/template">%s</script>';
+    /**
+     * The template HTML code.
+     *
+     * @since 3.2.0
+     */
+    const TEMPLATE_HTML = '<script id="%s" type="text/template">%s</script>';
 
-	/**
-	 * Get the button HTML.
-	 *
-	 * @since 3.2.0
-	 *
-	 * @param string $element_id The button element id.
-	 * @param string $label The button (translated) label.
-	 *
-	 * @return string The button HTML code.
-	 */
-	public function get_button_html( $element_id, $label ) {
+    /**
+     * Get the button HTML.
+     *
+     * @since 3.2.0
+     *
+     * @param string $element_id The button element id.
+     * @param string $label The button (translated) label.
+     *
+     * @return string The button HTML code.
+     */
+    public function get_button_html( $element_id, $label ) {
 
-		return sprintf( self::BUTTON_HTML, $element_id, esc_html( $label ) );
-	}
+        return sprintf( self::BUTTON_HTML, $element_id, esc_html( $label ) );
+    }
 
-	/**
-	 * Echo the button HTML.
-	 *
-	 * @since 3.2.0
-	 *
-	 * @param string $element_id The button element id.
-	 * @param string $label The button (translated) label.
-	 *
-	 * @return string The button HTML code.
-	 */
-	public function print_button( $element_id, $label ) {
+    /**
+     * Echo the button HTML.
+     *
+     * @since 3.2.0
+     *
+     * @param string $element_id The button element id.
+     * @param string $label The button (translated) label.
+     *
+     * @return string The button HTML code.
+     */
+    public function print_button( $element_id, $label ) {
 
-		echo( $this->get_button_html( $element_id, $label ) );
+        echo( $this->get_button_html( $element_id, $label ) );
 
-	}
+    }
 
-	/**
-	 * Get the HTML code for a template tag.
-	 *
-	 * @since 3.2.0
-	 *
-	 * @param string $element_id The element id.
-	 * @param string $body The element content.
-	 *
-	 * @return string The HTML code.
-	 */
-	public function get_template_html( $element_id, $body ) {
+    /**
+     * Get the HTML code for a template tag.
+     *
+     * @since 3.2.0
+     *
+     * @param string $element_id The element id.
+     * @param string $body The element content.
+     *
+     * @return string The HTML code.
+     */
+    public function get_template_html( $element_id, $body ) {
 
-		return sprintf( self::TEMPLATE_HTML, $element_id, $body );
-	}
+        return sprintf( self::TEMPLATE_HTML, $element_id, $body );
+    }
 
-	/**
-	 * Echo the HTML code for a template tag.
-	 *
-	 * @since 3.2.0
-	 *
-	 * @param string $element_id The element id.
-	 * @param string $body The element content.
-	 *
-	 * @return string The HTML code.
-	 */
-	public function print_template( $element_id, $body ) {
+    /**
+     * Echo the HTML code for a template tag.
+     *
+     * @since 3.2.0
+     *
+     * @param string $element_id The element id.
+     * @param string $body The element content.
+     *
+     * @return string The HTML code.
+     */
+    public function print_template( $element_id, $body ) {
 
-		echo( $this->get_template_html( $element_id, $body ) );
+        echo( $this->get_template_html( $element_id, $body ) );
 
-	}
+    }
 
 }

Spacing +8 added lines, -8 removed lines

@@ -31,9 +31,9 @@ 
 	 *
 	 * @return string The button HTML code.
 	 */
-	public function get_button_html( $element_id, $label ) {
+	public function get_button_html($element_id, $label) {
 
-		return sprintf( self::BUTTON_HTML, $element_id, esc_html( $label ) );
+		return sprintf(self::BUTTON_HTML, $element_id, esc_html($label));
 	}
 
 	/**
@@ -46,9 +46,9 @@ 
 	 *
 	 * @return string The button HTML code.
 	 */
-	public function print_button( $element_id, $label ) {
+	public function print_button($element_id, $label) {
 
-		echo( $this->get_button_html( $element_id, $label ) );
+		echo($this->get_button_html($element_id, $label));
 
 	}
 
@@ -62,9 +62,9 @@ 
 	 *
 	 * @return string The HTML code.
 	 */
-	public function get_template_html( $element_id, $body ) {
+	public function get_template_html($element_id, $body) {
 
-		return sprintf( self::TEMPLATE_HTML, $element_id, $body );
+		return sprintf(self::TEMPLATE_HTML, $element_id, $body);
 	}
 
 	/**
@@ -77,9 +77,9 @@ 
 	 *
 	 * @return string The HTML code.
 	 */
-	public function print_template( $element_id, $body ) {
+	public function print_template($element_id, $body) {
 
-		echo( $this->get_template_html( $element_id, $body ) );
+		echo($this->get_template_html($element_id, $body));
 
 	}
 

src/includes/class-wordlift-redirect-service.php

Doc Comments +1 added lines, -2 removed lines

@@ -115,9 +115,8 @@
 	 *
 	 * @since 3.2.0
 	 *
-	 * @param int $entity_id A post entity id.
 	 *
-	 * @return string permalink.
+	 * @return string[] permalink.
 	 */
 	public function allowed_redirect_hosts( $content ) {
 

Indentation +113 added lines, -113 removed lines

@@ -7,119 +7,119 @@
  */
 class Wordlift_Redirect_Service {
 
-	const LOD_ENDPOINT = 'http://www.lodview.it';
-	const LOD_HOST = 'www.lodview.it';
-
-	/**
-	 * The Log service.
-	 *
-	 * @since 3.2.0
-	 * @access private
-	 * @var \Wordlift_Log_Service $log_service The Log service.
-	 */
-	private $log_service;
-
-	/**
-	 * The Entity service.
-	 *
-	 * @since 3.2.0
-	 * @access private
-	 * @var \Wordlift_Entity_Service $entity_service The Entity service.
-	 */
-	private $entity_service;
-
-	/**
-	 * A singleton instance of the Redirect service (useful for unit tests).
-	 *
-	 * @since 3.2.0
-	 * @access private
-	 * @var \Wordlift_Redirect_Service $instance The singleton instance.
-	 */
-	private static $instance;
-
-	/**
-	 * Create a Wordlift_Redirect_Service instance.
-	 *
-	 * @since 3.2.0
-	 *
-	 * @param \Wordlift_Entity_Service $entity_service The Entity service.
-	 */
-	public function __construct( $entity_service ) {
-
-		$this->log_service = Wordlift_Log_Service::get_logger( 'Wordlift_Redirect_Service' );
-
-		$this->entity_service = $entity_service;
-
-		self::$instance = $this;
-
-	}
-
-	/**
-	 * Get the singleton instance of the Wordlift_Redirect_Service
-	 *
-	 * @since 3.2.0
-	 *
-	 * @return \Wordlift_Redirect_Service The singleton instance of the Wordlift_Redirect_Service.
-	 */
-	public static function get_instance() {
-
-		return self::$instance;
-	}
-
-	/**
-	 * Perform redirect depending on entity uri and target
-	 *
-	 * @since 3.2.0
-	 */
-	public function ajax_redirect() {
-
-		if ( !isset( $_GET['uri'] ) ) {
-			wp_die( 'Entity uri missing' );	
-		}
-		if ( !isset( $_GET['to'] ) ) {
-			wp_die( 'Redirect target missing' );	
-		}
-		// Get the entity uri
-		$entity_uri = $_GET['uri'];
-		// Get the redirect target
-		$target = $_GET['to'];
+    const LOD_ENDPOINT = 'http://www.lodview.it';
+    const LOD_HOST = 'www.lodview.it';
+
+    /**
+     * The Log service.
+     *
+     * @since 3.2.0
+     * @access private
+     * @var \Wordlift_Log_Service $log_service The Log service.
+     */
+    private $log_service;
+
+    /**
+     * The Entity service.
+     *
+     * @since 3.2.0
+     * @access private
+     * @var \Wordlift_Entity_Service $entity_service The Entity service.
+     */
+    private $entity_service;
+
+    /**
+     * A singleton instance of the Redirect service (useful for unit tests).
+     *
+     * @since 3.2.0
+     * @access private
+     * @var \Wordlift_Redirect_Service $instance The singleton instance.
+     */
+    private static $instance;
+
+    /**
+     * Create a Wordlift_Redirect_Service instance.
+     *
+     * @since 3.2.0
+     *
+     * @param \Wordlift_Entity_Service $entity_service The Entity service.
+     */
+    public function __construct( $entity_service ) {
+
+        $this->log_service = Wordlift_Log_Service::get_logger( 'Wordlift_Redirect_Service' );
+
+        $this->entity_service = $entity_service;
+
+        self::$instance = $this;
+
+    }
+
+    /**
+     * Get the singleton instance of the Wordlift_Redirect_Service
+     *
+     * @since 3.2.0
+     *
+     * @return \Wordlift_Redirect_Service The singleton instance of the Wordlift_Redirect_Service.
+     */
+    public static function get_instance() {
+
+        return self::$instance;
+    }
+
+    /**
+     * Perform redirect depending on entity uri and target
+     *
+     * @since 3.2.0
+     */
+    public function ajax_redirect() {
+
+        if ( !isset( $_GET['uri'] ) ) {
+            wp_die( 'Entity uri missing' );	
+        }
+        if ( !isset( $_GET['to'] ) ) {
+            wp_die( 'Redirect target missing' );	
+        }
+        // Get the entity uri
+        $entity_uri = $_GET['uri'];
+        // Get the redirect target
+        $target = $_GET['to'];
 		
-		if ( null === ( $entity_id = $this->entity_service->get_entity_post_by_uri( $entity_uri ) ) ) {
-    		wp_die( 'Entity not found' );
-		}
-
-		switch ( $target ) {
-			case 'edit':
-				$redirect_url = get_edit_post_link( $entity_id, 'none' );
-				break;
-			case 'lod':
-				$redirect_url = self::LOD_ENDPOINT . '/lodview/?IRI=' . urlencode( $entity_uri );
-				break;
-			case 'permalink':
-				$redirect_url = get_permalink( $entity_id );
-				break;
-			default:
- 				wp_die( 'Unsupported redirect target' );
-		}
-
-		// Perform the redirect
-		wp_safe_redirect( $redirect_url );
-		exit;
-	}
-
-	/**
-	 * Register custom allowed hosts.
-	 * @see https://developer.wordpress.org/reference/functions/wp_safe_redirect/
-	 *
-	 * @since 3.2.0
-	 *
-	 * @param int $entity_id A post entity id.
-	 *
-	 * @return string permalink.
-	 */
-	public function allowed_redirect_hosts( $content ) {
-
-		return array_merge( $content, array( self::LOD_HOST ) );
-	}
+        if ( null === ( $entity_id = $this->entity_service->get_entity_post_by_uri( $entity_uri ) ) ) {
+            wp_die( 'Entity not found' );
+        }
+
+        switch ( $target ) {
+            case 'edit':
+                $redirect_url = get_edit_post_link( $entity_id, 'none' );
+                break;
+            case 'lod':
+                $redirect_url = self::LOD_ENDPOINT . '/lodview/?IRI=' . urlencode( $entity_uri );
+                break;
+            case 'permalink':
+                $redirect_url = get_permalink( $entity_id );
+                break;
+            default:
+                 wp_die( 'Unsupported redirect target' );
+        }
+
+        // Perform the redirect
+        wp_safe_redirect( $redirect_url );
+        exit;
+    }
+
+    /**
+     * Register custom allowed hosts.
+     * @see https://developer.wordpress.org/reference/functions/wp_safe_redirect/
+     *
+     * @since 3.2.0
+     *
+     * @param int $entity_id A post entity id.
+     *
+     * @return string permalink.
+     */
+    public function allowed_redirect_hosts( $content ) {
+
+        return array_merge( $content, array( self::LOD_HOST ) );
+    }
 
 }

Spacing +16 added lines, -16 removed lines

@@ -44,9 +44,9 @@ 
 	 *
 	 * @param \Wordlift_Entity_Service $entity_service The Entity service.
 	 */
-	public function __construct( $entity_service ) {
+	public function __construct($entity_service) {
 
-		$this->log_service = Wordlift_Log_Service::get_logger( 'Wordlift_Redirect_Service' );
+		$this->log_service = Wordlift_Log_Service::get_logger('Wordlift_Redirect_Service');
 
 		$this->entity_service = $entity_service;
 
@@ -73,37 +73,37 @@ 
 	 */
 	public function ajax_redirect() {
 
-		if ( !isset( $_GET['uri'] ) ) {
-			wp_die( 'Entity uri missing' );	
+		if ( ! isset($_GET['uri'])) {
+			wp_die('Entity uri missing');	
 		}
-		if ( !isset( $_GET['to'] ) ) {
-			wp_die( 'Redirect target missing' );	
+		if ( ! isset($_GET['to'])) {
+			wp_die('Redirect target missing');	
 		}
 		// Get the entity uri
 		$entity_uri = $_GET['uri'];
 		// Get the redirect target
 		$target = $_GET['to'];
 		
-		if ( null === ( $entity_id = $this->entity_service->get_entity_post_by_uri( $entity_uri ) ) ) {
-    		wp_die( 'Entity not found' );
+		if (null === ($entity_id = $this->entity_service->get_entity_post_by_uri($entity_uri))) {
+    		wp_die('Entity not found');
 		}
 
-		switch ( $target ) {
+		switch ($target) {
 			case 'edit':
-				$redirect_url = get_edit_post_link( $entity_id, 'none' );
+				$redirect_url = get_edit_post_link($entity_id, 'none');
 				break;
 			case 'lod':
-				$redirect_url = self::LOD_ENDPOINT . '/lodview/?IRI=' . urlencode( $entity_uri );
+				$redirect_url = self::LOD_ENDPOINT.'/lodview/?IRI='.urlencode($entity_uri);
 				break;
 			case 'permalink':
-				$redirect_url = get_permalink( $entity_id );
+				$redirect_url = get_permalink($entity_id);
 				break;
 			default:
- 				wp_die( 'Unsupported redirect target' );
+ 				wp_die('Unsupported redirect target');
 		}
 
 		// Perform the redirect
-		wp_safe_redirect( $redirect_url );
+		wp_safe_redirect($redirect_url);
 		exit;
 	}
 
@@ -117,9 +117,9 @@ 
 	 *
 	 * @return string permalink.
 	 */
-	public function allowed_redirect_hosts( $content ) {
+	public function allowed_redirect_hosts($content) {
 
-		return array_merge( $content, array( self::LOD_HOST ) );
+		return array_merge($content, array(self::LOD_HOST));
 	}
 
 }

src/admin/class-wordlift-primashop-adapter.php

Indentation +24 added lines, -24 removed lines

@@ -7,33 +7,33 @@
  */
 class Wordlift_PrimaShop_Adapter {
 
-	/**
-	 * Create a Wordlift_PrimaShop_Adapter instance.
-	 *
-	 * @since 3.2.3
-	 */
-	public function __construct() {
+    /**
+     * Create a Wordlift_PrimaShop_Adapter instance.
+     *
+     * @since 3.2.3
+     */
+    public function __construct() {
 
-		// Tell WP (and PrimaShop) that we support the *prima-layout-settings*. This will display the Content Settings
-		// in the entity edit page.
-		add_post_type_support( Wordlift_Entity_Service::TYPE_NAME, 'prima-layout-settings' );
+        // Tell WP (and PrimaShop) that we support the *prima-layout-settings*. This will display the Content Settings
+        // in the entity edit page.
+        add_post_type_support( Wordlift_Entity_Service::TYPE_NAME, 'prima-layout-settings' );
 
-	}
+    }
 
-	/**
-	 * Intercept the <em>prima_metabox_entity_header_args</em> filter and return what a call to the related <em>post</em>
-	 * would have returned.
-	 *
-	 * @since 3.2.3
-	 *
-	 * @param array $meta The meta array.
-	 * @param string $ype The post type.
-	 *
-	 * @return array A meta array.
-	 */
-	function prima_metabox_entity_header_args( $meta, $ype ) {
+    /**
+     * Intercept the <em>prima_metabox_entity_header_args</em> filter and return what a call to the related <em>post</em>
+     * would have returned.
+     *
+     * @since 3.2.3
+     *
+     * @param array $meta The meta array.
+     * @param string $ype The post type.
+     *
+     * @return array A meta array.
+     */
+    function prima_metabox_entity_header_args( $meta, $ype ) {
 
-		return apply_filters( "prima_metabox_post_header_args", $meta, 'post' );
-	}
+        return apply_filters( "prima_metabox_post_header_args", $meta, 'post' );
+    }
 
 }

Spacing +3 added lines, -3 removed lines

@@ -16,7 +16,7 @@ 
 
 		// Tell WP (and PrimaShop) that we support the *prima-layout-settings*. This will display the Content Settings
 		// in the entity edit page.
-		add_post_type_support( Wordlift_Entity_Service::TYPE_NAME, 'prima-layout-settings' );
+		add_post_type_support(Wordlift_Entity_Service::TYPE_NAME, 'prima-layout-settings');
 
 	}
 
@@ -31,9 +31,9 @@ 
 	 *
 	 * @return array A meta array.
 	 */
-	function prima_metabox_entity_header_args( $meta, $ype ) {
+	function prima_metabox_entity_header_args($meta, $ype) {
 
-		return apply_filters( "prima_metabox_post_header_args", $meta, 'post' );
+		return apply_filters("prima_metabox_post_header_args", $meta, 'post');
 	}
 
 }