litefeel / writing-on-github

writing-on-github.php

Spacing +60 added lines, -60 removed lines

@@ -12,12 +12,12 @@ 
 
 // If the functions have already been autoloaded, don't reload.
 // This fixes function duplication during unit testing.
-$path = dirname( __FILE__ ) . '/vendor/autoload.php';
-if ( file_exists( $path ) ) {
-    require_once( $path );
+$path = dirname(__FILE__) . '/vendor/autoload.php';
+if (file_exists($path)) {
+    require_once($path);
 }
 
-add_action( 'plugins_loaded', array( new Writing_On_GitHub, 'boot' ) );
+add_action('plugins_loaded', array(new Writing_On_GitHub, 'boot'));
 
 class Writing_On_GitHub {
 
@@ -107,14 +107,14 @@ 
     public function __construct() {
         self::$instance = $this;
 
-        if ( is_admin() ) {
-            $this->admin = new Writing_On_GitHub_Admin( plugin_basename( __FILE__ ) );
+        if (is_admin()) {
+            $this->admin = new Writing_On_GitHub_Admin(plugin_basename(__FILE__));
         }
 
-        $this->controller = new Writing_On_GitHub_Controller( $this );
+        $this->controller = new Writing_On_GitHub_Controller($this);
 
-        if ( defined( 'WP_CLI' ) && WP_CLI ) {
-            WP_CLI::add_command( 'wogh', $this->cli() );
+        if (defined('WP_CLI') && WP_CLI) {
+            WP_CLI::add_command('wogh', $this->cli());
         }
     }
 
@@ -122,30 +122,30 @@ 
      * Attaches the plugin's hooks into WordPress.
      */
     public function boot() {
-        register_activation_hook( __FILE__, array( $this, 'activate' ) );
-        add_action( 'admin_notices', array( $this, 'activation_notice' ) );
+        register_activation_hook(__FILE__, array($this, 'activate'));
+        add_action('admin_notices', array($this, 'activation_notice'));
 
-        add_action( 'init', array( $this, 'l10n' ) );
+        add_action('init', array($this, 'l10n'));
 
         // Controller actions.
-        add_action( 'save_post', array( $this->controller, 'export_post' ) );
-        add_action( 'delete_post', array( $this->controller, 'delete_post' ) );
-        add_action( 'wp_ajax_nopriv_wogh_push_request', array( $this->controller, 'pull_posts' ) );
-        add_action( 'wogh_export', array( $this->controller, 'export_all' ), 10, 2 );
-        add_action( 'wogh_import', array( $this->controller, 'import_master' ), 10, 2 );
-        add_filter( 'get_edit_post_link', array( $this, 'edit_post_link' ), 10, 3 );
+        add_action('save_post', array($this->controller, 'export_post'));
+        add_action('delete_post', array($this->controller, 'delete_post'));
+        add_action('wp_ajax_nopriv_wogh_push_request', array($this->controller, 'pull_posts'));
+        add_action('wogh_export', array($this->controller, 'export_all'), 10, 2);
+        add_action('wogh_import', array($this->controller, 'import_master'), 10, 2);
+        add_filter('get_edit_post_link', array($this, 'edit_post_link'), 10, 3);
 
         // add_filter( 'wogh_post_meta', array( $this, 'ignore_post_meta' ), 10, 1 );
         // add_filter( 'wogh_pre_import_meta', array( $this, 'ignore_post_meta' ), 10, 1 );
-        add_filter( 'the_content', array( $this, 'the_content' ) );
+        add_filter('the_content', array($this, 'the_content'));
 
-        do_action( 'wogh_boot', $this );
+        do_action('wogh_boot', $this);
     }
 
     public function edit_post_link($link, $postID, $context) {
-        if ( ! wp_is_post_revision( $postID ) ) {
-            $post = new Writing_On_GitHub_Post( $postID, Writing_On_GitHub::$instance->api() );
-            if ( $post->is_on_github() ) {
+        if ( ! wp_is_post_revision($postID)) {
+            $post = new Writing_On_GitHub_Post($postID, Writing_On_GitHub::$instance->api());
+            if ($post->is_on_github()) {
                 return $post->github_edit_url();
             }
         }
@@ -204,29 +204,29 @@ 
      * Init i18n files
      */
     public function l10n() {
-        load_plugin_textdomain( self::$text_domain );
+        load_plugin_textdomain(self::$text_domain);
     }
 
     /**
      * Sets and kicks off the export cronjob
      */
-    public function start_export( $force = false ) {
-        $this->start_cron( 'export', $force );
+    public function start_export($force = false) {
+        $this->start_cron('export', $force);
     }
 
     /**
      * Sets and kicks off the import cronjob
      */
-    public function start_import( $force = false ) {
-        $this->start_cron( 'import', $force );
+    public function start_import($force = false) {
+        $this->start_cron('import', $force);
     }
 
     /**
      * Enables the admin notice on initial activation
      */
     public function activate() {
-        if ( 'yes' !== get_option( '_wogh_fully_exported' ) ) {
-            set_transient( '_wogh_activated', 'yes' );
+        if ('yes' !== get_option('_wogh_fully_exported')) {
+            set_transient('_wogh_activated', 'yes');
         }
     }
 
@@ -234,18 +234,18 @@ 
      * Displays the activation admin notice
      */
     public function activation_notice() {
-        if ( ! get_transient( '_wogh_activated' ) ) {
+        if ( ! get_transient('_wogh_activated')) {
             return;
         }
 
-        delete_transient( '_wogh_activated' );
+        delete_transient('_wogh_activated');
 
         ?><div class="updated">
             <p>
                 <?php
                     printf(
-                        __( 'To set up your site to sync with GitHub, update your <a href="%s">settings</a> and click "Export to GitHub."', 'writing-on-github' ),
-                        admin_url( 'options-general.php?page=' . static::$text_domain)
+                        __('To set up your site to sync with GitHub, update your <a href="%s">settings</a> and click "Export to GitHub."', 'writing-on-github'),
+                        admin_url('options-general.php?page=' . static::$text_domain)
                     );
                 ?>
             </p>
@@ -267,7 +267,7 @@ 
      * @return Writing_On_GitHub_CLI
      */
     public function cli() {
-        if ( ! $this->cli ) {
+        if ( ! $this->cli) {
             $this->cli = new Writing_On_GitHub_CLI;
         }
 
@@ -280,8 +280,8 @@ 
      * @return Writing_On_GitHub_Request
      */
     public function request() {
-        if ( ! $this->request ) {
-            $this->request = new Writing_On_GitHub_Request( $this );
+        if ( ! $this->request) {
+            $this->request = new Writing_On_GitHub_Request($this);
         }
 
         return $this->request;
@@ -293,8 +293,8 @@ 
      * @return Writing_On_GitHub_Response
      */
     public function response() {
-        if ( ! $this->response ) {
-            $this->response = new Writing_On_GitHub_Response( $this );
+        if ( ! $this->response) {
+            $this->response = new Writing_On_GitHub_Response($this);
         }
 
         return $this->response;
@@ -306,8 +306,8 @@ 
      * @return Writing_On_GitHub_Api
      */
     public function api() {
-        if ( ! $this->api ) {
-            $this->api = new Writing_On_GitHub_Api( $this );
+        if ( ! $this->api) {
+            $this->api = new Writing_On_GitHub_Api($this);
         }
 
         return $this->api;
@@ -319,8 +319,8 @@ 
      * @return Writing_On_GitHub_Import
      */
     public function import() {
-        if ( ! $this->import ) {
-            $this->import = new Writing_On_GitHub_Import( $this );
+        if ( ! $this->import) {
+            $this->import = new Writing_On_GitHub_Import($this);
         }
 
         return $this->import;
@@ -332,8 +332,8 @@ 
      * @return Writing_On_GitHub_Export
      */
     public function export() {
-        if ( ! $this->export ) {
-            $this->export = new Writing_On_GitHub_Export( $this );
+        if ( ! $this->export) {
+            $this->export = new Writing_On_GitHub_Export($this);
         }
 
         return $this->export;
@@ -345,7 +345,7 @@ 
      * @return Writing_On_GitHub_Semaphore
      */
     public function semaphore() {
-        if ( ! $this->semaphore ) {
+        if ( ! $this->semaphore) {
             $this->semaphore = new Writing_On_GitHub_Semaphore;
         }
 
@@ -358,8 +358,8 @@ 
      * @return Writing_On_GitHub_Database
      */
     public function database() {
-        if ( ! $this->database ) {
-            $this->database = new Writing_On_GitHub_Database( $this );
+        if ( ! $this->database) {
+            $this->database = new Writing_On_GitHub_Database($this);
         }
 
         return $this->database;
@@ -373,18 +373,18 @@ 
      * @param mixed $msg
      * @param string $write
      */
-    public static function write_log( $msg, $write = 'line' ) {
-        if ( defined( 'WP_CLI' ) && WP_CLI ) {
-            if ( is_array( $msg ) || is_object( $msg ) ) {
-                WP_CLI::print_value( $msg );
+    public static function write_log($msg, $write = 'line') {
+        if (defined('WP_CLI') && WP_CLI) {
+            if (is_array($msg) || is_object($msg)) {
+                WP_CLI::print_value($msg);
             } else {
-                WP_CLI::$write( $msg );
+                WP_CLI::$write($msg);
             }
-        } elseif ( true === WP_DEBUG ) {
-            if ( is_array( $msg ) || is_object( $msg ) ) {
-                error_log( print_r( $msg, true ) );
+        } elseif (true === WP_DEBUG) {
+            if (is_array($msg) || is_object($msg)) {
+                error_log(print_r($msg, true));
             } else {
-                error_log( $msg );
+                error_log($msg);
             }
         }
     }
@@ -395,10 +395,10 @@ 
      * @param bool   $force
      * @param string $type
      */
-    protected function start_cron( $type, $force = false ) {
-        update_option( '_wogh_' . $type . '_started', 'yes' );
+    protected function start_cron($type, $force = false) {
+        update_option('_wogh_' . $type . '_started', 'yes');
         $user_id = get_current_user_id();
-        wp_schedule_single_event( time(), 'wogh_' . $type . '', array( $user_id, $force ) );
+        wp_schedule_single_event(time(), 'wogh_' . $type . '', array($user_id, $force));
         spawn_cron();
     }
 }

lib/admin.php

Spacing +73 added lines, -73 removed lines

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

lib/controller.php

Spacing +76 added lines, -76 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);
     }
 
     /**
@@ -94,36 +94,36 @@ 
      * @param  boolean $force
      * @return boolean
      */
-    public function import_master( $user_id = 0, $force = false ) {
-        if ( ! $this->app->semaphore()->is_open() ) {
-            return $this->app->response()->error( new WP_Error(
+    public function import_master($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::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( $force );
+        $result = $this->app->import()->master($force);
 
         $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);
     }
 
     /**
@@ -133,34 +133,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);
         }
     }
 
@@ -173,28 +173,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);
     }
 
     /**
@@ -206,28 +206,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);
     }
 
     /**
@@ -236,8 +236,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);
         }
     }
 }

lib/import.php

Spacing +76 added lines, -76 removed lines

@@ -22,7 +22,7 @@ 
      *
      * @param Writing_On_GitHub $app Application container.
      */
-    public function __construct( Writing_On_GitHub $app ) {
+    public function __construct(Writing_On_GitHub $app) {
         $this->app = $app;
     }
 
@@ -32,24 +32,24 @@ 
      *
      * @return string|WP_Error
      */
-    public function payload( Writing_On_GitHub_Payload $payload ) {
+    public function payload(Writing_On_GitHub_Payload $payload) {
 
-        $result = $this->app->api()->fetch()->compare( $payload->get_before_commit_id() );
+        $result = $this->app->api()->fetch()->compare($payload->get_before_commit_id());
 
-        if ( is_wp_error( $result ) ) {
+        if (is_wp_error($result)) {
             /* @var WP_Error $result */
             return $result;
         }
 
-        if ( is_array( $result ) ) {
-            $result = $this->import_files( $result );
+        if (is_array($result)) {
+            $result = $this->import_files($result);
         }
 
-        if ( is_wp_error( $result ) ) {
+        if (is_wp_error($result)) {
             return $files;
         }
 
-        return __( 'Payload processed', 'writing-on-github' );
+        return __('Payload processed', 'writing-on-github');
     }
 
     /**
@@ -59,37 +59,37 @@ 
      *
      * @return true|WP_Error
      */
-    protected function import_files( $files, $force = false ) {
+    protected function import_files($files, $force = false) {
 
         $error = true;
 
-        foreach ( $files as $file ) {
-            if ( ! $this->importable_file( $file ) ) {
+        foreach ($files as $file) {
+            if ( ! $this->importable_file($file)) {
                 continue;
             }
 
-            $blob = $this->app->api()->fetch()->blob( $file );
+            $blob = $this->app->api()->fetch()->blob($file);
             // network error ?
-            if ( ! $blob instanceof Writing_On_GitHub_Blob ) {
+            if ( ! $blob instanceof Writing_On_GitHub_Blob) {
                 continue;
             }
 
             $is_remove = 'removed' == $file->status;
 
             $result = false;
-            if ( $this->importable_raw_file( $blob ) ) {
-                $result = $this->import_raw_file( $blob, $is_remove );
-            } elseif ( $this->importable_post( $blob ) ) {
-                if ( $is_remove ) {
-                    $result = $this->delete_post( $blob );
+            if ($this->importable_raw_file($blob)) {
+                $result = $this->import_raw_file($blob, $is_remove);
+            } elseif ($this->importable_post($blob)) {
+                if ($is_remove) {
+                    $result = $this->delete_post($blob);
                 } else {
-                    $result = $this->import_post( $blob, $force );
+                    $result = $this->import_post($blob, $force);
                 }
             }
 
-            if ( is_wp_error( $result ) ) {
+            if (is_wp_error($result)) {
                 /* @var WP_Error $result */
-                $error = wogh_append_error( $error, $result );
+                $error = wogh_append_error($error, $result);
             }
         }
 
@@ -102,24 +102,24 @@ 
      * @param  boolean $force
      * @return string|WP_Error
      */
-    public function master( $force = false ) {
+    public function master($force = false) {
         $result = $this->app->api()->fetch()->tree_recursive();
 
-        if ( is_wp_error( $result ) ) {
+        if (is_wp_error($result)) {
             /* @var WP_Error $result */
             return $result;
         }
 
-        if ( is_array( $result ) ) {
-            $result = $this->import_files( $result, $force );
+        if (is_array($result)) {
+            $result = $this->import_files($result, $force);
         }
 
-        if ( is_wp_error( $result ) ) {
+        if (is_wp_error($result)) {
             /* @var WP_Error $result */
             return $result;
         }
 
-        return __( 'Payload processed', 'writing-on-github' );
+        return __('Payload processed', 'writing-on-github');
     }
 
     /**
@@ -129,14 +129,14 @@ 
      *
      * @return bool
      */
-    protected function importable_file( Writing_On_GitHub_File_Info $file ) {
+    protected function importable_file(Writing_On_GitHub_File_Info $file) {
 
         $path = $file->path;
 
         // only _pages, _posts and images
-        $prefixs = array( '_pages/', '_posts/', 'images/');
+        $prefixs = array('_pages/', '_posts/', 'images/');
         foreach ($prefixs as $prefix) {
-            if ( ! strncasecmp($path, $prefix, strlen( $prefix ) ) ) {
+            if ( ! strncasecmp($path, $prefix, strlen($prefix))) {
                 return true;
             }
         }
@@ -150,7 +150,7 @@ 
      *
      * @return bool
      */
-    protected function importable_post( Writing_On_GitHub_Blob $blob ) {
+    protected function importable_post(Writing_On_GitHub_Blob $blob) {
         // global $wpdb;
 
         // // Skip the repo's readme.
@@ -163,7 +163,7 @@ 
         //  return false;
         // }
 
-        if ( ! $blob->has_frontmatter() ) {
+        if ( ! $blob->has_frontmatter()) {
             return false;
         }
 
@@ -175,13 +175,13 @@ 
      * @param  Writing_On_GitHub_Blob $blob
      * @return WP_Error|bool
      */
-    protected function delete_post( Writing_On_GitHub_Blob $blob ) {
+    protected function delete_post(Writing_On_GitHub_Blob $blob) {
         $id = $blob->id();
-        if ( empty( $id ) ) {
+        if (empty($id)) {
             return false;
         }
-        $result = $this->app->database()->delete_post( $id );
-        if ( is_wp_error( $result ) ) {
+        $result = $this->app->database()->delete_post($id);
+        if (is_wp_error($result)) {
             /* @var WP_Error $result */
             return $result;
         }
@@ -194,31 +194,31 @@ 
      * @param  boolean                $force
      * @return WP_Error|bool
      */
-    protected function import_post( Writing_On_GitHub_Blob $blob, $force = false ) {
-        $post = $this->blob_to_post( $blob, $force );
+    protected function import_post(Writing_On_GitHub_Blob $blob, $force = false) {
+        $post = $this->blob_to_post($blob, $force);
 
-        if ( ! $post instanceof Writing_On_GitHub_Post ) {
+        if ( ! $post instanceof Writing_On_GitHub_Post) {
             return false;
         }
 
-        $result = $this->app->database()->save_post( $post );
-        if ( is_wp_error( $result ) ) {
+        $result = $this->app->database()->save_post($post);
+        if (is_wp_error($result)) {
             /** @var WP_Error $result */
             return $result;
         }
 
-        if ( $post->is_new() ||
-                ! wogh_equal_front_matter( $post, $blob ) ) {
+        if ($post->is_new() ||
+                ! wogh_equal_front_matter($post, $blob)) {
 
-            $result = $this->app->export()->export_post( $post );
+            $result = $this->app->export()->export_post($post);
 
-            if ( is_wp_error( $result ) ) {
+            if (is_wp_error($result)) {
                 /** @var WP_Error $result */
                 return $result;
             }
         }
 
-        clean_post_cache( $post->id() );
+        clean_post_cache($post->id());
 
         return true;
     }
@@ -228,13 +228,13 @@ 
      * @param  Writing_On_GitHub_Blob $blob
      * @return bool
      */
-    protected function importable_raw_file( Writing_On_GitHub_Blob $blob ) {
-        if ( $blob->has_frontmatter() ) {
+    protected function importable_raw_file(Writing_On_GitHub_Blob $blob) {
+        if ($blob->has_frontmatter()) {
             return false;
         }
 
         // only images
-        if ( strncasecmp($blob->path(), 'images/', strlen('images/') ) != 0) {
+        if (strncasecmp($blob->path(), 'images/', strlen('images/')) != 0) {
             return false;
         }
 
@@ -246,16 +246,16 @@ 
      * @param  Writing_On_GitHub_Blob $blob
      * @param  bool                   $is_remove
      */
-    protected function import_raw_file( Writing_On_GitHub_Blob $blob, $is_remove ) {
+    protected function import_raw_file(Writing_On_GitHub_Blob $blob, $is_remove) {
         $arr = wp_upload_dir();
         $path = $arr['basedir'] . '/writing-on-github/' . $blob->path();
-        if ( $is_remove ) {
-            if ( file_exists($path) ) {
+        if ($is_remove) {
+            if (file_exists($path)) {
                 unlink($path);
             }
         } else {
             $dirname = dirname($path);
-            if ( ! file_exists($dirname) ) {
+            if ( ! file_exists($dirname)) {
                 wp_mkdir_p($dirname);
             }
 
@@ -272,58 +272,58 @@ 
      *
      * @return Writing_On_GitHub_Post|false
      */
-    protected function blob_to_post( Writing_On_GitHub_Blob $blob, $force = false ) {
-        $args = array( 'post_content' => $blob->content_import() );
+    protected function blob_to_post(Writing_On_GitHub_Blob $blob, $force = false) {
+        $args = array('post_content' => $blob->content_import());
         $meta = $blob->meta();
 
         $id = false;
 
-        if ( ! empty( $meta ) ) {
-            if ( array_key_exists( 'layout', $meta ) ) {
+        if ( ! empty($meta)) {
+            if (array_key_exists('layout', $meta)) {
                 $args['post_type'] = $meta['layout'];
-                unset( $meta['layout'] );
+                unset($meta['layout']);
             }
 
-            if ( array_key_exists( 'published', $meta ) ) {
+            if (array_key_exists('published', $meta)) {
                 $args['post_status'] = true === $meta['published'] ? 'publish' : 'draft';
-                unset( $meta['published'] );
+                unset($meta['published']);
             }
 
-            if ( array_key_exists( 'post_title', $meta ) ) {
+            if (array_key_exists('post_title', $meta)) {
                 $args['post_title'] = $meta['post_title'];
-                unset( $meta['post_title'] );
+                unset($meta['post_title']);
             }
 
-            if ( array_key_exists( 'post_name', $meta ) ) {
+            if (array_key_exists('post_name', $meta)) {
                 $args['post_name'] = $meta['post_name'];
-                unset( $meta['post_name'] );
+                unset($meta['post_name']);
             }
 
-            if ( array_key_exists( 'ID', $meta ) ) {
+            if (array_key_exists('ID', $meta)) {
                 $id = $args['ID'] = $meta['ID'];
                 $blob->set_id($id);
-                unset( $meta['ID'] );
+                unset($meta['ID']);
             }
         }
 
         $meta['_wogh_sha'] = $blob->sha();
 
-        if ( ! $force && $id ) {
-            $old_sha = get_post_meta( $id, '_wogh_sha', true );
-            $old_github_path = get_post_meta( $id, '_wogh_github_path', true );
+        if ( ! $force && $id) {
+            $old_sha = get_post_meta($id, '_wogh_sha', true);
+            $old_github_path = get_post_meta($id, '_wogh_github_path', true);
 
             // dont save post when has same sha
-            if ( $old_sha  && $old_sha == $meta['_wogh_sha'] &&
-                 $old_github_path && $old_github_path == $blob->path() ) {
+            if ($old_sha && $old_sha == $meta['_wogh_sha'] &&
+                 $old_github_path && $old_github_path == $blob->path()) {
                 return false;
             }
         }
 
-        $post = new Writing_On_GitHub_Post( $args, $this->app->api() );
-        $post->set_old_github_path( $blob->path() );
-        $post->set_meta( $meta );
-        $post->set_blob( $blob );
-        $blob->set_id( $post->id() );
+        $post = new Writing_On_GitHub_Post($args, $this->app->api());
+        $post->set_old_github_path($blob->path());
+        $post->set_meta($meta);
+        $post->set_blob($blob);
+        $blob->set_id($post->id());
 
         return $post;
     }

views/options.php

Spacing +15 added lines, -15 removed lines

@@ -6,35 +6,35 @@
 
 ?>
 <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( 'Export', 'writing-on-github' ); ?></th>
+                <th scope="row"><?php esc_html_e('Export', 'writing-on-github'); ?></th>
                 <td>
-                    <p><a href="<?php echo esc_url( add_query_arg( array( 'action' => 'export' ) ) ); ?>">
-                        <input type="button" class="button button-secondary" value="<?php esc_html_e( 'Export to GitHub', 'writing-on-github' ); ?>" />
+                    <p><a href="<?php echo esc_url(add_query_arg(array('action' => 'export'))); ?>">
+                        <input type="button" class="button button-secondary" value="<?php esc_html_e('Export to GitHub', 'writing-on-github'); ?>" />
                     </a></p>
-                    <p><a href="<?php echo esc_url( add_query_arg( array( 'action' => 'force_export' ) ) ); ?>">
-                        <input type="button" class="button button-secondary" value="<?php esc_html_e( 'Force export to GitHub', 'writing-on-github' ); ?>" />
+                    <p><a href="<?php echo esc_url(add_query_arg(array('action' => 'force_export'))); ?>">
+                        <input type="button" class="button button-secondary" value="<?php esc_html_e('Force export to GitHub', 'writing-on-github'); ?>" />
                     </a></p>
                 </td>
             </tr>
             <tr>
-                <th scope="row"><?php esc_html_e( 'Import', 'writing-on-github' ); ?></th>
+                <th scope="row"><?php esc_html_e('Import', 'writing-on-github'); ?></th>
                 <td>
-                    <p><a href="<?php echo esc_url( add_query_arg( array( 'action' => 'import' ) ) ); ?>">
-                        <input type="button" class="button button-secondary" value="<?php esc_html_e( 'Import from GitHub', 'writing-on-github' ); ?>" />
+                    <p><a href="<?php echo esc_url(add_query_arg(array('action' => 'import'))); ?>">
+                        <input type="button" class="button button-secondary" value="<?php esc_html_e('Import from GitHub', 'writing-on-github'); ?>" />
                     </a></p>
-                     <p><a href="<?php echo esc_url( add_query_arg( array( 'action' => 'force_import' ) ) ); ?>">
-                        <input type="button" class="button button-secondary" value="<?php esc_html_e( 'Force Import from GitHub', 'writing-on-github' ); ?>" />
+                     <p><a href="<?php echo esc_url(add_query_arg(array('action' => 'force_import'))); ?>">
+                        <input type="button" class="button button-secondary" value="<?php esc_html_e('Force Import from GitHub', 'writing-on-github'); ?>" />
                     </a></p>
                 </td>
             </tr>