LICENS-45 Resovle comments

src/Uplink/API/Validation_Response.php

@@ -497,13 +497,13 @@ public function to_wp_format() {
 		if ( empty( $this->response->auth_required ) || $this->resource->has_valid_auth_token( (array) $this->response->origin ) ) {
 			$info->download_link = isset($this->response->download_url) ? $this->response->download_url . '&pu_get_download=1' : '';
 		} else {
-			$url 		  = $this->response->origin->url;
+			$url          = $this->response->origin->url;
 			$query_params = [
 				'callback_uri' => urlencode( sprintf( '%s/%s', get_site_url(), Namespaces::get_hook_name( 'connect', '%TEXTDOMAIN%' ) ) ),
 				'refer'		   => urlencode( wp_get_referer() ),
 			];
 
-			$url 		  		 = sprintf( '%s/%s?%s', $url, Namespaces::get_hook_name( 'oauth_connect/login' ) , http_build_query( $query_params ) );
+			$url                 = sprintf( '%s/%s?%s', $url, Namespaces::get_hook_name( 'oauth_connect/login' ) , http_build_query( $query_params ) );
 			$info->api_upgrade   = sprintf(
 				esc_html__( 'Please <a href="%s">authenticate this plugin</a> to receive updates.', '%TEXTDOMAIN%' ),
 				$url

src/Uplink/Admin/Actions.php

@@ -13,17 +13,27 @@ class Actions {
 	const QUERY_VAR = 'stellarwp_action';
 
 	/**
+	 * Register handle route for connect/disconnect
+	 *
+	 * @since 1.0.1
+	 *
 	 * @return void
+	 *
 	 * @action init
 	 */
 	public function register_route() {
 		add_rewrite_endpoint( 'stellarwp', EP_ROOT, self::QUERY_VAR );
 	}
 
 	/**
+	 * Handle auth connect and disconnect request
+	 *
+	 * @since 1.0.1
+	 *
 	 * @param \WP $wp
 	 *
 	 * @return void
+	 *
 	 * @action parse_request
 	 */
 	public function handle_auth_request( $wp ) {
@@ -41,7 +51,9 @@ public function handle_auth_request( $wp ) {
 	}
 
 	/**
-	 * Remove auth tokens
+	 * Remove auth tokens and redirect back to settings page
+	 *
+	 * @since 1.0.1
 	 */
 	public function handle_disconnect() {
 		$license = $this->get_license_object();
@@ -56,6 +68,13 @@ public function handle_disconnect() {
 		exit();
 	}
 
+	/**
+	 * Save auth token and redirect back to referer URL
+	 *
+	 * @since 1.0.1
+	 *
+	 * @param array $args
+	 */
 	public function handle_connect( $args ) {
 		if ( empty( $args['token'] ) ) {
 			$url = $this->get_origin_url();
@@ -83,6 +102,13 @@ public function handle_connect( $args ) {
 		exit();
 	}
 
+	/**
+	 * Retrieve origin URL from server
+	 *
+	 * @since 1.0.1
+	 *
+	 * @return string
+	 */
 	protected function get_origin_url() {
 		$license = $this->get_license_object();
 		$api     = Config::get_container()->get( API\Client::class );
@@ -95,6 +121,13 @@ protected function get_origin_url() {
 		return '';
 	}
 
+	/**
+	 * Retrieve License
+	 *
+	 * @since 1.0.1
+	 *
+	 * @return mixed
+	 */
 	protected function get_license_object() {
 		$collection = Config::get_container()->get( Collection::class );
 		$plugin     = $collection->current();

src/Uplink/Admin/Auth.php

@@ -8,6 +8,13 @@
 
 class Auth {
 
+	/**
+	 * Return auth button html
+	 *
+	 * @since 1.0.1
+	 *
+	 * @return mixed
+	 */
 	public static function do_auth_html() {
 		$collection = Config::get_container()->get( Collection::class );
 		$plugin     = $collection->current();

src/Uplink/Admin/Provider.php

@@ -66,10 +66,12 @@ public function filter_plugins_api( $result, $action, $args ) {
 	/**
 	 * Handle auth and disconnect requests to origin
 	 *
+	 * @since 1.0.1
+	 *
 	 * @param \WP $wp
 	 */
 	public function auth_request( $wp ) {
-		if ( ! is_user_logged_in() ) {
+		if ( ! is_user_logged_in() || ! current_user_can( 'manage_options' ) ) {
 			return;
 		}
 		$this->container->get( Actions::class )->handle_auth_request( $wp );