search

tkem / mopidy-podcast

Mopidy extension for browsing and playing podcasts
Apache License 2.0
43 stars 3 forks source link

Support for <itunes:episode> #57

Closed Bougakov closed 4 years ago

Bougakov commented 4 years ago

Hi. Some podcasts turn out to have only a <itunes:episode>n</itunes:episode> and <itunes:type>episodic</itunes:type>, but not the date in their RSS feeds. Mopidy displays those in a random order.

Suggestion - if no date attribute is set, fall back to sorting by episode number.

tkem commented 4 years ago

Sounds reasonable, can you point me to any examples where this is the case?

Bougakov commented 4 years ago

About half of the podcasts I've tried to move to Mopidy were like that. For example, https://feeds.megaphone.fm/WWO6655869236 I don't know what "megaphone.fm" is, but RSS links on podcasts' pages mostly seem to point there.

tkem commented 4 years ago

@Bougakov: You're right that Mopidy-Podcast ignores the <itunes:episode> tags, but at least the podcast you linked to seems to provide proper <pubDate> tags. So these should be displayed "newest first", and not in "random order". I only checked with Mopidy-Mobile as a client, though.

Bougakov commented 4 years ago

Hmm, strange. Here is a piece of my conf file:

[podcast]
enabled = true
browse_root = Podcasts.opml
browse_order = desc
lookup_order = asc 
cache_size = 64
cache_ttl = 86400
timeout = 10

Here is my /etc/mopidy/podcast/Podcasts.opml

<?xml version="1.0" encoding="UTF-8"?>
<opml>
    <body>
        <outline text="Pivot" type="rss" xmlUrl='https://feeds.megaphone.fm/pivot' />
        <outline text="Prof G Show" type="rss" xmlUrl='https://feeds.megaphone.fm/WWO6655869236' />
        <outline text="WSJ What's new" type="rss" xmlUrl='https://video-api.wsj.com/podcast/rss/wsj/whats-news' />
    </body>
</opml>

mopidy-podcast is the newest version (3.0.0-1).

Let's check the https://feeds.megaphone.fm/WWO6655869236 The latest episode is: Benchmarking, Brand Strategy, and Starting Fresh, <pubDate>Thu, 23 Jul 2020 08:00:00 -0000. Strangely, it is displayed in the middle of the list:

image

The top entry is Get to a Platform with <pubDate>Thu, 28 May 2020 07:01:00 -0000</pubDate>

P.S. <channel> -> <title> doesn't appear to be parsed correctly as well...

tkem commented 4 years ago

I don't know about the Iris client, but when using the MPD frontend I get this:

$ mpc -f '%track%\t%date% %title%' ls Podcasts/Test
20/20   2020-07-23 Benchmarking, Brand Strategy, and Starting Fresh
19/20   2020-07-16 Predictions, COVID-19, and Loving Others
18/20   2020-07-09 Disruption is Due
17/20   2020-07-02 Getting Unstuck 
16/20   2020-06-25 Exploitation Economy
15/20   2020-06-18 Bring Forgiveness
14/20   2020-06-11 Be Quiet and Listen
13/20   2020-06-04 Slow Thinking
12/20   2020-05-28  Get to a Platform 
11/20   2020-05-21 Moral Illusions
10/20   2020-05-14 A Position of Strength 
9/20    2020-05-07 Examine Your Burn
8/20    2020-04-30  Stop Keeping Score 
7/20    2020-04-23 Post Corona: The Four
6/20    2020-04-16 The Essential Is Invisible
5/20    2020-04-09  Long on Humanity 
4/20    2020-04-02 Mind the Gap
3/20    2020-03-26 A Post-Corona World
2/20    2020-03-19 Love in the Time of Corona
1/20    2020-02-12 Official Trailer: The Prof G Show With Scott Galloway

I get the same result with Mopidy Mobile, so I suspect Iris is using its own sorting scheme:

image

Bougakov commented 4 years ago

Filed this bug to https://github.com/jaedb/Iris/issues/576

Thanks for helping to investigate this.