Silence a warning that happens when a user possibly has a cookie but …

…is not logged in.
metacpan · Nov 19, 2016 · e2eaa8d · e2eaa8d
1 parent 6ab87ab
commit e2eaa8d
Showing 1 changed file with 7 additions and 0 deletions.
diff --git a/lib/MetaCPAN/Web/Controller/Account/Favorite.pm b/lib/MetaCPAN/Web/Controller/Account/Favorite.pm
@@ -8,6 +8,13 @@ sub auto : Private {
 
     # Needed to clear the cache
     my $user = $c->model('API::User')->get_profile( $c->token )->recv;
+
+    # Probably we shouldn't be getting to this point without a defined user,
+    # but sometimes we do.  Might have been an issue with the v0 => v1
+    # migration.
+
+    $c->detach('/forbidden') unless $user;
+
     $c->stash->{user} = $user;
 
     return 1;