[geeklog-devtalk] geeklog-devel digest, Vol 1 #251 - 1 msg

[geeklog-devel-request at lists.geeklog.net]

Send geeklog-devel mailing list submissions to
	geeklog-devel at lists.geeklog.net

To subscribe or unsubscribe via the World Wide Web, visit
	http://lists.geeklog.net/listinfo/geeklog-devel
or, via email, send a message with subject or body 'help' to
	geeklog-devel-request at lists.geeklog.net

You can reach the person managing the list at
	geeklog-devel-admin at lists.geeklog.net

When replying, please edit your Subject line so it is more specific
than "Re: Contents of geeklog-devel digest..."


Today's Topics:

   1. Re: Testing of getimage.php (Dirk Haun)

--__--__--

Message: 1
From: "Dirk Haun" <dirk at haun-online.de>
To: <geeklog-devel at lists.geeklog.net>
Subject: Re: [geeklog-devel] Testing of getimage.php
Date: Sun, 11 Jan 2004 20:13:29 +0100
Organization: Terra Software Systems
Reply-To: geeklog-devel at lists.geeklog.net


>I have committed changes to CVS that use getimage.php.  That file allows 

>images outside of a webtree to be viewed (i.e. userphotos, article 

>images).


I've played a bit with it (userphotos only) and noticed a few minor things:

In "show" mode, getimage.php could display a proper error message.
Otherwise, it should at least send a "404 Not Found" header when the
image does not exist (added both).

The calls to header() (in the download class) don't need to end in a
"\n", so I removed them.

In Mozilla, right-clicking on the userphoto and selecting "View Image"
from the context menu prompts a dialog to download the image. This is
because of the "Content-Disposition:" header sent by the download class.
For an actual download, that is correct, but for an image? I guess this
could be an option, but I left it as it is.

Looks good 

-- 
http://www.haun-online.de/
http://www.macosx-faq.de/



--__--__--

_______________________________________________
geeklog-devel mailing list
geeklog-devel at lists.geeklog.net
http://lists.geeklog.net/listinfo/geeklog-devel


End of geeklog-devel Digest