Merge pull request #22 from anatolinicolae/patch-1

fix: empty option on fresh install leads to broken feed links

Author: pfefferle

includes/class-pubsubhubbub-admin.php

@@ -38,7 +38,7 @@ public static function register_settings() {
 				'type'              => 'string',
 				'description'       => __( 'The WebSub/PubSubHubbub endpoints', 'pubsubhubbub' ),
 				'show_in_rest'      => true,
-				'default'           => "https://pubsubhubbub.appspot.com\nhttps://pubsubhubbub.superfeedr.com\nhttps://websubhub.com/hub",
+				'default'           => implode( PHP_EOL, Pubsubhubbub::DEFAULT_HUBS ),
 				'sanitize_callback' => function ( $value ) {
 					$value = explode( PHP_EOL, $value );
 					$value = array_filter( array_map( 'trim', $value ) );

includes/class-pubsubhubbub-publisher.php

@@ -117,7 +117,13 @@ public static function publish_to_hub( $feed_urls ) {
 	 */
 	public static function get_hubs() {
 		$endpoints = get_option( 'pubsubhubbub_endpoints' );
-		$hub_urls  = explode( PHP_EOL, $endpoints );
+
+		// Fail with an empty array when option retrieval fails
+		if ( ! $endpoints ) {
+			$hub_urls = Pubsubhubbub::DEFAULT_HUBS;
+		} else {
+			$hub_urls = explode( PHP_EOL, $endpoints );
+		}
 
 		return apply_filters( 'pubsubhubbub_hub_urls', $hub_urls );
 	}

includes/class-pubsubhubbub.php

@@ -3,6 +3,12 @@
  * The WebSub/PubSubHubbub class
  */
 class Pubsubhubbub {
+	const DEFAULT_HUBS = array(
+		'https://pubsubhubbub.appspot.com',
+		'https://pubsubhubbub.superfeedr.com',
+		'https://websubhub.com/hub',
+	);
+
 	/**
 	 * Load the plugin textdomain.
 	 */

pubsubhubbub.php

@@ -31,7 +31,7 @@ function pubsubhubbub_init() {
 	 */
 	require_once( dirname( __FILE__ ) . '/includes/class-pubsubhubbub-admin.php' );
 
-	add_action( 'admin_init', array( 'PubSubHubbub_Admin', 'register_settings' ) );
+	add_action( 'init', array( 'PubSubHubbub_Admin', 'register_settings' ) );
 	add_action( 'admin_menu', array( 'Pubsubhubbub_Admin', 'add_plugin_menu' ) );
 
 	/**