Favorite model: check input args before URL concat

This will fix some log warnings.
metacpan · Jul 11, 2017 · 31a9d12 · 31a9d12
1 parent 8ccb288
commit 31a9d12
Showing 1 changed file with 2 additions and 0 deletions.
diff --git a/lib/MetaCPAN/Web/Model/API/Favorite.pm b/lib/MetaCPAN/Web/Model/API/Favorite.pm
@@ -23,6 +23,7 @@ sub get {
 
 sub by_user {
     my ( $self, $user, $size ) = @_;
+    return Future->done unless $user;
     $size ||= 250;
     my $ret
         = $self->request( "/favorite/by_user/$user", { size => $size } )
@@ -73,6 +74,7 @@ sub leaderboard {
 
 sub find_plussers {
     my ( $self, $distribution ) = @_;
+    return Future->done unless $distribution;
 
     # search for all users, match all according to the distribution.
     $self->request("/favorite/users_by_distribution/$distribution")->then(