Closed (fixed)
Project:
Drupal core
Version:
4.7.x-dev
Component:
aggregator.module
Priority:
Minor
Category:
Bug report
Assigned:
Reporter:
Created:
9 Jun 2006 at 13:36 UTC
Updated:
29 Dec 2006 at 12:15 UTC
Jump to comment: Most recent file
Comments
Comment #1
beginner commented+1 generally on the patch, but:
1) $read_more is not initialized and will create a Notice error when there is no more feeds to read.
2) cvs has this bug, too, and should be fixed first so that we are sure we won't forget to include the fix in the next version :)
3) diff from the root: http://drupal.org/patch
Comment #2
Zen commentedThanks for the review. Updated.
This patch also updates the title of the anchor tag to display a more relevant message.
-K
Comment #3
Zen commentedThis patch is against 4-7-4.
-K
Comment #4
heine commentedRolled a patch before I saw this issue, so you're not the only one considering this a minor bug.
Here's a patch against 5-dev.
$type will be either feed or category, so $read_more will be defined (as will $result). Text change is for another patch.
Code comment may be obvious.
Comment #5
bdragon commentedA side effect of having an empty block content is that such blocks will disappear from the layout.
This is a behavioral change, but one that I believe is more in line with the way the rest of Drupal works (i.e. if it isn't relevant, it isn't shown.)
All that being said, patch in #4 looks OK to me, works as advertised, and doesn't break anything. RTBC.
Comment #6
drummCOmmitted to HEAD.
Comment #7
(not verified) commentedComment #8
Zen commentedComment #9
killes@www.drop.org commentedapparently, I already applied this some tme ago.
Comment #10
(not verified) commented