Fixed a bug: we used to prefetch item_id, which gets discarded and then
grab a new one in the proc. Now we are relying on the caller to provide
item_id prefetched on the form. (This of course sucks donkeyballs but
at least it is not broken now).
Also using a character entity instead of $ in $Id$ to prevent keyword
expansion.
Plus, I added usage of the -message flag on the ad_returnedirect.
