Aggregator won't pull in descriptions if the content is just plainly in "content", and not in a summary. Simple patch to fix that.

P.S. I am not too familiar with the RSS/Atom RFC, so if someone has more insite, feel free to comment. but i think this fixs this issue/bug.

CommentFileSizeAuthor
#1 agg_fix_0.patch817 bytesSouvent22
agg_fix.patch816 bytesSouvent22

Comments

Souvent22’s picture

Souvent22 commented
StatusFileSize
new agg_fix_0.patch817 bytes

Re-roll. Spacing issue

m3avrck’s picture

m3avrck commented
Status: Active » Needs review

Makes sense to me ... often times feeds come out that don't have summaries but only have content. We shouldn't show just an empty aggregator item ... with title only, looks weird.

This accounts for that fact. +1

walkah’s picture

walkah commented
Status: Needs review » Reviewed & tested by the community

looks good - this is part of the atom 1.0 spec:

http://www.atomenabled.org/developers/syndication/atom-format-spec.php#e...

RTBC.

dries’s picture

dries commented

Committed to CVS HEAD. Thanks.

Souvent22’s picture

Souvent22 commented
Status: Reviewed & tested by the community » Fixed

Thanks Dries. Updating/Closing ticket.

m3avrck’s picture

m3avrck commented
Version: x.y.z » 4.7.3
Status: Fixed » Reviewed & tested by the community

Bug exists in 4.7 as well, simple patch should be backported for 4.7 :-)

killes@www.drop.org’s picture

killes@www.drop.org commented
Status: Reviewed & tested by the community » Fixed

backported

Anonymous’s picture

(not verified) commented
Status: Fixed » Closed (fixed)