Missing response elements in photos->by_id method.

[dbough]

Wondering if there is a reason a large portion of elements aren't being returned by the by_id subroutine in the Photos.pm module?

sub by_id {
    my ($self, $id) = @_;
    my $p = Flickr::API2::Photo->new( api => $self->api, id => $id );
    my $info = $p->info->{photo};
    $p->title($info->{title}{_content});
    $p->description($info->{description}{_content});
    $p->owner_name($info->{owner}{realname});
    $p->owner_id($info->{owner}{nsid});
    $p->path_alias($info->{owner}{username});

    return $p;
}

I've modified this (and some portion of Photo.pm) so that I can get some elements I need for a project. Is there another way to get this info that I've missed?:

sub by_id {
    my ($self, $id) = @_;
    my $p = Flickr::API2::Photo->new( api => $self->api, id => $id );
    my $info = $p->info->{photo};
    $p->title($info->{title}{_content});
    $p->description($info->{description}{_content});
    $p->owner_name($info->{owner}{realname});
    $p->owner_id($info->{owner}{nsid});
    $p->path_alias($info->{owner}{username});
    $p->license($info->{license});
    $p->tags($info->{tags}{tag});
    $p->dates($info->{dates});

    return $p;
}

If not, I can tweak it a bit more and submit a pull request.

[TJC]

A pull request would be great, thanks.

[TJC]

Thanks for providing the patch. I'll make a new release to CPAN soon.
Did you have any plans for further patches? If so, I'll hold off on the new release for a while.

[dbough]

The only other piece that I'd like to address is in issue #8. I'm not smart enough to determine how to do it w/in this framework though =). Is it something you can look at? I can write the code if you have an idea on a direction.

[TJC]

I have a couple of ideas around that, but won't have time to look at it until next week. I'll get back to it then, then.