toxblh/podcasts
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Added Feed cover extraction from itunes extension.

Browse Source
This commit is contained in:
Jordan Petridis 2017-10-20 00:59:58 +03:00
parent 590e99b7e8
commit 3d98bf12f3
No known key found for this signature in database
GPG Key ID: CEABAD9F5683B9A6
1 changed files with 20 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

24
hammond-data/src/feedparser.rs
View File

@ -6,6 +6,7 @@ use errors::*;
// TODO: look into how bad-utf8 is handled in rss crate, // TODO: look into how bad-utf8 is handled in rss crate,
// and figure if there is a need for checking before parsing. // and figure if there is a need for checking before parsing.
// TODO: Extend the support for parsing itunes extensions
pub fn parse_podcast(chan: &Channel, source_id: i32) -> Result<models::NewPodcast> { pub fn parse_podcast(chan: &Channel, source_id: i32) -> Result<models::NewPodcast> {
let title = chan.title().trim().to_owned(); let title = chan.title().trim().to_owned();
let link = chan.link().to_owned(); let link = chan.link().to_owned();
@ -13,7 +14,11 @@ pub fn parse_podcast(chan: &Channel, source_id: i32) -> Result<models::NewPodcas
// Some feeds miss baseurl and/or http:// // Some feeds miss baseurl and/or http://
// TODO: Sanitize the url, // TODO: Sanitize the url,
// could also be reuse to sanitize the new-url gui entrybox. // could also be reuse to sanitize the new-url gui entrybox.
let image_uri = chan.image().map(|foo| foo.url().to_owned()); let image_uri = if let Some(img) = chan.itunes_ext().map(|s| s.image()) {
img.map(|s| s.to_string())
} else {
chan.image().map(|foo| foo.url().to_owned())
};
let foo = models::NewPodcast { let foo = models::NewPodcast {
title, title,
@ -86,7 +91,15 @@ mod tests {
assert_eq!(pd.title, "Intercepted with Jeremy Scahill".to_string()); assert_eq!(pd.title, "Intercepted with Jeremy Scahill".to_string());
assert_eq!(pd.link, "https://theintercept.com/podcasts".to_string()); assert_eq!(pd.link, "https://theintercept.com/podcasts".to_string());
assert_eq!(pd.description, descr.to_string()); assert_eq!(pd.description, descr.to_string());
assert_eq!(pd.image_uri, None); assert_eq!(
pd.image_uri,
Some(
"http://static.megaphone.fm/podcasts/d5735a50-d904-11e6-8532-73c7de466ea6/image/\
uploads_2F1484252190700-qhn5krasklbce3dh-a797539282700ea0298a3a26f7e49b0b_\
2FIntercepted_COVER%2B_281_29.png"
.to_string()
)
);
} }
#[test] #[test]
@ -102,7 +115,10 @@ mod tests {
assert_eq!(pd.title, "The Breakthrough".to_string()); assert_eq!(pd.title, "The Breakthrough".to_string());
assert_eq!(pd.link, "http://www.propublica.org/podcast".to_string()); assert_eq!(pd.link, "http://www.propublica.org/podcast".to_string());
assert_eq!(pd.description, descr.to_string()); assert_eq!(pd.description, descr.to_string());
assert_eq!(pd.image_uri, None); assert_eq!(
pd.image_uri,
Some("http://www.propublica.org/images/podcast_logo_2.png".to_string())
);
} }
#[test] #[test]
@ -120,7 +136,7 @@ mod tests {
assert_eq!(pd.description, descr.to_string()); assert_eq!(pd.description, descr.to_string());
assert_eq!( assert_eq!(
pd.image_uri, pd.image_uri,
Some("http://michaeltunnell.com/images/linux-unplugged.jpg".to_string(),) Some("http://www.jupiterbroadcasting.com/images/LASUN-Badge1400.jpg".to_string(),)
); );
} }