summary |
shortlog |
log |
commit | commitdiff |
tree
raw |
patch |
inline | side by side (from parent 1:
007970e)
- remove code duplication
- allow easier implementation of asynchronous getters
my $video = shift;
my $dlfile;
my $dirname;
my $video = shift;
my $dlfile;
my $dirname;
$dlfile = sprintf($self->_getval('FILEPATTERN'),
$self->_encode($video->{'SOURCE'}),
$dlfile = sprintf($self->_getval('FILEPATTERN'),
$self->_encode($video->{'SOURCE'}),
if (exists($video->{'CONNECTOR'})) {
$self->selectconn($video->{'CONNECTOR'});
}
if (exists($video->{'CONNECTOR'})) {
$self->selectconn($video->{'CONNECTOR'});
}
- $self->debug('Going to download %s to %s', $video->{'DLURL'}, $dlfile);
- $res = $ua->mirror($video->{'DLURL'}, $dlfile);
- if (!$res->is_success()) {
- $self->error('Could not download %s to %s (%s)', $video->{'DLURL'}, $dlfile, $res->code());
- return 0;
- }
+ $self->_download( $video->{'DLURL'}, $dlfile, $video )
+ or return 0;
if (exists($video->{'DLURL_AUDIO'})) {
if (exists($video->{'DLURL_AUDIO'})) {
- $dlfile = $dlfile . ".audio";
- $self->debug('Going to download %s to %s', $video->{'DLURL_AUDIO'}, $dlfile);
-
- $res = $ua->mirror($video->{'DLURL_AUDIO'}, $dlfile);
- if (!$res->is_success()) {
- $self->error('Could not download %s to %s (%s)', $video->{'DLURL_AUDIO'}, $dlfile, $res->code());
- return 0;
- }
+ $self->_download( $video->{'DLURL_AUDIO'}, $dlfile . '.audio', $video )
+ or return 0;
+sub _download {
+ my $self = shift;
+ my $dlurl = shift;
+ my $dlfile = shift;
+ my $video = shift;
+ my $res;
+
+ $self->debug('Going to download %s to %s', $dlurl, $dlfile);
+
+ $res = $self->ua()->mirror($dlurl, $dlfile);
+
+ if (!$res->is_success()) {
+ $self->error('Could not download %s to %s (%s)', $dlurl, $dlfile, $res->code());
+ return 0;
+ }
+
+ return 1;
+}
+
sub _encode {
my $self = shift;
my $s = shift;
sub _encode {
my $self = shift;
my $s = shift;