dailymotion: Add a more generic way of parsing the JSON structure

diff --git a/videosite/DailyMotionGrabber.pm b/videosite/DailyMotionGrabber.pm
index 981fef7..a98fae6 100644 (file)
--- a/videosite/DailyMotionGrabber.pm
+++ b/videosite/DailyMotionGrabber.pm
@@ -77,7 +77,7 @@ sub _parse {
         if ($e->[0] =~ m|\.addVariable\("sequence",\s*"([^\"]+)"|) {
             my $sequence = $1;
             my $jsp = videosite::JSArrayParser->new();
-            my $main;
+            my $l;
             my $s;
 
             $sequence =~ s/%(..)/chr(hex($1))/ge;
@@ -93,31 +93,22 @@ sub _parse {
             } else {
                 $self->debug("Parsed sequence: %s", Dumper($sequence));
 
-                foreach (@{$sequence}) {
-                    if (exists($_->{'name'}) && ($_->{'name'} eq 'main')) {
-                        # Found main section
-                        $main = $_->{'layerList'};
-                    }
-                }
-                unless(defined($main)) {
-                    $self->error("Could not find layerList[main]");
+                $l = $self->_fetch_layer($sequence, "root/layerList", "background/sequenceList", "main/layerList", "video/param");
+                unless(defined($l)) {
+                    $self->error("Could not find video layer");
                     return undef;
                 }
 
-                foreach (@{$main}) {
-                    if (exists($_->{'name'}) && ($_->{'name'} eq 'video')) {
-                        # Found video section
-                        if (exists($_->{'param'}->{'hdURL'})) {
-                            $metadata->{'DLURL'} = $_->{'param'}->{'hdURL'};
-                        } elsif (exists($_->{'param'}->{'hqURL'})) {
-                            $metadata->{'DLURL'} = $_->{'param'}->{'hqURL'};
-                        } elsif (exists($_->{'param'}->{'hqURL'})) {
-                            $metadata->{'DLURL'} = $_->{'param'}->{'sdURL'};
-                        } else {
-                            $self->error("Video section found, but no URLs");
-                            return undef;
-                        }
-                    }
+                # Found video section
+                if (exists($l->{'videoPluginParameters'}->{'hdURL'})) {
+                    $metadata->{'DLURL'} = $l->{'videoPluginParameters'}->{'hdURL'};
+                } elsif (exists($l->{'videoPluginParameters'}->{'hqURL'})) {
+                    $metadata->{'DLURL'} = $l->{'videoPluginParameters'}->{'hqURL'};
+                } elsif (exists($l->{'videoPluginParameters'}->{'hqURL'})) {
+                    $metadata->{'DLURL'} = $l->{'videoPluginParameters'}->{'sdURL'};
+                } else {
+                    $self->error("Video section found, but no URLs");
+                    return undef;
                 }
             }
         }
@@ -131,4 +122,37 @@ sub _parse {
     return $metadata;
 }
 
+sub _fetch_layer {
+    my $self = shift;
+    my $sequence = shift;
+    my $point = shift;
+    my $next;
+    my @points = @_;
+    my $l;
+
+    $self->debug("Looking for %s in %s", $point, Dumper($sequence));
+
+    unless(defined($point)) {
+        $self->debug("Reached last point");
+        return $sequence;
+    }
+    ($point, $next) = split(/\//, $point, 2);
+
+    foreach (@{$sequence}) {
+        if (exists($_->{'name'}) and ($_->{'name'} eq $point)) {
+            if (exists($_->{$next})) {
+                $self->debug("Using %s in %s", $next, $point);
+                return $self->_fetch_layer($_->{$next}, @points);
+            } else {
+                $self->debug("%s found, but no %s", $point, $next);
+                return undef;
+            }
+
+        }
+    }
+
+    $self->debug("Could not find entry named %s", $point);
+    return undef;
+}
+
 1;