Make enclosure follow WikiLink LinkingRules.
parent
45d58983d2
commit
0e278c4a84
|
@ -122,6 +122,13 @@ sub preprocess (@) {
|
||||||
return "";
|
return "";
|
||||||
}
|
}
|
||||||
elsif ($key eq 'enclosure') {
|
elsif ($key eq 'enclosure') {
|
||||||
|
my $link=bestlink($page, $value);
|
||||||
|
if (! length $link) {
|
||||||
|
error gettext("enclosure not found")
|
||||||
|
}
|
||||||
|
add_depends($page, $link, deptype("presence"));
|
||||||
|
|
||||||
|
$value=urlto($link, $page);
|
||||||
$pagestate{$page}{meta}{enclosure}=$value;
|
$pagestate{$page}{meta}{enclosure}=$value;
|
||||||
}
|
}
|
||||||
elsif ($key eq 'author') {
|
elsif ($key eq 'author') {
|
||||||
|
@ -322,7 +329,6 @@ sub pagetemplate (@) {
|
||||||
}
|
}
|
||||||
|
|
||||||
if (exists $pagestate{$page}{meta}{enclosure}) {
|
if (exists $pagestate{$page}{meta}{enclosure}) {
|
||||||
# XXX what if the enclosure doesn't exist?
|
|
||||||
$template->param(enclosure => $pagestate{$page}{meta}{enclosure});
|
$template->param(enclosure => $pagestate{$page}{meta}{enclosure});
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
40
t/podcast.t
40
t/podcast.t
|
@ -3,16 +3,18 @@ use warnings;
|
||||||
use strict;
|
use strict;
|
||||||
|
|
||||||
BEGIN {
|
BEGIN {
|
||||||
eval q{use XML::Feed; use HTML::Parser};
|
eval q{use XML::Feed; use HTML::Parser; use HTML::LinkExtor};
|
||||||
if ($@) {
|
if ($@) {
|
||||||
eval q{use Test::More skip_all =>
|
eval q{use Test::More skip_all =>
|
||||||
"XML::Feed and/or HTML::Parser not available"};
|
"XML::Feed and/or HTML::Parser not available"};
|
||||||
}
|
}
|
||||||
else {
|
else {
|
||||||
eval q{use Test::More tests => 77};
|
eval q{use Test::More tests => 78};
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
use Cwd;
|
||||||
|
|
||||||
my $tmp = 't/tmp';
|
my $tmp = 't/tmp';
|
||||||
my $statedir = 't/tinypodcast/.ikiwiki';
|
my $statedir = 't/tinypodcast/.ikiwiki';
|
||||||
|
|
||||||
|
@ -23,8 +25,8 @@ sub simple_podcast {
|
||||||
push @command, qw(-set underlaydirbase=underlays -templatedir=templates);
|
push @command, qw(-set underlaydirbase=underlays -templatedir=templates);
|
||||||
push @command, "-url=$baseurl", qw(t/tinypodcast), "$tmp/out";
|
push @command, "-url=$baseurl", qw(t/tinypodcast), "$tmp/out";
|
||||||
|
|
||||||
ok(! system("mkdir $tmp"));
|
ok(! system("mkdir $tmp"), q{setup});
|
||||||
ok(! system(@command));
|
ok(! system(@command), q{build});
|
||||||
|
|
||||||
my %media_types = (
|
my %media_types = (
|
||||||
'simplepost' => undef,
|
'simplepost' => undef,
|
||||||
|
@ -86,7 +88,7 @@ sub simple_podcast {
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
ok(! system("rm -rf $tmp $statedir"));
|
ok(! system("rm -rf $tmp $statedir"), q{teardown});
|
||||||
}
|
}
|
||||||
|
|
||||||
sub single_page_html {
|
sub single_page_html {
|
||||||
|
@ -95,8 +97,9 @@ sub single_page_html {
|
||||||
push @command, qw(-set underlaydirbase=underlays -templatedir=templates);
|
push @command, qw(-set underlaydirbase=underlays -templatedir=templates);
|
||||||
push @command, qw(t/tinypodcast), "$tmp/out";
|
push @command, qw(t/tinypodcast), "$tmp/out";
|
||||||
|
|
||||||
ok(! system("mkdir $tmp"));
|
ok(! system("mkdir $tmp"), q{setup});
|
||||||
ok(! system(@command));
|
ok(! system(@command), q{build});
|
||||||
|
|
||||||
my $html = "$tmp/out/pianopost/index.html";
|
my $html = "$tmp/out/pianopost/index.html";
|
||||||
|
|
||||||
my $body = _extract_html_content($html, 'content');
|
my $body = _extract_html_content($html, 'content');
|
||||||
|
@ -105,10 +108,12 @@ sub single_page_html {
|
||||||
my $enclosure = _extract_html_content($html, 'enclosure');
|
my $enclosure = _extract_html_content($html, 'enclosure');
|
||||||
like($enclosure, qr/Download this episode/m, q{html enclosure});
|
like($enclosure, qr/Download this episode/m, q{html enclosure});
|
||||||
|
|
||||||
# XXX die if specified enclosure doesn't exist
|
my ($href) = _extract_html_links($html, 'piano');
|
||||||
|
ok(-f $href, q{html enclosure exists});
|
||||||
|
|
||||||
# XXX die if more than one enclosure is specified
|
# XXX die if more than one enclosure is specified
|
||||||
|
|
||||||
ok(! system("rm -rf $tmp $statedir"));
|
ok(! system("rm -rf $tmp $statedir"), q{teardown});
|
||||||
}
|
}
|
||||||
|
|
||||||
sub _extract_html_content {
|
sub _extract_html_content {
|
||||||
|
@ -139,5 +144,22 @@ sub _extract_html_content {
|
||||||
return $content;
|
return $content;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
sub _extract_html_links {
|
||||||
|
my ($file, $desired_value) = @_;
|
||||||
|
|
||||||
|
my @hrefs = ();
|
||||||
|
|
||||||
|
my $p = HTML::LinkExtor->new(sub {
|
||||||
|
my ($tag, %attr) = @_;
|
||||||
|
return if $tag ne 'a';
|
||||||
|
return unless $attr{href} =~ qr/$desired_value/;
|
||||||
|
push(@hrefs, values %attr);
|
||||||
|
}, getcwd() . '/' . $file);
|
||||||
|
|
||||||
|
$p->parse_file($file);
|
||||||
|
|
||||||
|
return @hrefs;
|
||||||
|
}
|
||||||
|
|
||||||
simple_podcast();
|
simple_podcast();
|
||||||
single_page_html();
|
single_page_html();
|
||||||
|
|
Loading…
Reference in New Issue