insideout10 / wordlift-plugin

src/admin/wordlift_admin_edit_post.php

Doc Comments +1 added lines, -1 removed lines

@@ -12,7 +12,7 @@
  * @param string $new_title Optional. New title.
  * @param string $new_slug Optional. New slug.
  *
- * @return The enhanced html.
+ * @return string enhanced html.
  */
 function wl_admin_permalink_html( $html, $post_id, $new_title, $new_slug ) {
 

Indentation +16 added lines, -16 removed lines

@@ -16,23 +16,23 @@
  */
 function wl_admin_permalink_html( $html, $post_id, $new_title, $new_slug ) {
 
-	// If the post is published, add the button to view Redlink's linked data.
-	if ( 'publish' == get_post_status( $post_id ) ) {
-		if ( $uri = wl_get_entity_uri( $post_id ) ) {
-			$uri_esc       = esc_attr( wl_get_entity_uri( $post_id ) );
-			$lod_view_href = 'http://lodview.it/lodview/?IRI=' . $uri_esc;
-			$html .= "<span id='view-post-btn'><a href='$lod_view_href' class='button button-small wl-button' target='_blank'>" .
-			         esc_html__( 'View Linked Data', 'wordlift' ) .
-			         "</a></span>\n";
-		}
-		$html .= "<span id='view-post-btn'><a href='" . WL_CONFIG_TEST_GOOGLE_RICH_SNIPPETS_URL .
-		         urlencode( get_permalink( $post_id ) ) .
-		         "' class='button button-small wl-button' target='_blank'>" .
-		         esc_html__( 'Test Google Rich Snippets', 'wordlift' ) .
-		         "</a></span>\n";
-	}
+    // If the post is published, add the button to view Redlink's linked data.
+    if ( 'publish' == get_post_status( $post_id ) ) {
+        if ( $uri = wl_get_entity_uri( $post_id ) ) {
+            $uri_esc       = esc_attr( wl_get_entity_uri( $post_id ) );
+            $lod_view_href = 'http://lodview.it/lodview/?IRI=' . $uri_esc;
+            $html .= "<span id='view-post-btn'><a href='$lod_view_href' class='button button-small wl-button' target='_blank'>" .
+                        esc_html__( 'View Linked Data', 'wordlift' ) .
+                        "</a></span>\n";
+        }
+        $html .= "<span id='view-post-btn'><a href='" . WL_CONFIG_TEST_GOOGLE_RICH_SNIPPETS_URL .
+                    urlencode( get_permalink( $post_id ) ) .
+                    "' class='button button-small wl-button' target='_blank'>" .
+                    esc_html__( 'Test Google Rich Snippets', 'wordlift' ) .
+                    "</a></span>\n";
+    }
 
-	return $html;
+    return $html;
 }
 
 add_filter( 'get_sample_permalink_html', 'wl_admin_permalink_html', PHP_INT_MAX, 4 );

Spacing +12 added lines, -12 removed lines

@@ -14,25 +14,25 @@
  *
  * @return The enhanced html.
  */
-function wl_admin_permalink_html( $html, $post_id, $new_title, $new_slug ) {
+function wl_admin_permalink_html($html, $post_id, $new_title, $new_slug) {
 
 	// If the post is published, add the button to view Redlink's linked data.
-	if ( 'publish' == get_post_status( $post_id ) ) {
-		if ( $uri = wl_get_entity_uri( $post_id ) ) {
-			$uri_esc       = esc_attr( wl_get_entity_uri( $post_id ) );
-			$lod_view_href = 'http://lodview.it/lodview/?IRI=' . $uri_esc;
-			$html .= "<span id='view-post-btn'><a href='$lod_view_href' class='button button-small wl-button' target='_blank'>" .
-			         esc_html__( 'View Linked Data', 'wordlift' ) .
+	if ('publish' == get_post_status($post_id)) {
+		if ($uri = wl_get_entity_uri($post_id)) {
+			$uri_esc       = esc_attr(wl_get_entity_uri($post_id));
+			$lod_view_href = 'http://lodview.it/lodview/?IRI='.$uri_esc;
+			$html .= "<span id='view-post-btn'><a href='$lod_view_href' class='button button-small wl-button' target='_blank'>".
+			         esc_html__('View Linked Data', 'wordlift').
 			         "</a></span>\n";
 		}
-		$html .= "<span id='view-post-btn'><a href='" . WL_CONFIG_TEST_GOOGLE_RICH_SNIPPETS_URL .
-		         urlencode( get_permalink( $post_id ) ) .
-		         "' class='button button-small wl-button' target='_blank'>" .
-		         esc_html__( 'Test Google Rich Snippets', 'wordlift' ) .
+		$html .= "<span id='view-post-btn'><a href='".WL_CONFIG_TEST_GOOGLE_RICH_SNIPPETS_URL.
+		         urlencode(get_permalink($post_id)).
+		         "' class='button button-small wl-button' target='_blank'>".
+		         esc_html__('Test Google Rich Snippets', 'wordlift').
 		         "</a></span>\n";
 	}
 
 	return $html;
 }
 
-add_filter( 'get_sample_permalink_html', 'wl_admin_permalink_html', PHP_INT_MAX, 4 );
+add_filter('get_sample_permalink_html', 'wl_admin_permalink_html', PHP_INT_MAX, 4);

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.
  */

Indentation +119 added lines, -119 removed lines

@@ -14,56 +14,56 @@ 
  */
 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::TYPE_NAME,
-		'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(
-			'taxonomy' => Wordlift_Entity_Types_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::TYPE_NAME,
+        '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(
+            'taxonomy' => Wordlift_Entity_Types_Taxonomy_Service::TAXONOMY_NAME,
+            'field'    => 'slug',
+            'terms'    => 'place'
+        )
+    ) );
 }
 
 /**
@@ -78,88 +78,88 @@ 
  */
 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;
 }
 
 /**
@@ -173,13 +173,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' );

Spacing +37 added lines, -37 removed lines

@@ -12,35 +12,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::TYPE_NAME,
 		'nopaging'    => true,
@@ -63,7 +63,7 @@ 
 			'field'    => 'slug',
 			'terms'    => 'place'
 		)
-	) );
+	));
 }
 
 /**
@@ -76,7 +76,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;
@@ -91,13 +91,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;
 		}
 
@@ -105,38 +105,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(
@@ -151,7 +151,7 @@ 
 
 		// Formatting boundaries in a Leaflet-like format (see LatLngBounds).
 		// http://leafletjs.com/reference.html#latlngbounds
-		$boundaries[] = array( $coordinates['latitude'], $coordinates['longitude'] );
+		$boundaries[] = array($coordinates['latitude'], $coordinates['longitude']);
 
 	}
 
@@ -174,16 +174,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

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/admin/wordlift_admin_save_post.php

Indentation +73 added lines, -73 removed lines

@@ -23,19 +23,19 @@ 
  */
 function wl_transition_post_status( $new_status, $old_status, $post ) {
 
-	// wl_write_log( "wl_transition_post_status [ new status :: $new_status ][ old status :: $old_status ][ post ID :: $post->ID ]" );
-
-	// transition from *published* to any other status: delete the post.
-	if ( 'publish' === $old_status && 'publish' !== $new_status ) {
-		rl_delete_post( $post );
-	}
-
-	// when a post is published, then all the referenced entities must be published.
-	if ( 'publish' !== $old_status && 'publish' === $new_status ) {
-		foreach ( wl_core_get_related_entity_ids( $post->ID ) as $entity_id ) {
-			wl_update_post_status( $entity_id, 'publish' );
-		}
-	}
+    // wl_write_log( "wl_transition_post_status [ new status :: $new_status ][ old status :: $old_status ][ post ID :: $post->ID ]" );
+
+    // transition from *published* to any other status: delete the post.
+    if ( 'publish' === $old_status && 'publish' !== $new_status ) {
+        rl_delete_post( $post );
+    }
+
+    // when a post is published, then all the referenced entities must be published.
+    if ( 'publish' !== $old_status && 'publish' === $new_status ) {
+        foreach ( wl_core_get_related_entity_ids( $post->ID ) as $entity_id ) {
+            wl_update_post_status( $entity_id, 'publish' );
+        }
+    }
 }
 
 // hook save events.
@@ -49,49 +49,49 @@ 
  */
 function rl_delete_post( $post ) {
 
-	$post_id = ( is_numeric( $post ) ? $post : $post->ID );
+    $post_id = ( is_numeric( $post ) ? $post : $post->ID );
 
-	// hide all entities that are not referenced by any published post.
-	foreach ( wl_core_get_related_entity_ids( $post_id ) as $entity_id ) {
+    // hide all entities that are not referenced by any published post.
+    foreach ( wl_core_get_related_entity_ids( $post_id ) as $entity_id ) {
 
-		// check if there is at least one referencing post published.
-		$is_published = array_reduce( wl_core_get_related_post_ids( $entity_id ), function ( $carry, $item ) {
+        // check if there is at least one referencing post published.
+        $is_published = array_reduce( wl_core_get_related_post_ids( $entity_id ), function ( $carry, $item ) {
                     $post = get_post( $item );
                     return ( $carry || ( 'publish' === $post->post_status ) );
-		} );
-		// set the entity to draft if no referencing posts are published.
-		if ( ! $is_published ) {
-			wl_update_post_status( $entity_id, 'draft' );
-		}
-	}
-
-	// get the entity URI (valid also for posts)
-	$uri_esc = wl_sparql_escape_uri( wl_get_entity_uri( $post_id ) );
-
-	wl_write_log( "rl_delete_post [ post id :: $post_id ][ uri esc :: $uri_esc ]" );
-
-	// create the SPARQL statement by joining the SPARQL prefixes and deleting any known predicate.
-	$stmt = rl_sparql_prefixes();
-	foreach ( wl_predicates() as $predicate ) {
-		$stmt .= "DELETE { <$uri_esc> $predicate ?o . } WHERE { <$uri_esc> $predicate ?o . };\n" .
-		         "DELETE { ?s $predicate <$uri_esc> . } WHERE { ?s $predicate <$uri_esc> . };\n";
-	}
-
-	// if the post is an entity and has exported properties, delete the related predicates.
-	if ( Wordlift_Entity_Service::TYPE_NAME === $post->post_type ) {
-		$type = wl_entity_type_taxonomy_get_type( $post->ID );
-
-		if ( isset( $type['custom_fields'] ) ) {
-			foreach ( $type['custom_fields'] as $field => $params ) {
-				// TODO: enclose in <> only if predicate starts with http(s)://
-				$predicate = '<' . $params['predicate'] . '>';
-				$stmt .= "DELETE { <$uri_esc> $predicate ?o . } WHERE { <$uri_esc> $predicate ?o . };\n";
-			}
-		}
-	}
-
-	// finally execute the query.
-	rl_execute_sparql_update_query( $stmt );
+        } );
+        // set the entity to draft if no referencing posts are published.
+        if ( ! $is_published ) {
+            wl_update_post_status( $entity_id, 'draft' );
+        }
+    }
+
+    // get the entity URI (valid also for posts)
+    $uri_esc = wl_sparql_escape_uri( wl_get_entity_uri( $post_id ) );
+
+    wl_write_log( "rl_delete_post [ post id :: $post_id ][ uri esc :: $uri_esc ]" );
+
+    // create the SPARQL statement by joining the SPARQL prefixes and deleting any known predicate.
+    $stmt = rl_sparql_prefixes();
+    foreach ( wl_predicates() as $predicate ) {
+        $stmt .= "DELETE { <$uri_esc> $predicate ?o . } WHERE { <$uri_esc> $predicate ?o . };\n" .
+                    "DELETE { ?s $predicate <$uri_esc> . } WHERE { ?s $predicate <$uri_esc> . };\n";
+    }
+
+    // if the post is an entity and has exported properties, delete the related predicates.
+    if ( Wordlift_Entity_Service::TYPE_NAME === $post->post_type ) {
+        $type = wl_entity_type_taxonomy_get_type( $post->ID );
+
+        if ( isset( $type['custom_fields'] ) ) {
+            foreach ( $type['custom_fields'] as $field => $params ) {
+                // TODO: enclose in <> only if predicate starts with http(s)://
+                $predicate = '<' . $params['predicate'] . '>';
+                $stmt .= "DELETE { <$uri_esc> $predicate ?o . } WHERE { <$uri_esc> $predicate ?o . };\n";
+            }
+        }
+    }
+
+    // finally execute the query.
+    rl_execute_sparql_update_query( $stmt );
 }
 
 /**
@@ -102,34 +102,34 @@ 
  */
 function wl_update_post_status( $post_id, $status ) {
 
-	wl_write_log( "wl_update_post_status [ post ID :: $post_id ][ status :: $status ]" );
+    wl_write_log( "wl_update_post_status [ post ID :: $post_id ][ status :: $status ]" );
 
-	global $wpdb;
+    global $wpdb;
 
-	if ( ! $post = get_post( $post_id ) ) {
-		return;
-	}
+    if ( ! $post = get_post( $post_id ) ) {
+        return;
+    }
 
-	if ( $status === $post->post_status ) {
-		return;
-	}
+    if ( $status === $post->post_status ) {
+        return;
+    }
 
-	$wpdb->update( $wpdb->posts, array( 'post_status' => $status ), array( 'ID' => $post->ID ) );
+    $wpdb->update( $wpdb->posts, array( 'post_status' => $status ), array( 'ID' => $post->ID ) );
 
-	clean_post_cache( $post->ID );
+    clean_post_cache( $post->ID );
 
-	$old_status        = $post->post_status;
-	$post->post_status = $status;
+    $old_status        = $post->post_status;
+    $post->post_status = $status;
 
-	wp_transition_post_status( $status, $old_status, $post );
+    wp_transition_post_status( $status, $old_status, $post );
 
-	/** This action is documented in wp-includes/post.php */
-	do_action( 'edit_post', $post->ID, $post );
-	/** This action is documented in wp-includes/post.php */
-	do_action( "save_post_{$post->post_type}", $post->ID, $post, true );
-	/** This action is documented in wp-includes/post.php */
-	do_action( 'wl_linked_data_save_post', $post->ID );
-	/** This action is documented in wp-includes/post.php */
-	do_action( 'wp_insert_post', $post->ID, $post, true );
+    /** This action is documented in wp-includes/post.php */
+    do_action( 'edit_post', $post->ID, $post );
+    /** This action is documented in wp-includes/post.php */
+    do_action( "save_post_{$post->post_type}", $post->ID, $post, true );
+    /** This action is documented in wp-includes/post.php */
+    do_action( 'wl_linked_data_save_post', $post->ID );
+    /** This action is documented in wp-includes/post.php */
+    do_action( 'wp_insert_post', $post->ID, $post, true );
 }
 

Spacing +36 added lines, -36 removed lines

@@ -21,25 +21,25 @@ 
  * @param string $old_status The old post status
  * @param array $post An array with the post data
  */
-function wl_transition_post_status( $new_status, $old_status, $post ) {
+function wl_transition_post_status($new_status, $old_status, $post) {
 
 	// wl_write_log( "wl_transition_post_status [ new status :: $new_status ][ old status :: $old_status ][ post ID :: $post->ID ]" );
 
 	// transition from *published* to any other status: delete the post.
-	if ( 'publish' === $old_status && 'publish' !== $new_status ) {
-		rl_delete_post( $post );
+	if ('publish' === $old_status && 'publish' !== $new_status) {
+		rl_delete_post($post);
 	}
 
 	// when a post is published, then all the referenced entities must be published.
-	if ( 'publish' !== $old_status && 'publish' === $new_status ) {
-		foreach ( wl_core_get_related_entity_ids( $post->ID ) as $entity_id ) {
-			wl_update_post_status( $entity_id, 'publish' );
+	if ('publish' !== $old_status && 'publish' === $new_status) {
+		foreach (wl_core_get_related_entity_ids($post->ID) as $entity_id) {
+			wl_update_post_status($entity_id, 'publish');
 		}
 	}
 }
 
 // hook save events.
-add_action( 'transition_post_status', 'wl_transition_post_status', 10, 3 );
+add_action('transition_post_status', 'wl_transition_post_status', 10, 3);
 
 
 /**
@@ -47,51 +47,51 @@ 
  *
  * @param array|int $post An array of post data
  */
-function rl_delete_post( $post ) {
+function rl_delete_post($post) {
 
-	$post_id = ( is_numeric( $post ) ? $post : $post->ID );
+	$post_id = (is_numeric($post) ? $post : $post->ID);
 
 	// hide all entities that are not referenced by any published post.
-	foreach ( wl_core_get_related_entity_ids( $post_id ) as $entity_id ) {
+	foreach (wl_core_get_related_entity_ids($post_id) as $entity_id) {
 
 		// check if there is at least one referencing post published.
-		$is_published = array_reduce( wl_core_get_related_post_ids( $entity_id ), function ( $carry, $item ) {
-                    $post = get_post( $item );
-                    return ( $carry || ( 'publish' === $post->post_status ) );
+		$is_published = array_reduce(wl_core_get_related_post_ids($entity_id), function($carry, $item) {
+                    $post = get_post($item);
+                    return ($carry || ('publish' === $post->post_status));
 		} );
 		// set the entity to draft if no referencing posts are published.
-		if ( ! $is_published ) {
-			wl_update_post_status( $entity_id, 'draft' );
+		if ( ! $is_published) {
+			wl_update_post_status($entity_id, 'draft');
 		}
 	}
 
 	// get the entity URI (valid also for posts)
-	$uri_esc = wl_sparql_escape_uri( wl_get_entity_uri( $post_id ) );
+	$uri_esc = wl_sparql_escape_uri(wl_get_entity_uri($post_id));
 
-	wl_write_log( "rl_delete_post [ post id :: $post_id ][ uri esc :: $uri_esc ]" );
+	wl_write_log("rl_delete_post [ post id :: $post_id ][ uri esc :: $uri_esc ]");
 
 	// create the SPARQL statement by joining the SPARQL prefixes and deleting any known predicate.
 	$stmt = rl_sparql_prefixes();
-	foreach ( wl_predicates() as $predicate ) {
-		$stmt .= "DELETE { <$uri_esc> $predicate ?o . } WHERE { <$uri_esc> $predicate ?o . };\n" .
+	foreach (wl_predicates() as $predicate) {
+		$stmt .= "DELETE { <$uri_esc> $predicate ?o . } WHERE { <$uri_esc> $predicate ?o . };\n".
 		         "DELETE { ?s $predicate <$uri_esc> . } WHERE { ?s $predicate <$uri_esc> . };\n";
 	}
 
 	// if the post is an entity and has exported properties, delete the related predicates.
-	if ( Wordlift_Entity_Service::TYPE_NAME === $post->post_type ) {
-		$type = wl_entity_type_taxonomy_get_type( $post->ID );
+	if (Wordlift_Entity_Service::TYPE_NAME === $post->post_type) {
+		$type = wl_entity_type_taxonomy_get_type($post->ID);
 
-		if ( isset( $type['custom_fields'] ) ) {
-			foreach ( $type['custom_fields'] as $field => $params ) {
+		if (isset($type['custom_fields'])) {
+			foreach ($type['custom_fields'] as $field => $params) {
 				// TODO: enclose in <> only if predicate starts with http(s)://
-				$predicate = '<' . $params['predicate'] . '>';
+				$predicate = '<'.$params['predicate'].'>';
 				$stmt .= "DELETE { <$uri_esc> $predicate ?o . } WHERE { <$uri_esc> $predicate ?o . };\n";
 			}
 		}
 	}
 
 	// finally execute the query.
-	rl_execute_sparql_update_query( $stmt );
+	rl_execute_sparql_update_query($stmt);
 }
 
 /**
@@ -100,36 +100,36 @@ 
  * @param int $post_id The post ID
  * @param string $status The new status
  */
-function wl_update_post_status( $post_id, $status ) {
+function wl_update_post_status($post_id, $status) {
 
-	wl_write_log( "wl_update_post_status [ post ID :: $post_id ][ status :: $status ]" );
+	wl_write_log("wl_update_post_status [ post ID :: $post_id ][ status :: $status ]");
 
 	global $wpdb;
 
-	if ( ! $post = get_post( $post_id ) ) {
+	if ( ! $post = get_post($post_id)) {
 		return;
 	}
 
-	if ( $status === $post->post_status ) {
+	if ($status === $post->post_status) {
 		return;
 	}
 
-	$wpdb->update( $wpdb->posts, array( 'post_status' => $status ), array( 'ID' => $post->ID ) );
+	$wpdb->update($wpdb->posts, array('post_status' => $status), array('ID' => $post->ID));
 
-	clean_post_cache( $post->ID );
+	clean_post_cache($post->ID);
 
 	$old_status        = $post->post_status;
 	$post->post_status = $status;
 
-	wp_transition_post_status( $status, $old_status, $post );
+	wp_transition_post_status($status, $old_status, $post);
 
 	/** This action is documented in wp-includes/post.php */
-	do_action( 'edit_post', $post->ID, $post );
+	do_action('edit_post', $post->ID, $post);
 	/** This action is documented in wp-includes/post.php */
-	do_action( "save_post_{$post->post_type}", $post->ID, $post, true );
+	do_action("save_post_{$post->post_type}", $post->ID, $post, true);
 	/** This action is documented in wp-includes/post.php */
-	do_action( 'wl_linked_data_save_post', $post->ID );
+	do_action('wl_linked_data_save_post', $post->ID);
 	/** This action is documented in wp-includes/post.php */
-	do_action( 'wp_insert_post', $post->ID, $post, true );
+	do_action('wp_insert_post', $post->ID, $post, true);
 }
 

src/includes/class-wordlift-deactivator.php

Indentation +9 added lines, -9 removed lines

@@ -22,15 +22,15 @@
  */
 class Wordlift_Deactivator {
 
-	/**
-	 * Short Description. (use period)
-	 *
-	 * Long Description.
-	 *
-	 * @since    1.0.0
-	 */
-	public static function deactivate() {
+    /**
+     * Short Description. (use period)
+     *
+     * Long Description.
+     *
+     * @since    1.0.0
+     */
+    public static function deactivate() {
 
-	}
+    }
 
 }

src/includes/class-wordlift-entity-types-taxonomy-service.php

Indentation +1 added lines, -1 removed lines

@@ -7,6 +7,6 @@
  */
 class Wordlift_Entity_Types_Taxonomy_Service {
 
-	const TAXONOMY_NAME = 'wl_entity_type';
+    const TAXONOMY_NAME = 'wl_entity_type';
 
 }

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/modules/core/wordlift_core.php

Spacing +6 added lines, -6 removed lines

@@ -1,19 +1,19 @@
 <?php
 
 // Include constants
-require_once( 'wordlift_core_constants.php' );
+require_once('wordlift_core_constants.php');
 
 // Include methods that deal with post/entity relations
-require_once( 'wordlift_core_post_entity_relations.php' );
+require_once('wordlift_core_post_entity_relations.php');
 
 // Include wl_schema api
-require_once( 'wordlift_core_schema_api.php' );
+require_once('wordlift_core_schema_api.php');
 
 // Include the Entity API which allow to query for entity posts.
-require_once( 'wordlift_core_entity_api.php' );
+require_once('wordlift_core_entity_api.php');
 
 // Include methods for Wordlift plugin install
-require_once( 'wordlift_core_install.php' );
+require_once('wordlift_core_install.php');
 
 // Include utility global functions 
-require_once( 'wordlift_core_functions.php' );
\ No newline at end of file
+require_once('wordlift_core_functions.php');
\ No newline at end of file