chaas/debshots
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages 1 Wiki Activity Actions

Properly return a 404 if an existing package has no screenshot

Browse source
This commit is contained in:
Christoph Haas 2016-06-16 10:42:04 +02:00
parent d7771f1a60
commit 5d856c61e6
1 changed files with 3 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

4
app/controllers/packages_controller.rb
View file

@ -82,7 +82,9 @@ class PackagesController < ApplicationController
return return
end end
@screenshot = @package.screenshots.first @screenshot = @package.screenshots.first
unless @screenshot.image.path
# Return a 404 if the package has no screenshots or the image was not found
unless @screenshot or @screenshot.image.path
thumbnail404 thumbnail404
return return
end end