litefeel / writing-on-github

lib/database.php

Spacing +109 added lines, -110 removed lines

@@ -21,21 +21,21 @@ 
      *
      * @var array
      */
-    protected $whitelisted_post_types = array( 'post', 'page' );
+    protected $whitelisted_post_types = array('post', 'page');
 
     /**
      * Currently whitelisted post statuses.
      *
      * @var array
      */
-    protected $whitelisted_post_statuses = array( 'publish' );
+    protected $whitelisted_post_statuses = array('publish');
 
     /**
      * Instantiates a new Database object.
      *
      * @param Writing_On_GitHub $app Application container.
      */
-    public function __construct( Writing_On_GitHub $app ) {
+    public function __construct(Writing_On_GitHub $app) {
         $this->app = $app;
     }
 
@@ -46,33 +46,33 @@ 
      *
      * @return Writing_On_GitHub_Post[]|WP_Error
      */
-    public function fetch_all_supported( $force = false ) {
-        $args  = array(
+    public function fetch_all_supported($force = false) {
+        $args = array(
             'post_type'   => $this->get_whitelisted_post_types(),
             'post_status' => $this->get_whitelisted_post_statuses(),
             'nopaging'    => true,
             'fields'      => 'ids',
         );
 
-        $query = new WP_Query( apply_filters( 'wogh_pre_fetch_all_supported', $args ) );
+        $query = new WP_Query(apply_filters('wogh_pre_fetch_all_supported', $args));
 
         $post_ids = $query->get_posts();
 
-        if ( ! $post_ids ) {
+        if ( ! $post_ids) {
             return new WP_Error(
                 'no_results',
-                __( 'Querying for supported posts returned no results.', 'writing-on-github' )
+                __('Querying for supported posts returned no results.', 'writing-on-github')
             );
         }
 
         /* @var Writing_On_GitHub_Post[] $results */
         $results = array();
-        foreach ( $post_ids as $post_id ) {
+        foreach ($post_ids as $post_id) {
             // Do not export posts that have already been exported
-            if ( $force || ! get_post_meta( $post_id, '_wogh_sha', true ) ||
-                 ! get_post_meta( $post_id, '_wogh_github_path', true ) ) {
+            if ($force || ! get_post_meta($post_id, '_wogh_sha', true) ||
+                 ! get_post_meta($post_id, '_wogh_github_path', true)) {
 
-                $results[] = new Writing_On_GitHub_Post( $post_id, $this->app->api() );
+                $results[] = new Writing_On_GitHub_Post($post_id, $this->app->api());
             }
         }
 
@@ -86,10 +86,10 @@ 
      *
      * @return WP_Error|Writing_On_GitHub_Post
      */
-    public function fetch_by_id( $post_id ) {
-        $post = new Writing_On_GitHub_Post( $post_id, $this->app->api() );
+    public function fetch_by_id($post_id) {
+        $post = new Writing_On_GitHub_Post($post_id, $this->app->api());
 
-        if ( ! $this->is_post_supported( $post ) ) {
+        if ( ! $this->is_post_supported($post)) {
             return new WP_Error(
                 'unsupported_post',
                 sprintf(
@@ -113,7 +113,7 @@ 
      *
      * @return string|WP_Error
      */
-    public function save_posts( array $posts ) {
+    public function save_posts(array $posts) {
 
         /**
          * Whether an error has occurred.
@@ -122,39 +122,38 @@ 
          */
         $error = false;
 
-        foreach ( $posts as $post ) {
-            $args = apply_filters( 'wogh_pre_import_args', $this->post_args( $post ), $post );
+        foreach ($posts as $post) {
+            $args = apply_filters('wogh_pre_import_args', $this->post_args($post), $post);
 
-            remove_filter( 'content_save_pre', 'wp_filter_post_kses' );
+            remove_filter('content_save_pre', 'wp_filter_post_kses');
             $post_id = $post->is_new() ?
-                wp_insert_post( $args, true ) :
-                wp_update_post( $args, true );
-            add_filter( 'content_save_pre', 'wp_filter_post_kses' );
+                wp_insert_post($args, true) : wp_update_post($args, true);
+            add_filter('content_save_pre', 'wp_filter_post_kses');
 
-            if ( is_wp_error( $post_id ) ) {
+            if (is_wp_error($post_id)) {
                 /* @var WP_Error $post_id */
-                $error = wogh_append_error( $error, $post_id );
+                $error = wogh_append_error($error, $post_id);
 
                 // Abort saving if updating the post fails.
                 continue;
             }
 
-            if ( $post->is_new() ) {
+            if ($post->is_new()) {
                 $author = false;
                 $meta = $post->get_meta();
-                if ( ! empty( $meta ) && ! empty( $meta['author'] ) ) {
+                if ( ! empty($meta) && ! empty($meta['author'])) {
                     $author = $meta['author'];
                 }
-                $user    = $this->fetch_commit_user( $author );
-                $user_id = is_wp_error( $user ) ? 0 : $user->ID;
-                $this->set_post_author( $post_id, $user_id );
+                $user    = $this->fetch_commit_user($author);
+                $user_id = is_wp_error($user) ? 0 : $user->ID;
+                $this->set_post_author($post_id, $user_id);
             }
 
-            $post->set_post( get_post( $post_id ) );
+            $post->set_post(get_post($post_id));
 
-            $meta = apply_filters( 'wogh_pre_import_meta', $post->get_meta(), $post );
+            $meta = apply_filters('wogh_pre_import_meta', $post->get_meta(), $post);
 
-            update_post_meta( $post_id, '_wogh_sha', $meta['_wogh_sha'] );
+            update_post_meta($post_id, '_wogh_sha', $meta['_wogh_sha']);
 
             // unset( $meta['tags'] );
             // unset( $meta['categories'] );
@@ -169,32 +168,32 @@ 
             // }
         }
 
-        if ( $error ) {
+        if ($error) {
             return $error;
         }
 
-        return __( 'Successfully saved posts.', 'writing-on-github' );
+        return __('Successfully saved posts.', 'writing-on-github');
     }
 
-    protected function post_args( $post ) {
+    protected function post_args($post) {
         $args = $post->get_args();
         $meta = $post->get_meta();
 
         // prevent backslash loss
-        $args['post_content'] = addslashes( $args['post_content'] );
+        $args['post_content'] = addslashes($args['post_content']);
 
         // update tags
-        if ( ! empty( $meta['tags'] ) ) {
+        if ( ! empty($meta['tags'])) {
             $args['tags_input'] = $meta['tags'];
         }
 
         // update categories
-        if ( ! empty( $meta['categories'] ) ) {
+        if ( ! empty($meta['categories'])) {
             $categories = $meta['categories'];
-            if ( ! is_array( $categories ) ) {
-                $categories = array( $categories );
+            if ( ! is_array($categories)) {
+                $categories = array($categories);
             }
-            $terms = get_terms( array(
+            $terms = get_terms(array(
                 'taxonomy' => 'category',
                 'fields' => 'id=>name',
                 'hide_empty' => 0,
@@ -202,22 +201,22 @@ 
                 )
             );
             $map = array();
-            foreach ( $categories as $name ) {
+            foreach ($categories as $name) {
                 $map[$name] = 1;
             }
 
             $ids = array();
-            if ( ! empty( $terms ) ) {
-                foreach ( $terms as $id => $name ) {
+            if ( ! empty($terms)) {
+                foreach ($terms as $id => $name) {
                     $ids[] = $id;
-                    unset( $map[$name] );
+                    unset($map[$name]);
                 }
             }
 
             // create new terms
-            if ( ! empty( $map ) ) {
-                foreach ( $map as $name => $value ) {
-                    $term = wp_insert_term( $name, 'category', array( 'parent' => 0 ) );
+            if ( ! empty($map)) {
+                foreach ($map as $name => $value) {
+                    $term = wp_insert_term($name, 'category', array('parent' => 0));
                     // array('term_id' => $term_id, 'term_taxonomy_id' => $tt_id);
                     $ids[] = $term['term_id'];
                 }
@@ -229,19 +228,19 @@ 
         return $args;
     }
 
-    private function get_post_id_by_filename( $filename, $pattern  ) {
-        preg_match( $pattern , $filename, $matches );
+    private function get_post_id_by_filename($filename, $pattern) {
+        preg_match($pattern, $filename, $matches);
         $title = $matches[4];
 
-        $query = new WP_Query( array(
+        $query = new WP_Query(array(
             'name'     => $title,
             'posts_per_page' => 1,
             'post_type' => $this->get_whitelisted_post_types(),
             'fields'         => 'ids',
-        ) );
+        ));
 
         $post_id = $query->get_posts();
-        $post_id = array_pop( $post_id );
+        $post_id = array_pop($post_id);
         return $post_id;
     }
 
@@ -252,55 +251,55 @@ 
      *
      * @return string|WP_Error
      */
-    public function delete_post_by_path( $path ) {
-        $query = new WP_Query( array(
+    public function delete_post_by_path($path) {
+        $query = new WP_Query(array(
             'meta_key'       => '_wogh_github_path',
             'meta_value'     => $path,
             'meta_compare'   => '=',
             'posts_per_page' => 1,
             'fields'         => 'ids',
-        ) );
+        ));
 
         $post_id = $query->get_posts();
-        $post_id = array_pop( $post_id );
+        $post_id = array_pop($post_id);
 
-        if ( ! $post_id ) {
-            $parts     = explode( '/', $path );
-            $filename  = array_pop( $parts );
-            $directory = $parts ? array_shift( $parts ) : '';
+        if ( ! $post_id) {
+            $parts     = explode('/', $path);
+            $filename  = array_pop($parts);
+            $directory = $parts ? array_shift($parts) : '';
 
-            if ( false !== strpos( $directory, 'post' ) ) {
-                $post_id = get_post_id_by_filename( $filename, '/([0-9]{4})-([0-9]{2})-([0-9]{2})-(.*)\.md/' );
+            if (false !== strpos($directory, 'post')) {
+                $post_id = get_post_id_by_filename($filename, '/([0-9]{4})-([0-9]{2})-([0-9]{2})-(.*)\.md/');
             }
 
-            if ( ! $post_id ) {
-                $post_id = get_post_id_by_filename( $filename, '/(.*)\.md/' );
+            if ( ! $post_id) {
+                $post_id = get_post_id_by_filename($filename, '/(.*)\.md/');
             }
         }
 
-        if ( ! $post_id ) {
+        if ( ! $post_id) {
             return new WP_Error(
                 'path_not_found',
                 sprintf(
-                    __( 'Post not found for path %s.', 'writing-on-github' ),
+                    __('Post not found for path %s.', 'writing-on-github'),
                     $path
                 )
             );
         }
 
-        $result = wp_delete_post( $post_id );
+        $result = wp_delete_post($post_id);
 
         // If deleting fails...
-        if ( false === $result ) {
-            $post = get_post( $post_id );
+        if (false === $result) {
+            $post = get_post($post_id);
 
             // ...and the post both exists and isn't in the trash...
-            if ( $post && 'trash' !== $post->post_status ) {
+            if ($post && 'trash' !== $post->post_status) {
                 // ... then something went wrong.
                 return new WP_Error(
                     'db_error',
                     sprintf(
-                        __( 'Failed to delete post ID %d.', 'writing-on-github' ),
+                        __('Failed to delete post ID %d.', 'writing-on-github'),
                         $post_id
                     )
                 );
@@ -308,25 +307,25 @@ 
         }
 
         return sprintf(
-            __( 'Successfully deleted post ID %d.', 'writing-on-github' ),
+            __('Successfully deleted post ID %d.', 'writing-on-github'),
             $post_id
         );
     }
 
-    public function delete_post( $post_id ) {
-        $result = wp_delete_post( $post_id );
+    public function delete_post($post_id) {
+        $result = wp_delete_post($post_id);
 
         // If deleting fails...
-        if ( false === $result ) {
-            $post = get_post( $post_id );
+        if (false === $result) {
+            $post = get_post($post_id);
 
             // ...and the post both exists and isn't in the trash...
-            if ( $post && 'trash' !== $post->post_status ) {
+            if ($post && 'trash' !== $post->post_status) {
                 // ... then something went wrong.
                 return new WP_Error(
                     'db_error',
                     sprintf(
-                        __( 'Failed to delete post ID %d.', 'writing-on-github' ),
+                        __('Failed to delete post ID %d.', 'writing-on-github'),
                         $post_id
                     )
                 );
@@ -334,7 +333,7 @@ 
         }
 
         return sprintf(
-            __( 'Successfully deleted post ID %d.', 'writing-on-github' ),
+            __('Successfully deleted post ID %d.', 'writing-on-github'),
             $post_id
         );
     }
@@ -345,7 +344,7 @@ 
      * @return array
      */
     protected function get_whitelisted_post_types() {
-        return apply_filters( 'wogh_whitelisted_post_types', $this->whitelisted_post_types );
+        return apply_filters('wogh_whitelisted_post_types', $this->whitelisted_post_types);
     }
 
     /**
@@ -354,7 +353,7 @@ 
      * @return array
      */
     protected function get_whitelisted_post_statuses() {
-        return apply_filters( 'wogh_whitelisted_post_statuses', $this->whitelisted_post_statuses );
+        return apply_filters('wogh_whitelisted_post_statuses', $this->whitelisted_post_statuses);
     }
 
     /**
@@ -364,12 +363,12 @@ 
      *
      * @return string Whitelist formatted for query
      */
-    protected function format_for_query( $whitelist ) {
-        foreach ( $whitelist as $key => $value ) {
-            $whitelist[ $key ] = "'$value'";
+    protected function format_for_query($whitelist) {
+        foreach ($whitelist as $key => $value) {
+            $whitelist[$key] = "'$value'";
         }
 
-        return implode( ', ', $whitelist );
+        return implode(', ', $whitelist);
     }
 
     /**
@@ -380,25 +379,25 @@ 
      *
      * @return boolean                          True if supported, false if not.
      */
-    protected function is_post_supported( Writing_On_GitHub_Post $post ) {
-        if ( wp_is_post_revision( $post->id ) ) {
+    protected function is_post_supported(Writing_On_GitHub_Post $post) {
+        if (wp_is_post_revision($post->id)) {
             return false;
         }
 
         // We need to allow trashed posts to be queried, but they are not whitelisted for export.
-        if ( ! in_array( $post->status(), $this->get_whitelisted_post_statuses() ) && 'trash' !== $post->status() ) {
+        if ( ! in_array($post->status(), $this->get_whitelisted_post_statuses()) && 'trash' !== $post->status()) {
             return false;
         }
 
-        if ( ! in_array( $post->type(), $this->get_whitelisted_post_types() ) ) {
+        if ( ! in_array($post->type(), $this->get_whitelisted_post_types())) {
             return false;
         }
 
-        if ( $post->has_password() ) {
+        if ($post->has_password()) {
             return false;
         }
 
-        return apply_filters( 'wogh_is_post_supported', true, $post );
+        return apply_filters('wogh_is_post_supported', true, $post);
     }
 
     /**
@@ -411,35 +410,35 @@ 
      *
      * @return WP_Error|WP_User
      */
-    protected function fetch_commit_user( $display_name ) {
+    protected function fetch_commit_user($display_name) {
         // If we can't find a user and a default hasn't been set,
         // we're just going to set the revision author to 0.
         $user = false;
 
-        if ( ! empty( $display_name ) ) {
-            $search_string = esc_attr( $display_name );
-            $query = new WP_User_Query( array(
+        if ( ! empty($display_name)) {
+            $search_string = esc_attr($display_name);
+            $query = new WP_User_Query(array(
                 'search'         => "{$search_string}",
                 'search_columns' => array(
                     'display_name',
                     'user_nicename',
                     'user_login',
                 )
-            ) );
+            ));
             $users = $query->get_results();
             $user = empty($users) ? false : $users[0];
         }
 
-        if ( ! $user ) {
+        if ( ! $user) {
             // Use the default user.
-            $user = get_user_by( 'id', (int) get_option( 'wogh_default_user' ) );
+            $user = get_user_by('id', (int) get_option('wogh_default_user'));
         }
 
-        if ( ! $user ) {
+        if ( ! $user) {
             return new WP_Error(
                 'user_not_found',
                 sprintf(
-                    __( 'Commit user not found for email %s', 'writing-on-github' ),
+                    __('Commit user not found for email %s', 'writing-on-github'),
                     $email
                 )
             );
@@ -491,7 +490,7 @@ 
      *
      * @return string|WP_Error
      */
-    protected function set_post_author( $post_id, $user_id ) {
+    protected function set_post_author($post_id, $user_id) {
         global $wpdb;
 
         $result = $wpdb->update(
@@ -502,25 +501,25 @@ 
             array(
                 'ID' => (int) $post_id,
             ),
-            array( '%d' ),
-            array( '%d' )
+            array('%d'),
+            array('%d')
         );
 
-        if ( false === $result ) {
-            return new WP_Error( 'db_error', $wpdb->last_error );
+        if (false === $result) {
+            return new WP_Error('db_error', $wpdb->last_error);
         }
 
-        if ( 0 === $result ) {
+        if (0 === $result) {
             return sprintf(
-                __( 'No change for post ID %d.', 'writing-on-github' ),
+                __('No change for post ID %d.', 'writing-on-github'),
                 $post_id
             );
         }
 
-        clean_post_cache( $post_id );
+        clean_post_cache($post_id);
 
         return sprintf(
-            __( 'Successfully updated post ID %d.', 'writing-on-github' ),
+            __('Successfully updated post ID %d.', 'writing-on-github'),
             $post_id
         );
     }

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

Spacing +11 added lines, -11 removed lines

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

lib/blob.php

Spacing +23 added lines, -23 removed lines

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