martin/rss-braider
1
0
Fork 0
mirror of https://gitlab.silvrtree.co.uk/martind2000/rss-braider.git synced 2025-02-10 20:39:15 +00:00
Code Issues Projects Releases Wiki Activity

Fixing thumbnail issue and unit test

Browse Source
This commit is contained in:
Kip Gebhardt 2014-12-26 15:57:23 -08:00
parent b8d8d12d79
commit 8fe8f3a175
2 changed files with 25 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

28
lib/RssBraider.js
View File

@ -55,7 +55,7 @@ RssBraider.prototype.processFeed = function(feed_name, format, callback)
if (url) {
var req = request(url);
logger.info("request to", url);
// logger.info("request to", url);
req.on('error', function (error) {
logger.error(error);
@ -202,8 +202,12 @@ RssBraider.prototype.processItem = function (item) {
// 'media:content'[0]'media:thumbnail'
// else
// 'media:thumbnail'
var thumbnail;
if (item['media:thumbnail'] && item['media:thumbnail']['#']) {
itemOptions.custom_elements.push({'media:thumbnail' : item['media:thumbnail']['#']});
thumbnail = {
'media:thumbnail': item['media:thumbnail']['#']
};
itemOptions.custom_elements.push(thumbnail);
} else {
if (item["media:content"]) {
var media_contents;
@ -218,10 +222,24 @@ RssBraider.prototype.processItem = function (item) {
media_contents[0]['media:thumbnail']['@'] &&
media_contents[0]['media:thumbnail']['@'].url) {
itemOptions.custom_elements.push({'media:thumbnail' : media_contents[0]['media:thumbnail']['@'].url});
thumbnail = {
'media:thumbnail' : [{
_attr: {
url: media_contents[0]['media:thumbnail']['@'].url
}
}]
};
// itemOptions.custom_elements.push({'media:thumbnail' : { url: media_contents[0]['media:thumbnail']['@'].url}} );
itemOptions.custom_elements.push(thumbnail);
} else {
itemOptions.custom_elements.push({'media:thumbnail' : media_contents[0]['@'].url});
thumbnail = {
'media:thumbnail' : [{
_attr: {
url: media_contents[0]['@'].url
}
}]
};
itemOptions.custom_elements.push(thumbnail);
}
}
}

3
test/expected_output/fileFeedOutput.xml
View File

@ -20,7 +20,8 @@
<div id='ctx-module' class='ctx-module-container ctx-clearfix'></div>]]></content:encoded>
<wfw:commentRss>http://ww2.kqed.org/arts/2014/12/20/rent-hike-for-dance-mission-theater-has-artists-worried-about-uncertain-future/feed/</wfw:commentRss>
<slash:comments>0</slash:comments>
<media:thumbnail>http://ww2.kqed.org/arts/wp-content/uploads/sites/2/2014/12/10232900-thumb.jpg</media:thumbnail>
<media:thumbnail url="http://ww2.kqed.org/arts/wp-content/uploads/sites/2/2014/12/10232900-thumb.jpg">
</media:thumbnail>
</item>
</channel>
</rss>