www

commit eb7556533adb53aec89184e824d705a9720cc3ed
parent 1b6dffb88640f1624c3a905dc520883fda5be082
Author: Dan Stillman <dstillman@zotero.org>
Date:   Wed, 13 Jul 2016 12:42:39 -0400

Merge pull request #1063 from adomasven/fix/use-link-for-id

Use feed item links for guid if id unavailable
Diffstat:
M
chrome/content/zotero/xpcom/feedReader.js
 | 
7
++++---

1 file changed, 4 insertions(+), 3 deletions(-)
diff --git a/chrome/content/zotero/xpcom/feedReader.js b/chrome/content/zotero/xpcom/feedReader.js
@@ -360,13 +360,14 @@ Zotero.FeedReader._processCreators = function(feedEntry, field, role) {
 Zotero.FeedReader._getFeedItem = function(feedEntry, feedInfo) {
 	// ID is not required, but most feeds have these and we have to rely on them
 	// to handle updating properly
-	if (!feedEntry.id) {
-		Zotero.debug("FeedReader: Feed item missing an ID");
+	// Can probably fall back to links on missing id - unlikely to change
+	if (!feedEntry.id && !feedEntry.link) {
+		Zotero.debug("FeedReader: Feed item missing an ID or link - discarding");
 		return;
 	}
 	
 	let item = {
-		guid: feedEntry.id
+		guid: feedEntry.id || feedEntry.link.spec
 	};
 			
 	if (feedEntry.title) item.title = Zotero.FeedReader._getRichText(feedEntry.title, 'title');