litefeel / writing-on-github

views/textarea-setting-field.php

Spacing +3 added lines, -3 removed lines

@@ -5,8 +5,8 @@
  */
 
 ?>
-<?php $value = get_option( $args['name'], $args['default'] ); ?>
-<textarea name="<?php echo esc_attr( $args['name'] ); ?>" id="<?php echo esc_attr( $args['name'] ); ?>" rows="10" cols="40">
-<?php echo esc_attr( $value ); ?>
+<?php $value = get_option($args['name'], $args['default']); ?>
+<textarea name="<?php echo esc_attr($args['name']); ?>" id="<?php echo esc_attr($args['name']); ?>" rows="10" cols="40">
+<?php echo esc_attr($value); ?>
 </textarea>
 <p class="description"><?php echo $args['help_text']; ?></p>

views/options.php

Spacing +12 added lines, -12 removed lines

@@ -6,27 +6,27 @@
 
 ?>
 <div class="wrap">
-	<h2><?php esc_html_e( 'Writing On GitHub', 'writing-on-github' ); ?></h2>
+	<h2><?php esc_html_e('Writing On GitHub', 'writing-on-github'); ?></h2>
 
 	<form method="post" action="options.php">
-		<?php settings_fields( Writing_On_GitHub::$text_domain ); ?>
-		<?php do_settings_sections( Writing_On_GitHub::$text_domain ); ?>
+		<?php settings_fields(Writing_On_GitHub::$text_domain); ?>
+		<?php do_settings_sections(Writing_On_GitHub::$text_domain); ?>
 		<table class="form-table">
 			<tr>
-				<th scope="row"><?php esc_html_e( 'Webhook callback', 'writing-on-github' ); ?></th>
-				<td><code><?php echo esc_url( admin_url( 'admin-ajax.php' ) ); ?>?action=wogh_push_request</code></td>
+				<th scope="row"><?php esc_html_e('Webhook callback', 'writing-on-github'); ?></th>
+				<td><code><?php echo esc_url(admin_url('admin-ajax.php')); ?>?action=wogh_push_request</code></td>
 			</tr>
 			<tr>
-				<th scope="row"><?php esc_html_e( 'Bulk actions', 'writing-on-github' ); ?></th>
+				<th scope="row"><?php esc_html_e('Bulk actions', 'writing-on-github'); ?></th>
 				<td>
-					<a href="<?php echo esc_url( add_query_arg( array( 'action' => 'export' ) ) ); ?>">
-						<?php esc_html_e( 'Export to GitHub', 'writing-on-github' ); ?>
+					<a href="<?php echo esc_url(add_query_arg(array('action' => 'export'))); ?>">
+						<?php esc_html_e('Export to GitHub', 'writing-on-github'); ?>
 					</a> |
-                    <a href="<?php echo esc_url( add_query_arg( array( 'action' => 'force_export' ) ) ); ?>">
-                        <?php esc_html_e( 'Force export to GitHub', 'writing-on-github' ); ?>
+                    <a href="<?php echo esc_url(add_query_arg(array('action' => 'force_export'))); ?>">
+                        <?php esc_html_e('Force export to GitHub', 'writing-on-github'); ?>
                     </a> |
-					<a href="<?php echo esc_url( add_query_arg( array( 'action' => 'import' ) ) ); ?>">
-						<?php esc_html_e( 'Import from GitHub', 'writing-on-github' ); ?>
+					<a href="<?php echo esc_url(add_query_arg(array('action' => 'import'))); ?>">
+						<?php esc_html_e('Import from GitHub', 'writing-on-github'); ?>
 					</a>
 				</td>
 		</table>

views/user-setting-field.php

Spacing +5 added lines, -5 removed lines

@@ -1,8 +1,8 @@
-<?php $value = get_option( $args['name'], $args['default'] ); ?>
-<select name="<?php echo esc_attr( $args['name'] ) ?>" id="<?php echo esc_attr( $args['name'] ) ?>">
-	<?php foreach ( get_users() as $user ) : ?>
-		<option value="<?php echo esc_attr( $user->ID ); ?>"<?php echo (int) $value === $user->ID ? ' selected' : '';?>>
-			<?php echo esc_html( $user->display_name ); ?>
+<?php $value = get_option($args['name'], $args['default']); ?>
+<select name="<?php echo esc_attr($args['name']) ?>" id="<?php echo esc_attr($args['name']) ?>">
+	<?php foreach (get_users() as $user) : ?>
+		<option value="<?php echo esc_attr($user->ID); ?>"<?php echo (int) $value === $user->ID ? ' selected' : ''; ?>>
+			<?php echo esc_html($user->display_name); ?>
 		</option>
 	<?php endforeach; ?>
 </select>

views/checkbox-setting-field.php

Spacing +2 added lines, -2 removed lines

@@ -5,6 +5,6 @@
  */
 
 ?>
-<?php $value = get_option( $args['name'], $args['default'] ); ?>
-<input type="checkbox" name="<?php echo esc_attr( $args['name'] ); ?>" id="<?php echo esc_attr( $args['name'] ); ?>" value="yes" <?php echo 'yes' === $value ? 'checked' : '';  ?> />
+<?php $value = get_option($args['name'], $args['default']); ?>
+<input type="checkbox" name="<?php echo esc_attr($args['name']); ?>" id="<?php echo esc_attr($args['name']); ?>" value="yes" <?php echo 'yes' === $value ? 'checked' : ''; ?> />
 <p class="description"><?php echo $args['help_text']; ?></p>

views/setting-field.php

Spacing +2 added lines, -2 removed lines

@@ -5,6 +5,6 @@
  */
 
 ?>
-<?php $value = get_option( $args['name'], $args['default'] ); ?>
-<input name="<?php echo esc_attr( $args['name'] ); ?>" id="<?php echo esc_attr( $args['name'] ); ?>" type="<?php echo 'wogh_secret' === $args['name'] ? 'password' : 'text'; ?>" value="<?php echo esc_attr( $value ); ?>" />
+<?php $value = get_option($args['name'], $args['default']); ?>
+<input name="<?php echo esc_attr($args['name']); ?>" id="<?php echo esc_attr($args['name']); ?>" type="<?php echo 'wogh_secret' === $args['name'] ? 'password' : 'text'; ?>" value="<?php echo esc_attr($value); ?>" />
 <p class="description"><?php echo $args['help_text']; ?></p>

lib/client/base.php

Indentation +200 added lines, -200 removed lines

@@ -9,204 +9,204 @@
  */
 class Writing_On_GitHub_Base_Client {
 
-    const HOST_OPTION_KEY   = 'wogh_host';
-    const TOKEN_OPTION_KEY  = 'wogh_oauth_token';
-    const REPO_OPTION_KEY   = 'wogh_repository';
-    const BRANCH_OPTION_KEY = 'wogh_branch';
-
-    /**
-     * Application container.
-     *
-     * @var Writing_On_GitHub
-     */
-    protected $app;
-
-    /**
-     * Instantiates a new Api object.
-     *
-     * @param Writing_On_GitHub $app Application container.
-     */
-    public function __construct( Writing_On_GitHub $app ) {
-        $this->app = $app;
-    }
-
-    /**
-     * Generic GitHub API interface and response handler
-     *
-     * @param string $method HTTP method.
-     * @param string $endpoint API endpoint.
-     * @param array  $body Request body.
-     *
-     * @return stdClass|WP_Error
-     */
-    protected function call( $method, $endpoint, $body = array() ) {
-        if ( is_wp_error( $error = $this->can_call() ) ) {
-            /*　@var WP_Error $error */
-            return $error;
-        }
-
-        $args = array(
-            'method'  => $method,
-            'headers' => array(
-                'Authorization' => 'token ' . $this->oauth_token(),
-            ),
-        );
-
-        if ( 'GET' !== $method ) {
-            $args['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 ) );
-
-        if ( '2' !== substr( $status, 0, 1 ) && '3' !== substr( $status, 0, 1 ) ) {
-            return new WP_Error(
-                strtolower( str_replace( ' ', '_', $status ) ),
-                sprintf(
-                    __( 'Method %s to endpoint %s failed with error: %s', 'writing-on-github' ),
-                    $method,
-                    $endpoint,
-                    ( $body && $body->message ) ? $body->message : 'Unknown error'
-                )
-            );
-        }
-
-        return $body;
-    }
-
-    /**
-     * Validates whether the Api object can make a call.
-     *
-     * @return true|WP_Error
-     */
-    protected function can_call() {
-        if ( ! $this->oauth_token() ) {
-            return new WP_Error(
-                'missing_token',
-                __( 'Writing On GitHub needs an auth token. Please update your settings.', 'writing-on-github' )
-            );
-        }
-
-        $repo = $this->repository();
-
-        if ( ! $repo ) {
-            return new WP_Error(
-                'missing_repository',
-                __( 'Writing On GitHub needs a repository. Please update your settings.', 'writing-on-github' )
-            );
-        }
-
-        $parts = explode( '/', $repo );
-
-        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' )
-            );
-        }
-
-        return true;
-    }
-
-    /**
-     * Returns the repository to sync with
-     *
-     * @return string
-     */
-    public function repository() {
-        return (string) get_option( self::REPO_OPTION_KEY );
-    }
-
-    /**
-     * Returns the user's oauth token
-     *
-     * @return string
-     */
-    public function oauth_token() {
-        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 );
-    }
-
-    public function branch() {
-        $branch = get_option( self::BRANCH_OPTION_KEY );
-        return $branch ? $branch : 'master';
-    }
-
-    /**
-     * API endpoint for the master branch reference
-     */
-    public function reference_endpoint() {
-        $url = $this->api_base() . '/repos/';
-        $url = $url . $this->repository() . '/git/refs/heads/' . $this->branch();
-
-        return $url;
-    }
-
-    /**
-     * Api to get and create commits
-     */
-    public function commit_endpoint() {
-        $url = $this->api_base() . '/repos/';
-        $url = $url . $this->repository() . '/git/commits';
-
-        return $url;
-    }
-
-    /**
-     * Api to compare commits
-     */
-    public function compare_endpoint() {
-        $url = $this->api_base() . '/repos/';
-        $url = $url . $this->repository() . '/compare';
-
-        return $url;
-    }
-
-    /**
-     * Api to get and create trees
-     */
-    public function tree_endpoint() {
-        $url = $this->api_base() . '/repos/';
-        $url = $url . $this->repository() . '/git/trees';
-
-        return $url;
-    }
-
-    /**
-     * Builds the proper blob API endpoint for a given post
-     *
-     * Returns String the relative API call path
-     */
-    public function blob_endpoint() {
-        $url = $this->api_base() . '/repos/';
-        $url = $url . $this->repository() . '/git/blobs';
-
-        return $url;
-    }
-
-    /**
-     * Builds the proper content API endpoint for a given post
-     *
-     * Returns String the relative API call path
-     */
-    public function content_endpoint( $path = false ) {
-        $url = $this->api_base() . '/repos/';
-        $url = $url . $this->repository() . '/contents';
-
-        if ( ! empty($path) ) {
-            $url .= '/' . $path;
-        }
-
-        return $url;
-    }
+	const HOST_OPTION_KEY   = 'wogh_host';
+	const TOKEN_OPTION_KEY  = 'wogh_oauth_token';
+	const REPO_OPTION_KEY   = 'wogh_repository';
+	const BRANCH_OPTION_KEY = 'wogh_branch';
+
+	/**
+	 * Application container.
+	 *
+	 * @var Writing_On_GitHub
+	 */
+	protected $app;
+
+	/**
+	 * Instantiates a new Api object.
+	 *
+	 * @param Writing_On_GitHub $app Application container.
+	 */
+	public function __construct( Writing_On_GitHub $app ) {
+		$this->app = $app;
+	}
+
+	/**
+	 * Generic GitHub API interface and response handler
+	 *
+	 * @param string $method HTTP method.
+	 * @param string $endpoint API endpoint.
+	 * @param array  $body Request body.
+	 *
+	 * @return stdClass|WP_Error
+	 */
+	protected function call( $method, $endpoint, $body = array() ) {
+		if ( is_wp_error( $error = $this->can_call() ) ) {
+			/*　@var WP_Error $error */
+			return $error;
+		}
+
+		$args = array(
+			'method'  => $method,
+			'headers' => array(
+				'Authorization' => 'token ' . $this->oauth_token(),
+			),
+		);
+
+		if ( 'GET' !== $method ) {
+			$args['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 ) );
+
+		if ( '2' !== substr( $status, 0, 1 ) && '3' !== substr( $status, 0, 1 ) ) {
+			return new WP_Error(
+				strtolower( str_replace( ' ', '_', $status ) ),
+				sprintf(
+					__( 'Method %s to endpoint %s failed with error: %s', 'writing-on-github' ),
+					$method,
+					$endpoint,
+					( $body && $body->message ) ? $body->message : 'Unknown error'
+				)
+			);
+		}
+
+		return $body;
+	}
+
+	/**
+	 * Validates whether the Api object can make a call.
+	 *
+	 * @return true|WP_Error
+	 */
+	protected function can_call() {
+		if ( ! $this->oauth_token() ) {
+			return new WP_Error(
+				'missing_token',
+				__( 'Writing On GitHub needs an auth token. Please update your settings.', 'writing-on-github' )
+			);
+		}
+
+		$repo = $this->repository();
+
+		if ( ! $repo ) {
+			return new WP_Error(
+				'missing_repository',
+				__( 'Writing On GitHub needs a repository. Please update your settings.', 'writing-on-github' )
+			);
+		}
+
+		$parts = explode( '/', $repo );
+
+		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' )
+			);
+		}
+
+		return true;
+	}
+
+	/**
+	 * Returns the repository to sync with
+	 *
+	 * @return string
+	 */
+	public function repository() {
+		return (string) get_option( self::REPO_OPTION_KEY );
+	}
+
+	/**
+	 * Returns the user's oauth token
+	 *
+	 * @return string
+	 */
+	public function oauth_token() {
+		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 );
+	}
+
+	public function branch() {
+		$branch = get_option( self::BRANCH_OPTION_KEY );
+		return $branch ? $branch : 'master';
+	}
+
+	/**
+	 * API endpoint for the master branch reference
+	 */
+	public function reference_endpoint() {
+		$url = $this->api_base() . '/repos/';
+		$url = $url . $this->repository() . '/git/refs/heads/' . $this->branch();
+
+		return $url;
+	}
+
+	/**
+	 * Api to get and create commits
+	 */
+	public function commit_endpoint() {
+		$url = $this->api_base() . '/repos/';
+		$url = $url . $this->repository() . '/git/commits';
+
+		return $url;
+	}
+
+	/**
+	 * Api to compare commits
+	 */
+	public function compare_endpoint() {
+		$url = $this->api_base() . '/repos/';
+		$url = $url . $this->repository() . '/compare';
+
+		return $url;
+	}
+
+	/**
+	 * Api to get and create trees
+	 */
+	public function tree_endpoint() {
+		$url = $this->api_base() . '/repos/';
+		$url = $url . $this->repository() . '/git/trees';
+
+		return $url;
+	}
+
+	/**
+	 * Builds the proper blob API endpoint for a given post
+	 *
+	 * Returns String the relative API call path
+	 */
+	public function blob_endpoint() {
+		$url = $this->api_base() . '/repos/';
+		$url = $url . $this->repository() . '/git/blobs';
+
+		return $url;
+	}
+
+	/**
+	 * Builds the proper content API endpoint for a given post
+	 *
+	 * Returns String the relative API call path
+	 */
+	public function content_endpoint( $path = false ) {
+		$url = $this->api_base() . '/repos/';
+		$url = $url . $this->repository() . '/contents';
+
+		if ( ! empty($path) ) {
+			$url .= '/' . $path;
+		}
+
+		return $url;
+	}
 }

Spacing +25 added lines, -25 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())) {
             /*　@var WP_Error $error */
             return $error;
         }
@@ -52,25 +52,25 @@ 
             ),
         );
 
-        if ( 'GET' !== $method ) {
-            $args['body'] = json_encode( $body );
+        if ('GET' !== $method) {
+            $args['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'
+                    ($body && $body->message) ? $body->message : 'Unknown error'
                 )
             );
         }
@@ -84,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')
             );
         }
 
@@ -118,7 +118,7 @@ 
      * @return string
      */
     public function repository() {
-        return (string) get_option( self::REPO_OPTION_KEY );
+        return (string) get_option(self::REPO_OPTION_KEY);
     }
 
     /**
@@ -127,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';
     }
 
@@ -199,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/client/persist.php

Indentation +60 added lines, -60 removed lines

@@ -9,75 +9,75 @@
  */
 class Writing_On_GitHub_Persist_Client extends Writing_On_GitHub_Base_Client {
 
-    /**
-     * Get the data for the current user.
-     *
-     * @return array
-     */
-    protected function export_user() {
-        $user_id = get_current_user_id();
-        $user = get_userdata( $user_id );
+	/**
+	 * Get the data for the current user.
+	 *
+	 * @return array
+	 */
+	protected function export_user() {
+		$user_id = get_current_user_id();
+		$user = get_userdata( $user_id );
 
-        if ( $user ) {
-            return array(
-                'name'  => $user->display_name,
-                'email' => $user->user_email,
-            );
-        }
+		if ( $user ) {
+			return array(
+				'name'  => $user->display_name,
+				'email' => $user->user_email,
+			);
+		}
 
-        return false;
-    }
+		return false;
+	}
 
-    /**
-     * Delete the file.
-     *
-     * @return array
-     */
-    public function delete_file( $path, $sha, $message ) {
-        $body = new stdClass();
-        $body->message = $message;
-        $body->sha = $sha;
-        $body->branch = $this->branch();
+	/**
+	 * Delete the file.
+	 *
+	 * @return array
+	 */
+	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() ) {
-            $body->author = $author;
-        }
+		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 );
+	}
 
-    /**
-     * Create the file.
-     *
-     * @return array
-     */
-    public function create_file( $blob, $message ) {
-        $body = $blob->to_body();
-        $body->message = $message;
-        $body->branch = $this->branch();
-        unset($body->sha);
+	/**
+	 * Create the file.
+	 *
+	 * @return array
+	 */
+	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() ) {
-            $body->author = $author;
-        }
+		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 );
+	}
 
-    /**
-     * Update the file.
-     *
-     * @return array
-     */
-    public function update_file( $blob, $message ) {
-        $body = $blob->to_body();
-        $body->message = $message;
-        $body->branch = $this->branch();
+	/**
+	 * Update the file.
+	 *
+	 * @return array
+	 */
+	public function update_file( $blob, $message ) {
+		$body = $blob->to_body();
+		$body->message = $message;
+		$body->branch = $this->branch();
 
-        if ( $author = $this->export_user() ) {
-            $body->author = $author;
-        }
+		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 );
+	}
 }

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/response.php

Indentation +70 added lines, -70 removed lines

@@ -9,87 +9,87 @@
  */
 class Writing_On_GitHub_Response {
 
-    /**
-     * Application container.
-     *
-     * @var Writing_On_GitHub
-     */
-    protected $app;
+	/**
+	 * Application container.
+	 *
+	 * @var Writing_On_GitHub
+	 */
+	protected $app;
 
-    /**
-     * Writing_On_GitHub_Response constructor.
-     *
-     * @param Writing_On_GitHub $app Application container.
-     */
-    public function __construct( Writing_On_GitHub $app ) {
-        $this->app = $app;
-    }
+	/**
+	 * Writing_On_GitHub_Response constructor.
+	 *
+	 * @param Writing_On_GitHub $app Application container.
+	 */
+	public function __construct( Writing_On_GitHub $app ) {
+		$this->app = $app;
+	}
 
-    /**
-     * Writes to the log and returns the error response.
-     *
-     * @param WP_Error $error Error to respond with.
-     *
-     * @return false
-     */
-    public function error( WP_Error $error ) {
-        global $wp_version;
+	/**
+	 * Writes to the log and returns the error response.
+	 *
+	 * @param WP_Error $error Error to respond with.
+	 *
+	 * @return false
+	 */
+	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 ) {
-            /**
-             * 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', '<' ) ) {
-                $result = array();
+		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', '<' ) ) {
+				$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;
-            }
+				$error = $result;
+			}
 
-            wp_send_json_error( $error );
-        }
+			wp_send_json_error( $error );
+		}
 
-        return false;
-    }
+		return false;
+	}
 
-    /**
-     * Writes to the log and returns the success response.
-     *
-     * @param string $success Success message to respond with.
-     *
-     * @return true
-     */
-    public function success( $success ) {
-        $this->log( $success );
+	/**
+	 * Writes to the log and returns the success response.
+	 *
+	 * @param string $success Success message to respond with.
+	 *
+	 * @return true
+	 */
+	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;
-    }
+		return true;
+	}
 
-    /**
-     * Writes a log message.
-     *
-     * Can extract a message from WP_Error object.
-     *
-     * @param string|WP_Error $msg Message to log.
-     */
-    protected function log( $msg ) {
-        if ( is_wp_error( $msg ) ) {
-            $msg = $msg->get_error_message();
-        }
+	/**
+	 * Writes a log message.
+	 *
+	 * Can extract a message from WP_Error object.
+	 *
+	 * @param string|WP_Error $msg Message to log.
+	 */
+	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 );
+	}
 }

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/controller.php

Indentation +230 added lines, -230 removed lines

@@ -9,234 +9,234 @@
  */
 class Writing_On_GitHub_Controller {
 
-    /**
-     * Application container.
-     *
-     * @var Writing_On_GitHub
-     */
-    public $app;
-
-    /**
-     * Instantiates a new Controller object
-     *
-     * @param Writing_On_GitHub $app Applicatio container.
-     */
-    public function __construct( Writing_On_GitHub $app ) {
-        $this->app = $app;
-    }
-
-    /**
-     * Webhook callback as triggered from GitHub push.
-     *
-     * Reads the Webhook payload and syncs posts as necessary.
-     *
-     * @return boolean
-     */
-    public function pull_posts() {
-        $this->set_ajax();
-        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()' )
-            ) );
-        }
-
-        if ( ! $this->app->request()->is_secret_valid() ) {
-            return $this->app->response()->error( new WP_Error(
-                'invalid_headers',
-                __( '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' ) );
-        }
-
-        // push
-        if ( ! $this->app->request()->is_push() ) {
-            return $this->app->response()->error( new WP_Error(
-                'invalid_headers',
-                __( '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(
-                'invalid_payload',
-                sprintf(
-                    __( "%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' ) );
-
-        $result = $this->app->import()->payload( $payload );
-
-        $this->app->semaphore()->unlock();
-
-        if ( is_wp_error( $result ) ) {
-            /*　@var WP_Error $result */
-            return $this->app->response()->error( $result );
-        }
-
-        return $this->app->response()->success( $result );
-    }
-
-    /**
-     * Imports posts from the current master branch.
-     *
-     * @return boolean
-     */
-    public function import_master( $user_id = 0 ) {
-        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()' )
-            ) );
-        }
-
-        $this->app->semaphore()->lock();
-        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 );
-        }
-
-        $result = $this->app->import()->master();
-
-        $this->app->semaphore()->unlock();
-
-        if ( is_wp_error( $result ) ) {
-            /*　@var WP_Error $result */
-            update_option( '_wogh_import_error', $result->get_error_message() );
-
-            return $this->app->response()->error( $result );
-        }
-
-        update_option( '_wogh_import_complete', 'yes' );
-
-        return $this->app->response()->success( $result );
-    }
-
-    /**
-     * Export all the posts in the database to GitHub.
-     *
-     * @param  int        $user_id
-     * @param  boolean    $force
-     * @return boolean
-     */
-    public function export_all( $user_id = 0, $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()' )
-            ) );
-        }
-
-        $this->app->semaphore()->lock();
-
-        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 ) ) {
-            /*　@var WP_Error $result */
-            update_option( '_wogh_export_error', $result->get_error_message() );
-
-            return $this->app->response()->error( $result );
-        } else {
-            update_option( '_wogh_export_complete', 'yes' );
-            update_option( '_wogh_fully_exported', 'yes' );
-
-            return $this->app->response()->success( $result );
-        }
-    }
-
-    /**
-     * Exports a single post to GitHub by ID.
-     *
-     * Called on the save_post hook.
-     *
-     * @param int $post_id Post ID.
-     *
-     * @return boolean
-     */
-    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(
-                'semaphore_locked',
-                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 );
-        $this->app->semaphore()->unlock();
-
-        if ( is_wp_error( $result ) ) {
-            /*　@var WP_Error $result */
-            return $this->app->response()->error( $result );
-        }
-
-        return $this->app->response()->success( $result );
-    }
-
-    /**
-     * Removes the post from the tree.
-     *
-     * Called the delete_post hook.
-     *
-     * @param int $post_id Post ID.
-     *
-     * @return boolean
-     */
-    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(
-                'semaphore_locked',
-                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 );
-        $this->app->semaphore()->unlock();
-
-        if ( is_wp_error( $result ) ) {
-            /*　@var WP_Error $result */
-            return $this->app->response()->error( $result );
-        }
-
-        return $this->app->response()->success( $result );
-    }
-
-    /**
-     * Indicates we're running our own AJAX hook
-     * and thus should respond with JSON, rather
-     * than just returning data.
-     */
-    protected function set_ajax() {
-        if ( ! defined( 'WOGH_AJAX' ) ) {
-            define( 'WOGH_AJAX', true );
-        }
-    }
+	/**
+	 * Application container.
+	 *
+	 * @var Writing_On_GitHub
+	 */
+	public $app;
+
+	/**
+	 * Instantiates a new Controller object
+	 *
+	 * @param Writing_On_GitHub $app Applicatio container.
+	 */
+	public function __construct( Writing_On_GitHub $app ) {
+		$this->app = $app;
+	}
+
+	/**
+	 * Webhook callback as triggered from GitHub push.
+	 *
+	 * Reads the Webhook payload and syncs posts as necessary.
+	 *
+	 * @return boolean
+	 */
+	public function pull_posts() {
+		$this->set_ajax();
+		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()' )
+			) );
+		}
+
+		if ( ! $this->app->request()->is_secret_valid() ) {
+			return $this->app->response()->error( new WP_Error(
+				'invalid_headers',
+				__( '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' ) );
+		}
+
+		// push
+		if ( ! $this->app->request()->is_push() ) {
+			return $this->app->response()->error( new WP_Error(
+				'invalid_headers',
+				__( '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(
+				'invalid_payload',
+				sprintf(
+					__( "%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' ) );
+
+		$result = $this->app->import()->payload( $payload );
+
+		$this->app->semaphore()->unlock();
+
+		if ( is_wp_error( $result ) ) {
+			/*　@var WP_Error $result */
+			return $this->app->response()->error( $result );
+		}
+
+		return $this->app->response()->success( $result );
+	}
+
+	/**
+	 * Imports posts from the current master branch.
+	 *
+	 * @return boolean
+	 */
+	public function import_master( $user_id = 0 ) {
+		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()' )
+			) );
+		}
+
+		$this->app->semaphore()->lock();
+		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 );
+		}
+
+		$result = $this->app->import()->master();
+
+		$this->app->semaphore()->unlock();
+
+		if ( is_wp_error( $result ) ) {
+			/*　@var WP_Error $result */
+			update_option( '_wogh_import_error', $result->get_error_message() );
+
+			return $this->app->response()->error( $result );
+		}
+
+		update_option( '_wogh_import_complete', 'yes' );
+
+		return $this->app->response()->success( $result );
+	}
+
+	/**
+	 * Export all the posts in the database to GitHub.
+	 *
+	 * @param  int        $user_id
+	 * @param  boolean    $force
+	 * @return boolean
+	 */
+	public function export_all( $user_id = 0, $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()' )
+			) );
+		}
+
+		$this->app->semaphore()->lock();
+
+		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 ) ) {
+			/*　@var WP_Error $result */
+			update_option( '_wogh_export_error', $result->get_error_message() );
+
+			return $this->app->response()->error( $result );
+		} else {
+			update_option( '_wogh_export_complete', 'yes' );
+			update_option( '_wogh_fully_exported', 'yes' );
+
+			return $this->app->response()->success( $result );
+		}
+	}
+
+	/**
+	 * Exports a single post to GitHub by ID.
+	 *
+	 * Called on the save_post hook.
+	 *
+	 * @param int $post_id Post ID.
+	 *
+	 * @return boolean
+	 */
+	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(
+				'semaphore_locked',
+				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 );
+		$this->app->semaphore()->unlock();
+
+		if ( is_wp_error( $result ) ) {
+			/*　@var WP_Error $result */
+			return $this->app->response()->error( $result );
+		}
+
+		return $this->app->response()->success( $result );
+	}
+
+	/**
+	 * Removes the post from the tree.
+	 *
+	 * Called the delete_post hook.
+	 *
+	 * @param int $post_id Post ID.
+	 *
+	 * @return boolean
+	 */
+	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(
+				'semaphore_locked',
+				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 );
+		$this->app->semaphore()->unlock();
+
+		if ( is_wp_error( $result ) ) {
+			/*　@var WP_Error $result */
+			return $this->app->response()->error( $result );
+		}
+
+		return $this->app->response()->success( $result );
+	}
+
+	/**
+	 * Indicates we're running our own AJAX hook
+	 * and thus should respond with JSON, rather
+	 * than just returning data.
+	 */
+	protected function set_ajax() {
+		if ( ! defined( 'WOGH_AJAX' ) ) {
+			define( 'WOGH_AJAX', true );
+		}
+	}
 }

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,58 +34,58 @@ 
      */
     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 ) ) {
+        if (is_wp_error($result)) {
             /*　@var WP_Error $result */
-            return $this->app->response()->error( $result );
+            return $this->app->response()->error($result);
         }
 
-        return $this->app->response()->success( $result );
+        return $this->app->response()->success($result);
     }
 
     /**
@@ -93,36 +93,36 @@ 
      *
      * @return boolean
      */
-    public function import_master( $user_id = 0 ) {
-        if ( ! $this->app->semaphore()->is_open() ) {
-            return $this->app->response()->error( new WP_Error(
+    public function import_master($user_id = 0) {
+        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);
     }
 
     /**
@@ -132,34 +132,34 @@ 
      * @param  boolean    $force
      * @return boolean
      */
-    public function export_all( $user_id = 0, $force = false ) {
-        if ( ! $this->app->semaphore()->is_open() ) {
-            return $this->app->response()->error( new WP_Error(
+    public function export_all($user_id = 0, $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 ) ) {
+        if (is_wp_error($result)) {
             /*　@var WP_Error $result */
-            update_option( '_wogh_export_error', $result->get_error_message() );
+            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);
         }
     }
 
@@ -172,28 +172,28 @@ 
      *
      * @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 ) ) {
+        if (is_wp_error($result)) {
             /*　@var WP_Error $result */
-            return $this->app->response()->error( $result );
+            return $this->app->response()->error($result);
         }
 
-        return $this->app->response()->success( $result );
+        return $this->app->response()->success($result);
     }
 
     /**
@@ -205,28 +205,28 @@ 
      *
      * @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 ) ) {
+        if (is_wp_error($result)) {
             /*　@var WP_Error $result */
-            return $this->app->response()->error( $result );
+            return $this->app->response()->error($result);
         }
 
-        return $this->app->response()->success( $result );
+        return $this->app->response()->success($result);
     }
 
     /**
@@ -235,8 +235,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);
         }
     }
 }