@@ -5,8 +5,8 @@ |
||
| 5 | 5 | */ |
| 6 | 6 | |
| 7 | 7 | ?> |
| 8 | -<?php $value = get_option( $args['name'], $args['default'] ); ?> |
|
| 9 | -<textarea name="<?php echo esc_attr( $args['name'] ); ?>" id="<?php echo esc_attr( $args['name'] ); ?>" rows="10" cols="40"> |
|
| 10 | -<?php echo esc_attr( $value ); ?> |
|
| 8 | +<?php $value = get_option($args['name'], $args['default']); ?> |
|
| 9 | +<textarea name="<?php echo esc_attr($args['name']); ?>" id="<?php echo esc_attr($args['name']); ?>" rows="10" cols="40"> |
|
| 10 | +<?php echo esc_attr($value); ?> |
|
| 11 | 11 | </textarea> |
| 12 | 12 | <p class="description"><?php echo $args['help_text']; ?></p> |
@@ -1,8 +1,8 @@ |
||
| 1 | -<?php $value = get_option( $args['name'], $args['default'] ); ?> |
|
| 2 | -<select name="<?php echo esc_attr( $args['name'] ) ?>" id="<?php echo esc_attr( $args['name'] ) ?>"> |
|
| 3 | - <?php foreach ( get_users() as $user ) : ?> |
|
| 4 | - <option value="<?php echo esc_attr( $user->ID ); ?>"<?php echo (int) $value === $user->ID ? ' selected' : '';?>> |
|
| 5 | - <?php echo esc_html( $user->display_name ); ?> |
|
| 1 | +<?php $value = get_option($args['name'], $args['default']); ?> |
|
| 2 | +<select name="<?php echo esc_attr($args['name']) ?>" id="<?php echo esc_attr($args['name']) ?>"> |
|
| 3 | + <?php foreach (get_users() as $user) : ?> |
|
| 4 | + <option value="<?php echo esc_attr($user->ID); ?>"<?php echo (int) $value === $user->ID ? ' selected' : ''; ?>> |
|
| 5 | + <?php echo esc_html($user->display_name); ?> |
|
| 6 | 6 | </option> |
| 7 | 7 | <?php endforeach; ?> |
| 8 | 8 | </select> |
@@ -5,6 +5,6 @@ |
||
| 5 | 5 | */ |
| 6 | 6 | |
| 7 | 7 | ?> |
| 8 | -<?php $value = get_option( $args['name'], $args['default'] ); ?> |
|
| 9 | -<input type="checkbox" name="<?php echo esc_attr( $args['name'] ); ?>" id="<?php echo esc_attr( $args['name'] ); ?>" value="yes" <?php echo 'yes' === $value ? 'checked' : ''; ?> /> |
|
| 8 | +<?php $value = get_option($args['name'], $args['default']); ?> |
|
| 9 | +<input type="checkbox" name="<?php echo esc_attr($args['name']); ?>" id="<?php echo esc_attr($args['name']); ?>" value="yes" <?php echo 'yes' === $value ? 'checked' : ''; ?> /> |
|
| 10 | 10 | <p class="description"><?php echo $args['help_text']; ?></p> |
@@ -5,6 +5,6 @@ |
||
| 5 | 5 | */ |
| 6 | 6 | |
| 7 | 7 | ?> |
| 8 | -<?php $value = get_option( $args['name'], $args['default'] ); ?> |
|
| 9 | -<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 ); ?>" /> |
|
| 8 | +<?php $value = get_option($args['name'], $args['default']); ?> |
|
| 9 | +<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); ?>" /> |
|
| 10 | 10 | <p class="description"><?php echo $args['help_text']; ?></p> |
@@ -4,7 +4,7 @@ discard block |
||
| 4 | 4 | */ |
| 5 | 5 | $_tests_dir = getenv( 'WP_TESTS_DIR' ); |
| 6 | 6 | if ( ! $_tests_dir ) { |
| 7 | - $_tests_dir = '/tmp/wordpress-tests-lib'; |
|
| 7 | + $_tests_dir = '/tmp/wordpress-tests-lib'; |
|
| 8 | 8 | } |
| 9 | 9 | // Give access to tests_add_filter() function. |
| 10 | 10 | require_once $_tests_dir . '/includes/functions.php'; |
@@ -15,8 +15,8 @@ discard block |
||
| 15 | 15 | * Manually load the plugin being tested. |
| 16 | 16 | */ |
| 17 | 17 | function _manually_load_plugin() { |
| 18 | - require dirname( __FILE__ ) . '/../../writing-on-github.php'; |
|
| 19 | - remove_action( 'plugins_loaded', array( Writing_On_GitHub::$instance, 'boot' ) ); |
|
| 18 | + require dirname( __FILE__ ) . '/../../writing-on-github.php'; |
|
| 19 | + remove_action( 'plugins_loaded', array( Writing_On_GitHub::$instance, 'boot' ) ); |
|
| 20 | 20 | } |
| 21 | 21 | tests_add_filter( 'muplugins_loaded', '_manually_load_plugin' ); |
| 22 | 22 | // Start up the WP testing environment. |
@@ -2,24 +2,24 @@ |
||
| 2 | 2 | /** |
| 3 | 3 | * PHPUnit bootstrap file |
| 4 | 4 | */ |
| 5 | -$_tests_dir = getenv( 'WP_TESTS_DIR' ); |
|
| 6 | -if ( ! $_tests_dir ) { |
|
| 5 | +$_tests_dir = getenv('WP_TESTS_DIR'); |
|
| 6 | +if ( ! $_tests_dir) { |
|
| 7 | 7 | $_tests_dir = '/tmp/wordpress-tests-lib'; |
| 8 | 8 | } |
| 9 | 9 | // Give access to tests_add_filter() function. |
| 10 | 10 | require_once $_tests_dir . '/includes/functions.php'; |
| 11 | 11 | |
| 12 | -define( 'WRITING_ON_GITHUB_TEST', true ); |
|
| 12 | +define('WRITING_ON_GITHUB_TEST', true); |
|
| 13 | 13 | |
| 14 | 14 | /** |
| 15 | 15 | * Manually load the plugin being tested. |
| 16 | 16 | */ |
| 17 | 17 | function _manually_load_plugin() { |
| 18 | - require dirname( __FILE__ ) . '/../../writing-on-github.php'; |
|
| 19 | - remove_action( 'plugins_loaded', array( Writing_On_GitHub::$instance, 'boot' ) ); |
|
| 18 | + require dirname(__FILE__) . '/../../writing-on-github.php'; |
|
| 19 | + remove_action('plugins_loaded', array(Writing_On_GitHub::$instance, 'boot')); |
|
| 20 | 20 | } |
| 21 | -tests_add_filter( 'muplugins_loaded', '_manually_load_plugin' ); |
|
| 21 | +tests_add_filter('muplugins_loaded', '_manually_load_plugin'); |
|
| 22 | 22 | // Start up the WP testing environment. |
| 23 | 23 | require $_tests_dir . '/includes/bootstrap.php'; |
| 24 | 24 | |
| 25 | -error_reporting( E_ALL ^ E_DEPRECATED ); |
|
| 25 | +error_reporting(E_ALL ^ E_DEPRECATED); |
|
@@ -9,204 +9,204 @@ |
||
| 9 | 9 | */ |
| 10 | 10 | class Writing_On_GitHub_Base_Client { |
| 11 | 11 | |
| 12 | - const HOST_OPTION_KEY = 'wogh_host'; |
|
| 13 | - const TOKEN_OPTION_KEY = 'wogh_oauth_token'; |
|
| 14 | - const REPO_OPTION_KEY = 'wogh_repository'; |
|
| 15 | - const BRANCH_OPTION_KEY = 'wogh_branch'; |
|
| 16 | - |
|
| 17 | - /** |
|
| 18 | - * Application container. |
|
| 19 | - * |
|
| 20 | - * @var Writing_On_GitHub |
|
| 21 | - */ |
|
| 22 | - protected $app; |
|
| 23 | - |
|
| 24 | - /** |
|
| 25 | - * Instantiates a new Api object. |
|
| 26 | - * |
|
| 27 | - * @param Writing_On_GitHub $app Application container. |
|
| 28 | - */ |
|
| 29 | - public function __construct( Writing_On_GitHub $app ) { |
|
| 30 | - $this->app = $app; |
|
| 31 | - } |
|
| 32 | - |
|
| 33 | - /** |
|
| 34 | - * Generic GitHub API interface and response handler |
|
| 35 | - * |
|
| 36 | - * @param string $method HTTP method. |
|
| 37 | - * @param string $endpoint API endpoint. |
|
| 38 | - * @param array $body Request body. |
|
| 39 | - * |
|
| 40 | - * @return stdClass|WP_Error |
|
| 41 | - */ |
|
| 42 | - protected function call( $method, $endpoint, $body = array() ) { |
|
| 43 | - if ( is_wp_error( $error = $this->can_call() ) ) { |
|
| 44 | - /* @var WP_Error $error */ |
|
| 45 | - return $error; |
|
| 46 | - } |
|
| 47 | - |
|
| 48 | - $args = array( |
|
| 49 | - 'method' => $method, |
|
| 50 | - 'headers' => array( |
|
| 51 | - 'Authorization' => 'token ' . $this->oauth_token(), |
|
| 52 | - ), |
|
| 53 | - ); |
|
| 54 | - |
|
| 55 | - if ( 'GET' !== $method ) { |
|
| 56 | - $args['body'] = json_encode( $body ); |
|
| 57 | - } |
|
| 58 | - |
|
| 59 | - // $tmpbody = isset( $args['body'] ) ? $args['body'] : ''; |
|
| 60 | - // error_log( "writing-on-github-call $method $endpoint $tmpbody" ); |
|
| 61 | - |
|
| 62 | - $response = wp_remote_request( $endpoint, $args ); |
|
| 63 | - $status = wp_remote_retrieve_header( $response, 'status' ); |
|
| 64 | - $body = json_decode( wp_remote_retrieve_body( $response ) ); |
|
| 65 | - |
|
| 66 | - if ( '2' !== substr( $status, 0, 1 ) && '3' !== substr( $status, 0, 1 ) ) { |
|
| 67 | - return new WP_Error( |
|
| 68 | - strtolower( str_replace( ' ', '_', $status ) ), |
|
| 69 | - sprintf( |
|
| 70 | - __( 'Method %s to endpoint %s failed with error: %s', 'writing-on-github' ), |
|
| 71 | - $method, |
|
| 72 | - $endpoint, |
|
| 73 | - ( $body && $body->message ) ? $body->message : 'Unknown error' |
|
| 74 | - ) |
|
| 75 | - ); |
|
| 76 | - } |
|
| 77 | - |
|
| 78 | - return $body; |
|
| 79 | - } |
|
| 80 | - |
|
| 81 | - /** |
|
| 82 | - * Validates whether the Api object can make a call. |
|
| 83 | - * |
|
| 84 | - * @return true|WP_Error |
|
| 85 | - */ |
|
| 86 | - protected function can_call() { |
|
| 87 | - if ( ! $this->oauth_token() ) { |
|
| 88 | - return new WP_Error( |
|
| 89 | - 'missing_token', |
|
| 90 | - __( 'Writing On GitHub needs an auth token. Please update your settings.', 'writing-on-github' ) |
|
| 91 | - ); |
|
| 92 | - } |
|
| 93 | - |
|
| 94 | - $repo = $this->repository(); |
|
| 95 | - |
|
| 96 | - if ( ! $repo ) { |
|
| 97 | - return new WP_Error( |
|
| 98 | - 'missing_repository', |
|
| 99 | - __( 'Writing On GitHub needs a repository. Please update your settings.', 'writing-on-github' ) |
|
| 100 | - ); |
|
| 101 | - } |
|
| 102 | - |
|
| 103 | - $parts = explode( '/', $repo ); |
|
| 104 | - |
|
| 105 | - if ( 2 !== count( $parts ) ) { |
|
| 106 | - return new WP_Error( |
|
| 107 | - 'malformed_repository', |
|
| 108 | - __( 'Writing On GitHub needs a properly formed repository. Please update your settings.', 'writing-on-github' ) |
|
| 109 | - ); |
|
| 110 | - } |
|
| 111 | - |
|
| 112 | - return true; |
|
| 113 | - } |
|
| 114 | - |
|
| 115 | - /** |
|
| 116 | - * Returns the repository to sync with |
|
| 117 | - * |
|
| 118 | - * @return string |
|
| 119 | - */ |
|
| 120 | - public function repository() { |
|
| 121 | - return (string) get_option( self::REPO_OPTION_KEY ); |
|
| 122 | - } |
|
| 123 | - |
|
| 124 | - /** |
|
| 125 | - * Returns the user's oauth token |
|
| 126 | - * |
|
| 127 | - * @return string |
|
| 128 | - */ |
|
| 129 | - public function oauth_token() { |
|
| 130 | - return (string) get_option( self::TOKEN_OPTION_KEY ); |
|
| 131 | - } |
|
| 132 | - |
|
| 133 | - /** |
|
| 134 | - * Returns the GitHub host to sync with (for GitHub Enterprise support) |
|
| 135 | - */ |
|
| 136 | - public function api_base() { |
|
| 137 | - return get_option( self::HOST_OPTION_KEY ); |
|
| 138 | - } |
|
| 139 | - |
|
| 140 | - public function branch() { |
|
| 141 | - $branch = get_option( self::BRANCH_OPTION_KEY ); |
|
| 142 | - return $branch ? $branch : 'master'; |
|
| 143 | - } |
|
| 144 | - |
|
| 145 | - /** |
|
| 146 | - * API endpoint for the master branch reference |
|
| 147 | - */ |
|
| 148 | - public function reference_endpoint() { |
|
| 149 | - $url = $this->api_base() . '/repos/'; |
|
| 150 | - $url = $url . $this->repository() . '/git/refs/heads/' . $this->branch(); |
|
| 151 | - |
|
| 152 | - return $url; |
|
| 153 | - } |
|
| 154 | - |
|
| 155 | - /** |
|
| 156 | - * Api to get and create commits |
|
| 157 | - */ |
|
| 158 | - public function commit_endpoint() { |
|
| 159 | - $url = $this->api_base() . '/repos/'; |
|
| 160 | - $url = $url . $this->repository() . '/git/commits'; |
|
| 161 | - |
|
| 162 | - return $url; |
|
| 163 | - } |
|
| 164 | - |
|
| 165 | - /** |
|
| 166 | - * Api to compare commits |
|
| 167 | - */ |
|
| 168 | - public function compare_endpoint() { |
|
| 169 | - $url = $this->api_base() . '/repos/'; |
|
| 170 | - $url = $url . $this->repository() . '/compare'; |
|
| 171 | - |
|
| 172 | - return $url; |
|
| 173 | - } |
|
| 174 | - |
|
| 175 | - /** |
|
| 176 | - * Api to get and create trees |
|
| 177 | - */ |
|
| 178 | - public function tree_endpoint() { |
|
| 179 | - $url = $this->api_base() . '/repos/'; |
|
| 180 | - $url = $url . $this->repository() . '/git/trees'; |
|
| 181 | - |
|
| 182 | - return $url; |
|
| 183 | - } |
|
| 184 | - |
|
| 185 | - /** |
|
| 186 | - * Builds the proper blob API endpoint for a given post |
|
| 187 | - * |
|
| 188 | - * Returns String the relative API call path |
|
| 189 | - */ |
|
| 190 | - public function blob_endpoint() { |
|
| 191 | - $url = $this->api_base() . '/repos/'; |
|
| 192 | - $url = $url . $this->repository() . '/git/blobs'; |
|
| 193 | - |
|
| 194 | - return $url; |
|
| 195 | - } |
|
| 196 | - |
|
| 197 | - /** |
|
| 198 | - * Builds the proper content API endpoint for a given post |
|
| 199 | - * |
|
| 200 | - * Returns String the relative API call path |
|
| 201 | - */ |
|
| 202 | - public function content_endpoint( $path = false ) { |
|
| 203 | - $url = $this->api_base() . '/repos/'; |
|
| 204 | - $url = $url . $this->repository() . '/contents'; |
|
| 205 | - |
|
| 206 | - if ( ! empty($path) ) { |
|
| 207 | - $url .= '/' . $path; |
|
| 208 | - } |
|
| 209 | - |
|
| 210 | - return $url; |
|
| 211 | - } |
|
| 12 | + const HOST_OPTION_KEY = 'wogh_host'; |
|
| 13 | + const TOKEN_OPTION_KEY = 'wogh_oauth_token'; |
|
| 14 | + const REPO_OPTION_KEY = 'wogh_repository'; |
|
| 15 | + const BRANCH_OPTION_KEY = 'wogh_branch'; |
|
| 16 | + |
|
| 17 | + /** |
|
| 18 | + * Application container. |
|
| 19 | + * |
|
| 20 | + * @var Writing_On_GitHub |
|
| 21 | + */ |
|
| 22 | + protected $app; |
|
| 23 | + |
|
| 24 | + /** |
|
| 25 | + * Instantiates a new Api object. |
|
| 26 | + * |
|
| 27 | + * @param Writing_On_GitHub $app Application container. |
|
| 28 | + */ |
|
| 29 | + public function __construct( Writing_On_GitHub $app ) { |
|
| 30 | + $this->app = $app; |
|
| 31 | + } |
|
| 32 | + |
|
| 33 | + /** |
|
| 34 | + * Generic GitHub API interface and response handler |
|
| 35 | + * |
|
| 36 | + * @param string $method HTTP method. |
|
| 37 | + * @param string $endpoint API endpoint. |
|
| 38 | + * @param array $body Request body. |
|
| 39 | + * |
|
| 40 | + * @return stdClass|WP_Error |
|
| 41 | + */ |
|
| 42 | + protected function call( $method, $endpoint, $body = array() ) { |
|
| 43 | + if ( is_wp_error( $error = $this->can_call() ) ) { |
|
| 44 | + /* @var WP_Error $error */ |
|
| 45 | + return $error; |
|
| 46 | + } |
|
| 47 | + |
|
| 48 | + $args = array( |
|
| 49 | + 'method' => $method, |
|
| 50 | + 'headers' => array( |
|
| 51 | + 'Authorization' => 'token ' . $this->oauth_token(), |
|
| 52 | + ), |
|
| 53 | + ); |
|
| 54 | + |
|
| 55 | + if ( 'GET' !== $method ) { |
|
| 56 | + $args['body'] = json_encode( $body ); |
|
| 57 | + } |
|
| 58 | + |
|
| 59 | + // $tmpbody = isset( $args['body'] ) ? $args['body'] : ''; |
|
| 60 | + // error_log( "writing-on-github-call $method $endpoint $tmpbody" ); |
|
| 61 | + |
|
| 62 | + $response = wp_remote_request( $endpoint, $args ); |
|
| 63 | + $status = wp_remote_retrieve_header( $response, 'status' ); |
|
| 64 | + $body = json_decode( wp_remote_retrieve_body( $response ) ); |
|
| 65 | + |
|
| 66 | + if ( '2' !== substr( $status, 0, 1 ) && '3' !== substr( $status, 0, 1 ) ) { |
|
| 67 | + return new WP_Error( |
|
| 68 | + strtolower( str_replace( ' ', '_', $status ) ), |
|
| 69 | + sprintf( |
|
| 70 | + __( 'Method %s to endpoint %s failed with error: %s', 'writing-on-github' ), |
|
| 71 | + $method, |
|
| 72 | + $endpoint, |
|
| 73 | + ( $body && $body->message ) ? $body->message : 'Unknown error' |
|
| 74 | + ) |
|
| 75 | + ); |
|
| 76 | + } |
|
| 77 | + |
|
| 78 | + return $body; |
|
| 79 | + } |
|
| 80 | + |
|
| 81 | + /** |
|
| 82 | + * Validates whether the Api object can make a call. |
|
| 83 | + * |
|
| 84 | + * @return true|WP_Error |
|
| 85 | + */ |
|
| 86 | + protected function can_call() { |
|
| 87 | + if ( ! $this->oauth_token() ) { |
|
| 88 | + return new WP_Error( |
|
| 89 | + 'missing_token', |
|
| 90 | + __( 'Writing On GitHub needs an auth token. Please update your settings.', 'writing-on-github' ) |
|
| 91 | + ); |
|
| 92 | + } |
|
| 93 | + |
|
| 94 | + $repo = $this->repository(); |
|
| 95 | + |
|
| 96 | + if ( ! $repo ) { |
|
| 97 | + return new WP_Error( |
|
| 98 | + 'missing_repository', |
|
| 99 | + __( 'Writing On GitHub needs a repository. Please update your settings.', 'writing-on-github' ) |
|
| 100 | + ); |
|
| 101 | + } |
|
| 102 | + |
|
| 103 | + $parts = explode( '/', $repo ); |
|
| 104 | + |
|
| 105 | + if ( 2 !== count( $parts ) ) { |
|
| 106 | + return new WP_Error( |
|
| 107 | + 'malformed_repository', |
|
| 108 | + __( 'Writing On GitHub needs a properly formed repository. Please update your settings.', 'writing-on-github' ) |
|
| 109 | + ); |
|
| 110 | + } |
|
| 111 | + |
|
| 112 | + return true; |
|
| 113 | + } |
|
| 114 | + |
|
| 115 | + /** |
|
| 116 | + * Returns the repository to sync with |
|
| 117 | + * |
|
| 118 | + * @return string |
|
| 119 | + */ |
|
| 120 | + public function repository() { |
|
| 121 | + return (string) get_option( self::REPO_OPTION_KEY ); |
|
| 122 | + } |
|
| 123 | + |
|
| 124 | + /** |
|
| 125 | + * Returns the user's oauth token |
|
| 126 | + * |
|
| 127 | + * @return string |
|
| 128 | + */ |
|
| 129 | + public function oauth_token() { |
|
| 130 | + return (string) get_option( self::TOKEN_OPTION_KEY ); |
|
| 131 | + } |
|
| 132 | + |
|
| 133 | + /** |
|
| 134 | + * Returns the GitHub host to sync with (for GitHub Enterprise support) |
|
| 135 | + */ |
|
| 136 | + public function api_base() { |
|
| 137 | + return get_option( self::HOST_OPTION_KEY ); |
|
| 138 | + } |
|
| 139 | + |
|
| 140 | + public function branch() { |
|
| 141 | + $branch = get_option( self::BRANCH_OPTION_KEY ); |
|
| 142 | + return $branch ? $branch : 'master'; |
|
| 143 | + } |
|
| 144 | + |
|
| 145 | + /** |
|
| 146 | + * API endpoint for the master branch reference |
|
| 147 | + */ |
|
| 148 | + public function reference_endpoint() { |
|
| 149 | + $url = $this->api_base() . '/repos/'; |
|
| 150 | + $url = $url . $this->repository() . '/git/refs/heads/' . $this->branch(); |
|
| 151 | + |
|
| 152 | + return $url; |
|
| 153 | + } |
|
| 154 | + |
|
| 155 | + /** |
|
| 156 | + * Api to get and create commits |
|
| 157 | + */ |
|
| 158 | + public function commit_endpoint() { |
|
| 159 | + $url = $this->api_base() . '/repos/'; |
|
| 160 | + $url = $url . $this->repository() . '/git/commits'; |
|
| 161 | + |
|
| 162 | + return $url; |
|
| 163 | + } |
|
| 164 | + |
|
| 165 | + /** |
|
| 166 | + * Api to compare commits |
|
| 167 | + */ |
|
| 168 | + public function compare_endpoint() { |
|
| 169 | + $url = $this->api_base() . '/repos/'; |
|
| 170 | + $url = $url . $this->repository() . '/compare'; |
|
| 171 | + |
|
| 172 | + return $url; |
|
| 173 | + } |
|
| 174 | + |
|
| 175 | + /** |
|
| 176 | + * Api to get and create trees |
|
| 177 | + */ |
|
| 178 | + public function tree_endpoint() { |
|
| 179 | + $url = $this->api_base() . '/repos/'; |
|
| 180 | + $url = $url . $this->repository() . '/git/trees'; |
|
| 181 | + |
|
| 182 | + return $url; |
|
| 183 | + } |
|
| 184 | + |
|
| 185 | + /** |
|
| 186 | + * Builds the proper blob API endpoint for a given post |
|
| 187 | + * |
|
| 188 | + * Returns String the relative API call path |
|
| 189 | + */ |
|
| 190 | + public function blob_endpoint() { |
|
| 191 | + $url = $this->api_base() . '/repos/'; |
|
| 192 | + $url = $url . $this->repository() . '/git/blobs'; |
|
| 193 | + |
|
| 194 | + return $url; |
|
| 195 | + } |
|
| 196 | + |
|
| 197 | + /** |
|
| 198 | + * Builds the proper content API endpoint for a given post |
|
| 199 | + * |
|
| 200 | + * Returns String the relative API call path |
|
| 201 | + */ |
|
| 202 | + public function content_endpoint( $path = false ) { |
|
| 203 | + $url = $this->api_base() . '/repos/'; |
|
| 204 | + $url = $url . $this->repository() . '/contents'; |
|
| 205 | + |
|
| 206 | + if ( ! empty($path) ) { |
|
| 207 | + $url .= '/' . $path; |
|
| 208 | + } |
|
| 209 | + |
|
| 210 | + return $url; |
|
| 211 | + } |
|
| 212 | 212 | } |
@@ -26,7 +26,7 @@ discard block |
||
| 26 | 26 | * |
| 27 | 27 | * @param Writing_On_GitHub $app Application container. |
| 28 | 28 | */ |
| 29 | - public function __construct( Writing_On_GitHub $app ) { |
|
| 29 | + public function __construct(Writing_On_GitHub $app) { |
|
| 30 | 30 | $this->app = $app; |
| 31 | 31 | } |
| 32 | 32 | |
@@ -39,8 +39,8 @@ discard block |
||
| 39 | 39 | * |
| 40 | 40 | * @return stdClass|WP_Error |
| 41 | 41 | */ |
| 42 | - protected function call( $method, $endpoint, $body = array() ) { |
|
| 43 | - if ( is_wp_error( $error = $this->can_call() ) ) { |
|
| 42 | + protected function call($method, $endpoint, $body = array()) { |
|
| 43 | + if (is_wp_error($error = $this->can_call())) { |
|
| 44 | 44 | /* @var WP_Error $error */ |
| 45 | 45 | return $error; |
| 46 | 46 | } |
@@ -52,25 +52,25 @@ discard block |
||
| 52 | 52 | ), |
| 53 | 53 | ); |
| 54 | 54 | |
| 55 | - if ( 'GET' !== $method ) { |
|
| 56 | - $args['body'] = json_encode( $body ); |
|
| 55 | + if ('GET' !== $method) { |
|
| 56 | + $args['body'] = json_encode($body); |
|
| 57 | 57 | } |
| 58 | 58 | |
| 59 | 59 | // $tmpbody = isset( $args['body'] ) ? $args['body'] : ''; |
| 60 | 60 | // error_log( "writing-on-github-call $method $endpoint $tmpbody" ); |
| 61 | 61 | |
| 62 | - $response = wp_remote_request( $endpoint, $args ); |
|
| 63 | - $status = wp_remote_retrieve_header( $response, 'status' ); |
|
| 64 | - $body = json_decode( wp_remote_retrieve_body( $response ) ); |
|
| 62 | + $response = wp_remote_request($endpoint, $args); |
|
| 63 | + $status = wp_remote_retrieve_header($response, 'status'); |
|
| 64 | + $body = json_decode(wp_remote_retrieve_body($response)); |
|
| 65 | 65 | |
| 66 | - if ( '2' !== substr( $status, 0, 1 ) && '3' !== substr( $status, 0, 1 ) ) { |
|
| 66 | + if ('2' !== substr($status, 0, 1) && '3' !== substr($status, 0, 1)) { |
|
| 67 | 67 | return new WP_Error( |
| 68 | - strtolower( str_replace( ' ', '_', $status ) ), |
|
| 68 | + strtolower(str_replace(' ', '_', $status)), |
|
| 69 | 69 | sprintf( |
| 70 | - __( 'Method %s to endpoint %s failed with error: %s', 'writing-on-github' ), |
|
| 70 | + __('Method %s to endpoint %s failed with error: %s', 'writing-on-github'), |
|
| 71 | 71 | $method, |
| 72 | 72 | $endpoint, |
| 73 | - ( $body && $body->message ) ? $body->message : 'Unknown error' |
|
| 73 | + ($body && $body->message) ? $body->message : 'Unknown error' |
|
| 74 | 74 | ) |
| 75 | 75 | ); |
| 76 | 76 | } |
@@ -84,28 +84,28 @@ discard block |
||
| 84 | 84 | * @return true|WP_Error |
| 85 | 85 | */ |
| 86 | 86 | protected function can_call() { |
| 87 | - if ( ! $this->oauth_token() ) { |
|
| 87 | + if ( ! $this->oauth_token()) { |
|
| 88 | 88 | return new WP_Error( |
| 89 | 89 | 'missing_token', |
| 90 | - __( 'Writing On GitHub needs an auth token. Please update your settings.', 'writing-on-github' ) |
|
| 90 | + __('Writing On GitHub needs an auth token. Please update your settings.', 'writing-on-github') |
|
| 91 | 91 | ); |
| 92 | 92 | } |
| 93 | 93 | |
| 94 | 94 | $repo = $this->repository(); |
| 95 | 95 | |
| 96 | - if ( ! $repo ) { |
|
| 96 | + if ( ! $repo) { |
|
| 97 | 97 | return new WP_Error( |
| 98 | 98 | 'missing_repository', |
| 99 | - __( 'Writing On GitHub needs a repository. Please update your settings.', 'writing-on-github' ) |
|
| 99 | + __('Writing On GitHub needs a repository. Please update your settings.', 'writing-on-github') |
|
| 100 | 100 | ); |
| 101 | 101 | } |
| 102 | 102 | |
| 103 | - $parts = explode( '/', $repo ); |
|
| 103 | + $parts = explode('/', $repo); |
|
| 104 | 104 | |
| 105 | - if ( 2 !== count( $parts ) ) { |
|
| 105 | + if (2 !== count($parts)) { |
|
| 106 | 106 | return new WP_Error( |
| 107 | 107 | 'malformed_repository', |
| 108 | - __( 'Writing On GitHub needs a properly formed repository. Please update your settings.', 'writing-on-github' ) |
|
| 108 | + __('Writing On GitHub needs a properly formed repository. Please update your settings.', 'writing-on-github') |
|
| 109 | 109 | ); |
| 110 | 110 | } |
| 111 | 111 | |
@@ -118,7 +118,7 @@ discard block |
||
| 118 | 118 | * @return string |
| 119 | 119 | */ |
| 120 | 120 | public function repository() { |
| 121 | - return (string) get_option( self::REPO_OPTION_KEY ); |
|
| 121 | + return (string) get_option(self::REPO_OPTION_KEY); |
|
| 122 | 122 | } |
| 123 | 123 | |
| 124 | 124 | /** |
@@ -127,18 +127,18 @@ discard block |
||
| 127 | 127 | * @return string |
| 128 | 128 | */ |
| 129 | 129 | public function oauth_token() { |
| 130 | - return (string) get_option( self::TOKEN_OPTION_KEY ); |
|
| 130 | + return (string) get_option(self::TOKEN_OPTION_KEY); |
|
| 131 | 131 | } |
| 132 | 132 | |
| 133 | 133 | /** |
| 134 | 134 | * Returns the GitHub host to sync with (for GitHub Enterprise support) |
| 135 | 135 | */ |
| 136 | 136 | public function api_base() { |
| 137 | - return get_option( self::HOST_OPTION_KEY ); |
|
| 137 | + return get_option(self::HOST_OPTION_KEY); |
|
| 138 | 138 | } |
| 139 | 139 | |
| 140 | 140 | public function branch() { |
| 141 | - $branch = get_option( self::BRANCH_OPTION_KEY ); |
|
| 141 | + $branch = get_option(self::BRANCH_OPTION_KEY); |
|
| 142 | 142 | return $branch ? $branch : 'master'; |
| 143 | 143 | } |
| 144 | 144 | |
@@ -199,11 +199,11 @@ discard block |
||
| 199 | 199 | * |
| 200 | 200 | * Returns String the relative API call path |
| 201 | 201 | */ |
| 202 | - public function content_endpoint( $path = false ) { |
|
| 202 | + public function content_endpoint($path = false) { |
|
| 203 | 203 | $url = $this->api_base() . '/repos/'; |
| 204 | 204 | $url = $url . $this->repository() . '/contents'; |
| 205 | 205 | |
| 206 | - if ( ! empty($path) ) { |
|
| 206 | + if ( ! empty($path)) { |
|
| 207 | 207 | $url .= '/' . $path; |
| 208 | 208 | } |
| 209 | 209 | |
@@ -9,75 +9,75 @@ |
||
| 9 | 9 | */ |
| 10 | 10 | class Writing_On_GitHub_Persist_Client extends Writing_On_GitHub_Base_Client { |
| 11 | 11 | |
| 12 | - /** |
|
| 13 | - * Get the data for the current user. |
|
| 14 | - * |
|
| 15 | - * @return array |
|
| 16 | - */ |
|
| 17 | - protected function export_user() { |
|
| 18 | - $user_id = get_current_user_id(); |
|
| 19 | - $user = get_userdata( $user_id ); |
|
| 12 | + /** |
|
| 13 | + * Get the data for the current user. |
|
| 14 | + * |
|
| 15 | + * @return array |
|
| 16 | + */ |
|
| 17 | + protected function export_user() { |
|
| 18 | + $user_id = get_current_user_id(); |
|
| 19 | + $user = get_userdata( $user_id ); |
|
| 20 | 20 | |
| 21 | - if ( $user ) { |
|
| 22 | - return array( |
|
| 23 | - 'name' => $user->display_name, |
|
| 24 | - 'email' => $user->user_email, |
|
| 25 | - ); |
|
| 26 | - } |
|
| 21 | + if ( $user ) { |
|
| 22 | + return array( |
|
| 23 | + 'name' => $user->display_name, |
|
| 24 | + 'email' => $user->user_email, |
|
| 25 | + ); |
|
| 26 | + } |
|
| 27 | 27 | |
| 28 | - return false; |
|
| 29 | - } |
|
| 28 | + return false; |
|
| 29 | + } |
|
| 30 | 30 | |
| 31 | - /** |
|
| 32 | - * Delete the file. |
|
| 33 | - * |
|
| 34 | - * @return array |
|
| 35 | - */ |
|
| 36 | - public function delete_file( $path, $sha, $message ) { |
|
| 37 | - $body = new stdClass(); |
|
| 38 | - $body->message = $message; |
|
| 39 | - $body->sha = $sha; |
|
| 40 | - $body->branch = $this->branch(); |
|
| 31 | + /** |
|
| 32 | + * Delete the file. |
|
| 33 | + * |
|
| 34 | + * @return array |
|
| 35 | + */ |
|
| 36 | + public function delete_file( $path, $sha, $message ) { |
|
| 37 | + $body = new stdClass(); |
|
| 38 | + $body->message = $message; |
|
| 39 | + $body->sha = $sha; |
|
| 40 | + $body->branch = $this->branch(); |
|
| 41 | 41 | |
| 42 | - if ( $author = $this->export_user() ) { |
|
| 43 | - $body->author = $author; |
|
| 44 | - } |
|
| 42 | + if ( $author = $this->export_user() ) { |
|
| 43 | + $body->author = $author; |
|
| 44 | + } |
|
| 45 | 45 | |
| 46 | - return $this->call( 'DELETE', $this->content_endpoint( $path ), $body ); |
|
| 47 | - } |
|
| 46 | + return $this->call( 'DELETE', $this->content_endpoint( $path ), $body ); |
|
| 47 | + } |
|
| 48 | 48 | |
| 49 | - /** |
|
| 50 | - * Create the file. |
|
| 51 | - * |
|
| 52 | - * @return array |
|
| 53 | - */ |
|
| 54 | - public function create_file( $blob, $message ) { |
|
| 55 | - $body = $blob->to_body(); |
|
| 56 | - $body->message = $message; |
|
| 57 | - $body->branch = $this->branch(); |
|
| 58 | - unset($body->sha); |
|
| 49 | + /** |
|
| 50 | + * Create the file. |
|
| 51 | + * |
|
| 52 | + * @return array |
|
| 53 | + */ |
|
| 54 | + public function create_file( $blob, $message ) { |
|
| 55 | + $body = $blob->to_body(); |
|
| 56 | + $body->message = $message; |
|
| 57 | + $body->branch = $this->branch(); |
|
| 58 | + unset($body->sha); |
|
| 59 | 59 | |
| 60 | - if ( $author = $this->export_user() ) { |
|
| 61 | - $body->author = $author; |
|
| 62 | - } |
|
| 60 | + if ( $author = $this->export_user() ) { |
|
| 61 | + $body->author = $author; |
|
| 62 | + } |
|
| 63 | 63 | |
| 64 | - return $this->call( 'PUT', $this->content_endpoint( $blob->path() ), $body ); |
|
| 65 | - } |
|
| 64 | + return $this->call( 'PUT', $this->content_endpoint( $blob->path() ), $body ); |
|
| 65 | + } |
|
| 66 | 66 | |
| 67 | - /** |
|
| 68 | - * Update the file. |
|
| 69 | - * |
|
| 70 | - * @return array |
|
| 71 | - */ |
|
| 72 | - public function update_file( $blob, $message ) { |
|
| 73 | - $body = $blob->to_body(); |
|
| 74 | - $body->message = $message; |
|
| 75 | - $body->branch = $this->branch(); |
|
| 67 | + /** |
|
| 68 | + * Update the file. |
|
| 69 | + * |
|
| 70 | + * @return array |
|
| 71 | + */ |
|
| 72 | + public function update_file( $blob, $message ) { |
|
| 73 | + $body = $blob->to_body(); |
|
| 74 | + $body->message = $message; |
|
| 75 | + $body->branch = $this->branch(); |
|
| 76 | 76 | |
| 77 | - if ( $author = $this->export_user() ) { |
|
| 78 | - $body->author = $author; |
|
| 79 | - } |
|
| 77 | + if ( $author = $this->export_user() ) { |
|
| 78 | + $body->author = $author; |
|
| 79 | + } |
|
| 80 | 80 | |
| 81 | - return $this->call( 'PUT', $this->content_endpoint( $blob->path() ), $body ); |
|
| 82 | - } |
|
| 81 | + return $this->call( 'PUT', $this->content_endpoint( $blob->path() ), $body ); |
|
| 82 | + } |
|
| 83 | 83 | } |
@@ -16,9 +16,9 @@ discard block |
||
| 16 | 16 | */ |
| 17 | 17 | protected function export_user() { |
| 18 | 18 | $user_id = get_current_user_id(); |
| 19 | - $user = get_userdata( $user_id ); |
|
| 19 | + $user = get_userdata($user_id); |
|
| 20 | 20 | |
| 21 | - if ( $user ) { |
|
| 21 | + if ($user) { |
|
| 22 | 22 | return array( |
| 23 | 23 | 'name' => $user->display_name, |
| 24 | 24 | 'email' => $user->user_email, |
@@ -33,17 +33,17 @@ discard block |
||
| 33 | 33 | * |
| 34 | 34 | * @return array |
| 35 | 35 | */ |
| 36 | - public function delete_file( $path, $sha, $message ) { |
|
| 36 | + public function delete_file($path, $sha, $message) { |
|
| 37 | 37 | $body = new stdClass(); |
| 38 | 38 | $body->message = $message; |
| 39 | 39 | $body->sha = $sha; |
| 40 | 40 | $body->branch = $this->branch(); |
| 41 | 41 | |
| 42 | - if ( $author = $this->export_user() ) { |
|
| 42 | + if ($author = $this->export_user()) { |
|
| 43 | 43 | $body->author = $author; |
| 44 | 44 | } |
| 45 | 45 | |
| 46 | - return $this->call( 'DELETE', $this->content_endpoint( $path ), $body ); |
|
| 46 | + return $this->call('DELETE', $this->content_endpoint($path), $body); |
|
| 47 | 47 | } |
| 48 | 48 | |
| 49 | 49 | /** |
@@ -51,17 +51,17 @@ discard block |
||
| 51 | 51 | * |
| 52 | 52 | * @return array |
| 53 | 53 | */ |
| 54 | - public function create_file( $blob, $message ) { |
|
| 54 | + public function create_file($blob, $message) { |
|
| 55 | 55 | $body = $blob->to_body(); |
| 56 | 56 | $body->message = $message; |
| 57 | 57 | $body->branch = $this->branch(); |
| 58 | 58 | unset($body->sha); |
| 59 | 59 | |
| 60 | - if ( $author = $this->export_user() ) { |
|
| 60 | + if ($author = $this->export_user()) { |
|
| 61 | 61 | $body->author = $author; |
| 62 | 62 | } |
| 63 | 63 | |
| 64 | - return $this->call( 'PUT', $this->content_endpoint( $blob->path() ), $body ); |
|
| 64 | + return $this->call('PUT', $this->content_endpoint($blob->path()), $body); |
|
| 65 | 65 | } |
| 66 | 66 | |
| 67 | 67 | /** |
@@ -69,15 +69,15 @@ discard block |
||
| 69 | 69 | * |
| 70 | 70 | * @return array |
| 71 | 71 | */ |
| 72 | - public function update_file( $blob, $message ) { |
|
| 72 | + public function update_file($blob, $message) { |
|
| 73 | 73 | $body = $blob->to_body(); |
| 74 | 74 | $body->message = $message; |
| 75 | 75 | $body->branch = $this->branch(); |
| 76 | 76 | |
| 77 | - if ( $author = $this->export_user() ) { |
|
| 77 | + if ($author = $this->export_user()) { |
|
| 78 | 78 | $body->author = $author; |
| 79 | 79 | } |
| 80 | 80 | |
| 81 | - return $this->call( 'PUT', $this->content_endpoint( $blob->path() ), $body ); |
|
| 81 | + return $this->call('PUT', $this->content_endpoint($blob->path()), $body); |
|
| 82 | 82 | } |
| 83 | 83 | } |
@@ -9,87 +9,87 @@ |
||
| 9 | 9 | */ |
| 10 | 10 | class Writing_On_GitHub_Response { |
| 11 | 11 | |
| 12 | - /** |
|
| 13 | - * Application container. |
|
| 14 | - * |
|
| 15 | - * @var Writing_On_GitHub |
|
| 16 | - */ |
|
| 17 | - protected $app; |
|
| 12 | + /** |
|
| 13 | + * Application container. |
|
| 14 | + * |
|
| 15 | + * @var Writing_On_GitHub |
|
| 16 | + */ |
|
| 17 | + protected $app; |
|
| 18 | 18 | |
| 19 | - /** |
|
| 20 | - * Writing_On_GitHub_Response constructor. |
|
| 21 | - * |
|
| 22 | - * @param Writing_On_GitHub $app Application container. |
|
| 23 | - */ |
|
| 24 | - public function __construct( Writing_On_GitHub $app ) { |
|
| 25 | - $this->app = $app; |
|
| 26 | - } |
|
| 19 | + /** |
|
| 20 | + * Writing_On_GitHub_Response constructor. |
|
| 21 | + * |
|
| 22 | + * @param Writing_On_GitHub $app Application container. |
|
| 23 | + */ |
|
| 24 | + public function __construct( Writing_On_GitHub $app ) { |
|
| 25 | + $this->app = $app; |
|
| 26 | + } |
|
| 27 | 27 | |
| 28 | - /** |
|
| 29 | - * Writes to the log and returns the error response. |
|
| 30 | - * |
|
| 31 | - * @param WP_Error $error Error to respond with. |
|
| 32 | - * |
|
| 33 | - * @return false |
|
| 34 | - */ |
|
| 35 | - public function error( WP_Error $error ) { |
|
| 36 | - global $wp_version; |
|
| 28 | + /** |
|
| 29 | + * Writes to the log and returns the error response. |
|
| 30 | + * |
|
| 31 | + * @param WP_Error $error Error to respond with. |
|
| 32 | + * |
|
| 33 | + * @return false |
|
| 34 | + */ |
|
| 35 | + public function error( WP_Error $error ) { |
|
| 36 | + global $wp_version; |
|
| 37 | 37 | |
| 38 | - $this->log( $error ); |
|
| 38 | + $this->log( $error ); |
|
| 39 | 39 | |
| 40 | - if ( defined( 'DOING_AJAX' ) && DOING_AJAX && defined( 'WOGH_AJAX' ) && WOGH_AJAX ) { |
|
| 41 | - /** |
|
| 42 | - * WordPress 4.1.0 introduced allowing WP_Error objects to be |
|
| 43 | - * passed directly into `wp_send_json_error`. This shims in |
|
| 44 | - * compatibility for older versions. We're currently supporting 3.9+. |
|
| 45 | - */ |
|
| 46 | - if ( version_compare( $wp_version, '4.1.0', '<' ) ) { |
|
| 47 | - $result = array(); |
|
| 40 | + if ( defined( 'DOING_AJAX' ) && DOING_AJAX && defined( 'WOGH_AJAX' ) && WOGH_AJAX ) { |
|
| 41 | + /** |
|
| 42 | + * WordPress 4.1.0 introduced allowing WP_Error objects to be |
|
| 43 | + * passed directly into `wp_send_json_error`. This shims in |
|
| 44 | + * compatibility for older versions. We're currently supporting 3.9+. |
|
| 45 | + */ |
|
| 46 | + if ( version_compare( $wp_version, '4.1.0', '<' ) ) { |
|
| 47 | + $result = array(); |
|
| 48 | 48 | |
| 49 | - foreach ( $error->errors as $code => $messages ) { |
|
| 50 | - foreach ( $messages as $message ) { |
|
| 51 | - $result[] = array( 'code' => $code, 'message' => $message ); |
|
| 52 | - } |
|
| 53 | - } |
|
| 49 | + foreach ( $error->errors as $code => $messages ) { |
|
| 50 | + foreach ( $messages as $message ) { |
|
| 51 | + $result[] = array( 'code' => $code, 'message' => $message ); |
|
| 52 | + } |
|
| 53 | + } |
|
| 54 | 54 | |
| 55 | - $error = $result; |
|
| 56 | - } |
|
| 55 | + $error = $result; |
|
| 56 | + } |
|
| 57 | 57 | |
| 58 | - wp_send_json_error( $error ); |
|
| 59 | - } |
|
| 58 | + wp_send_json_error( $error ); |
|
| 59 | + } |
|
| 60 | 60 | |
| 61 | - return false; |
|
| 62 | - } |
|
| 61 | + return false; |
|
| 62 | + } |
|
| 63 | 63 | |
| 64 | - /** |
|
| 65 | - * Writes to the log and returns the success response. |
|
| 66 | - * |
|
| 67 | - * @param string $success Success message to respond with. |
|
| 68 | - * |
|
| 69 | - * @return true |
|
| 70 | - */ |
|
| 71 | - public function success( $success ) { |
|
| 72 | - $this->log( $success ); |
|
| 64 | + /** |
|
| 65 | + * Writes to the log and returns the success response. |
|
| 66 | + * |
|
| 67 | + * @param string $success Success message to respond with. |
|
| 68 | + * |
|
| 69 | + * @return true |
|
| 70 | + */ |
|
| 71 | + public function success( $success ) { |
|
| 72 | + $this->log( $success ); |
|
| 73 | 73 | |
| 74 | - if ( defined( 'DOING_AJAX' ) && DOING_AJAX && defined( 'WOGH_AJAX' ) && WOGH_AJAX ) { |
|
| 75 | - wp_send_json_success( $success ); |
|
| 76 | - } |
|
| 74 | + if ( defined( 'DOING_AJAX' ) && DOING_AJAX && defined( 'WOGH_AJAX' ) && WOGH_AJAX ) { |
|
| 75 | + wp_send_json_success( $success ); |
|
| 76 | + } |
|
| 77 | 77 | |
| 78 | - return true; |
|
| 79 | - } |
|
| 78 | + return true; |
|
| 79 | + } |
|
| 80 | 80 | |
| 81 | - /** |
|
| 82 | - * Writes a log message. |
|
| 83 | - * |
|
| 84 | - * Can extract a message from WP_Error object. |
|
| 85 | - * |
|
| 86 | - * @param string|WP_Error $msg Message to log. |
|
| 87 | - */ |
|
| 88 | - protected function log( $msg ) { |
|
| 89 | - if ( is_wp_error( $msg ) ) { |
|
| 90 | - $msg = $msg->get_error_message(); |
|
| 91 | - } |
|
| 81 | + /** |
|
| 82 | + * Writes a log message. |
|
| 83 | + * |
|
| 84 | + * Can extract a message from WP_Error object. |
|
| 85 | + * |
|
| 86 | + * @param string|WP_Error $msg Message to log. |
|
| 87 | + */ |
|
| 88 | + protected function log( $msg ) { |
|
| 89 | + if ( is_wp_error( $msg ) ) { |
|
| 90 | + $msg = $msg->get_error_message(); |
|
| 91 | + } |
|
| 92 | 92 | |
| 93 | - Writing_On_GitHub::write_log( $msg ); |
|
| 94 | - } |
|
| 93 | + Writing_On_GitHub::write_log( $msg ); |
|
| 94 | + } |
|
| 95 | 95 | } |
@@ -21,7 +21,7 @@ discard block |
||
| 21 | 21 | * |
| 22 | 22 | * @param Writing_On_GitHub $app Application container. |
| 23 | 23 | */ |
| 24 | - public function __construct( Writing_On_GitHub $app ) { |
|
| 24 | + public function __construct(Writing_On_GitHub $app) { |
|
| 25 | 25 | $this->app = $app; |
| 26 | 26 | } |
| 27 | 27 | |
@@ -32,30 +32,30 @@ discard block |
||
| 32 | 32 | * |
| 33 | 33 | * @return false |
| 34 | 34 | */ |
| 35 | - public function error( WP_Error $error ) { |
|
| 35 | + public function error(WP_Error $error) { |
|
| 36 | 36 | global $wp_version; |
| 37 | 37 | |
| 38 | - $this->log( $error ); |
|
| 38 | + $this->log($error); |
|
| 39 | 39 | |
| 40 | - if ( defined( 'DOING_AJAX' ) && DOING_AJAX && defined( 'WOGH_AJAX' ) && WOGH_AJAX ) { |
|
| 40 | + if (defined('DOING_AJAX') && DOING_AJAX && defined('WOGH_AJAX') && WOGH_AJAX) { |
|
| 41 | 41 | /** |
| 42 | 42 | * WordPress 4.1.0 introduced allowing WP_Error objects to be |
| 43 | 43 | * passed directly into `wp_send_json_error`. This shims in |
| 44 | 44 | * compatibility for older versions. We're currently supporting 3.9+. |
| 45 | 45 | */ |
| 46 | - if ( version_compare( $wp_version, '4.1.0', '<' ) ) { |
|
| 46 | + if (version_compare($wp_version, '4.1.0', '<')) { |
|
| 47 | 47 | $result = array(); |
| 48 | 48 | |
| 49 | - foreach ( $error->errors as $code => $messages ) { |
|
| 50 | - foreach ( $messages as $message ) { |
|
| 51 | - $result[] = array( 'code' => $code, 'message' => $message ); |
|
| 49 | + foreach ($error->errors as $code => $messages) { |
|
| 50 | + foreach ($messages as $message) { |
|
| 51 | + $result[] = array('code' => $code, 'message' => $message); |
|
| 52 | 52 | } |
| 53 | 53 | } |
| 54 | 54 | |
| 55 | 55 | $error = $result; |
| 56 | 56 | } |
| 57 | 57 | |
| 58 | - wp_send_json_error( $error ); |
|
| 58 | + wp_send_json_error($error); |
|
| 59 | 59 | } |
| 60 | 60 | |
| 61 | 61 | return false; |
@@ -68,11 +68,11 @@ discard block |
||
| 68 | 68 | * |
| 69 | 69 | * @return true |
| 70 | 70 | */ |
| 71 | - public function success( $success ) { |
|
| 72 | - $this->log( $success ); |
|
| 71 | + public function success($success) { |
|
| 72 | + $this->log($success); |
|
| 73 | 73 | |
| 74 | - if ( defined( 'DOING_AJAX' ) && DOING_AJAX && defined( 'WOGH_AJAX' ) && WOGH_AJAX ) { |
|
| 75 | - wp_send_json_success( $success ); |
|
| 74 | + if (defined('DOING_AJAX') && DOING_AJAX && defined('WOGH_AJAX') && WOGH_AJAX) { |
|
| 75 | + wp_send_json_success($success); |
|
| 76 | 76 | } |
| 77 | 77 | |
| 78 | 78 | return true; |
@@ -85,11 +85,11 @@ discard block |
||
| 85 | 85 | * |
| 86 | 86 | * @param string|WP_Error $msg Message to log. |
| 87 | 87 | */ |
| 88 | - protected function log( $msg ) { |
|
| 89 | - if ( is_wp_error( $msg ) ) { |
|
| 88 | + protected function log($msg) { |
|
| 89 | + if (is_wp_error($msg)) { |
|
| 90 | 90 | $msg = $msg->get_error_message(); |
| 91 | 91 | } |
| 92 | 92 | |
| 93 | - Writing_On_GitHub::write_log( $msg ); |
|
| 93 | + Writing_On_GitHub::write_log($msg); |
|
| 94 | 94 | } |
| 95 | 95 | } |
@@ -9,150 +9,150 @@ |
||
| 9 | 9 | */ |
| 10 | 10 | class Writing_On_GitHub_CLI extends WP_CLI_Command { |
| 11 | 11 | |
| 12 | - /** |
|
| 13 | - * Application container. |
|
| 14 | - * |
|
| 15 | - * @var Writing_On_GitHub |
|
| 16 | - */ |
|
| 17 | - protected $app; |
|
| 18 | - |
|
| 19 | - /** |
|
| 20 | - * Grab the Application container on instantiation. |
|
| 21 | - */ |
|
| 22 | - public function __construct() { |
|
| 23 | - $this->app = Writing_On_GitHub::$instance; |
|
| 24 | - } |
|
| 25 | - |
|
| 26 | - /** |
|
| 27 | - * Exports an individual post |
|
| 28 | - * all your posts to GitHub |
|
| 29 | - * |
|
| 30 | - * ## OPTIONS |
|
| 31 | - * |
|
| 32 | - * <post_id|all> |
|
| 33 | - * : The post ID to export or 'all' for full site |
|
| 34 | - * |
|
| 35 | - * <user_id> |
|
| 36 | - * : The user ID you'd like to save the commit as |
|
| 37 | - * |
|
| 38 | - * ## EXAMPLES |
|
| 39 | - * |
|
| 40 | - * wp wogh export all 1 |
|
| 41 | - * wp wogh export 1 1 |
|
| 42 | - * |
|
| 43 | - * @synopsis <post_id|all> <user_id> |
|
| 44 | - * |
|
| 45 | - * @param array $args Command arguments. |
|
| 46 | - */ |
|
| 47 | - public function export( $args ) { |
|
| 48 | - list( $post_id, $user_id ) = $args; |
|
| 49 | - |
|
| 50 | - if ( ! is_numeric( $user_id ) ) { |
|
| 51 | - WP_CLI::error( __( 'Invalid user ID', 'writing-on-github' ) ); |
|
| 52 | - } |
|
| 53 | - |
|
| 54 | - $this->app->export()->set_user( $user_id ); |
|
| 55 | - |
|
| 56 | - if ( 'all' === $post_id ) { |
|
| 57 | - WP_CLI::line( __( 'Starting full export to GitHub.', 'writing-on-github' ) ); |
|
| 58 | - $this->app->controller()->export_all(); |
|
| 59 | - } elseif ( is_numeric( $post_id ) ) { |
|
| 60 | - WP_CLI::line( |
|
| 61 | - sprintf( |
|
| 62 | - __( 'Exporting post ID to GitHub: %d', 'writing-on-github' ), |
|
| 63 | - $post_id |
|
| 64 | - ) |
|
| 65 | - ); |
|
| 66 | - $this->app->controller()->export_post( (int) $post_id ); |
|
| 67 | - } else { |
|
| 68 | - WP_CLI::error( __( 'Invalid post ID', 'writing-on-github' ) ); |
|
| 69 | - } |
|
| 70 | - } |
|
| 71 | - |
|
| 72 | - /** |
|
| 73 | - * Imports the post in your GitHub repo |
|
| 74 | - * into your WordPress blog |
|
| 75 | - * |
|
| 76 | - * ## OPTIONS |
|
| 77 | - * |
|
| 78 | - * <user_id> |
|
| 79 | - * : The user ID you'd like to save the commit as |
|
| 80 | - * |
|
| 81 | - * ## EXAMPLES |
|
| 82 | - * |
|
| 83 | - * wp wogh import 1 |
|
| 84 | - * |
|
| 85 | - * @synopsis <user_id> |
|
| 86 | - * |
|
| 87 | - * @param array $args Command arguments. |
|
| 88 | - */ |
|
| 89 | - public function import( $args ) { |
|
| 90 | - list( $user_id ) = $args; |
|
| 91 | - |
|
| 92 | - if ( ! is_numeric( $user_id ) ) { |
|
| 93 | - WP_CLI::error( __( 'Invalid user ID', 'writing-on-github' ) ); |
|
| 94 | - } |
|
| 95 | - |
|
| 96 | - update_option( '_wogh_export_user_id', (int) $user_id ); |
|
| 97 | - |
|
| 98 | - WP_CLI::line( __( 'Starting import from GitHub.', 'writing-on-github' ) ); |
|
| 99 | - |
|
| 100 | - $this->app->controller()->import_master(); |
|
| 101 | - } |
|
| 102 | - |
|
| 103 | - /** |
|
| 104 | - * Fetches the provided sha or the repository's |
|
| 105 | - * master branch and caches it. |
|
| 106 | - * |
|
| 107 | - * ## OPTIONS |
|
| 108 | - * |
|
| 109 | - * <user_id> |
|
| 110 | - * : The user ID you'd like to save the commit as |
|
| 111 | - * |
|
| 112 | - * ## EXAMPLES |
|
| 113 | - * |
|
| 114 | - * wp wogh prime --branch=master |
|
| 115 | - * wp wogh prime --sha=<commit_sha> |
|
| 116 | - * |
|
| 117 | - * @synopsis [--sha=<commit_sha>] [--branch] |
|
| 118 | - * |
|
| 119 | - * @param array $args Command arguments. |
|
| 120 | - * @param array $assoc_args Command associated arguments. |
|
| 121 | - */ |
|
| 122 | - public function prime( $args, $assoc_args ) { |
|
| 123 | - if ( isset( $assoc_args['branch'] ) ) { |
|
| 124 | - WP_CLI::line( __( 'Starting branch import.', 'writing-on-github' ) ); |
|
| 125 | - |
|
| 126 | - $commit = $this->app->api()->fetch()->master(); |
|
| 127 | - |
|
| 128 | - if ( is_wp_error( $commit ) ) { |
|
| 129 | - WP_CLI::error( |
|
| 130 | - sprintf( |
|
| 131 | - __( 'Failed to import and cache branch with error: %s', 'writing-on-github' ), |
|
| 132 | - $commit->get_error_message() |
|
| 133 | - ) |
|
| 134 | - ); |
|
| 135 | - } else { |
|
| 136 | - WP_CLI::success( |
|
| 137 | - sprintf( |
|
| 138 | - __( 'Successfully imported and cached commit %s from branch.', 'writing-on-github' ), |
|
| 139 | - $commit->sha() |
|
| 140 | - ) |
|
| 141 | - ); |
|
| 142 | - } |
|
| 143 | - } else if ( isset( $assoc_args['sha'] ) ) { |
|
| 144 | - WP_CLI::line( 'Starting sha import.' ); |
|
| 145 | - |
|
| 146 | - $commit = $this->app->api()->fetch()->commit( $assoc_args['sha'] ); |
|
| 147 | - |
|
| 148 | - WP_CLI::success( |
|
| 149 | - sprintf( |
|
| 150 | - __( 'Successfully imported and cached commit %s.', 'writing-on-github' ), |
|
| 151 | - $commit->sha() |
|
| 152 | - ) |
|
| 153 | - ); |
|
| 154 | - } else { |
|
| 155 | - WP_CLI::error( 'Invalid fetch.' ); |
|
| 156 | - } |
|
| 157 | - } |
|
| 12 | + /** |
|
| 13 | + * Application container. |
|
| 14 | + * |
|
| 15 | + * @var Writing_On_GitHub |
|
| 16 | + */ |
|
| 17 | + protected $app; |
|
| 18 | + |
|
| 19 | + /** |
|
| 20 | + * Grab the Application container on instantiation. |
|
| 21 | + */ |
|
| 22 | + public function __construct() { |
|
| 23 | + $this->app = Writing_On_GitHub::$instance; |
|
| 24 | + } |
|
| 25 | + |
|
| 26 | + /** |
|
| 27 | + * Exports an individual post |
|
| 28 | + * all your posts to GitHub |
|
| 29 | + * |
|
| 30 | + * ## OPTIONS |
|
| 31 | + * |
|
| 32 | + * <post_id|all> |
|
| 33 | + * : The post ID to export or 'all' for full site |
|
| 34 | + * |
|
| 35 | + * <user_id> |
|
| 36 | + * : The user ID you'd like to save the commit as |
|
| 37 | + * |
|
| 38 | + * ## EXAMPLES |
|
| 39 | + * |
|
| 40 | + * wp wogh export all 1 |
|
| 41 | + * wp wogh export 1 1 |
|
| 42 | + * |
|
| 43 | + * @synopsis <post_id|all> <user_id> |
|
| 44 | + * |
|
| 45 | + * @param array $args Command arguments. |
|
| 46 | + */ |
|
| 47 | + public function export( $args ) { |
|
| 48 | + list( $post_id, $user_id ) = $args; |
|
| 49 | + |
|
| 50 | + if ( ! is_numeric( $user_id ) ) { |
|
| 51 | + WP_CLI::error( __( 'Invalid user ID', 'writing-on-github' ) ); |
|
| 52 | + } |
|
| 53 | + |
|
| 54 | + $this->app->export()->set_user( $user_id ); |
|
| 55 | + |
|
| 56 | + if ( 'all' === $post_id ) { |
|
| 57 | + WP_CLI::line( __( 'Starting full export to GitHub.', 'writing-on-github' ) ); |
|
| 58 | + $this->app->controller()->export_all(); |
|
| 59 | + } elseif ( is_numeric( $post_id ) ) { |
|
| 60 | + WP_CLI::line( |
|
| 61 | + sprintf( |
|
| 62 | + __( 'Exporting post ID to GitHub: %d', 'writing-on-github' ), |
|
| 63 | + $post_id |
|
| 64 | + ) |
|
| 65 | + ); |
|
| 66 | + $this->app->controller()->export_post( (int) $post_id ); |
|
| 67 | + } else { |
|
| 68 | + WP_CLI::error( __( 'Invalid post ID', 'writing-on-github' ) ); |
|
| 69 | + } |
|
| 70 | + } |
|
| 71 | + |
|
| 72 | + /** |
|
| 73 | + * Imports the post in your GitHub repo |
|
| 74 | + * into your WordPress blog |
|
| 75 | + * |
|
| 76 | + * ## OPTIONS |
|
| 77 | + * |
|
| 78 | + * <user_id> |
|
| 79 | + * : The user ID you'd like to save the commit as |
|
| 80 | + * |
|
| 81 | + * ## EXAMPLES |
|
| 82 | + * |
|
| 83 | + * wp wogh import 1 |
|
| 84 | + * |
|
| 85 | + * @synopsis <user_id> |
|
| 86 | + * |
|
| 87 | + * @param array $args Command arguments. |
|
| 88 | + */ |
|
| 89 | + public function import( $args ) { |
|
| 90 | + list( $user_id ) = $args; |
|
| 91 | + |
|
| 92 | + if ( ! is_numeric( $user_id ) ) { |
|
| 93 | + WP_CLI::error( __( 'Invalid user ID', 'writing-on-github' ) ); |
|
| 94 | + } |
|
| 95 | + |
|
| 96 | + update_option( '_wogh_export_user_id', (int) $user_id ); |
|
| 97 | + |
|
| 98 | + WP_CLI::line( __( 'Starting import from GitHub.', 'writing-on-github' ) ); |
|
| 99 | + |
|
| 100 | + $this->app->controller()->import_master(); |
|
| 101 | + } |
|
| 102 | + |
|
| 103 | + /** |
|
| 104 | + * Fetches the provided sha or the repository's |
|
| 105 | + * master branch and caches it. |
|
| 106 | + * |
|
| 107 | + * ## OPTIONS |
|
| 108 | + * |
|
| 109 | + * <user_id> |
|
| 110 | + * : The user ID you'd like to save the commit as |
|
| 111 | + * |
|
| 112 | + * ## EXAMPLES |
|
| 113 | + * |
|
| 114 | + * wp wogh prime --branch=master |
|
| 115 | + * wp wogh prime --sha=<commit_sha> |
|
| 116 | + * |
|
| 117 | + * @synopsis [--sha=<commit_sha>] [--branch] |
|
| 118 | + * |
|
| 119 | + * @param array $args Command arguments. |
|
| 120 | + * @param array $assoc_args Command associated arguments. |
|
| 121 | + */ |
|
| 122 | + public function prime( $args, $assoc_args ) { |
|
| 123 | + if ( isset( $assoc_args['branch'] ) ) { |
|
| 124 | + WP_CLI::line( __( 'Starting branch import.', 'writing-on-github' ) ); |
|
| 125 | + |
|
| 126 | + $commit = $this->app->api()->fetch()->master(); |
|
| 127 | + |
|
| 128 | + if ( is_wp_error( $commit ) ) { |
|
| 129 | + WP_CLI::error( |
|
| 130 | + sprintf( |
|
| 131 | + __( 'Failed to import and cache branch with error: %s', 'writing-on-github' ), |
|
| 132 | + $commit->get_error_message() |
|
| 133 | + ) |
|
| 134 | + ); |
|
| 135 | + } else { |
|
| 136 | + WP_CLI::success( |
|
| 137 | + sprintf( |
|
| 138 | + __( 'Successfully imported and cached commit %s from branch.', 'writing-on-github' ), |
|
| 139 | + $commit->sha() |
|
| 140 | + ) |
|
| 141 | + ); |
|
| 142 | + } |
|
| 143 | + } else if ( isset( $assoc_args['sha'] ) ) { |
|
| 144 | + WP_CLI::line( 'Starting sha import.' ); |
|
| 145 | + |
|
| 146 | + $commit = $this->app->api()->fetch()->commit( $assoc_args['sha'] ); |
|
| 147 | + |
|
| 148 | + WP_CLI::success( |
|
| 149 | + sprintf( |
|
| 150 | + __( 'Successfully imported and cached commit %s.', 'writing-on-github' ), |
|
| 151 | + $commit->sha() |
|
| 152 | + ) |
|
| 153 | + ); |
|
| 154 | + } else { |
|
| 155 | + WP_CLI::error( 'Invalid fetch.' ); |
|
| 156 | + } |
|
| 157 | + } |
|
| 158 | 158 | } |
@@ -44,28 +44,28 @@ discard block |
||
| 44 | 44 | * |
| 45 | 45 | * @param array $args Command arguments. |
| 46 | 46 | */ |
| 47 | - public function export( $args ) { |
|
| 48 | - list( $post_id, $user_id ) = $args; |
|
| 47 | + public function export($args) { |
|
| 48 | + list($post_id, $user_id) = $args; |
|
| 49 | 49 | |
| 50 | - if ( ! is_numeric( $user_id ) ) { |
|
| 51 | - WP_CLI::error( __( 'Invalid user ID', 'writing-on-github' ) ); |
|
| 50 | + if ( ! is_numeric($user_id)) { |
|
| 51 | + WP_CLI::error(__('Invalid user ID', 'writing-on-github')); |
|
| 52 | 52 | } |
| 53 | 53 | |
| 54 | - $this->app->export()->set_user( $user_id ); |
|
| 54 | + $this->app->export()->set_user($user_id); |
|
| 55 | 55 | |
| 56 | - if ( 'all' === $post_id ) { |
|
| 57 | - WP_CLI::line( __( 'Starting full export to GitHub.', 'writing-on-github' ) ); |
|
| 56 | + if ('all' === $post_id) { |
|
| 57 | + WP_CLI::line(__('Starting full export to GitHub.', 'writing-on-github')); |
|
| 58 | 58 | $this->app->controller()->export_all(); |
| 59 | - } elseif ( is_numeric( $post_id ) ) { |
|
| 59 | + } elseif (is_numeric($post_id)) { |
|
| 60 | 60 | WP_CLI::line( |
| 61 | 61 | sprintf( |
| 62 | - __( 'Exporting post ID to GitHub: %d', 'writing-on-github' ), |
|
| 62 | + __('Exporting post ID to GitHub: %d', 'writing-on-github'), |
|
| 63 | 63 | $post_id |
| 64 | 64 | ) |
| 65 | 65 | ); |
| 66 | - $this->app->controller()->export_post( (int) $post_id ); |
|
| 66 | + $this->app->controller()->export_post((int) $post_id); |
|
| 67 | 67 | } else { |
| 68 | - WP_CLI::error( __( 'Invalid post ID', 'writing-on-github' ) ); |
|
| 68 | + WP_CLI::error(__('Invalid post ID', 'writing-on-github')); |
|
| 69 | 69 | } |
| 70 | 70 | } |
| 71 | 71 | |
@@ -86,16 +86,16 @@ discard block |
||
| 86 | 86 | * |
| 87 | 87 | * @param array $args Command arguments. |
| 88 | 88 | */ |
| 89 | - public function import( $args ) { |
|
| 90 | - list( $user_id ) = $args; |
|
| 89 | + public function import($args) { |
|
| 90 | + list($user_id) = $args; |
|
| 91 | 91 | |
| 92 | - if ( ! is_numeric( $user_id ) ) { |
|
| 93 | - WP_CLI::error( __( 'Invalid user ID', 'writing-on-github' ) ); |
|
| 92 | + if ( ! is_numeric($user_id)) { |
|
| 93 | + WP_CLI::error(__('Invalid user ID', 'writing-on-github')); |
|
| 94 | 94 | } |
| 95 | 95 | |
| 96 | - update_option( '_wogh_export_user_id', (int) $user_id ); |
|
| 96 | + update_option('_wogh_export_user_id', (int) $user_id); |
|
| 97 | 97 | |
| 98 | - WP_CLI::line( __( 'Starting import from GitHub.', 'writing-on-github' ) ); |
|
| 98 | + WP_CLI::line(__('Starting import from GitHub.', 'writing-on-github')); |
|
| 99 | 99 | |
| 100 | 100 | $this->app->controller()->import_master(); |
| 101 | 101 | } |
@@ -119,40 +119,40 @@ discard block |
||
| 119 | 119 | * @param array $args Command arguments. |
| 120 | 120 | * @param array $assoc_args Command associated arguments. |
| 121 | 121 | */ |
| 122 | - public function prime( $args, $assoc_args ) { |
|
| 123 | - if ( isset( $assoc_args['branch'] ) ) { |
|
| 124 | - WP_CLI::line( __( 'Starting branch import.', 'writing-on-github' ) ); |
|
| 122 | + public function prime($args, $assoc_args) { |
|
| 123 | + if (isset($assoc_args['branch'])) { |
|
| 124 | + WP_CLI::line(__('Starting branch import.', 'writing-on-github')); |
|
| 125 | 125 | |
| 126 | 126 | $commit = $this->app->api()->fetch()->master(); |
| 127 | 127 | |
| 128 | - if ( is_wp_error( $commit ) ) { |
|
| 128 | + if (is_wp_error($commit)) { |
|
| 129 | 129 | WP_CLI::error( |
| 130 | 130 | sprintf( |
| 131 | - __( 'Failed to import and cache branch with error: %s', 'writing-on-github' ), |
|
| 131 | + __('Failed to import and cache branch with error: %s', 'writing-on-github'), |
|
| 132 | 132 | $commit->get_error_message() |
| 133 | 133 | ) |
| 134 | 134 | ); |
| 135 | 135 | } else { |
| 136 | 136 | WP_CLI::success( |
| 137 | 137 | sprintf( |
| 138 | - __( 'Successfully imported and cached commit %s from branch.', 'writing-on-github' ), |
|
| 138 | + __('Successfully imported and cached commit %s from branch.', 'writing-on-github'), |
|
| 139 | 139 | $commit->sha() |
| 140 | 140 | ) |
| 141 | 141 | ); |
| 142 | 142 | } |
| 143 | - } else if ( isset( $assoc_args['sha'] ) ) { |
|
| 144 | - WP_CLI::line( 'Starting sha import.' ); |
|
| 143 | + } else if (isset($assoc_args['sha'])) { |
|
| 144 | + WP_CLI::line('Starting sha import.'); |
|
| 145 | 145 | |
| 146 | - $commit = $this->app->api()->fetch()->commit( $assoc_args['sha'] ); |
|
| 146 | + $commit = $this->app->api()->fetch()->commit($assoc_args['sha']); |
|
| 147 | 147 | |
| 148 | 148 | WP_CLI::success( |
| 149 | 149 | sprintf( |
| 150 | - __( 'Successfully imported and cached commit %s.', 'writing-on-github' ), |
|
| 150 | + __('Successfully imported and cached commit %s.', 'writing-on-github'), |
|
| 151 | 151 | $commit->sha() |
| 152 | 152 | ) |
| 153 | 153 | ); |
| 154 | 154 | } else { |
| 155 | - WP_CLI::error( 'Invalid fetch.' ); |
|
| 155 | + WP_CLI::error('Invalid fetch.'); |
|
| 156 | 156 | } |
| 157 | 157 | } |
| 158 | 158 | } |