Merge pull request #279 from 10up/fix/rss-title-filter

Fix feed title output & allow feed title to be filtered.

README.md

@@ -111,6 +111,30 @@ function podcasting_feed_episodes_per_page( $qty ) {
 
 ```
 
+## Customize the RSS feed title
+
+The `<title>` element of the RSS feed can be adjusted using the `simple_podcasting_feed_title` filter.
+
+```php
+<?php
+
+add_filter( 'simple_podcasting_feed_title', 'podcasting_feed_update_feed_title', 10, 2 );
+
+/**
+ * Filter the name of the of the feed channel
+ *
+ * @param $output Output to be modified.
+ * @param $term WP_Term object representing the podcast
+ * @return string
+ */
+function podcasting_feed_update_feed_title( $output, $term ) {
+	$term_name = $term->name;
+
+	return '10up Presents: ' . $term_name;
+}
+
+```
+
 ## Customize RSS feed
 
 If you want to modify RSS feed items output, there is a filter for that:

includes/customize-feed.php

@@ -45,12 +45,13 @@ function bloginfo_rss_name( $output ) {
 	$title = get_term_meta( $term->term_id, 'podcasting_title', true );
 	if ( empty( $title ) ) {
 		$title = get_bloginfo( 'name' );
-		$title = "$title » {$term->name}";
+		$output = "$title » {$term->name}";
 	} else {
 		$output = $title;
 	}
 
-	return $output;
+	return apply_filters( 'simple_podcasting_feed_title', $output, $term );
+
 }
 add_filter( 'wp_title_rss', __NAMESPACE__ . '\bloginfo_rss_name' );
 

tests/unit/test-customize-feed.php

@@ -88,7 +88,7 @@ public function test_pre_get_posts_no_feed() {
 	}
 
 	public function test_pre_get_posts_feed() {
-		 $query_mock = \Mockery::mock( '\WP_Query' );
+		$query_mock = \Mockery::mock( '\WP_Query' );
 
 		$query_mock->shouldReceive( 'is_feed' )->andReturn( true );
 
@@ -176,6 +176,31 @@ function ( $text, $num_words, $more ) {
 		}
 	}
 
+	public function test_rss_title_can_be_filtered() {
+		$queried_object = (object) array(
+			'term_id' => 42,
+			'name' => 'Original Podcast Name'
+		);
+		\WP_Mock::userFunction( 'get_queried_object' )
+			->andReturn( $queried_object );
+
+		\WP_Mock::userFunction( 'get_bloginfo' )
+			->with( 'name' )
+			->andReturn( 'Blogname' );
+
+
+		\WP_Mock::onFilter( 'simple_podcasting_feed_title' )
+			->with( 'Blogname » Original Podcast Name', $queried_object )
+			->reply( 'Filtered Podcast Title' );
+
+		$this->assertEquals(
+			'Filtered Podcast Title',
+			tenup_podcasting\bloginfo_rss_name( 'Podcast Title' ),
+			'tenup_podcasting\bloginfo_rss_name() should return the filtered value.'
+		);
+
+	}
+
 	public function data_provider_for_test_feed_item() {
 		return array(
 			'Term not found'              => array(