Problem/Motivation

The URL for episodes on Buzzsprout has been updated. As shown in their supported WP plugin:
https://plugins.trac.wordpress.org/changeset/3161694/

Steps to reproduce

Eg. go to the podcast that this person suggested #3456494: Improve the "Configuration & Use" section in README.md on their rather Buzzsprout specific readme change. It was https://www.buzzsprout.com/1209764/10394989 when they wrote it, but it now redirects to https://www.buzzsprout.com/1209764/episodes/10394989

Proposed resolution

Make the minimum viable change to the regex to match the updated URL pattern.

Remaining tasks

Merge request to follow.

CommentFileSizeAuthor
#7 media_remote_3482122_buzzsprout.patch3.22 KBdgroene
Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

Comments

ekes created an issue. See original summary.

ekes’s picture

Status: Active » Needs review
ekes’s picture

OK a bit more than the minimum change to the regex. Add the test and user feedback too.

marcoscano’s picture

Thanks for filing this and for the fix.
Is there a scenario where old URLs should be supported too? In other words, should be change the regex to the new format or should we just accept the new format as well?

ekes’s picture

Thanks for filing this and for the fix.
Is there a scenario where old URLs should be supported too? In other words, should be change the regex to the new format or should we just accept the new format as well?

I think I was planning that (?:\/episodes) but then didn't follow through ?. As seen in the in the update from Buzzsprout themselves it's only the new format for their content. In our case however, there will be entities with the old pattern that have it in the field with the old URL, no point in updating that. Sorry kinda rushing about here today.

dgroene’s picture

StatusFileSize
new3.22 KB

Patch for version 1.9.1 (same as MR)

dgroene’s picture

  • marcoscano committed 0d066b74 on 1.x authored by ekes
    Issue #3482122 by ekes, dgroene, marcoscano: Buzzsprout episode URL...
marcoscano’s picture

Status: Needs review » Fixed

Sorry for the late reply. Committed, thanks! 👍

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.