litefeel / writing-on-github

lib/client/fetch.php

Spacing +19 added lines, -19 removed lines

@@ -16,13 +16,13 @@ 
 	 *
 	 * @return Writing_On_GitHub_File_Info[]|WP_Error
 	 */
-	public function compare( $sha ) {
+	public function compare($sha) {
 		// https://api.github.com/repos/litefeel/testwpsync/compare/861f87e8851b8debb78db548269d29f8da4d94ac...master
 		$endpoint = $this->compare_endpoint();
 		$branch = $this->branch();
-		$data = $this->call( 'GET', "$endpoint/$sha...$branch" );
+		$data = $this->call('GET', "$endpoint/$sha...$branch");
 
-		if ( is_wp_error( $data ) ) {
+		if (is_wp_error($data)) {
 			return $data;
 		}
 
@@ -44,13 +44,13 @@ 
 	 *
 	 * @return mixed
 	 */
-	public function remote_contents( $post ) {
-		return $this->call( 'GET', $this->content_endpoint( $post->github_path() ) );
+	public function remote_contents($post) {
+		return $this->call('GET', $this->content_endpoint($post->github_path()));
 	}
 
-	public function exists( $path ) {
-		$result = $this->call( 'GET', $this->content_endpoint( $path ) );
-		if ( is_wp_error( $result ) ) {
+	public function exists($path) {
+		$result = $this->call('GET', $this->content_endpoint($path));
+		if (is_wp_error($result)) {
 			return false;
 		}
 		return true;
@@ -63,27 +63,27 @@ 
 	 *
 	 * @return Writing_On_GitHub_File_Info[]|WP_Error
 	 */
-	public function tree_recursive( $sha = '_default' ) {
+	public function tree_recursive($sha = '_default') {
 
-		if ( '_default' === $sha ) {
+		if ('_default' === $sha) {
 			$sha = $this->branch();
 		}
 
-		$data = $this->call( 'GET', $this->tree_endpoint() . '/' . $sha . '?recursive=1' );
+		$data = $this->call('GET', $this->tree_endpoint() . '/' . $sha . '?recursive=1');
 
-		if ( is_wp_error( $data ) ) {
+		if (is_wp_error($data)) {
 			return $data;
 		}
 
 		$files = array();
 
-		foreach ( $data->tree as $index => $thing ) {
+		foreach ($data->tree as $index => $thing) {
 			// We need to remove the trees because
 			// the recursive tree includes both
 			// the subtrees as well the subtrees' blobs.
-			if ( 'blob' === $thing->type ) {
+			if ('blob' === $thing->type) {
 				$thing->status = '';
-				$files[] = new Writing_On_GitHub_File_Info( $thing );
+				$files[] = new Writing_On_GitHub_File_Info($thing);
 			}
 		}
 
@@ -97,14 +97,14 @@ 
 	 *
 	 * @return Writing_On_GitHub_Blob|WP_Error
 	 */
-	public function blob( Writing_On_GitHub_File_Info $fileinfo ) {
-		$data = $this->call( 'GET', $this->blob_endpoint() . '/' . $fileinfo->sha );
+	public function blob(Writing_On_GitHub_File_Info $fileinfo) {
+		$data = $this->call('GET', $this->blob_endpoint() . '/' . $fileinfo->sha);
 
-		if ( is_wp_error( $data ) ) {
+		if (is_wp_error($data)) {
 			return $data;
 		}
 
 		$data->path = $fileinfo->path;
-		return new Writing_On_GitHub_Blob( $data );
+		return new Writing_On_GitHub_Blob($data);
 	}
 }

lib/database.php

Indentation +45 added lines, -45 removed lines

@@ -42,10 +42,10 @@ 
 	/**
 	 * Queries the database for all of the supported posts.
 	 *
-     * @param  bool $force
-     *
-     * @return Writing_On_GitHub_Post[]|WP_Error
-     */
+	 * @param  bool $force
+	 *
+	 * @return Writing_On_GitHub_Post[]|WP_Error
+	 */
 	public function fetch_all_supported( $force = false ) {
 		$args  = array(
 			'post_type'   => $this->get_whitelisted_post_types(),
@@ -187,45 +187,45 @@ 
 
 		// update tags
 		if ( isset( $meta['tags'] ) && $meta['tags'] ) {
-		    $args['tags_input'] = $meta['tags'];
+			$args['tags_input'] = $meta['tags'];
 		}
 
 		// update categories
 		if ( isset( $meta['categories'] ) && $meta['categories'] ) {
-		    $categories = $meta['categories'];
-		    if (!is_array($categories)) {
-		        $categories = array($categories);
-		    }
-		    $terms = get_terms(array(
-		        'taxonomy' => 'category',
-		        'fields' => 'id=>name',
-		        'hide_empty' => 0,
-		        'name' => $categories
-		        )
-		    );
-		    $map = array();
-		    foreach ($categories as $name) {
-		        $map[$name] = 1;
-		    }
-
-		    $ids = array();
-		    if (!empty($terms)) {
-		        foreach ($terms as $id => $name) {
-		            $ids[] = $id;
-		            unset($map[$name]);
-		        }
-		    }
-
-		    // create new terms
-		    if (!empty($map)) {
-		        foreach ($map as $name => $value) {
-		            $term = wp_insert_term($name, 'category', array('parent' => 0));
-		            // array('term_id' => $term_id, 'term_taxonomy_id' => $tt_id);
-		            $ids[] = $term['term_id'];
-		        }
-		    }
-
-		    $args['post_category'] = $ids;
+			$categories = $meta['categories'];
+			if (!is_array($categories)) {
+				$categories = array($categories);
+			}
+			$terms = get_terms(array(
+				'taxonomy' => 'category',
+				'fields' => 'id=>name',
+				'hide_empty' => 0,
+				'name' => $categories
+				)
+			);
+			$map = array();
+			foreach ($categories as $name) {
+				$map[$name] = 1;
+			}
+
+			$ids = array();
+			if (!empty($terms)) {
+				foreach ($terms as $id => $name) {
+					$ids[] = $id;
+					unset($map[$name]);
+				}
+			}
+
+			// create new terms
+			if (!empty($map)) {
+				foreach ($map as $name => $value) {
+					$term = wp_insert_term($name, 'category', array('parent' => 0));
+					// array('term_id' => $term_id, 'term_taxonomy_id' => $tt_id);
+					$ids[] = $term['term_id'];
+				}
+			}
+
+			$args['post_category'] = $ids;
 		}
 
 		return $args;
@@ -427,12 +427,12 @@ 
 		if ( ! empty( $display_name ) ) {
 			$search_string = esc_attr( $display_name );
 			$query = new WP_User_Query( array(
-			    'search'         => "{$search_string}",
-			    'search_columns' => array(
-			        'display_name',
-			        'user_nicename',
-			        'user_login',
-			    )
+				'search'         => "{$search_string}",
+				'search_columns' => array(
+					'display_name',
+					'user_nicename',
+					'user_login',
+				)
 			) );
 			$users = $query->get_results();
 			$user = empty($users) ? false : $users[0];

Spacing +115 added lines, -116 removed lines

@@ -21,21 +21,21 @@ 
 	 *
 	 * @var array
 	 */
-	protected $whitelisted_post_types = array( 'post', 'page' );
+	protected $whitelisted_post_types = array('post', 'page');
 
 	/**
 	 * Currently whitelisted post statuses.
 	 *
 	 * @var array
 	 */
-	protected $whitelisted_post_statuses = array( 'publish' );
+	protected $whitelisted_post_statuses = array('publish');
 
 	/**
 	 * Instantiates a new Database object.
 	 *
 	 * @param Writing_On_GitHub $app Application container.
 	 */
-	public function __construct( Writing_On_GitHub $app ) {
+	public function __construct(Writing_On_GitHub $app) {
 		$this->app = $app;
 	}
 
@@ -46,32 +46,32 @@ 
      *
      * @return Writing_On_GitHub_Post[]|WP_Error
      */
-	public function fetch_all_supported( $force = false ) {
-		$args  = array(
+	public function fetch_all_supported($force = false) {
+		$args = array(
 			'post_type'   => $this->get_whitelisted_post_types(),
 			'post_status' => $this->get_whitelisted_post_statuses(),
 			'nopaging'    => true,
 			'fields'      => 'ids',
 		);
 
-		$query = new WP_Query( apply_filters( 'wogh_pre_fetch_all_supported', $args ) );
+		$query = new WP_Query(apply_filters('wogh_pre_fetch_all_supported', $args));
 
 		$post_ids = $query->get_posts();
 
-		if ( ! $post_ids ) {
+		if ( ! $post_ids) {
 			return new WP_Error(
 				'no_results',
-				__( 'Querying for supported posts returned no results.', 'writing-on-github' )
+				__('Querying for supported posts returned no results.', 'writing-on-github')
 			);
 		}
 
 		$results = array();
-		foreach ( $post_ids as $post_id ) {
+		foreach ($post_ids as $post_id) {
 			// Do not export posts that have already been exported
-			if ( $force || ! get_post_meta( $post_id, '_wogh_sha', true ) ||
-				 ! get_post_meta( $post_id, '_wogh_github_path', true ) ) {
+			if ($force || ! get_post_meta($post_id, '_wogh_sha', true) ||
+				 ! get_post_meta($post_id, '_wogh_github_path', true)) {
 
-				$results[] = new Writing_On_GitHub_Post( $post_id, $this->app->api() );
+				$results[] = new Writing_On_GitHub_Post($post_id, $this->app->api());
 			}
 		}
 
@@ -85,10 +85,10 @@ 
 	 *
 	 * @return WP_Error|Writing_On_GitHub_Post
 	 */
-	public function fetch_by_id( $post_id ) {
-		$post = new Writing_On_GitHub_Post( $post_id, $this->app->api() );
+	public function fetch_by_id($post_id) {
+		$post = new Writing_On_GitHub_Post($post_id, $this->app->api());
 
-		if ( ! $this->is_post_supported( $post ) ) {
+		if ( ! $this->is_post_supported($post)) {
 			return new WP_Error(
 				'unsupported_post',
 				sprintf(
@@ -112,7 +112,7 @@ 
 	 *
 	 * @return string|WP_Error
 	 */
-	public function save_posts( array $posts ) {
+	public function save_posts(array $posts) {
 
 		/**
 		 * Whether an error has occurred.
@@ -121,20 +121,19 @@ 
 		 */
 		$error = false;
 
-		foreach ( $posts as $post ) {
-			$args = apply_filters( 'wogh_pre_import_args', $this->post_args( $post ), $post );
+		foreach ($posts as $post) {
+			$args = apply_filters('wogh_pre_import_args', $this->post_args($post), $post);
 
-			remove_filter( 'content_save_pre', 'wp_filter_post_kses' );
+			remove_filter('content_save_pre', 'wp_filter_post_kses');
 			$post_id = $post->is_new() ?
-				wp_insert_post( $args, true ) :
-				wp_update_post( $args, true );
-			add_filter( 'content_save_pre', 'wp_filter_post_kses' );
+				wp_insert_post($args, true) : wp_update_post($args, true);
+			add_filter('content_save_pre', 'wp_filter_post_kses');
 
-			if ( is_wp_error( $post_id ) ) {
-				if ( ! $error ) {
+			if (is_wp_error($post_id)) {
+				if ( ! $error) {
 					$error = $post_id;
 				} else {
-					$error->add( $post_id->get_error_code(), $post_id->get_error_message() );
+					$error->add($post_id->get_error_code(), $post_id->get_error_message());
 				}
 
 				// Abort saving if updating the post fails.
@@ -143,20 +142,20 @@ 
 
 			// $this->set_revision_author( $post_id, $user_id );
 
-			if ( $post->is_new() ) {
+			if ($post->is_new()) {
 				$author = false;
 				$meta = $post->get_meta();
-				if ( ! empty( $meta ) && ! empty( $meta['author'] ) ) {
+				if ( ! empty($meta) && ! empty($meta['author'])) {
 					$author = $meta['author'];
 				}
-				$user    = $this->fetch_commit_user( $author );
-				$user_id = ! is_wp_error( $user ) ? $user->ID : 0;
-				$this->set_post_author( $post_id, $user_id );
+				$user    = $this->fetch_commit_user($author);
+				$user_id = ! is_wp_error($user) ? $user->ID : 0;
+				$this->set_post_author($post_id, $user_id);
 			}
 
-			$post->set_post( get_post( $post_id ) );
+			$post->set_post(get_post($post_id));
 
-			$meta = apply_filters( 'wogh_pre_import_meta', $post->get_meta(), $post );
+			$meta = apply_filters('wogh_pre_import_meta', $post->get_meta(), $post);
 
 			// unset( $meta['tags'] );
 			// unset( $meta['categories'] );
@@ -171,14 +170,14 @@ 
 			// }
 		}
 
-		if ( $error ) {
+		if ($error) {
 			return $error;
 		}
 
-		return __( 'Successfully saved posts.', 'writing-on-github' );
+		return __('Successfully saved posts.', 'writing-on-github');
 	}
 
-	protected function post_args( $post ) {
+	protected function post_args($post) {
 		$args = $post->get_args();
 		$meta = $post->get_meta();
 
@@ -186,14 +185,14 @@ 
 		$args['post_content'] = addslashes($args['post_content']);
 
 		// update tags
-		if ( isset( $meta['tags'] ) && $meta['tags'] ) {
+		if (isset($meta['tags']) && $meta['tags']) {
 		    $args['tags_input'] = $meta['tags'];
 		}
 
 		// update categories
-		if ( isset( $meta['categories'] ) && $meta['categories'] ) {
+		if (isset($meta['categories']) && $meta['categories']) {
 		    $categories = $meta['categories'];
-		    if (!is_array($categories)) {
+		    if ( ! is_array($categories)) {
 		        $categories = array($categories);
 		    }
 		    $terms = get_terms(array(
@@ -209,7 +208,7 @@ 
 		    }
 
 		    $ids = array();
-		    if (!empty($terms)) {
+		    if ( ! empty($terms)) {
 		        foreach ($terms as $id => $name) {
 		            $ids[] = $id;
 		            unset($map[$name]);
@@ -217,7 +216,7 @@ 
 		    }
 
 		    // create new terms
-		    if (!empty($map)) {
+		    if ( ! empty($map)) {
 		        foreach ($map as $name => $value) {
 		            $term = wp_insert_term($name, 'category', array('parent' => 0));
 		            // array('term_id' => $term_id, 'term_taxonomy_id' => $tt_id);
@@ -238,77 +237,77 @@ 
 	 *
 	 * @return string|WP_Error
 	 */
-	public function delete_post_by_path( $path ) {
-		$query = new WP_Query( array(
+	public function delete_post_by_path($path) {
+		$query = new WP_Query(array(
 			'meta_key'       => '_wogh_github_path',
 			'meta_value'     => $path,
 			'meta_compare'   => '=',
 			'posts_per_page' => 1,
 			'fields'         => 'ids',
-		) );
+		));
 
 		$post_id = $query->get_posts();
-		$post_id = array_pop( $post_id );
+		$post_id = array_pop($post_id);
 
-		if ( ! $post_id ) {
-			$parts     = explode( '/', $path );
-			$filename  = array_pop( $parts );
-			$directory = $parts ? array_shift( $parts ) : '';
+		if ( ! $post_id) {
+			$parts     = explode('/', $path);
+			$filename  = array_pop($parts);
+			$directory = $parts ? array_shift($parts) : '';
 
-			if ( false !== strpos( $directory, 'post' ) ) {
-				preg_match( '/([0-9]{4})-([0-9]{2})-([0-9]{2})-(.*)\.md/', $filename, $matches );
+			if (false !== strpos($directory, 'post')) {
+				preg_match('/([0-9]{4})-([0-9]{2})-([0-9]{2})-(.*)\.md/', $filename, $matches);
 				$title = $matches[4];
 
-				$query = new WP_Query( array(
+				$query = new WP_Query(array(
 					'name'     => $title,
 					'posts_per_page' => 1,
 					'post_type' => $this->get_whitelisted_post_types(),
 					'fields'         => 'ids',
-				) );
+				));
 
 				$post_id = $query->get_posts();
-				$post_id = array_pop( $post_id );
+				$post_id = array_pop($post_id);
 			}
 
-			if ( ! $post_id ) {
-				preg_match( '/(.*)\.md/', $filename, $matches );
+			if ( ! $post_id) {
+				preg_match('/(.*)\.md/', $filename, $matches);
 				$title = $matches[1];
 
-				$query = new WP_Query( array(
+				$query = new WP_Query(array(
 					'name'     => $title,
 					'posts_per_page' => 1,
 					'post_type' => $this->get_whitelisted_post_types(),
 					'fields'         => 'ids',
-				) );
+				));
 
 				$post_id = $query->get_posts();
-				$post_id = array_pop( $post_id );
+				$post_id = array_pop($post_id);
 			}
 		}
 
-		if ( ! $post_id ) {
+		if ( ! $post_id) {
 			return new WP_Error(
 				'path_not_found',
 				sprintf(
-					__( 'Post not found for path %s.', 'writing-on-github' ),
+					__('Post not found for path %s.', 'writing-on-github'),
 					$path
 				)
 			);
 		}
 
-		$result = wp_delete_post( $post_id );
+		$result = wp_delete_post($post_id);
 
 		// If deleting fails...
-		if ( false === $result ) {
-			$post = get_post( $post_id );
+		if (false === $result) {
+			$post = get_post($post_id);
 
 			// ...and the post both exists and isn't in the trash...
-			if ( $post && 'trash' !== $post->post_status ) {
+			if ($post && 'trash' !== $post->post_status) {
 				// ... then something went wrong.
 				return new WP_Error(
 					'db_error',
 					sprintf(
-						__( 'Failed to delete post ID %d.', 'writing-on-github' ),
+						__('Failed to delete post ID %d.', 'writing-on-github'),
 						$post_id
 					)
 				);
@@ -316,25 +315,25 @@ 
 		}
 
 		return sprintf(
-			__( 'Successfully deleted post ID %d.', 'writing-on-github' ),
+			__('Successfully deleted post ID %d.', 'writing-on-github'),
 			$post_id
 		);
 	}
 
-	public function delete_post( $post_id ) {
-		$result = wp_delete_post( $post_id );
+	public function delete_post($post_id) {
+		$result = wp_delete_post($post_id);
 
 		// If deleting fails...
-		if ( false === $result ) {
-			$post = get_post( $post_id );
+		if (false === $result) {
+			$post = get_post($post_id);
 
 			// ...and the post both exists and isn't in the trash...
-			if ( $post && 'trash' !== $post->post_status ) {
+			if ($post && 'trash' !== $post->post_status) {
 				// ... then something went wrong.
 				return new WP_Error(
 					'db_error',
 					sprintf(
-						__( 'Failed to delete post ID %d.', 'writing-on-github' ),
+						__('Failed to delete post ID %d.', 'writing-on-github'),
 						$post_id
 					)
 				);
@@ -342,7 +341,7 @@ 
 		}
 
 		return sprintf(
-			__( 'Successfully deleted post ID %d.', 'writing-on-github' ),
+			__('Successfully deleted post ID %d.', 'writing-on-github'),
 			$post_id
 		);
 	}
@@ -353,7 +352,7 @@ 
 	 * @return array
 	 */
 	protected function get_whitelisted_post_types() {
-		return apply_filters( 'wogh_whitelisted_post_types', $this->whitelisted_post_types );
+		return apply_filters('wogh_whitelisted_post_types', $this->whitelisted_post_types);
 	}
 
 	/**
@@ -362,7 +361,7 @@ 
 	 * @return array
 	 */
 	protected function get_whitelisted_post_statuses() {
-		return apply_filters( 'wogh_whitelisted_post_statuses', $this->whitelisted_post_statuses );
+		return apply_filters('wogh_whitelisted_post_statuses', $this->whitelisted_post_statuses);
 	}
 
 	/**
@@ -372,12 +371,12 @@ 
 	 *
 	 * @return string Whitelist formatted for query
 	 */
-	protected function format_for_query( $whitelist ) {
-		foreach ( $whitelist as $key => $value ) {
-			$whitelist[ $key ] = "'$value'";
+	protected function format_for_query($whitelist) {
+		foreach ($whitelist as $key => $value) {
+			$whitelist[$key] = "'$value'";
 		}
 
-		return implode( ', ', $whitelist );
+		return implode(', ', $whitelist);
 	}
 
 	/**
@@ -388,25 +387,25 @@ 
 	 *
 	 * @return boolean                          True if supported, false if not.
 	 */
-	protected function is_post_supported( Writing_On_GitHub_Post $post ) {
-		if ( wp_is_post_revision( $post->id ) ) {
+	protected function is_post_supported(Writing_On_GitHub_Post $post) {
+		if (wp_is_post_revision($post->id)) {
 			return false;
 		}
 
 		// We need to allow trashed posts to be queried, but they are not whitelisted for export.
-		if ( ! in_array( $post->status(), $this->get_whitelisted_post_statuses() ) && 'trash' !== $post->status() ) {
+		if ( ! in_array($post->status(), $this->get_whitelisted_post_statuses()) && 'trash' !== $post->status()) {
 			return false;
 		}
 
-		if ( ! in_array( $post->type(), $this->get_whitelisted_post_types() ) ) {
+		if ( ! in_array($post->type(), $this->get_whitelisted_post_types())) {
 			return false;
 		}
 
-		if ( $post->has_password() ) {
+		if ($post->has_password()) {
 			return false;
 		}
 
-		return apply_filters( 'wogh_is_post_supported', true, $post );
+		return apply_filters('wogh_is_post_supported', true, $post);
 	}
 
 	/**
@@ -419,35 +418,35 @@ 
 	 *
 	 * @return WP_Error|WP_User
 	 */
-	protected function fetch_commit_user( $display_name ) {
+	protected function fetch_commit_user($display_name) {
 		// If we can't find a user and a default hasn't been set,
 		// we're just going to set the revision author to 0.
 		$user = false;
 
-		if ( ! empty( $display_name ) ) {
-			$search_string = esc_attr( $display_name );
-			$query = new WP_User_Query( array(
+		if ( ! empty($display_name)) {
+			$search_string = esc_attr($display_name);
+			$query = new WP_User_Query(array(
 			    'search'         => "{$search_string}",
 			    'search_columns' => array(
 			        'display_name',
 			        'user_nicename',
 			        'user_login',
 			    )
-			) );
+			));
 			$users = $query->get_results();
 			$user = empty($users) ? false : $users[0];
 		}
 
-		if ( ! $user ) {
+		if ( ! $user) {
 			// Use the default user.
-			$user = get_user_by( 'id', (int) get_option( 'wogh_default_user' ) );
+			$user = get_user_by('id', (int) get_option('wogh_default_user'));
 		}
 
-		if ( ! $user ) {
+		if ( ! $user) {
 			return new WP_Error(
 				'user_not_found',
 				sprintf(
-					__( 'Commit user not found for email %s', 'writing-on-github' ),
+					__('Commit user not found for email %s', 'writing-on-github'),
 					$email
 				)
 			);
@@ -465,28 +464,28 @@ 
 	 *
 	 * @return string|WP_Error
 	 */
-	protected function set_revision_author( $post_id, $user_id ) {
-		$revision = wp_get_post_revisions( $post_id );
+	protected function set_revision_author($post_id, $user_id) {
+		$revision = wp_get_post_revisions($post_id);
 
-		if ( ! $revision ) {
-			$new_revision = wp_save_post_revision( $post_id );
+		if ( ! $revision) {
+			$new_revision = wp_save_post_revision($post_id);
 
-			if ( ! $new_revision || is_wp_error( $new_revision ) ) {
-				return new WP_Error( 'db_error', 'There was a problem saving a new revision.' );
+			if ( ! $new_revision || is_wp_error($new_revision)) {
+				return new WP_Error('db_error', 'There was a problem saving a new revision.');
 			}
 
 			// `wp_save_post_revision` returns the ID, whereas `get_post_revision` returns the whole object
 			// in order to be consistent, let's make sure we have the whole object before continuing.
-			$revision = get_post( $new_revision );
+			$revision = get_post($new_revision);
 
-			if ( ! $revision ) {
-				return new WP_Error( 'db_error', 'There was a problem retrieving the newly recreated revision.' );
+			if ( ! $revision) {
+				return new WP_Error('db_error', 'There was a problem retrieving the newly recreated revision.');
 			}
 		} else {
-			$revision = array_shift( $revision );
+			$revision = array_shift($revision);
 		}
 
-		return $this->set_post_author( $revision->ID, $user_id );
+		return $this->set_post_author($revision->ID, $user_id);
 	}
 
 	/**
@@ -499,7 +498,7 @@ 
 	 *
 	 * @return string|WP_Error
 	 */
-	protected function set_post_author( $post_id, $user_id ) {
+	protected function set_post_author($post_id, $user_id) {
 		global $wpdb;
 
 		$result = $wpdb->update(
@@ -510,25 +509,25 @@ 
 			array(
 				'ID' => (int) $post_id,
 			),
-			array( '%d' ),
-			array( '%d' )
+			array('%d'),
+			array('%d')
 		);
 
-		if ( false === $result ) {
-			return new WP_Error( 'db_error', $wpdb->last_error );
+		if (false === $result) {
+			return new WP_Error('db_error', $wpdb->last_error);
 		}
 
-		if ( 0 === $result ) {
+		if (0 === $result) {
 			return sprintf(
-				__( 'No change for post ID %d.', 'writing-on-github' ),
+				__('No change for post ID %d.', 'writing-on-github'),
 				$post_id
 			);
 		}
 
-		clean_post_cache( $post_id );
+		clean_post_cache($post_id);
 
 		return sprintf(
-			__( 'Successfully updated post ID %d.', 'writing-on-github' ),
+			__('Successfully updated post ID %d.', 'writing-on-github'),
 			$post_id
 		);
 	}
@@ -541,7 +540,7 @@ 
 	 *
 	 * @return bool|int
 	 */
-	public function set_post_sha( $post, $sha ) {
-		return update_post_meta( $post->id, '_wogh_sha', $sha );
+	public function set_post_sha($post, $sha) {
+		return update_post_meta($post->id, '_wogh_sha', $sha);
 	}
 }

lib/controller.php

Indentation +6 added lines, -6 removed lines

@@ -113,8 +113,8 @@ 
 		$this->app->semaphore()->unlock();
 
 		if ( is_wp_error( $result ) ) {
-            //@var WP_Error $result
-            update_option( '_wogh_import_error', $result->get_error_message() );
+			//@var WP_Error $result
+			update_option( '_wogh_import_error', $result->get_error_message() );
 
 			return $this->app->response()->error( $result );
 		}
@@ -127,10 +127,10 @@ 
 	/**
 	 * Export all the posts in the database to GitHub.
 	 *
-     * @param  int        $user_id
-     * @param  boolean    $force
-     * @return boolean
-     */
+	 * @param  int        $user_id
+	 * @param  boolean    $force
+	 * @return boolean
+	 */
 	public function export_all( $user_id, $force = false ) {
 		if ( ! $this->app->semaphore()->is_open() ) {
 			return $this->app->response()->error( new WP_Error(

Spacing +75 added lines, -75 removed lines

@@ -21,7 +21,7 @@ 
 	 *
 	 * @param Writing_On_GitHub $app Applicatio container.
 	 */
-	public function __construct( Writing_On_GitHub $app ) {
+	public function __construct(Writing_On_GitHub $app) {
 		$this->app = $app;
 	}
 
@@ -34,57 +34,57 @@ 
 	 */
 	public function pull_posts() {
 		$this->set_ajax();
-		if ( ! $this->app->semaphore()->is_open() ) {
-			return $this->app->response()->error( new WP_Error(
+		if ( ! $this->app->semaphore()->is_open()) {
+			return $this->app->response()->error(new WP_Error(
 				'semaphore_locked',
-				sprintf( __( '%s : Semaphore is locked, import/export already in progress.', 'writing-on-github' ), 'Controller::pull_posts()' )
-			) );
+				sprintf(__('%s : Semaphore is locked, import/export already in progress.', 'writing-on-github'), 'Controller::pull_posts()')
+			));
 		}
 
-		if ( ! $this->app->request()->is_secret_valid() ) {
-			return $this->app->response()->error( new WP_Error(
+		if ( ! $this->app->request()->is_secret_valid()) {
+			return $this->app->response()->error(new WP_Error(
 				'invalid_headers',
-				__( 'Failed to validate secret.', 'writing-on-github' )
-			) );
+				__('Failed to validate secret.', 'writing-on-github')
+			));
 		}
 
 		// ping
-		if ( $this->app->request()->is_ping() ) {
-			return $this->app->response()->success( __( 'Wordpress is ready.', 'writing-on-github' ) );
+		if ($this->app->request()->is_ping()) {
+			return $this->app->response()->success(__('Wordpress is ready.', 'writing-on-github'));
 		}
 
 		// push
-		if ( ! $this->app->request()->is_push() ) {
-			return $this->app->response()->error( new WP_Error(
+		if ( ! $this->app->request()->is_push()) {
+			return $this->app->response()->error(new WP_Error(
 				'invalid_headers',
-				__( 'Failed to validate webhook event.', 'writing-on-github' )
-			) );
+				__('Failed to validate webhook event.', 'writing-on-github')
+			));
 		}
 		$payload = $this->app->request()->payload();
 
-		if ( ! $payload->should_import() ) {
-			return $this->app->response()->error( new WP_Error(
+		if ( ! $payload->should_import()) {
+			return $this->app->response()->error(new WP_Error(
 				'invalid_payload',
 				sprintf(
-					__( "%s won't be imported.", 'writing-on-github' ),
-					strtolower( $payload->get_commit_id() ) ? : '[Missing Commit ID]'
+					__("%s won't be imported.", 'writing-on-github'),
+					strtolower($payload->get_commit_id()) ?: '[Missing Commit ID]'
 				)
-			) );
+			));
 		}
 
 		$this->app->semaphore()->lock();
-		remove_action( 'save_post', array( $this, 'export_post' ) );
-		remove_action( 'delete_post', array( $this, 'delete_post' ) );
+		remove_action('save_post', array($this, 'export_post'));
+		remove_action('delete_post', array($this, 'delete_post'));
 
-		$result = $this->app->import()->payload( $payload );
+		$result = $this->app->import()->payload($payload);
 
 		$this->app->semaphore()->unlock();
 
-		if ( is_wp_error( $result ) ) {
-			return $this->app->response()->error( $result );
+		if (is_wp_error($result)) {
+			return $this->app->response()->error($result);
 		}
 
-		return $this->app->response()->success( $result );
+		return $this->app->response()->success($result);
 	}
 
 	/**
@@ -92,36 +92,36 @@ 
 	 *
 	 * @return boolean
 	 */
-	public function import_master( $user_id ) {
-		if ( ! $this->app->semaphore()->is_open() ) {
-			return $this->app->response()->error( new WP_Error(
+	public function import_master($user_id) {
+		if ( ! $this->app->semaphore()->is_open()) {
+			return $this->app->response()->error(new WP_Error(
 				'semaphore_locked',
-				sprintf( __( '%s : Semaphore is locked, import/export already in progress.', 'writing-on-github' ), 'Controller::import_master()' )
-			) );
+				sprintf(__('%s : Semaphore is locked, import/export already in progress.', 'writing-on-github'), 'Controller::import_master()')
+			));
 		}
 
 		$this->app->semaphore()->lock();
-		remove_action( 'save_post', array( $this, 'export_post' ) );
-		remove_action( 'save_post', array( $this, 'delete_post' ) );
+		remove_action('save_post', array($this, 'export_post'));
+		remove_action('save_post', array($this, 'delete_post'));
 
-		if ( $user_id ) {
-			wp_set_current_user( $user_id );
+		if ($user_id) {
+			wp_set_current_user($user_id);
 		}
 
 		$result = $this->app->import()->master();
 
 		$this->app->semaphore()->unlock();
 
-		if ( is_wp_error( $result ) ) {
+		if (is_wp_error($result)) {
             //@var WP_Error $result
-            update_option( '_wogh_import_error', $result->get_error_message() );
+            update_option('_wogh_import_error', $result->get_error_message());
 
-			return $this->app->response()->error( $result );
+			return $this->app->response()->error($result);
 		}
 
-		update_option( '_wogh_import_complete', 'yes' );
+		update_option('_wogh_import_complete', 'yes');
 
-		return $this->app->response()->success( $result );
+		return $this->app->response()->success($result);
 	}
 
 	/**
@@ -131,33 +131,33 @@ 
      * @param  boolean    $force
      * @return boolean
      */
-	public function export_all( $user_id, $force = false ) {
-		if ( ! $this->app->semaphore()->is_open() ) {
-			return $this->app->response()->error( new WP_Error(
+	public function export_all($user_id, $force = false) {
+		if ( ! $this->app->semaphore()->is_open()) {
+			return $this->app->response()->error(new WP_Error(
 				'semaphore_locked',
-				sprintf( __( '%s : Semaphore is locked, import/export already in progress.', 'writing-on-github' ), 'Controller::export_all()' )
-			) );
+				sprintf(__('%s : Semaphore is locked, import/export already in progress.', 'writing-on-github'), 'Controller::export_all()')
+			));
 		}
 
 		$this->app->semaphore()->lock();
 
-		if ( $user_id ) {
-			wp_set_current_user( $user_id );
+		if ($user_id) {
+			wp_set_current_user($user_id);
 		}
 
 		$result = $this->app->export()->full($force);
 		$this->app->semaphore()->unlock();
 
 		// Maybe move option updating out of this class/upgrade message display?
-		if ( is_wp_error( $result ) ) {
-			update_option( '_wogh_export_error', $result->get_error_message() );
+		if (is_wp_error($result)) {
+			update_option('_wogh_export_error', $result->get_error_message());
 
-			return $this->app->response()->error( $result );
+			return $this->app->response()->error($result);
 		} else {
-			update_option( '_wogh_export_complete', 'yes' );
-			update_option( '_wogh_fully_exported', 'yes' );
+			update_option('_wogh_export_complete', 'yes');
+			update_option('_wogh_fully_exported', 'yes');
 
-			return $this->app->response()->success( $result );
+			return $this->app->response()->success($result);
 		}
 	}
 
@@ -170,27 +170,27 @@ 
 	 *
 	 * @return boolean
 	 */
-	public function export_post( $post_id ) {
-		if ( wp_is_post_revision( $post_id ) ) {
+	public function export_post($post_id) {
+		if (wp_is_post_revision($post_id)) {
 			return;
 		}
 
-		if ( ! $this->app->semaphore()->is_open() ) {
-			return $this->app->response()->error( new WP_Error(
+		if ( ! $this->app->semaphore()->is_open()) {
+			return $this->app->response()->error(new WP_Error(
 				'semaphore_locked',
-				sprintf( __( '%s : Semaphore is locked, import/export already in progress.', 'writing-on-github' ), 'Controller::export_post()' )
-			) );
+				sprintf(__('%s : Semaphore is locked, import/export already in progress.', 'writing-on-github'), 'Controller::export_post()')
+			));
 		}
 
 		$this->app->semaphore()->lock();
-		$result = $this->app->export()->update( $post_id );
+		$result = $this->app->export()->update($post_id);
 		$this->app->semaphore()->unlock();
 
-		if ( is_wp_error( $result ) ) {
-			return $this->app->response()->error( $result );
+		if (is_wp_error($result)) {
+			return $this->app->response()->error($result);
 		}
 
-		return $this->app->response()->success( $result );
+		return $this->app->response()->success($result);
 	}
 
 	/**
@@ -202,27 +202,27 @@ 
 	 *
 	 * @return boolean
 	 */
-	public function delete_post( $post_id ) {
-		if ( wp_is_post_revision( $post_id ) ) {
+	public function delete_post($post_id) {
+		if (wp_is_post_revision($post_id)) {
 			return;
 		}
 
-		if ( ! $this->app->semaphore()->is_open() ) {
-			return $this->app->response()->error( new WP_Error(
+		if ( ! $this->app->semaphore()->is_open()) {
+			return $this->app->response()->error(new WP_Error(
 				'semaphore_locked',
-				sprintf( __( '%s : Semaphore is locked, import/export already in progress.', 'writing-on-github' ), 'Controller::delete_post()' )
-			) );
+				sprintf(__('%s : Semaphore is locked, import/export already in progress.', 'writing-on-github'), 'Controller::delete_post()')
+			));
 		}
 
 		$this->app->semaphore()->lock();
-		$result = $this->app->export()->delete( $post_id );
+		$result = $this->app->export()->delete($post_id);
 		$this->app->semaphore()->unlock();
 
-		if ( is_wp_error( $result ) ) {
-			return $this->app->response()->error( $result );
+		if (is_wp_error($result)) {
+			return $this->app->response()->error($result);
 		}
 
-		return $this->app->response()->success( $result );
+		return $this->app->response()->success($result);
 	}
 
 	/**
@@ -231,8 +231,8 @@ 
 	 * than just returning data.
 	 */
 	protected function set_ajax() {
-		if ( ! defined( 'WOGH_AJAX' ) ) {
-			define( 'WOGH_AJAX', true );
+		if ( ! defined('WOGH_AJAX')) {
+			define('WOGH_AJAX', true);
 		}
 	}
 }