Merge 7cb29e8 into 7bf793d

xwp · Aug 6, 2020 · b3630a6 · b3630a6
2 parents 7bf793d + 7cb29e8
commit b3630a6
Show file tree

Hide file tree

Showing 2 changed files with 25 additions and 6 deletions.
diff --git a/php/class-settings.php b/php/class-settings.php
@@ -444,14 +444,13 @@ public function get_access_token( $code ) {
 	 * @return mixed
 	 */
 	public function get_client_id( $access_token ) {
-		$url      = get_home_url( null, '/' );
-		$name     = get_bloginfo( 'name' );
-		$response = wp_remote_post(
+		$site_data = $this->get_site_data();
+		$response  = wp_remote_post(
 			'https://api.unsplash.com/clients',
 			[
 				'body'    => [
-					'name'        => $name,
-					'description' => sprintf( 'Wordpress Oauth Client application for: %1$s - %2$s', $name, $url ),
+					'name'        => $site_data['name'],
+					'description' => sprintf( 'Wordpress Oauth Client application for: %1$s - %2$s', $site_data['name'], $site_data['url'] ),
 				],
 				'headers' => [
 					'Authorization' => 'Bearer ' . $access_token,
@@ -504,7 +503,8 @@ public function access_key_render() {
 	 */
 	public function get_credentials() {
 		$options        = get_option( 'unsplash_settings' );
-		$site_name_slug = sanitize_title_with_dashes( get_bloginfo( 'name' ) );
+		$site_data      = $this->get_site_data();
+		$site_name_slug = sanitize_title_with_dashes( $site_data['name'] );
 
 		$credentials = [
 			'applicationId' => ! empty( $options['access_key'] ) ? $this->decrypt( $options['access_key'] ) : getenv( 'UNSPLASH_ACCESS_KEY' ),
@@ -521,4 +521,22 @@ public function get_credentials() {
 
 		return $credentials;
 	}
+
+	/**
+	 * Get site name and url. If site name is empty, fallback to domain with dashes.
+	 *
+	 * @return array
+	 */
+	public function get_site_data() {
+		$data = [
+			'url'  => get_home_url( null, '/' ),
+			'name' => get_bloginfo( 'name' ),
+		];
+		if ( ! $data['name'] && $data['url'] ) {
+			$url          = wp_parse_url( $data['url'] );
+			$data['name'] = sanitize_title_with_dashes( $url['host'] );
+		}
+
+		return $data;
+	}
 }
diff --git a/tests/phpunit/php/class-test-settings.php b/tests/phpunit/php/class-test-settings.php
@@ -704,6 +704,7 @@ public function test_access_key_render() {
 	 * Test get_credentials.
 	 *
 	 * @covers ::get_credentials()
+	 * @covers ::get_site_data()
 	 */
 	public function test_get_credentials() {
 		$credentials = $this->settings->get_credentials();