User tests: Successful: Unsuccessful:
Pull Request resolves #47759
Fixed the tag feed (RSS) to properly respect the Show Intro Text article parameter when displaying content items.
File changed: components/com_tags/views/tag/FeedView.php
What was changed:
The tag feed previously used only core_body from the UCM content table, which contains only the intro text. Full text after Read More was completely missing from the feed.
The fix:
com_content.article)fulltext and attribs from the #__content table for all articles in a single optimized queryattribs to determine the show_intro settingfeed_summary (com_content parameter: 0 = intro only, 1 = full text)show_intro (article parameter: 1 = show intro, 0 = hide intro)show_intro setting if not set at article levelPrerequisites:
Step 1: Configure Article Options
Step 2: Create an article with Read More and assign a tag
Step 3: Create a Tagged Items menu item (if not already exists)
Step 4: Access the RSS feed
Use the URL of your tagged items menu item with feed parameters:
http://localhost/tags-rss-feed-test-menu?format=feed&type=rss
Or use the direct route:
http://localhost/index.php?option=com_tags&view=tag&id=XX&format=feed&type=rss
Step 5: Observe the feed output
Look at the <description> tag of your article in the feed.
<description><![CDATA[<p>AA</p>]]></description>Only the intro text (AA) appears. The full text (BB) is completely missing from the feed.
<description><![CDATA[<p>BB</p>]]></description>Only the full text (BB) appears. The intro text (AA) is correctly hidden because Show Intro Text = Hide.
Test with Show Intro Text = Show:
Expected result:
<description><![CDATA[<p>AA</p><p>BB</p>]]></description>Both intro text (AA) and full text (BB) appear.
The fix uses a single optimized query to fetch fulltext and attribs for all articles in the feed, regardless of the number of articles. This avoids N+1 query issues and maintains good performance even with larger feed limits.
com_content.article items are affected by this fixshow_intro setting and the global com_content defaultPlease select:
| Status | New | ⇒ | Pending |
| Category | ⇒ | com_tags Front End |
| Labels |
Added:
PR-5.4-dev
|
||
Having a query in the view is a no go. We might let it as it is and don't fix this at all.
I think you are right. It should be at the model, no?
I have fixed it.
Having a query in the view is a no go. We might let it as it is and don't fix this at all.