Add rss/channel/itunes:summary handling

[CJxD]

We noticed that some podcasts don't have descriptions but do have itunes summaries.

Here's a file that can be used in a test: http://netstorage.discovery.com/ahc/podcasts/2016/ahc-ato-podcastrss.xml

[thp]

From above that block:

# If you modify the mapping, don't forget to also update the documentation
# section "Supported XML Elements and Attributes" in doc/index.rst

Apart from that, it's good. It might cause some feeds that have both description and itunes:summary with different content to have different results after parsing, though.

[auouymous]

@thp Could we add an optional or secondary flag to set_podcast_attr() so it only sets the value if None? Then description could be the primary field and summary a secondary, no matter which order they are found in the feed.

[CJxD]

Just got back to this! Sorry for the delay. Yes I can look into a suitable fallback behaviour :)

[CJxD]

Here's a question, the episode description behaviour is similar to what I have suggested here so I wonder if this change-if-none behaviour should also be applied in this case.

I also noted how HTML is handled in description but not in the summary according to this. Is it accurate? Should the same happen for channel/description?

**rss/channel/item/description**
    Episode description.
    If it contains html, it's returned as description_html.
    Otherwise it's returned as description (whitespace is squashed).
    See Mozilla's article `Why RSS Content Module is Popular`

**rss/channel/item/itunes:summary**
    Episode description (whitespace is squashed).

[auouymous]

Here's a question, the episode description behaviour is similar to what I have suggested here so I wonder if this change-if-none behaviour should also be applied in this case.

Yes. It would have the same issue, unless it already only changes if none.

I also noted how HTML is handled in description but not in the summary according to this. Is it accurate? Should the same happen for channel/description?

It could if the RSS spec supports HTML in channel descriptions. gPodder only supports HTML descriptions for episodes and would need to detect an HTML channel description and strip it.

[thp]

Bump; is there anything left to do here or can this be merged? :)

[CJxD]

I'll take another pass at this next week :)

[CJxD]

I've updated this MR to include:

1. Support for the overwrite=True flag in PodcastAttrs
2. Behaviour to default to reading the description field in case both itunes:summary and description are present
3. Tests to confirm the behaviour
4. A version number bump for the package

[thp]

This is super clean now, just need to remove the version bump (it's done separately as part of releasing, not in code changes in branches). Thanks!

[thp]

Merged, thanks!

[CJxD]

Hey @thp - will there be a new release soon? I'd like to update the library in my project :)

[thp]

Hey @thp - will there be a new release soon? I'd like to update the library in my project :)

Done: 5ac16fa