Feature PR-5.4-dev Pending

User tests: Successful: Unsuccessful:

avatar CSGoat0
CSGoat0
10 May 2026

Pull Request resolves #41445

  • I read the Generative AI policy and my contribution is either not created with the help of AI or is compatible with the policy and GNU/GPL 2 or later.

Summary of Changes

Modified the category feed view to respect the Show Intro Text article parameter when Include in Feed = Full Text is selected.

File changed: components/com_content/views/category/FeedView.php (line 60)

Before:

$item->description .= ($params->get('feed_summary', 0) ? $item->introtext . $item->fulltext : $item->introtext);

After:

$item->description .= $item->params->get('show_intro', 1) ? $item->introtext : '';
$item->description .= $params->get('feed_summary', 0) ? $item->fulltext : '';

Previously, the feed ignored the Show Intro Text = Hide setting and always included intro text when Full Text was selected. Now the feed checks the article's show_intro parameter and only includes intro text when it is set to Show.

Testing Instructions

Prerequisites:

  • Joomla 5.4 or later
  • Category blog menu item created
  • RSS feeds enabled

Step 1: Configure Article Options

  1. Log in to Joomla Administrator
  2. Navigate to Content → Articles → Options
  3. Click the Articles tab
  4. Set Show Intro Text to Hide
  5. Click the Integration tab
  6. Set Include in Feed to Full Text
  7. Click Save & Close

Step 2: Create an article with Read More

  1. Navigate to Content → Articles
  2. Click New
  3. Title: "Test RSS Article"
  4. In the content area:
    • Type "AA" (intro text)
    • Click the Read More toggle (CMS Content → Read More)
    • After the read more break, type "BB" (full text)
  5. Click Save & Close

Step 3: Access the RSS feed

Use the URL of your category blog menu item with feed parameters, for example:

http://localhost/your-category-alias?format=feed&type=rss

Or use the direct route:

http://localhost/index.php?option=com_content&view=category&id=YOUR_CATEGORY_ID&format=feed&type=rss

Step 4: Observe the feed output

Look at the <description> tag of your article in the feed.

Actual result BEFORE applying this Pull Request

<description><![CDATA[<p>AA</p>

<p>BB</p>]]></description>

Both intro text (AA) and full text (BB) appear, even though Show Intro Text = Hide is selected.

Expected result AFTER applying this Pull Request

<description><![CDATA[<p>BB</p>]]></description>

Only the full text (BB) appears. The intro text (AA) is correctly hidden.

Additional Notes

  • The HTML article view already respects the Show Intro Text setting (only BB appears)
  • This PR brings the RSS feed behavior in line with the HTML view
  • The same issue exists in the Featured Articles feed and will be addressed in a separate PR

Link to documentations

Please select:

  • Documentation link for guide.joomla.org:
  • No documentation changes for guide.joomla.org needed
  • Pull Request link for manual.joomla.org:
  • No documentation changes for manual.joomla.org needed
avatar CSGoat0 CSGoat0 - open - 10 May 2026
avatar CSGoat0 CSGoat0 - change - 10 May 2026
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 10 May 2026
Category Front End com_content
avatar CSGoat0 CSGoat0 - change - 10 May 2026
Title
[5.4] Fix Article Feed Intro Text With Include In Feed
[5.4] Fix Category Feed Intro Text With Include In Feed
avatar CSGoat0 CSGoat0 - edited - 10 May 2026
avatar CSGoat0 CSGoat0 - change - 10 May 2026
The description was changed
avatar CSGoat0 CSGoat0 - edited - 10 May 2026
avatar QuyTon
QuyTon - comment - 13 May 2026

Please refactor code as #47758 for consistency.

avatar CSGoat0 CSGoat0 - change - 13 May 2026
Labels Added: PR-5.4-dev
avatar CSGoat0 CSGoat0 - change - 13 May 2026
The description was changed
avatar CSGoat0 CSGoat0 - edited - 13 May 2026
avatar CSGoat0
CSGoat0 - comment - 13 May 2026

Please refactor code as #47758 for consistency.

Done.

avatar CSGoat0 CSGoat0 - change - 13 May 2026
The description was changed
avatar CSGoat0 CSGoat0 - edited - 13 May 2026
avatar CSGoat0
CSGoat0 - comment - 13 May 2026

Please refactor code as #47758 for consistency.

I have similar section in this PR #47761.
If I refactor it, it will look like this:

$description = ($feedSummary ? ($showIntro ? $item->core_body : '') . $row->fulltext : ($showIntro ? $item->core_body : ''));

Which is more complex even when keeping $showIntro.
What do you suggest? I would leave it as it is.

avatar QuyTon
QuyTon - comment - 13 May 2026

Maybe one of these:

description = ($showIntro ? $item->core_body : '') . ($feedSummary ? $row->fulltext : '');
or

description  = $showIntro ? $item->core_body : '';
description .= $feedSummary ? $row->fulltext : '';
avatar rdeutz
rdeutz - comment - 13 May 2026

@CSGoat0 could you plese rebase your PR to 6.2. We discussed it today in the maintainers meeting and decided that it is more a feature than a bug fix. Thanks.

avatar CSGoat0
CSGoat0 - comment - 13 May 2026

@CSGoat0 could you plese rebase your PR to 6.2. We discussed it today in the maintainers meeting and decided that it is more a feature than a bug fix. Thanks.

Hello,
thank you for the feedback, I respect the decision.

I originally targeted 5.4 because this was reported and confirmed as a bug by both a user and a maintainer as the RSS feed output differs from the HTML view. However, I understand that different perspectives exist on whether this is a bug or a feature.

Since rebasing to 6.2 would require some work on my end, would it be acceptable if I close these 2 PRs and create 2 new ones against 6.2 instead? That would be cleaner for me.

avatar richard67
richard67 - comment - 13 May 2026

Since rebasing to 6.2 would require some work on my end, would it be acceptable if I close these 2 PRs and create 2 new ones against 6.2 instead? That would be cleaner for me.

@CSGoat0 That would also be ok. Ideally you leave a comment in the old PR which refers to the new PR, and in the description of the new PR you mention that is is a redo of the old PR, and then close the old PR.

Thanks for your understanding.

avatar CSGoat0
CSGoat0 - comment - 13 May 2026

Since rebasing to 6.2 would require some work on my end, would it be acceptable if I close these 2 PRs and create 2 new ones against 6.2 instead? That would be cleaner for me.

@CSGoat0 That would also be ok. Ideally you leave a comment in the old PR which refers to the new PR, and in the description of the new PR you mention that is is a redo of the old PR, and then close the old PR.

Thanks for your understanding.

Regarding my third PR (Tag Feed fix #47761), I believe this one is different from the other two.

The Tag Feed was losing content entirely — only AA appeared (just the intro), and BB (the whole content body) was completely missing from the feed. This is not a display preference issue; the full text was never retrieved from the database. My fix restores this missing data.

Given that this is a data loss bug, would it be acceptable for 5.4? Or should I also move it to 6.2?

avatar CSGoat0
CSGoat0 - comment - 15 May 2026

I am asking as it's from the same family but with a different output. Also, no body have mentioned if i should keep it at 5.4 or not that's it.
So, i will be waiting for directions.

avatar CSGoat0
CSGoat0 - comment - 17 May 2026

Closing in favor of PR #47788

avatar CSGoat0 CSGoat0 - change - 17 May 2026
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2026-05-17 14:55:37
Closed_By CSGoat0
Labels Added: Feature
avatar CSGoat0 CSGoat0 - close - 17 May 2026

Add a Comment

Login with GitHub to post a comment