731101 - Move image fallback logic out of htaccess

Status: RESOLVED FIXED

(addons.mozilla.org Graveyard :: Public Pages, defect)

Description

[Wil Clouser [:clouserw]]

Right now if an image doesn't exist on disk we show a fallback image.  There's no reason for this to live in apache.  If a $object doesn't have an image, send the fallback image URL.  If someone manually goes to a bad image, they'll get a 404, as expected.  Please avoid bug 731089 and bug 731097 when you're doing this or they'll be conflicts.  Rewrites:

>    ## fall userpics on anon
>    RewriteCond %{REQUEST_FILENAME} !-f
>    RewriteRule ^img/uploads/userpics/.* /img/anon_user.png [L]
>
>    ## fall addon icons on default
>    RewriteCond %{REQUEST_FILENAME} !-f
>    RewriteRule ^img/uploads/addon_icons/.* /img/default_icon.png [L]
>
>    ## fall collection icons on collection.png
>    RewriteCond %{REQUEST_FILENAME} !-f
>    RewriteRule ^img/uploads/collection_icons/.* /img/collection.png [L]

Comment 1

[Wil Clouser [:clouserw]]

(In reply to Wil Clouser [:clouserw] from comment #0)
> Right now if an image doesn't exist on disk we show a fallback image. 
> There's no reason for this to live in apache.  If a $object doesn't have an
> image, send the fallback image URL.  If someone manually goes to a bad
> image, they'll get a 404, as expected.  Please avoid bug 731089 and bug
> 731097 when you're doing this or they'll be conflicts.  Rewrites:
> 
> >    ## fall userpics on anon
> >    RewriteCond %{REQUEST_FILENAME} !-f
> >    RewriteRule ^img/uploads/userpics/.* /img/anon_user.png [L]

apps/users/models.py line 189

> >    ## fall addon icons on default
> >    RewriteCond %{REQUEST_FILENAME} !-f
> >    RewriteRule ^img/uploads/addon_icons/.* /img/default_icon.png [L]

This is settings.ADDON_ICONS_DEFAULT_URL.  We're good there.

> >    ## fall collection icons on collection.png
> >    RewriteCond %{REQUEST_FILENAME} !-f
> >    RewriteRule ^img/uploads/collection_icons/.* /img/collection.png [L]

apps/bandwagon/models.py line 264



We're good here.