Skip to content
Snippets Groups Projects
Commit b821af43 authored by Rob Swindell's avatar Rob Swindell :speech_balloon:
Browse files

Merge branch 'rss-atom' into 'master'

Fix massive regression introduced in previous commit.

Iterate over RSS feed channels and items in a less ugly way.
Don't iterate over all item children and cherry-pick the ones
we want, just target the desired elements instead.

Remove the "This is shitty" comment. It's still shitty overall,
but that particular shitty thing has been de-shitted.

See merge request sbbs/sbbs!5
parents c16a8735 3605fed1
Branches
Tags
1 merge request!5Fix massive regression introduced in previous commit.
Pipeline #16 passed
......@@ -109,111 +109,53 @@
Array of { type : String, url : String, length : Number } objects
for any <enclosure> elements in the item.
Item.extra (Object)
If the item/entry contains additional elements not provided for
above, they are tacked on to this object in case you may need
to access them. Presumably these will all be E4X XML objects.
*/
load("http.js");
// This is really not the way to do things, but meh.
var toLocal = function(xmlObject) {
for each(var element in xmlObject) {
element.setName(element.localName());
toLocal(element);
}
}
var Feed = function (url, follow_redirects) {
var Item = function Item(xmlObject) {
const Item = function (i) {
this.id = "";
this.title = "";
this.date = "";
this.author = "";
this.body = "";
this.content = "";
this.link = "";
this.id = i.guid.length() ? i.guid[0].toString() : (i.id.length() ? i.id[0].toString() : '');
this.title = ''; // uh ...
this.date = i.pubDate.length() ? i.pubDate[0].toString() : (i.updated.length() ? i.updated[0].toString() : '');
this.author = i.author.length() ? i.author.toString() : '';
this.body = i.description.length() ? i.description[0].toString() : (i.summary.length() ? i.summary[0].toString() : '');
this.content = i.encoded.length() ? i.encoded.toString() : '';
this.link = i.link.length() ? skipsp(truncsp(i.link[0].toString())) : '';
this.enclosures = [];
this.extra = {};
for each(var element in xmlObject) {
switch (element.name()) {
case 'guid':
case 'id':
this.id = element;
break;
case 'pubDate':
case 'updated':
this.date = element;
break;
case 'author':
this.author = element.text();
break;
case 'description':
case 'summary':
this.body = element;
break;
case 'link':
this.link = skipsp(truncsp(element.text()));
break;
case 'encoded':
this.content = element;
break;
case 'enclosure':
var enclosures = i.enclosure.length();
for (var n = 0; n < enclosures; n++) {
this.enclosures.push({
type: element.attribute('type'),
url: element.attribute("url"),
length: parseInt(element.attribute('length'), 10),
type: i.enclosure[n].attribute('type'),
url: i.enclosure[n].attribute('url'),
length: parseInt(i.enclosure[n].attribute('length'), 10),
});
break;
default:
this.extra[element.name()] = element;
}
}
}
var Channel = function (xmlObject) {
const Channel = function (c) {
this.title = "a";
this.description = "";
this.link = "";
this.updated = "";
this.title = c.title.length() ? c.title[0].toString() : '';
this.description = c.description.length() ? c.description[0].toString() : (c.subtitle.length() ? c.subtitle[0].toString() : '');
this.link = c.link.length() ? skipsp(truncsp(c.link[0].toString())) : '';
this.updated = c.lastBuildDate.length() ? c.lastBuildDate[0].toString() : (c.updated.length() ? c.updated[0].toString() : '');
this.items = [];
if (typeof xmlObject.title != "undefined") this.title = xmlObject.title;
if (typeof xmlObject.description != "undefined") {
this.description = xmlObject.description;
} else if (typeof xmlObject.subtitle != "undefined") {
this.description = xmlObject.subtitle;
var items = c.item.length();
for (var n = 0; n < items; n++) {
this.items.push(new Item(c.item[n]));
}
// To do: deal with multiple links
if (typeof xmlObject.link != "undefined") this.link = skipsp(truncsp(xmlObject.link.text()));
if (typeof xmlObject.lastBuildDate != "undefined") {
this.updated = xmlObject.lastBuildDate;
} else if (typeof xmlObject.updated != "undefined") {
this.updated = xmlObject.updated;
var entries = c.entry.length();
for (var n = 0; n < entries; n++) {
this.items.push(new Item(c.entry[n]));
}
var items = xmlObject.elements("item");
for each(var item in items) {
this.items.push(new Item(item));
}
var entries = xmlObject.elements("entry");
for each(var entry in entries) {
this.items.push(new Item(entry));
}
}
const Feed = function (url, follow_redirects) {
this.channels = [];
......@@ -221,14 +163,17 @@ var Feed = function (url, follow_redirects) {
var httpRequest = new HTTPRequest();
httpRequest.follow_redirects = follow_redirects || 0;
var response = httpRequest.Get(url);
if (typeof response == "undefined" || response == "") throw "Empty response from server.";
if (typeof response == "undefined" || response == "") {
throw new Error('Empty response from server.');
}
var feed = new XML(response.replace(/^<\?xml.*\?>/g, ""));
toLocal(feed); // This is shitty
httpRequest = undefined;
response = undefined;
switch (feed.localName()) {
case "rss":
var channels = feed.elements("channel");
for each(var element in channels) {
this.channels.push(new Channel(element));
var channels = feed.channel.length();
for (var n = 0; n < channels; n++) {
this.channels.push(new Channel(feed.channel[n]));
}
break;
case "feed":
......@@ -237,6 +182,7 @@ var Feed = function (url, follow_redirects) {
default:
break;
}
feed = undefined;
}
this.load();
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment