Skip to content

Commit 890a19b

Browse files
authored
Merge pull request #1253 from hydephp/add-default-rss-feed-value-to-config-stub
Add default RSS feed description value to config stub
2 parents 98be626 + 7524c67 commit 890a19b

File tree

5 files changed

+9
-5
lines changed

5 files changed

+9
-5
lines changed

RELEASE_NOTES.md

+1-1
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@ This serves two purposes:
1313
- for new features.
1414

1515
### Changed
16-
- for changes in existing functionality.
16+
- Added default RSS feed description value to config stub.
1717

1818
### Deprecated
1919
- for soon-to-be removed features.

config/hyde.php

+2-2
Original file line numberDiff line numberDiff line change
@@ -114,8 +114,8 @@
114114
// What filename should the RSS file use?
115115
'rss_filename' => 'feed.xml',
116116

117-
// The channel description. By default this is "Site Name + RSS Feed".
118-
// 'rss_description' => '',
117+
// The channel description.
118+
'rss_description' => env('SITE_NAME', 'HydePHP').' RSS Feed',
119119

120120
/*
121121
|--------------------------------------------------------------------------

packages/framework/config/hyde.php

+2-2
Original file line numberDiff line numberDiff line change
@@ -114,8 +114,8 @@
114114
// What filename should the RSS file use?
115115
'rss_filename' => 'feed.xml',
116116

117-
// The channel description. By default this is "Site Name + RSS Feed".
118-
// 'rss_description' => '',
117+
// The channel description.
118+
'rss_description' => env('SITE_NAME', 'HydePHP').' RSS Feed',
119119

120120
/*
121121
|--------------------------------------------------------------------------

packages/framework/tests/Feature/GlobalMetadataBagTest.php

+3
Original file line numberDiff line numberDiff line change
@@ -84,6 +84,9 @@ public function test_site_metadata_rss_feed_uses_configured_site_name()
8484
config(['hyde.url' => 'foo']);
8585
config(['hyde.name' => 'Site']);
8686
config(['hyde.generate_rss_feed' => true]);
87+
$config = config('hyde');
88+
unset($config['rss_description']);
89+
config(['hyde' => $config]);
8790
$this->file('_posts/foo.md');
8891

8992
$this->assertEquals('<link rel="alternate" href="foo/feed.xml" type="application/rss+xml" title="Site RSS Feed">', GlobalMetadataBag::make()->render());

packages/framework/tests/Feature/Services/RssFeedServiceTest.php

+1
Original file line numberDiff line numberDiff line change
@@ -39,6 +39,7 @@ public function test_xml_channel_element_has_required_elements()
3939
{
4040
config(['hyde.name' => 'Test Blog']);
4141
config(['hyde.url' => 'https://example.com']);
42+
config(['hyde.rss_description' => 'Test Blog RSS Feed']);
4243

4344
$service = new RssFeedGenerator();
4445
$this->assertTrue(property_exists($service->getXmlElement()->channel, 'title'));

0 commit comments

Comments
 (0)