The 'feed_item_length' setting ("Feed content: Global setting for the default display of content items in each feed - Titles only, Titles plus teaser, Full text") is currently broken.
The current code in node_feed() grabs the value, but does nothing with it.
With the new 'Body as field', those options are now display settings for the 'body' field in 'rss' build mode : hidden, default, summary.
With no field UI in core for now, we could programmatically update all instances of the body field when the setting is changed, but this doesn't play nicely when you enable the CCK UI and can define different settings for different node types.
So I guess the question is: what do we want to do with this feature ?
| Comment | File | Size | Author |
|---|---|---|---|
| #24 | Screen Shot 2021-06-28 at 8.38.31 PM.png | 31.8 KB | pameeela |
| #10 | feed-item-length-493674-10.patch | 4.21 KB | eporama |
| #2 | 493674-aggregator-feed-item-length.patch | 3.99 KB | damien tournoud |
Comments
Comment #1
yched commentedCritical, not as in 'urgent', but as in 'needs to be solved before a release'.
Comment #2
damien tournoud commentedI think it makes sense to move that settings back into the aggregator.
Comment #4
yched commentedField UI is now in. Users can choose which display settings they want for the 'RSS build mode' field by field, including body.
So agreed on the approach where the 'length of text to display' variable is limited to aggregator feeds (listing "external" content with no fields on them - although in Feed API, agregated (non-node) items could be fieldable too, I guess).
However :
+++ modules/node/node.module
@@ -2065,9 +2065,7 @@ function node_feed($nids = FALSE, $channel = array()) {
- $item_length = variable_get('feed_item_length', 'fulltext');
$item_length is used below (are we displaying full objects or only titles). It's nothing more than a boolean now.
This review is powered by Dreditor.
Comment #5
sun.core commentedNot critical. Please read and understand Priority levels of Issues, thanks.
Comment #6
grendzy commentedGot bit by this today. It's quite confusing that the teaser setting at /admin/config/services/rss-publishing has no effect. This also seems to impact views.
Comment #7
jodeygrist commentedYeah confirmed, views can't display just the title plus teaser, you end up with the full body.
Comment #8
sheila8stamp commentedsubscribing
Comment #9
agileware commentedThis appears to be causing the complete body text to be displayed in RSS feeds instead of just the teaser, which is not good.
Subscribe.
Comment #10
eporama commentedJust to sum up yched's point, the setting can be ignored for your own RSS feed as now Field UI will let you set the field formatter for each field per content type.
Visit /admin/structure/types/manage/article/display and click on the "Custom Display Settings" to enable the RSS view mode on the article content type, for example. Then you can set "Trimmed" or "Summary or Trimmed" which will be in effect for the default rss.xml and views based feeds.
Seems slightly more difficult now as you have to change those per field per content type if you want teasers. Might be nice to have this as the "default" setting for RSS instead of using the per content-type "default" formatter. Then you can override individual types and fields against this setting rather than having to change each content type.
I think yched's comment in #4 leads to the fact that that whole section about
$item_lengthcould be removed as well, since one of the display formatters you can choose for a field is "<hidden>" which if you set for all fields on a content type has the same effect as "Title only".Comment #11
thumb commentedsubscribe
Comment #12
catch#1010766: Global feed setting has no effect was duplicate.
Comment #13
petergw commentedUnfortunately the fix using a change to the Custom Display Settings of say an Article does not work in an Omega child theme. It works fine in Bartik. So I for one would still like it to be fixed. Afterall, the original design (of now 3 years ago) was that we could use Teasers in an RSS. No one really wants to use the whole article. And the title alone is pretty base.
Comment #14
ianthomas_ukSetting needs work because the patch is for D7 and this is targeted at D8 (as per backport policy)
Comment #15
lokapujyaI think the RSS feed display already works in D8. The body display type is set to default. The Default display seems to also need to be set to "Default". Then, I set the RSS Global Setting to one of: "Titles Only", "Titles plus teaser", or "Full text". Then, the RSS.xml conforms to the Global Settings.
When in a view, RSS display types are adhered to. I can set the RSS display's body to use the summary, trimmed, full text, or invisible.
However, if I have the RSS display's summary set to "default", views seems to just be using the full content. Instead, views should drill down to the defaults set in RSS Global settings (or first drill down to the default display and then -if the default display is set to default- then it should use the Global setting. So, is that the bug?
Comment #24
pameeela commentedA lot of changes since this issue was created - Field UI in core!!
Anyway it works fine now. You can either use a view mode for RSS, or you can opt to 'Use site default RSS settings'. Tested both and all works as expected.