favorite/agg_by_distributions: 'user' --> optional param

read_param is meant for mandatory params as it detaches when value is not found. In this case, 'user' is an optional param and always single-value, so we don't need to use it.
metacpan · Jul 27, 2017 · 7b6608e · 7b6608e
1 parent 1a472ad
commit 7b6608e
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/lib/MetaCPAN/Server/Controller/Favorite.pm b/lib/MetaCPAN/Server/Controller/Favorite.pm
@@ -79,7 +79,7 @@ sub agg_by_distributions : Path('agg_by_distributions') : Args(0) {
     my ( $self, $c ) = @_;
 
     my $distributions = $c->read_param('distribution');
-    my $user          = $c->read_param('user');
+    my $user          = $c->req->param('user');           # optional
     my $data
         = $self->model($c)->agg_by_distributions( $distributions, $user );