litefeel / writing-on-github

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;

lib/fileinfo.php

Spacing +3 added lines, -3 removed lines

@@ -6,13 +6,13 @@
  */
 class Writing_On_GitHub_File_Info {
 
-	public function __construct( stdClass $data ) {
+	public function __construct(stdClass $data) {
 		$this->sha 			= $data->sha;
 		$this->path 		= $data->path;
-		$this->status 		= $data->status;
+		$this->status = $data->status;
 	}
 
 	public $sha;
 	public $path;
-	public $status;  // added removed modified
+	public $status; // added removed modified
 }

lib/semaphore.php

Spacing +4 added lines, -4 removed lines

@@ -28,7 +28,7 @@ 
 	 * Clean up the old values on instantiation.
 	 */
 	public function __construct() {
-		delete_option( self::KEY );
+		delete_option(self::KEY);
 	}
 
 	/**
@@ -40,7 +40,7 @@ 
 	 * @return bool
 	 */
 	public function is_open() {
-		if ( self::VALUE_LOCKED === get_transient( self::KEY ) ) {
+		if (self::VALUE_LOCKED === get_transient(self::KEY)) {
 			return false;
 		}
 
@@ -51,13 +51,13 @@ 
 	 * Enables the push lock.
 	 */
 	public function lock() {
-		set_transient( self::KEY, self::VALUE_LOCKED, MINUTE_IN_SECONDS );
+		set_transient(self::KEY, self::VALUE_LOCKED, MINUTE_IN_SECONDS);
 	}
 
 	/**
 	 * Disables the push lock.
 	 */
 	public function unlock() {
-		set_transient( self::KEY, self::VALUE_UNLOCKED, MINUTE_IN_SECONDS );
+		set_transient(self::KEY, self::VALUE_UNLOCKED, MINUTE_IN_SECONDS);
 	}
 }

lib/cli.php

Spacing +28 added lines, -28 removed lines

@@ -44,28 +44,28 @@ 
 	 *
 	 * @param array $args Command arguments.
 	 */
-	public function export( $args ) {
-		list( $post_id, $user_id ) = $args;
+	public function export($args) {
+		list($post_id, $user_id) = $args;
 
-		if ( ! is_numeric( $user_id ) ) {
-			WP_CLI::error( __( 'Invalid user ID', 'writing-on-github' ) );
+		if ( ! is_numeric($user_id)) {
+			WP_CLI::error(__('Invalid user ID', 'writing-on-github'));
 		}
 
-		$this->app->export()->set_user( $user_id );
+		$this->app->export()->set_user($user_id);
 
-		if ( 'all' === $post_id ) {
-			WP_CLI::line( __( 'Starting full export to GitHub.', 'writing-on-github' ) );
+		if ('all' === $post_id) {
+			WP_CLI::line(__('Starting full export to GitHub.', 'writing-on-github'));
 			$this->app->controller()->export_all();
-		} elseif ( is_numeric( $post_id ) ) {
+		} elseif (is_numeric($post_id)) {
 			WP_CLI::line(
 				sprintf(
-					__( 'Exporting post ID to GitHub: %d', 'writing-on-github' ),
+					__('Exporting post ID to GitHub: %d', 'writing-on-github'),
 					$post_id
 				)
 			);
-			$this->app->controller()->export_post( (int) $post_id );
+			$this->app->controller()->export_post((int) $post_id);
 		} else {
-			WP_CLI::error( __( 'Invalid post ID', 'writing-on-github' ) );
+			WP_CLI::error(__('Invalid post ID', 'writing-on-github'));
 		}
 	}
 
@@ -86,16 +86,16 @@ 
 	 *
 	 * @param array $args Command arguments.
 	 */
-	public function import( $args ) {
-		list( $user_id ) = $args;
+	public function import($args) {
+		list($user_id) = $args;
 
-		if ( ! is_numeric( $user_id ) ) {
-			WP_CLI::error( __( 'Invalid user ID', 'writing-on-github' ) );
+		if ( ! is_numeric($user_id)) {
+			WP_CLI::error(__('Invalid user ID', 'writing-on-github'));
 		}
 
-		update_option( '_wogh_export_user_id', (int) $user_id );
+		update_option('_wogh_export_user_id', (int) $user_id);
 
-		WP_CLI::line( __( 'Starting import from GitHub.', 'writing-on-github' ) );
+		WP_CLI::line(__('Starting import from GitHub.', 'writing-on-github'));
 
 		$this->app->controller()->import_master();
 	}
@@ -119,40 +119,40 @@ 
 	 * @param array $args Command arguments.
 	 * @param array $assoc_args Command associated arguments.
 	 */
-	public function prime( $args, $assoc_args ) {
-		if ( isset( $assoc_args['branch'] ) ) {
-			WP_CLI::line( __( 'Starting branch import.', 'writing-on-github' ) );
+	public function prime($args, $assoc_args) {
+		if (isset($assoc_args['branch'])) {
+			WP_CLI::line(__('Starting branch import.', 'writing-on-github'));
 
 			$commit = $this->app->api()->fetch()->master();
 
-			if ( is_wp_error( $commit ) ) {
+			if (is_wp_error($commit)) {
 				WP_CLI::error(
 					sprintf(
-						__( 'Failed to import and cache branch with error: %s', 'writing-on-github' ),
+						__('Failed to import and cache branch with error: %s', 'writing-on-github'),
 						$commit->get_error_message()
 					)
 				);
 			} else {
 				WP_CLI::success(
 					sprintf(
-						__( 'Successfully imported and cached commit %s from branch.', 'writing-on-github' ),
+						__('Successfully imported and cached commit %s from branch.', 'writing-on-github'),
 						$commit->sha()
 					)
 				);
 			}
-		} else if ( isset( $assoc_args['sha'] ) ) {
-			WP_CLI::line( 'Starting sha import.' );
+		} else if (isset($assoc_args['sha'])) {
+			WP_CLI::line('Starting sha import.');
 
-			$commit = $this->app->api()->fetch()->commit( $assoc_args['sha'] );
+			$commit = $this->app->api()->fetch()->commit($assoc_args['sha']);
 
 			WP_CLI::success(
 				sprintf(
-					__( 'Successfully imported and cached commit %s.', 'writing-on-github' ),
+					__('Successfully imported and cached commit %s.', 'writing-on-github'),
 					$commit->sha()
 				)
 			);
 		} else {
-			WP_CLI::error( 'Invalid fetch.' );
+			WP_CLI::error('Invalid fetch.');
 		}
 	}
 }

lib/post.php

Indentation +3 added lines, -3 removed lines

@@ -376,9 +376,9 @@
 		if ( empty($this->post->post_excerpt) ) {
 			unset($meta['post_excerpt']);
 		}
-        if ( 'yes' == get_option('wogh_ignore_author') ) {
-            unset($meta['author']);
-        }
+		if ( 'yes' == get_option('wogh_ignore_author') ) {
+			unset($meta['author']);
+		}
 
 		//convert traditional post_meta values, hide hidden values, skip already populated values
 		// foreach ( get_post_custom( $this->id ) as $key => $value ) {

Spacing +62 added lines, -62 removed lines

@@ -60,26 +60,26 @@ 
 	 *
 	 * @todo remove database operations from this method
 	 */
-	public function __construct( $id_or_args, Writing_On_GitHub_Api $api ) {
+	public function __construct($id_or_args, Writing_On_GitHub_Api $api) {
 		$this->api = $api;
 
-		if ( is_numeric( $id_or_args ) ) {
+		if (is_numeric($id_or_args)) {
 			$this->id   = (int) $id_or_args;
-			$this->post = get_post( $this->id );
+			$this->post = get_post($this->id);
 			$this->new  = false;
 		}
 
-		if ( is_array( $id_or_args ) ) {
+		if (is_array($id_or_args)) {
 			$this->args = $id_or_args;
 
-			if ( isset( $this->args['ID'] ) ) {
-				$this->post = get_post( $this->args['ID'] );
+			if (isset($this->args['ID'])) {
+				$this->post = get_post($this->args['ID']);
 
-				if ( $this->post ) {
+				if ($this->post) {
 					$this->id  = $this->post->ID;
 					$this->new = false;
 				} else {
-					unset( $this->args['ID'] );
+					unset($this->args['ID']);
 				}
 			}
 		}
@@ -115,7 +115,7 @@ 
 	 * @return bool
 	 */
 	public function has_password() {
-		return ! empty( $this->post->post_password );
+		return ! empty($this->post->post_password);
 	}
 
 	/**
@@ -123,9 +123,9 @@ 
 	 */
 	public function github_content() {
 		$content = $this->front_matter() . $this->post_content();
-		$ending  = apply_filters( 'wogh_line_endings', "\n" );
+		$ending  = apply_filters('wogh_line_endings', "\n");
 
-		return preg_replace( '~(*BSR_ANYCRLF)\R~', $ending, $content );
+		return preg_replace('~(*BSR_ANYCRLF)\R~', $ending, $content);
 	}
 
 	/**
@@ -134,7 +134,7 @@ 
 	 * Returns String the YAML frontmatter, ready to be written to the file
 	 */
 	public function front_matter() {
-		return "---\n" . spyc_dump( $this->meta() ) . "---\n";
+		return "---\n" . spyc_dump($this->meta()) . "---\n";
 	}
 
 	/**
@@ -145,24 +145,24 @@ 
 	public function post_content() {
 		$content = $this->post->post_content;
 
-		if ( function_exists( 'wpmarkdown_html_to_markdown' ) ) {
-			$content = wpmarkdown_html_to_markdown( $content );
-		} else if ( class_exists( 'WPCom_Markdown' ) ) {
-			if ( WPCom_Markdown::get_instance()->is_markdown( $this->post->ID ) ) {
+		if (function_exists('wpmarkdown_html_to_markdown')) {
+			$content = wpmarkdown_html_to_markdown($content);
+		} else if (class_exists('WPCom_Markdown')) {
+			if (WPCom_Markdown::get_instance()->is_markdown($this->post->ID)) {
 				$content = $this->post->post_content_filtered;
 			}
 		}
 
-		return apply_filters( 'wogh_content_export', $content, $this );
+		return apply_filters('wogh_content_export', $content, $this);
 	}
 
 	public function old_github_path() {
 		return $this->old_github_path;
 	}
 
-	public function set_old_github_path( $path ) {
+	public function set_old_github_path($path) {
 		$this->old_github_path = $path;
-		update_post_meta( $this->id, '_wogh_github_path', $path);
+		update_post_meta($this->id, '_wogh_github_path', $path);
 	}
 
 
@@ -184,13 +184,13 @@ 
 	 * @return string
 	 */
 	public function github_directory() {
-		if ( 'publish' !== $this->status() ) {
-			return apply_filters( 'wogh_directory_unpublished', '_drafts/', $this );
+		if ('publish' !== $this->status()) {
+			return apply_filters('wogh_directory_unpublished', '_drafts/', $this);
 		}
 
 		$name = '';
 
-		switch ( $this->type() ) {
+		switch ($this->type()) {
 			case 'post':
 				$name = 'posts';
 				break;
@@ -198,35 +198,35 @@ 
 				$name = 'pages';
 				break;
 			default:
-				$obj = get_post_type_object( $this->type() );
+				$obj = get_post_type_object($this->type());
 
-				if ( $obj ) {
-					$name = strtolower( $obj->labels->name );
+				if ($obj) {
+					$name = strtolower($obj->labels->name);
 				}
 
-				if ( ! $name ) {
+				if ( ! $name) {
 					$name = '';
 				}
 		}
 
-		if ( $name ) {
+		if ($name) {
 			$name = '_' . $name . '/';
 		}
 
-		return apply_filters( 'wogh_directory_published', $name, $this );
+		return apply_filters('wogh_directory_published', $name, $this);
 	}
 
 	/**
 	 * Build GitHub filename based on post
 	 */
 	public function github_filename() {
-		if ( 'post' === $this->type() ) {
-			$filename = get_the_time( 'Y-m-d-', $this->id ) . $this->get_name() . '.md';
+		if ('post' === $this->type()) {
+			$filename = get_the_time('Y-m-d-', $this->id) . $this->get_name() . '.md';
 		} else {
 			$filename = $this->get_name() . '.md';
 		}
 
-		return apply_filters( 'wogh_filename', $filename, $this );
+		return apply_filters('wogh_filename', $filename, $this);
 	}
 
 	/**
@@ -235,11 +235,11 @@ 
 	 * @return string
 	 */
 	protected function get_name() {
-		if ( '' !== $this->name() ) {
+		if ('' !== $this->name()) {
 			return $this->name();
 		}
 
-		return sanitize_title( get_the_title( $this->post ) );
+		return sanitize_title(get_the_title($this->post));
 	}
 
 	/**
@@ -247,9 +247,9 @@ 
 	 * @return boolean
 	 */
 	public function is_on_github() {
-		$sha = get_post_meta( $this->id, '_wogh_sha', true );
-		$github_path = get_post_meta( $this->id, '_wogh_github_path', true );
-		if ( $sha && $github_path ) {
+		$sha = get_post_meta($this->id, '_wogh_sha', true);
+		$github_path = get_post_meta($this->id, '_wogh_github_path', true);
+		if ($sha && $github_path) {
 			return true;
 		}
 		return false;
@@ -261,7 +261,7 @@ 
 	 * @return string
 	 */
 	public function github_view_url() {
-		$github_path = get_post_meta( $this->id, '_wogh_github_path', true );
+		$github_path = get_post_meta($this->id, '_wogh_github_path', true);
 		$repository = $this->api->fetch()->repository();
 		$branch = $this->api->fetch()->branch();
 
@@ -274,7 +274,7 @@ 
 	 * @return string
 	 */
 	public function github_edit_url() {
-		$github_path = get_post_meta( $this->id, '_wogh_github_path', true );
+		$github_path = get_post_meta($this->id, '_wogh_github_path', true);
 		$repository = $this->api->fetch()->repository();
 		$branch = $this->api->fetch()->branch();
 
@@ -288,9 +288,9 @@ 
 	 *
 	 * @return string
 	 */
-	public function get_directory_from_path( $path ) {
-		$directory = explode( '/', $path );
-		$directory = count( $directory ) > 0 ? $directory[0] : '';
+	public function get_directory_from_path($path) {
+		$directory = explode('/', $path);
+		$directory = count($directory) > 0 ? $directory[0] : '';
 
 		return $directory;
 	}
@@ -301,11 +301,11 @@ 
 	 * Returns Array an array containing the author name and email
 	 */
 	public function last_modified_author() {
-		if ( $last_id = get_post_meta( $this->id, '_edit_last', true ) ) {
-			$user = get_userdata( $last_id );
+		if ($last_id = get_post_meta($this->id, '_edit_last', true)) {
+			$user = get_userdata($last_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);
 			}
 		}
 
@@ -319,7 +319,7 @@ 
 	 * Returns String the sha1 hash
 	 */
 	public function sha() {
-		$sha = get_post_meta( $this->id, '_wogh_sha', true );
+		$sha = get_post_meta($this->id, '_wogh_sha', true);
 
 		// If we've done a full export and we have no sha
 		// then we should try a live check to see if it exists.
@@ -335,7 +335,7 @@ 
 		// }
 
 		// if the sha still doesn't exist, then it's empty
-		if ( ! $sha || is_wp_error( $sha ) ) {
+		if ( ! $sha || is_wp_error($sha)) {
 			$sha = '';
 		}
 
@@ -347,8 +347,8 @@ 
 	 *
 	 * @param string $sha
 	 */
-	public function set_sha( $sha ) {
-		update_post_meta( $this->id, '_wogh_sha', $sha );
+	public function set_sha($sha) {
+		update_post_meta($this->id, '_wogh_sha', $sha);
 	}
 
 	/**
@@ -359,24 +359,24 @@ 
 	public function meta() {
 		$meta = array(
 			'ID'           => $this->id,
-			'post_title'   => get_the_title( $this->post ),
+			'post_title'   => get_the_title($this->post),
 			'post_name'    => $this->post->post_name,
-			'author'       => ( $author = get_userdata( $this->post->post_author ) ) ? $author->display_name : '',
+			'author'       => ($author = get_userdata($this->post->post_author)) ? $author->display_name : '',
 			'post_date'    => $this->post->post_date,
 			'post_excerpt' => $this->post->post_excerpt,
-			'layout'       => get_post_type( $this->post ),
-			'link'		   => get_permalink( $this->post ),
+			'layout'       => get_post_type($this->post),
+			'link'		   => get_permalink($this->post),
 			'published'    => 'publish' === $this->status() ? true : false,
-			'tags'         => wp_get_post_tags( $this->id, array( 'fields' => 'names' ) ),
-			'categories'   => wp_get_post_categories( $this->id, array( 'fields' => 'names' ) )
+			'tags'         => wp_get_post_tags($this->id, array('fields' => 'names')),
+			'categories'   => wp_get_post_categories($this->id, array('fields' => 'names'))
 		);
-		if ( empty($this->post->post_name) ) {
+		if (empty($this->post->post_name)) {
 			unset($meta['post_name']);
 		}
-		if ( empty($this->post->post_excerpt) ) {
+		if (empty($this->post->post_excerpt)) {
 			unset($meta['post_excerpt']);
 		}
-        if ( 'yes' == get_option('wogh_ignore_author') ) {
+        if ('yes' == get_option('wogh_ignore_author')) {
             unset($meta['author']);
         }
 
@@ -391,7 +391,7 @@ 
 
 		// }
 
-		return apply_filters( 'wogh_post_meta', $meta, $this );
+		return apply_filters('wogh_post_meta', $meta, $this);
 	}
 
 	/**
@@ -408,7 +408,7 @@ 
 	 *
 	 * @param array $meta
 	 */
-	public function set_meta( $meta ) {
+	public function set_meta($meta) {
 		$this->meta = $meta;
 	}
 
@@ -437,7 +437,7 @@ 
 	 *
 	 * @return $this
 	 */
-	public function set_post( WP_Post $post ) {
+	public function set_post(WP_Post $post) {
 		$this->post = $post;
 		$this->id   = $post->ID;
 
@@ -456,6 +456,6 @@ 
 		$data->content = $this->github_content();
 		$data->sha     = $this->sha();
 
-		return new Writing_On_GitHub_Blob( $data );
+		return new Writing_On_GitHub_Blob($data);
 	}
 }

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>