Allow trailing feed metadata after items

Author: danmactough

README.md

@@ -113,9 +113,16 @@ You can also check out this nice [working implementation](https://github.com/scr
 
 - `addmeta` - Set to `false` to override Feedparser's default behavior, which
   is to add the feed's `meta` information to each article.
+  Feed metadata is available as soon as Feedparser has enough information to
+  emit the first article. While bad practice and borderline pathological, feeds
+  can legally include additional channel metadata after articles, so the `meta`
+  object may be enriched until the stream ends. If you need complete metadata,
+  also handle the `meta` event and keep the emitted object until the stream ends.
+  If you only need the metadata available when each article streams, you can
+  use `item.meta` as usual.
 
 - `feedurl` - The url (string) of the feed. FeedParser is very good at
-  resolving relative urls in feeds, including those embedded in HTML content 
+  resolving relative urls in feeds, including those embedded in HTML content
   fields. But some feeds use relative urls without declaring the `xml:base`
   attribute any place in the feed. This is perfectly valid, but we don't know
   the feed's url before we start parsing the feed and trying to resolve those

lib/feedparser.js

@@ -363,11 +363,10 @@ FeedParser.prototype.handleCloseTag = function (el) {
     }
     if (this.meta.author && !item.author) item.author = this.meta.author;
     this.push(item);
-  } else if (!this.meta.title && // We haven't yet parsed all the metadata
-    (node['#name'] === 'channel' ||
+  } else if (node['#name'] === 'channel' ||
       node['#name'] === 'feed' ||
       (node['#local'] === 'channel' && (node['#prefix'] === '' || node['#type'] === 'rdf')) ||
-      (node['#local'] === 'feed' && (node['#prefix'] === '' || node['#type'] === 'atom')))) {
+      (node['#local'] === 'feed' && (node['#prefix'] === '' || node['#type'] === 'atom'))) {
     _.assign(this.meta, this.handleMeta(n, this.meta['#type'], this.options));
     if (!this._emitted_meta) {
       this.emit('meta', this.meta);
@@ -503,6 +502,8 @@ FeedParser.prototype.handleMeta = function handleMeta (node, type, options) {
   Object.keys(node).forEach((name) => {
     var el = node[name];
 
+    if (name === 'item' || name === 'entry' || name.match(/:(item|entry)$/)) return;
+
     if (normalize) {
       switch (name) {
       case ('title'):

test/feeds/rss-with-trailing-meta.xml

@@ -0,0 +1,14 @@
+<?xml version="1.0"?>
+<rss version="2.0" xmlns:itunes="http://www.itunes.com/dtds/podcast-1.0.dtd">
+  <channel>
+    <title>Trailing Meta Feed</title>
+    <link>https://example.com/</link>
+    <description>Feed metadata can appear after items.</description>
+    <item>
+      <title>First item</title>
+      <link>https://example.com/items/1</link>
+      <guid>https://example.com/items/1</guid>
+    </item>
+    <itunes:category text="Music"/>
+  </channel>
+</rss>

test/trailing-meta.js

@@ -0,0 +1,62 @@
+describe('trailing metadata', function () {
+
+  var feed = __dirname + '/feeds/rss-with-trailing-meta.xml';
+
+  it('should include channel metadata that appears after items in final meta', function (done) {
+    var feedparser = new FeedParser();
+    var items = [];
+    var meta;
+    var metaEvents = 0;
+
+    fs.createReadStream(feed).pipe(feedparser)
+      .on('error', function (err) {
+        assert.ifError(err);
+        done(err);
+      })
+      .on('meta', function (_meta) {
+        meta = _meta;
+        metaEvents++;
+      })
+      .on('readable', function () {
+        var item;
+        while ((item = this.read()) !== null) {
+          items.push(item);
+        }
+      })
+      .on('end', function () {
+        assert.strictEqual(metaEvents, 1);
+        assert.deepStrictEqual(feedparser.meta.categories, ['Music']);
+        assert.strictEqual(meta, feedparser.meta);
+        assert.strictEqual(items.length, 1);
+        assert.strictEqual(items[0].meta, feedparser.meta);
+        assert.deepStrictEqual(items[0].meta.categories, ['Music']);
+        assert.strictEqual(feedparser.meta['rss:item'], undefined);
+        done();
+      });
+  });
+
+  it('should skip items in final native meta when normalize is false', function (done) {
+    var feedparser = new FeedParser({ normalize: false });
+
+    fs.createReadStream(feed).pipe(feedparser)
+      .on('error', function (err) {
+        assert.ifError(err);
+        done(err);
+      })
+      .on('readable', function () {
+        var item;
+        while ((item = this.read()) !== null) {
+          assert(item);
+        }
+      })
+      .on('end', function () {
+        assert.strictEqual(feedparser.meta.title, undefined);
+        assert.strictEqual(feedparser.meta['rss:title']['#'], 'Trailing Meta Feed');
+        assert.strictEqual(feedparser.meta['itunes:category']['@'].text, 'Music');
+        assert.strictEqual(feedparser.meta['rss:item'], undefined);
+        assert.strictEqual(feedparser.meta.item, undefined);
+        done();
+      });
+  });
+
+});