Merge pull request #1803 from metacpan/oalders/silence-concat-warning

Silence a warning that happens when a user possibly has a cookie but …
metacpan · Nov 19, 2016 · 2e10399 · 2e10399
2 parents e25a01a + e2eaa8d
commit 2e10399
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;