r7159 - dumbhippo/trunk/server/src/com/dumbhippo/dm

[commits mugshot org]

Author: otaylor
Date: 2008-01-09 12:04:07 -0600 (Wed, 09 Jan 2008)
New Revision: 7159

Modified:
   dumbhippo/trunk/server/src/com/dumbhippo/dm/FeedWrapper.java
Log:
Fix a bug where after an incremental feed update, subsequent from-scratch
 fetches got too few items.


Modified: dumbhippo/trunk/server/src/com/dumbhippo/dm/FeedWrapper.java
===================================================================
--- dumbhippo/trunk/server/src/com/dumbhippo/dm/FeedWrapper.java	2008-01-09 16:55:42 UTC (rev 7158)
+++ dumbhippo/trunk/server/src/com/dumbhippo/dm/FeedWrapper.java	2008-01-09 18:04:07 UTC (rev 7159)
@@ -156,8 +156,11 @@
 					return;
 				}
 			}
-	
-			if (cachedFeed != null)
+
+			// We only should update the maxFetched if we didn't specify a minTimestamp
+			// when fetching; if we specified a minTimestamp, we have no idea how many
+			// items actually were in the feed beyond the ones we got
+			if (cachedFeed != null && minTimestamp <= 0)
 				cachedFeed.addToMaxfetched(start + max);
 		}