litefeel / writing-on-github

lib/request.php

Spacing +11 added lines, -11 removed lines

@@ -34,7 +34,7 @@ 
 	 *
 	 * @param Writing_On_GitHub $app Application container.
 	 */
-	public function __construct( Writing_On_GitHub $app ) {
+	public function __construct(Writing_On_GitHub $app) {
 		$this->app = $app;
 	}
 
@@ -49,8 +49,8 @@ 
 		$this->raw_data = $this->read_raw_data();
 
 		// Validate request secret.
-		$hash = hash_hmac( 'sha1', $this->raw_data, $this->secret() );
-		if ( 'sha1=' . $hash !== $headers['X-Hub-Signature'] ) {
+		$hash = hash_hmac('sha1', $this->raw_data, $this->secret());
+		if ('sha1=' . $hash !== $headers['X-Hub-Signature']) {
 			return false;
 		}
 
@@ -90,7 +90,7 @@ 
 	 * @return Writing_On_GitHub_Payload
 	 */
 	public function payload() {
-		return new Writing_On_GitHub_Payload( $this->app, $this->raw_data );
+		return new Writing_On_GitHub_Payload($this->app, $this->raw_data);
 	}
 
 	/**
@@ -101,11 +101,11 @@ 
 	 * @return array
 	 */
 	protected function headers() {
-		if ( $this->headers ) {
+		if ($this->headers) {
 			return $this->headers;
 		}
 
-		if ( function_exists( 'getallheaders' ) ) {
+		if (function_exists('getallheaders')) {
 
 			$this->headers = getallheaders();
 			return $this->headers;
@@ -115,9 +115,9 @@ 
 		 * @see http://www.php.net/manual/en/function.getallheaders.php
 		 */
 		$this->headers = array();
-		foreach ( $_SERVER as $name => $value ) {
-			if ( 'HTTP_' === substr( $name, 0, 5 ) ) {
-				$this->headers[ str_replace( ' ', '-', ucwords( strtolower( str_replace( '_', ' ', substr( $name, 5 ) ) ) ) ) ] = $value;
+		foreach ($_SERVER as $name => $value) {
+			if ('HTTP_' === substr($name, 0, 5)) {
+				$this->headers[str_replace(' ', '-', ucwords(strtolower(str_replace('_', ' ', substr($name, 5)))))] = $value;
 			}
 		}
 
@@ -130,7 +130,7 @@ 
 	 * @return string
 	 */
 	protected function read_raw_data() {
-		return file_get_contents( 'php://input' );
+		return file_get_contents('php://input');
 	}
 
 	/**
@@ -139,6 +139,6 @@ 
 	 * @return string
 	 */
 	protected function secret() {
-		return get_option( 'wogh_secret' );
+		return get_option('wogh_secret');
 	}
 }

lib/export.php

Indentation +18 added lines, -18 removed lines

@@ -29,9 +29,9 @@ 
 	/**
 	 * Updates all of the current posts in the database on master.
 	 *
-     * @param  bool    $force_all
-     * @return string|WP_Error
-     */
+	 * @param  bool    $force_all
+	 * @return string|WP_Error
+	 */
 	public function full( $force_all = false ) {
 		$posts = $this->app->database()->fetch_all_supported($force_all);
 
@@ -39,24 +39,24 @@ 
 			return $posts;
 		}
 
-        $error = false;
+		$error = false;
 
-        foreach ( $posts as $post ) {
-            $result = $this->update( $post->id() );
-            if ( is_wp_error( $result ) ) {
-                if ( $error ) {
-                    $error->add( $result->get_error_code(), $result->get_error_message() );
-                } else {
-                    $error = $result;
-                }
-            }
-        }
+		foreach ( $posts as $post ) {
+			$result = $this->update( $post->id() );
+			if ( is_wp_error( $result ) ) {
+				if ( $error ) {
+					$error->add( $result->get_error_code(), $result->get_error_message() );
+				} else {
+					$error = $result;
+				}
+			}
+		}
 
-        if ( is_wp_error( $error ) ) {
-            return $error;
-        }
+		if ( is_wp_error( $error ) ) {
+			return $error;
+		}
 
-        return __( 'Export to GitHub completed successfully.', 'writing-on-github' );
+		return __( 'Export to GitHub completed successfully.', 'writing-on-github' );
 	}
 
 

Spacing +57 added lines, -57 removed lines

@@ -22,7 +22,7 @@ 
 	 *
 	 * @param Writing_On_GitHub $app Application container.
 	 */
-	public function __construct( Writing_On_GitHub $app ) {
+	public function __construct(Writing_On_GitHub $app) {
 		$this->app = $app;
 	}
 
@@ -32,31 +32,31 @@ 
      * @param  bool    $force_all
      * @return string|WP_Error
      */
-	public function full( $force_all = false ) {
+	public function full($force_all = false) {
 		$posts = $this->app->database()->fetch_all_supported($force_all);
 
-		if ( is_wp_error( $posts ) ) {
+		if (is_wp_error($posts)) {
 			return $posts;
 		}
 
         $error = false;
 
-        foreach ( $posts as $post ) {
-            $result = $this->update( $post->id() );
-            if ( is_wp_error( $result ) ) {
-                if ( $error ) {
-                    $error->add( $result->get_error_code(), $result->get_error_message() );
+        foreach ($posts as $post) {
+            $result = $this->update($post->id());
+            if (is_wp_error($result)) {
+                if ($error) {
+                    $error->add($result->get_error_code(), $result->get_error_message());
                 } else {
                     $error = $result;
                 }
             }
         }
 
-        if ( is_wp_error( $error ) ) {
+        if (is_wp_error($error)) {
             return $error;
         }
 
-        return __( 'Export to GitHub completed successfully.', 'writing-on-github' );
+        return __('Export to GitHub completed successfully.', 'writing-on-github');
 	}
 
 
@@ -65,10 +65,10 @@ 
 	 * @param  int  $post_id
 	 * @return boolean
 	 */
-	protected function github_path( $post_id ) {
-		$github_path = get_post_meta( $post_id, '_wogh_github_path', true );
+	protected function github_path($post_id) {
+		$github_path = get_post_meta($post_id, '_wogh_github_path', true);
 
-		if ( $github_path && $this->app->api()->fetch()->exists( $github_path ) ) {
+		if ($github_path && $this->app->api()->fetch()->exists($github_path)) {
 			return $github_path;
 		}
 
@@ -82,29 +82,29 @@ 
 	 *
 	 * @return string|WP_Error
 	 */
-	public function update( $post_id ) {
-		$post = $this->app->database()->fetch_by_id( $post_id );
+	public function update($post_id) {
+		$post = $this->app->database()->fetch_by_id($post_id);
 
-		if ( is_wp_error( $post ) ) {
+		if (is_wp_error($post)) {
 			return $post;
 		}
 
-		if ( 'trash' === $post->status() ) {
-			return $this->delete( $post_id );
+		if ('trash' === $post->status()) {
+			return $this->delete($post_id);
 		}
 
-		if ( $old_github_path = $this->github_path( $post->id() ) ) {
+		if ($old_github_path = $this->github_path($post->id())) {
 			error_log("old_github_path: $old_github_path");
 			$post->set_old_github_path($old_github_path);
 		}
 
-		$result = $this->new_posts( array( $post ) );
+		$result = $this->new_posts(array($post));
 
-		if ( is_wp_error( $result ) ) {
+		if (is_wp_error($result)) {
 			return $result;
 		}
 
-		return __( 'Export to GitHub completed successfully.', 'writing-on-github' );
+		return __('Export to GitHub completed successfully.', 'writing-on-github');
 	}
 
 	/**
@@ -114,16 +114,16 @@ 
 	 *
 	 * @return string|WP_Error
 	 */
-	public function new_posts( array $posts ) {
+	public function new_posts(array $posts) {
 		$error = false;
 
 		$persist = $this->app->api()->persist();
 
-		foreach ( $posts as $post ) {
-			$result = $this->new_post( $post, $persist );
-			if ( is_wp_error( $result ) ) {
-				if ( $error ) {
-					$error->add( $result->get_error_code(), $result->get_error_message() );
+		foreach ($posts as $post) {
+			$result = $this->new_post($post, $persist);
+			if (is_wp_error($result)) {
+				if ($error) {
+					$error->add($result->get_error_code(), $result->get_error_message());
 				} else {
 					$error = $result;
 				}
@@ -134,7 +134,7 @@ 
 
 		// $result = $this->app->api()->persist()->commit( $master );
 
-		if ( is_wp_error( $error ) ) {
+		if (is_wp_error($error)) {
 			return $error;
 		}
 
@@ -142,13 +142,13 @@ 
 		return true;
 	}
 
-	protected function new_post( $post, $persist ) {
+	protected function new_post($post, $persist) {
 		$github_path = $post->github_path();
 		$old_github_path = $post->old_github_path();
 		$blob = $post->to_blob();
 		$result = false;
 
-		if ( $old_github_path && $old_github_path != $github_path ) {
+		if ($old_github_path && $old_github_path != $github_path) {
 			// rename
 			$message = apply_filters(
 				'wogh_commit_msg_move_post',
@@ -156,20 +156,20 @@ 
 					'Move %s to %s via WordPress at %s (%s)',
 					$old_github_path, $github_path,
 					site_url(),
-					get_bloginfo( 'name' )
+					get_bloginfo('name')
 				)
 			) . $this->get_commit_msg_tag();
 
-			$result = $persist->delete_file( $post->old_github_path(), $blob->sha(), $message );
-			if ( is_wp_error( $result ) ) {
+			$result = $persist->delete_file($post->old_github_path(), $blob->sha(), $message);
+			if (is_wp_error($result)) {
 				return $result;
 			}
 
-			$result = $persist->create_file( $blob, $message );
-			if ( is_wp_error( $result ) ) {
+			$result = $persist->create_file($blob, $message);
+			if (is_wp_error($result)) {
 				return $result;
 			}
-		} elseif ( ! $old_github_path ) {
+		} elseif ( ! $old_github_path) {
 			// create new
 			$message = apply_filters(
 				'wogh_commit_msg_new_post',
@@ -177,14 +177,14 @@ 
 					'Create new post %s from WordPress at %s (%s)',
 					$github_path,
 					site_url(),
-					get_bloginfo( 'name' )
+					get_bloginfo('name')
 				)
 			) . $this->get_commit_msg_tag();
-			$result = $persist->create_file( $blob, $message );
-			if ( is_wp_error( $result ) ) {
+			$result = $persist->create_file($blob, $message);
+			if (is_wp_error($result)) {
 				return $result;
 			}
-		} elseif ( $old_github_path && $old_github_path == $github_path ) {
+		} elseif ($old_github_path && $old_github_path == $github_path) {
 			// update
 			$message = apply_filters(
 				'wogh_commit_msg_update_post',
@@ -192,11 +192,11 @@ 
 					'Update post %s from WordPress at %s (%s)',
 					$github_path,
 					site_url(),
-					get_bloginfo( 'name' )
+					get_bloginfo('name')
 				)
 			) . $this->get_commit_msg_tag();
-			$result = $persist->update_file( $blob, $message );
-			if ( is_wp_error( $result ) ) {
+			$result = $persist->update_file($blob, $message);
+			if (is_wp_error($result)) {
 				return $result;
 			}
 		}
@@ -215,14 +215,14 @@ 
 	 *
 	 * @return string|WP_Error
 	 */
-	public function delete( $post_id ) {
-		$post = $this->app->database()->fetch_by_id( $post_id );
+	public function delete($post_id) {
+		$post = $this->app->database()->fetch_by_id($post_id);
 
-		if ( is_wp_error( $post ) ) {
+		if (is_wp_error($post)) {
 			return $post;
 		}
 
-		$github_path = get_post_meta( $post_id, '_wogh_github_path', true );
+		$github_path = get_post_meta($post_id, '_wogh_github_path', true);
 
 		$message = apply_filters(
 			'wogh_commit_msg_delete',
@@ -230,18 +230,18 @@ 
 				'Deleting %s via WordPress at %s (%s)',
 				$github_path,
 				site_url(),
-				get_bloginfo( 'name' )
+				get_bloginfo('name')
 			),
 			$post
 		) . $this->get_commit_msg_tag();
 
-		$result = $this->app->api()->persist()->delete_file( $github_path, $post->sha(), $message );
+		$result = $this->app->api()->persist()->delete_file($github_path, $post->sha(), $message);
 
-		if ( is_wp_error( $result ) ) {
+		if (is_wp_error($result)) {
 			return $result;
 		}
 
-		return __( 'Export to GitHub completed successfully.', 'writing-on-github' );
+		return __('Export to GitHub completed successfully.', 'writing-on-github');
 	}
 
 
@@ -252,8 +252,8 @@ 
 	 *
 	 * @return bool
 	 */
-	public function set_user( $user_id ) {
-		return update_option( self::EXPORT_USER_OPTION, (int) $user_id );
+	public function set_user($user_id) {
+		return update_option(self::EXPORT_USER_OPTION, (int) $user_id);
 	}
 
 	/**
@@ -262,10 +262,10 @@ 
 	 * @return string
 	 */
 	protected function get_commit_msg_tag() {
-		$tag = apply_filters( 'wogh_commit_msg_tag', 'wogh' );
+		$tag = apply_filters('wogh_commit_msg_tag', 'wogh');
 
-		if ( ! $tag ) {
-			throw new Exception( __( 'Commit message tag not set. Filter `wogh_commit_msg_tag` misconfigured.', 'writing-on-github' ) );
+		if ( ! $tag) {
+			throw new Exception(__('Commit message tag not set. Filter `wogh_commit_msg_tag` misconfigured.', 'writing-on-github'));
 		}
 
 		return ' - ' . $tag;

lib/client/persist.php

Spacing +11 added lines, -11 removed lines

@@ -16,9 +16,9 @@ 
 	 */
 	protected function export_user() {
 		$user_id = get_current_user_id();
-		$user = get_userdata( $user_id );
+		$user = get_userdata($user_id);
 
-		if ( $user ) {
+		if ($user) {
 			return array(
 				'name'  => $user->display_name,
 				'email' => $user->user_email,
@@ -33,17 +33,17 @@ 
 	 *
 	 * @return array
 	 */
-	public function delete_file( $path, $sha, $message ) {
+	public function delete_file($path, $sha, $message) {
 		$body = new stdClass();
 		$body->message = $message;
 		$body->sha = $sha;
 		$body->branch = $this->branch();
 
-		if ( $author = $this->export_user() ) {
+		if ($author = $this->export_user()) {
 			$body->author = $author;
 		}
 
-		return $this->call( 'DELETE', $this->content_endpoint( $path ), $body );
+		return $this->call('DELETE', $this->content_endpoint($path), $body);
 	}
 
 	/**
@@ -51,17 +51,17 @@ 
 	 *
 	 * @return array
 	 */
-	public function create_file( $blob, $message ) {
+	public function create_file($blob, $message) {
 		$body = $blob->to_body();
 		$body->message = $message;
 		$body->branch = $this->branch();
 		unset($body->sha);
 
-		if ( $author = $this->export_user() ) {
+		if ($author = $this->export_user()) {
 			$body->author = $author;
 		}
 
-		return $this->call( 'PUT', $this->content_endpoint( $blob->path() ), $body );
+		return $this->call('PUT', $this->content_endpoint($blob->path()), $body);
 	}
 
 	/**
@@ -69,15 +69,15 @@ 
 	 *
 	 * @return array
 	 */
-	public function update_file( $blob, $message ) {
+	public function update_file($blob, $message) {
 		$body = $blob->to_body();
 		$body->message = $message;
 		$body->branch = $this->branch();
 
-		if ( $author = $this->export_user() ) {
+		if ($author = $this->export_user()) {
 			$body->author = $author;
 		}
 
-		return $this->call( 'PUT', $this->content_endpoint( $blob->path() ), $body );
+		return $this->call('PUT', $this->content_endpoint($blob->path()), $body);
 	}
 }

lib/client/base.php

Spacing +25 added lines, -26 removed lines

@@ -26,7 +26,7 @@ 
 	 *
 	 * @param Writing_On_GitHub $app Application container.
 	 */
-	public function __construct( Writing_On_GitHub $app ) {
+	public function __construct(Writing_On_GitHub $app) {
 		$this->app = $app;
 	}
 
@@ -39,8 +39,8 @@ 
 	 *
 	 * @return stdClass|WP_Error
 	 */
-	protected function call( $method, $endpoint, $body = array() ) {
-		if ( is_wp_error( $error = $this->can_call() ) ) {
+	protected function call($method, $endpoint, $body = array()) {
+		if (is_wp_error($error = $this->can_call())) {
 			return $error;
 		}
 
@@ -51,24 +51,23 @@ 
 			),
 		);
 
-		if ( 'GET' !== $method ) {
-			$args['body'] = function_exists( 'wp_json_encode' ) ?
-				wp_json_encode( $body ) :
-				json_encode( $body );
+		if ('GET' !== $method) {
+			$args['body'] = function_exists('wp_json_encode') ?
+				wp_json_encode($body) : json_encode($body);
 		}
 
 		$tmpbody = isset($args['body']) ? $args['body'] : '';
 		error_log("writing-on-github-call $method $endpoint $tmpbody");
 
-		$response = wp_remote_request( $endpoint, $args );
-		$status   = wp_remote_retrieve_header( $response, 'status' );
-		$body     = json_decode( wp_remote_retrieve_body( $response ) );
+		$response = wp_remote_request($endpoint, $args);
+		$status   = wp_remote_retrieve_header($response, 'status');
+		$body     = json_decode(wp_remote_retrieve_body($response));
 
-		if ( '2' !== substr( $status, 0, 1 ) && '3' !== substr( $status, 0, 1 ) ) {
+		if ('2' !== substr($status, 0, 1) && '3' !== substr($status, 0, 1)) {
 			return new WP_Error(
-				strtolower( str_replace( ' ', '_', $status ) ),
+				strtolower(str_replace(' ', '_', $status)),
 				sprintf(
-					__( 'Method %s to endpoint %s failed with error: %s', 'writing-on-github' ),
+					__('Method %s to endpoint %s failed with error: %s', 'writing-on-github'),
 					$method,
 					$endpoint,
 					$body && $body->message ? $body->message : 'Unknown error'
@@ -85,28 +84,28 @@ 
 	 * @return true|WP_Error
 	 */
 	protected function can_call() {
-		if ( ! $this->oauth_token() ) {
+		if ( ! $this->oauth_token()) {
 			return new WP_Error(
 				'missing_token',
-				__( 'Writing On GitHub needs an auth token. Please update your settings.', 'writing-on-github' )
+				__('Writing On GitHub needs an auth token. Please update your settings.', 'writing-on-github')
 			);
 		}
 
 		$repo = $this->repository();
 
-		if ( ! $repo ) {
+		if ( ! $repo) {
 			return new WP_Error(
 				'missing_repository',
-				__( 'Writing On GitHub needs a repository. Please update your settings.', 'writing-on-github' )
+				__('Writing On GitHub needs a repository. Please update your settings.', 'writing-on-github')
 			);
 		}
 
-		$parts = explode( '/', $repo );
+		$parts = explode('/', $repo);
 
-		if ( 2 !== count( $parts ) ) {
+		if (2 !== count($parts)) {
 			return new WP_Error(
 				'malformed_repository',
-				__( 'Writing On GitHub needs a properly formed repository. Please update your settings.', 'writing-on-github' )
+				__('Writing On GitHub needs a properly formed repository. Please update your settings.', 'writing-on-github')
 			);
 		}
 
@@ -119,7 +118,7 @@ 
 	 * @return string
 	 */
 	public function repository() {
-		return (string) get_option( self::REPO_OPTION_KEY );
+		return (string) get_option(self::REPO_OPTION_KEY);
 	}
 
 	/**
@@ -128,18 +127,18 @@ 
 	 * @return string
 	 */
 	public function oauth_token() {
-		return (string) get_option( self::TOKEN_OPTION_KEY );
+		return (string) get_option(self::TOKEN_OPTION_KEY);
 	}
 
 	/**
 	 * Returns the GitHub host to sync with (for GitHub Enterprise support)
 	 */
 	public function api_base() {
-		return get_option( self::HOST_OPTION_KEY );
+		return get_option(self::HOST_OPTION_KEY);
 	}
 
 	public function branch() {
-		$branch = get_option( self::BRANCH_OPTION_KEY );
+		$branch = get_option(self::BRANCH_OPTION_KEY);
 		return $branch ? $branch : 'master';
 	}
 
@@ -200,11 +199,11 @@ 
 	 *
 	 * Returns String the relative API call path
 	 */
-	public function content_endpoint( $path = false ) {
+	public function content_endpoint($path = false) {
 		$url = $this->api_base() . '/repos/';
 		$url = $url . $this->repository() . '/contents';
 
-		if ( ! empty($path) ) {
+		if ( ! empty($path)) {
 			$url .= '/' . $path;
 		}
 

lib/response.php

Spacing +16 added lines, -16 removed lines

@@ -21,7 +21,7 @@ 
 	 *
 	 * @param Writing_On_GitHub $app Application container.
 	 */
-	public function __construct( Writing_On_GitHub $app ) {
+	public function __construct(Writing_On_GitHub $app) {
 		$this->app = $app;
 	}
 
@@ -32,30 +32,30 @@ 
 	 *
 	 * @return false
 	 */
-	public function error( WP_Error $error ) {
+	public function error(WP_Error $error) {
 		global $wp_version;
 
-		$this->log( $error );
+		$this->log($error);
 
-		if ( defined( 'DOING_AJAX' ) && DOING_AJAX && defined( 'WOGH_AJAX' ) && WOGH_AJAX ) {
+		if (defined('DOING_AJAX') && DOING_AJAX && defined('WOGH_AJAX') && WOGH_AJAX) {
 			/**
 			 * WordPress 4.1.0 introduced allowing WP_Error objects to be
 			 * passed directly into `wp_send_json_error`. This shims in
 			 * compatibility for older versions. We're currently supporting 3.9+.
 			 */
-			if ( version_compare( $wp_version, '4.1.0', '<' ) ) {
+			if (version_compare($wp_version, '4.1.0', '<')) {
 				$result = array();
 
-				foreach ( $error->errors as $code => $messages ) {
-					foreach ( $messages as $message ) {
-						$result[] = array( 'code' => $code, 'message' => $message );
+				foreach ($error->errors as $code => $messages) {
+					foreach ($messages as $message) {
+						$result[] = array('code' => $code, 'message' => $message);
 					}
 				}
 
 				$error = $result;
 			}
 
-			wp_send_json_error( $error );
+			wp_send_json_error($error);
 		}
 
 		return false;
@@ -68,11 +68,11 @@ 
 	 *
 	 * @return true
 	 */
-	public function success( $success ) {
-		$this->log( $success );
+	public function success($success) {
+		$this->log($success);
 
-		if ( defined( 'DOING_AJAX' ) && DOING_AJAX && defined( 'WOGH_AJAX' ) && WOGH_AJAX ) {
-			wp_send_json_success( $success );
+		if (defined('DOING_AJAX') && DOING_AJAX && defined('WOGH_AJAX') && WOGH_AJAX) {
+			wp_send_json_success($success);
 		}
 
 		return true;
@@ -85,11 +85,11 @@ 
 	 *
 	 * @param string|WP_Error $msg Message to log.
 	 */
-	protected function log( $msg ) {
-		if ( is_wp_error( $msg ) ) {
+	protected function log($msg) {
+		if (is_wp_error($msg)) {
 			$msg = $msg->get_error_message();
 		}
 
-		Writing_On_GitHub::write_log( $msg );
+		Writing_On_GitHub::write_log($msg);
 	}
 }

lib/admin.php

Indentation +18 added lines, -18 removed lines

@@ -84,13 +84,13 @@ 
 			)
 		);
 
-        register_setting( Writing_On_GitHub::$text_domain, 'wogh_ignore_author' );
-        add_settings_field( 'wogh_ignore_author', __( 'Ignore author', 'writing-on-github' ), array( &$this, 'checkbox_field_callback' ), Writing_On_GitHub::$text_domain, 'general', array(
-                'default'   => '',
-                'name'      => 'wogh_ignore_author',
-                'help_text' => __( 'Do not export author and do not use author info from GitHub.', 'writing-on-github' ),
-            )
-        );
+		register_setting( Writing_On_GitHub::$text_domain, 'wogh_ignore_author' );
+		add_settings_field( 'wogh_ignore_author', __( 'Ignore author', 'writing-on-github' ), array( &$this, 'checkbox_field_callback' ), Writing_On_GitHub::$text_domain, 'general', array(
+				'default'   => '',
+				'name'      => 'wogh_ignore_author',
+				'help_text' => __( 'Do not export author and do not use author info from GitHub.', 'writing-on-github' ),
+			)
+		);
 
 		// register_setting( Writing_On_GitHub::$text_domain, 'wogh_ignore_metas' );
 		// add_settings_field( 'wogh_ignore_metas', __( 'Ignore post metas', 'writing-on-github' ), array( &$this, 'textarea_field_callback' ), Writing_On_GitHub::$text_domain, 'general', array(
@@ -128,14 +128,14 @@ 
 		include dirname( dirname( __FILE__ ) ) . '/views/textarea-setting-field.php';
 	}
 
-    /**
-     * Callback to render the checkbox field.
-     *
-     * @param array $args Field arguments.
-     */
-    public function checkbox_field_callback( $args ) {
-        include dirname( dirname( __FILE__ ) ) . '/views/checkbox-setting-field.php';
-    }
+	/**
+	 * Callback to render the checkbox field.
+	 *
+	 * @param array $args Field arguments.
+	 */
+	public function checkbox_field_callback( $args ) {
+		include dirname( dirname( __FILE__ ) ) . '/views/checkbox-setting-field.php';
+	}
 
 	/**
 	 * Displays settings messages from background processes
@@ -221,9 +221,9 @@ 
 			Writing_On_GitHub::$instance->start_export();
 		}
 
-        if ( 'force_export' === $_GET['action'] ) {
-            Writing_On_GitHub::$instance->start_export(true);
-        }
+		if ( 'force_export' === $_GET['action'] ) {
+			Writing_On_GitHub::$instance->start_export(true);
+		}
 
 		if ( 'import' === $_GET['action'] ) {
 			Writing_On_GitHub::$instance->start_import();

Spacing +63 added lines, -63 removed lines

@@ -13,16 +13,16 @@ 
 	 * Hook into GitHub API
 	 */
 	public function __construct() {
-		add_action( 'admin_menu', array( $this, 'add_admin_menu' ) );
-		add_action( 'admin_init', array( $this, 'register_settings' ) );
-		add_action( 'current_screen', array( $this, 'trigger_cron' ) );
+		add_action('admin_menu', array($this, 'add_admin_menu'));
+		add_action('admin_init', array($this, 'register_settings'));
+		add_action('current_screen', array($this, 'trigger_cron'));
 	}
 
 	/**
 	 * Callback to render the settings page view
 	 */
 	public function settings_page() {
-		include dirname( dirname( __FILE__ ) ) . '/views/options.php';
+		include dirname(dirname(__FILE__)) . '/views/options.php';
 	}
 
 	/**
@@ -32,63 +32,63 @@ 
 		add_settings_section(
 			'general',
 			'General Settings',
-			array( $this, 'section_callback' ),
+			array($this, 'section_callback'),
 			Writing_On_GitHub::$text_domain
 		);
 
-		register_setting( Writing_On_GitHub::$text_domain, 'wogh_host' );
-		add_settings_field( 'wogh_host', __( 'GitHub hostname', 'writing-on-github' ), array( $this, 'field_callback' ), Writing_On_GitHub::$text_domain, 'general', array(
+		register_setting(Writing_On_GitHub::$text_domain, 'wogh_host');
+		add_settings_field('wogh_host', __('GitHub hostname', 'writing-on-github'), array($this, 'field_callback'), Writing_On_GitHub::$text_domain, 'general', array(
 				'default'   => 'https://api.github.com',
 				'name'      => 'wogh_host',
-				'help_text' => __( 'The GitHub host to use. This only needs to be changed to support a GitHub Enterprise installation.', 'writing-on-github' ),
+				'help_text' => __('The GitHub host to use. This only needs to be changed to support a GitHub Enterprise installation.', 'writing-on-github'),
 			)
 		);
 
-		register_setting( Writing_On_GitHub::$text_domain, 'wogh_repository' );
-		add_settings_field( 'wogh_repository', __( 'Repository', 'writing-on-github' ), array( $this, 'field_callback' ), Writing_On_GitHub::$text_domain, 'general', array(
+		register_setting(Writing_On_GitHub::$text_domain, 'wogh_repository');
+		add_settings_field('wogh_repository', __('Repository', 'writing-on-github'), array($this, 'field_callback'), Writing_On_GitHub::$text_domain, 'general', array(
 				'default'   => '',
 				'name'      => 'wogh_repository',
-				'help_text' => __( 'The GitHub repository to commit to, with owner (<code>[OWNER]/[REPOSITORY]</code>), e.g., <code>github/hubot.github.com</code>. The repository should contain an initial commit, which is satisfied by including a README when you create the repository on GitHub.', 'writing-on-github' ),
+				'help_text' => __('The GitHub repository to commit to, with owner (<code>[OWNER]/[REPOSITORY]</code>), e.g., <code>github/hubot.github.com</code>. The repository should contain an initial commit, which is satisfied by including a README when you create the repository on GitHub.', 'writing-on-github'),
 			)
 		);
 
-		register_setting( Writing_On_GitHub::$text_domain, 'wogh_branch' );
-		add_settings_field( 'wogh_branch', __( 'Branch', 'writing-on-github' ), array( $this, 'field_callback' ), Writing_On_GitHub::$text_domain, 'general', array(
+		register_setting(Writing_On_GitHub::$text_domain, 'wogh_branch');
+		add_settings_field('wogh_branch', __('Branch', 'writing-on-github'), array($this, 'field_callback'), Writing_On_GitHub::$text_domain, 'general', array(
 				'default'   => 'master',
 				'name'      => 'wogh_branch',
-				'help_text' => __( 'The GitHub branch to commit to, default is master.', 'writing-on-github' ),
+				'help_text' => __('The GitHub branch to commit to, default is master.', 'writing-on-github'),
 			)
 		);
 
-		register_setting( Writing_On_GitHub::$text_domain, 'wogh_oauth_token' );
-		add_settings_field( 'wogh_oauth_token', __( 'Oauth Token', 'writing-on-github' ), array( $this, 'field_callback' ), Writing_On_GitHub::$text_domain, 'general', array(
+		register_setting(Writing_On_GitHub::$text_domain, 'wogh_oauth_token');
+		add_settings_field('wogh_oauth_token', __('Oauth Token', 'writing-on-github'), array($this, 'field_callback'), Writing_On_GitHub::$text_domain, 'general', array(
 				'default'   => '',
 				'name'      => 'wogh_oauth_token',
-				'help_text' => __( "A <a href='https://github.com/settings/tokens/new'>personal oauth token</a> with <code>public_repo</code> scope.", 'writing-on-github' ),
+				'help_text' => __("A <a href='https://github.com/settings/tokens/new'>personal oauth token</a> with <code>public_repo</code> scope.", 'writing-on-github'),
 			)
 		);
 
-		register_setting( Writing_On_GitHub::$text_domain, 'wogh_secret' );
-		add_settings_field( 'wogh_secret', __( 'Webhook Secret', 'writing-on-github' ), array( $this, 'field_callback' ), Writing_On_GitHub::$text_domain, 'general', array(
+		register_setting(Writing_On_GitHub::$text_domain, 'wogh_secret');
+		add_settings_field('wogh_secret', __('Webhook Secret', 'writing-on-github'), array($this, 'field_callback'), Writing_On_GitHub::$text_domain, 'general', array(
 				'default'   => '',
 				'name'      => 'wogh_secret',
-				'help_text' => __( "The webhook's secret phrase. This should be password strength, as it is used to verify the webhook's payload.", 'writing-on-github' ),
+				'help_text' => __("The webhook's secret phrase. This should be password strength, as it is used to verify the webhook's payload.", 'writing-on-github'),
 			)
 		);
 
-		register_setting( Writing_On_GitHub::$text_domain, 'wogh_default_user' );
-		add_settings_field( 'wogh_default_user', __( 'Default Import User', 'writing-on-github' ), array( &$this, 'user_field_callback' ), Writing_On_GitHub::$text_domain, 'general', array(
+		register_setting(Writing_On_GitHub::$text_domain, 'wogh_default_user');
+		add_settings_field('wogh_default_user', __('Default Import User', 'writing-on-github'), array(&$this, 'user_field_callback'), Writing_On_GitHub::$text_domain, 'general', array(
 				'default'   => '',
 				'name'      => 'wogh_default_user',
-				'help_text' => __( 'The fallback user for import, in case Writing On GitHub cannot find the committer in the database.', 'writing-on-github' ),
+				'help_text' => __('The fallback user for import, in case Writing On GitHub cannot find the committer in the database.', 'writing-on-github'),
 			)
 		);
 
-        register_setting( Writing_On_GitHub::$text_domain, 'wogh_ignore_author' );
-        add_settings_field( 'wogh_ignore_author', __( 'Ignore author', 'writing-on-github' ), array( &$this, 'checkbox_field_callback' ), Writing_On_GitHub::$text_domain, 'general', array(
+        register_setting(Writing_On_GitHub::$text_domain, 'wogh_ignore_author');
+        add_settings_field('wogh_ignore_author', __('Ignore author', 'writing-on-github'), array(&$this, 'checkbox_field_callback'), Writing_On_GitHub::$text_domain, 'general', array(
                 'default'   => '',
                 'name'      => 'wogh_ignore_author',
-                'help_text' => __( 'Do not export author and do not use author info from GitHub.', 'writing-on-github' ),
+                'help_text' => __('Do not export author and do not use author info from GitHub.', 'writing-on-github'),
             )
         );
 
@@ -106,8 +106,8 @@ 
 	 *
 	 * @param array $args Field arguments.
 	 */
-	public function field_callback( $args ) {
-		include dirname( dirname( __FILE__ ) ) . '/views/setting-field.php';
+	public function field_callback($args) {
+		include dirname(dirname(__FILE__)) . '/views/setting-field.php';
 	}
 
 	/**
@@ -115,8 +115,8 @@ 
 	 *
 	 * @param array $args Field arguments.
 	 */
-	public function user_field_callback( $args ) {
-		include dirname( dirname( __FILE__ ) ) . '/views/user-setting-field.php';
+	public function user_field_callback($args) {
+		include dirname(dirname(__FILE__)) . '/views/user-setting-field.php';
 	}
 
 	/**
@@ -124,8 +124,8 @@ 
 	 *
 	 * @param array $args Field arguments.
 	 */
-	public function textarea_field_callback( $args ) {
-		include dirname( dirname( __FILE__ ) ) . '/views/textarea-setting-field.php';
+	public function textarea_field_callback($args) {
+		include dirname(dirname(__FILE__)) . '/views/textarea-setting-field.php';
 	}
 
     /**
@@ -133,58 +133,58 @@ 
      *
      * @param array $args Field arguments.
      */
-    public function checkbox_field_callback( $args ) {
-        include dirname( dirname( __FILE__ ) ) . '/views/checkbox-setting-field.php';
+    public function checkbox_field_callback($args) {
+        include dirname(dirname(__FILE__)) . '/views/checkbox-setting-field.php';
     }
 
 	/**
 	 * Displays settings messages from background processes
 	 */
 	public function section_callback() {
-		if ( get_current_screen()->id !== 'settings_page_' . Writing_On_GitHub::$text_domain ) {
+		if (get_current_screen()->id !== 'settings_page_' . Writing_On_GitHub::$text_domain) {
 			return;
 		}
 
-		if ( 'yes' === get_option( '_wogh_export_started' ) ) { ?>
+		if ('yes' === get_option('_wogh_export_started')) { ?>
 			<div class="updated">
-				<p><?php esc_html_e( 'Export to GitHub started.', 'writing-on-github' ); ?></p>
+				<p><?php esc_html_e('Export to GitHub started.', 'writing-on-github'); ?></p>
 			</div><?php
-			delete_option( '_wogh_export_started' );
+			delete_option('_wogh_export_started');
 		}
 
-		if ( $message = get_option( '_wogh_export_error' ) ) { ?>
+		if ($message = get_option('_wogh_export_error')) { ?>
 			<div class="error">
-				<p><?php esc_html_e( 'Export to GitHub failed with error:', 'writing-on-github' ); ?> <?php echo esc_html( $message );?></p>
+				<p><?php esc_html_e('Export to GitHub failed with error:', 'writing-on-github'); ?> <?php echo esc_html($message); ?></p>
 			</div><?php
-			delete_option( '_wogh_export_error' );
+			delete_option('_wogh_export_error');
 		}
 
-		if ( 'yes' === get_option( '_wogh_export_complete' ) ) { ?>
+		if ('yes' === get_option('_wogh_export_complete')) { ?>
 			<div class="updated">
-				<p><?php esc_html_e( 'Export to GitHub completed successfully.', 'writing-on-github' );?></p>
+				<p><?php esc_html_e('Export to GitHub completed successfully.', 'writing-on-github'); ?></p>
 			</div><?php
-			delete_option( '_wogh_export_complete' );
+			delete_option('_wogh_export_complete');
 		}
 
-		if ( 'yes' === get_option( '_wogh_import_started' ) ) { ?>
+		if ('yes' === get_option('_wogh_import_started')) { ?>
 			<div class="updated">
-			<p><?php esc_html_e( 'Import from GitHub started.', 'writing-on-github' ); ?></p>
+			<p><?php esc_html_e('Import from GitHub started.', 'writing-on-github'); ?></p>
 			</div><?php
-			delete_option( '_wogh_import_started' );
+			delete_option('_wogh_import_started');
 		}
 
-		if ( $message = get_option( '_wogh_import_error' ) ) { ?>
+		if ($message = get_option('_wogh_import_error')) { ?>
 			<div class="error">
-			<p><?php esc_html_e( 'Import from GitHub failed with error:', 'writing-on-github' ); ?> <?php echo esc_html( $message );?></p>
+			<p><?php esc_html_e('Import from GitHub failed with error:', 'writing-on-github'); ?> <?php echo esc_html($message); ?></p>
 			</div><?php
-			delete_option( '_wogh_import_error' );
+			delete_option('_wogh_import_error');
 		}
 
-		if ( 'yes' === get_option( '_wogh_import_complete' ) ) { ?>
+		if ('yes' === get_option('_wogh_import_complete')) { ?>
 			<div class="updated">
-			<p><?php esc_html_e( 'Import from GitHub completed successfully.', 'writing-on-github' );?></p>
+			<p><?php esc_html_e('Import from GitHub completed successfully.', 'writing-on-github'); ?></p>
 			</div><?php
-			delete_option( '_wogh_import_complete' );
+			delete_option('_wogh_import_complete');
 		}
 	}
 
@@ -193,11 +193,11 @@ 
 	 */
 	public function add_admin_menu() {
 		add_options_page(
-			__( 'Writing On GitHub', 'writing-on-github' ),
-			__( 'Writing On GitHub', 'writing-on-github' ),
+			__('Writing On GitHub', 'writing-on-github'),
+			__('Writing On GitHub', 'writing-on-github'),
 			'manage_options',
 			Writing_On_GitHub::$text_domain,
-			array( $this, 'settings_page' )
+			array($this, 'settings_page')
 		);
 	}
 
@@ -205,31 +205,31 @@ 
 	 * Admin callback to trigger import/export because WordPress admin routing lol
 	 */
 	public function trigger_cron() {
-		if ( ! current_user_can( 'manage_options' ) ) {
+		if ( ! current_user_can('manage_options')) {
 			return;
 		}
 
-		if ( get_current_screen()->id !== 'settings_page_' . Writing_On_GitHub::$text_domain ) {
+		if (get_current_screen()->id !== 'settings_page_' . Writing_On_GitHub::$text_domain) {
 			return;
 		}
 
-		if ( ! isset( $_GET['action'] ) ) {
+		if ( ! isset($_GET['action'])) {
 			return;
 		}
 
-		if ( 'export' === $_GET['action'] ) {
+		if ('export' === $_GET['action']) {
 			Writing_On_GitHub::$instance->start_export();
 		}
 
-        if ( 'force_export' === $_GET['action'] ) {
+        if ('force_export' === $_GET['action']) {
             Writing_On_GitHub::$instance->start_export(true);
         }
 
-		if ( 'import' === $_GET['action'] ) {
+		if ('import' === $_GET['action']) {
 			Writing_On_GitHub::$instance->start_import();
 		}
 
-		wp_redirect( admin_url( 'options-general.php?page=writing-on-github' ) );
+		wp_redirect(admin_url('options-general.php?page=writing-on-github'));
 		die;
 	}
 }

lib/blob.php

Indentation +3 added lines, -3 removed lines

@@ -152,9 +152,9 @@
 			array_pop( $matches );
 
 			$meta = spyc_load( $matches[2] );
-            if ( 'yes' == get_option('wogh_ignore_author') ) {
-                unset($meta['author']);
-            }
+			if ( 'yes' == get_option('wogh_ignore_author') ) {
+				unset($meta['author']);
+			}
 			// if ( isset( $meta['link'] ) ) {
 			// 	$meta['link'] = str_replace( home_url(), '', $meta['link'] );
 			// }

Spacing +23 added lines, -23 removed lines

@@ -49,8 +49,8 @@ 
 	 *
 	 * @param stdClass $data Raw blob data.
 	 */
-	public function __construct( stdClass $data ) {
-		$this->interpret_data( $data );
+	public function __construct(stdClass $data) {
+		$this->interpret_data($data);
 	}
 
 	public function id() {
@@ -78,12 +78,12 @@ 
 	 *
 	 * @return $this
 	 */
-	public function set_content( $content, $base64 = false ) {
-		if ( $base64 ) {
-			$content = base64_decode( $content );
+	public function set_content($content, $base64 = false) {
+		if ($base64) {
+			$content = base64_decode($content);
 		}
 
-		$this->frontmatter = '---' === substr( $this->content = $content, 0, 3 ) ? true : false;
+		$this->frontmatter = '---' === substr($this->content = $content, 0, 3) ? true : false;
 
 		return $this;
 	}
@@ -122,20 +122,20 @@ 
 	public function content_import() {
 		$content = $this->content();
 
-		if ( $this->has_frontmatter() ) {
+		if ($this->has_frontmatter()) {
 			// Break out content.
-			preg_match( '/(^---(.*?)---$)?(.*)/ms', $content, $matches );
-			$content = array_pop( $matches );
+			preg_match('/(^---(.*?)---$)?(.*)/ms', $content, $matches);
+			$content = array_pop($matches);
 		}
 
-		if ( function_exists( 'wpmarkdown_markdown_to_html' ) ) {
-			$content = wpmarkdown_markdown_to_html( $content );
+		if (function_exists('wpmarkdown_markdown_to_html')) {
+			$content = wpmarkdown_markdown_to_html($content);
 		}
 
 		/**
 		 * Filters the content for import.
 		 */
-		return apply_filters( 'wogh_content_import', trim( $content ) );
+		return apply_filters('wogh_content_import', trim($content));
 	}
 
 	/**
@@ -146,13 +146,13 @@ 
 	public function meta() {
 		$meta = array();
 
-		if ( $this->has_frontmatter() ) {
+		if ($this->has_frontmatter()) {
 			// Break out meta, if present.
-			preg_match( '/(^---(.*?)---$)?(.*)/ms', $this->content(), $matches );
-			array_pop( $matches );
+			preg_match('/(^---(.*?)---$)?(.*)/ms', $this->content(), $matches);
+			array_pop($matches);
 
-			$meta = spyc_load( $matches[2] );
-            if ( 'yes' == get_option('wogh_ignore_author') ) {
+			$meta = spyc_load($matches[2]);
+            if ('yes' == get_option('wogh_ignore_author')) {
                 unset($meta['author']);
             }
 			// if ( isset( $meta['link'] ) ) {
@@ -198,7 +198,7 @@ 
 		// $data->type = 'blob';
 
 		$data->path = $this->path();
-		$data->content = base64_encode( $this->content() );
+		$data->content = base64_encode($this->content());
 		$data->sha = $this->sha;
 
 		return $data;
@@ -207,13 +207,13 @@ 
 	/**
 	 * Interprets the blob's data into properties.
 	 */
-	protected function interpret_data( $data ) {
-		$this->sha  = isset( $data->sha  ) ? $data->sha  : '';
-		$this->path = isset( $data->path ) ? $data->path : '';
+	protected function interpret_data($data) {
+		$this->sha  = isset($data->sha) ? $data->sha : '';
+		$this->path = isset($data->path) ? $data->path : '';
 
 		$this->set_content(
-			isset( $data->content ) ? trim( $data->content ) : '',
-			isset( $data->encoding ) && 'base64' === $data->encoding ? true : false
+			isset($data->content) ? trim($data->content) : '',
+			isset($data->encoding) && 'base64' === $data->encoding ? true : false
 		);
 	}
 }

lib/payload.php

Spacing +11 added lines, -11 removed lines

@@ -29,9 +29,9 @@ 
 	 * @param Writing_On_GitHub $app      Application container.
 	 * @param string                $raw_data Raw request data.
 	 */
-	public function __construct( Writing_On_GitHub $app, $raw_data ) {
+	public function __construct(Writing_On_GitHub $app, $raw_data) {
 		$this->app  = $app;
-		$this->data = json_decode( $raw_data );
+		$this->data = json_decode($raw_data);
 	}
 
 	/**
@@ -41,32 +41,32 @@ 
 	 */
 	public function should_import() {
 		// @todo how do we get this without importing the whole api object just for this?
-		if ( strtolower( $this->data->repository->full_name ) !== strtolower( $this->app->api()->fetch()->repository() ) ) {
+		if (strtolower($this->data->repository->full_name) !== strtolower($this->app->api()->fetch()->repository())) {
 			return false;
 		}
 
 		// The last term in the ref is the payload_branch name.
-		$refs   = explode( '/', $this->data->ref );
-		$payload_branch = array_pop( $refs );
+		$refs   = explode('/', $this->data->ref);
+		$payload_branch = array_pop($refs);
 
 		$branch = $this->app->api()->fetch()->branch();
 
-		if ( $branch !== $payload_branch ) {
+		if ($branch !== $payload_branch) {
 			return false;
 		}
 
 		// We add a tag to commits we push out, so we shouldn't pull them in again.
-		$tag = apply_filters( 'wogh_commit_msg_tag', 'wogh' );
+		$tag = apply_filters('wogh_commit_msg_tag', 'wogh');
 
-		if ( ! $tag ) {
-			throw new Exception( __( 'Commit message tag not set. Filter `wogh_commit_msg_tag` misconfigured.', 'writing-on-github' ) );
+		if ( ! $tag) {
+			throw new Exception(__('Commit message tag not set. Filter `wogh_commit_msg_tag` misconfigured.', 'writing-on-github'));
 		}
 
-		if ( $tag === substr( $this->message(), -1 * strlen( $tag ) ) ) {
+		if ($tag === substr($this->message(), -1 * strlen($tag))) {
 			return false;
 		}
 
-		if ( ! $this->get_commit_id() ) {
+		if ( ! $this->get_commit_id()) {
 			return false;
 		}
 

lib/api.php

Spacing +5 added lines, -5 removed lines

@@ -35,7 +35,7 @@ 
 	 *
 	 * @param Writing_On_GitHub $app Application container.
 	 */
-	public function __construct( Writing_On_GitHub $app ) {
+	public function __construct(Writing_On_GitHub $app) {
 		$this->app = $app;
 	}
 
@@ -45,8 +45,8 @@ 
 	 * @return Writing_On_GitHub_Fetch_Client
 	 */
 	public function fetch() {
-		if ( ! $this->fetch ) {
-			$this->fetch = new Writing_On_GitHub_Fetch_Client( $this->app );
+		if ( ! $this->fetch) {
+			$this->fetch = new Writing_On_GitHub_Fetch_Client($this->app);
 		}
 
 		return $this->fetch;
@@ -58,8 +58,8 @@ 
 	 * @return Writing_On_GitHub_Persist_Client
 	 */
 	public function persist() {
-		if ( ! $this->persist ) {
-			$this->persist = new Writing_On_GitHub_Persist_Client( $this->app );
+		if ( ! $this->persist) {
+			$this->persist = new Writing_On_GitHub_Persist_Client($this->app);
 		}
 
 		return $this->persist;