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

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

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

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/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/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;