Issue #971812 by mfb, twistor: aggregator ignores Atom id element, which is equivalen...
authorGábor Hojtsy
Wed, 23 Nov 2011 10:54:23 +0000 (11:54 +0100)
committerGábor Hojtsy
Wed, 23 Nov 2011 10:54:23 +0000 (11:54 +0100)
modules/aggregator/aggregator.module

index 59a4fdb..0afb9f5 100644 (file)
@@ -751,7 +751,17 @@ function aggregator_parse_feed(&$data, $feed) {
     else {
       $link = $feed['link'];
     }
-    $guid = isset($item['GUID']) ? $item['GUID'] : '';
+
+    // Atom feeds use ID rather than GUID.
+    if (isset($item['GUID'])) {
+      $guid = $item['GUID'];
+    }
+    elseif (isset($item['ID'])) {
+      $guid = $item['ID'];
+    }
+    else {
+      $guid = '';
+    }
 
     // Atom feeds have a CONTENT and/or SUMMARY tag instead of a DESCRIPTION tag.
     if (!empty($item['CONTENT:ENCODED'])) {