- Issue created by @thatguy
- 🇫🇮Finland thatguy
Patch based on the core issue where the header is added in the render-function
- Status changed to Needs review
10 months ago 8:13am 6 March 2024 - Open on Drupal.org →Core: 10.2.x + Environment: PHP 8.2 & MySQL 8last update
10 months ago Waiting for branch to pass - 🇺🇸United States DamienMcKenna NH, USA
Thanks for opening the issue and for providing the patch.
I think the indentation needs to be adjusted, but otherwise it looks fine.
The last submitted patch, 2: views-rss-content-type-header-fix-3425960-2.patch, failed testing. View results →
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.- Status changed to Needs review
10 months ago 3:34pm 28 March 2024 - Open on Drupal.org →Core: 10.2.x + Environment: PHP 8.2 & MySQL 8last update
10 months ago Waiting for branch to pass - Status changed to RTBC
8 months ago 8:03am 17 May 2024 Tested patch #6 with D10.2.5, php8.2 and views_rss 2.2, it works for me as well.
-
DamienMcKenna →
committed fedbbf0b on 8.x-2.x
Issue #3425960 by thatguy, edwardsay, ivnish, DamienMcKenna, misterdidi...
-
DamienMcKenna →
committed fedbbf0b on 8.x-2.x
- Status changed to Fixed
8 months ago 9:23pm 23 May 2024 Automatically closed - issue fixed for 2 weeks with no activity.