A bug in get_iplayer-3.01?

[Vangelis forthnet]

On Mon Jun 5 14:51:45 BST 2017, RS wrote:

> I take it the answer is that substitute needs to be called
> before create_dir, so that the sub-directory name
> is sanitised by removal of the /
> before the sub-directory is created.

 Hello Richard and many thanks for your analysis :-)
I am otherwise occupied lately, apologies I can't
contribute to the list to the extent I'd wish for...

Remember, I DO NOT KNOW perl, but patching the code like:

 # Create a subdir for programme sorting option
 elsif ( $opt->{subdir} ) {
  my $subdir = $prog->substitute( $opt->{subdirformat} || '<longname>', 1 );
+# Replace forwardslashes in $subdir with _
+$subdir =~ s/\//_/g;
  $prog->{dir} = File::Spec->catdir($prog->{dir}, $subdir);

has allowed for:

get_iplayer --type=tv --pid=b08r69t1 --subdir -i | FindStr dir =>

dir:            D:\Vangelis\iPlayer Recordings\Vets_24_7_Series_4

and

get_iplayer --type=tv --pid=b08r69t1 --subdir -i | FindStr filename =>

filename:       D:\Vangelis\iPlayer 
Recordings\Vets_24_7_Series_4\Vets_24_7_Seri
es_4_-_4._Episode_4_b08r69t1_default.EXT

 Obviously, someone with proper perl-fu can write
less crude code to deal with this bug; tested on
Windows only; am confident the maintainer
will address this in next release...

Best regards,
Vangelis.