In most uses of this page, Status is '' so %additional warns

3db519e8 refactored code into a MapStatus method, but it removed the
check to see if Status had a value.  Since most uses of this page
explicitly set Status = '' (since Status isn't changing) it causes a
warning in the logs any time you reply to an Incident.

MapStatus does 'return unless $Status', so you're guaranteed to get the
odd number of elements in a hash warning on reply.
Use the new MakeClicky cache added to RT

On a new enough version of RT, this subroutine will be executed on any
MakeClicky call.  This allows us to cache for 6 hours the results of
generating links to Lookup.html (in particular, on tickets with lots of
IPs in history, this is a huge win).  Even on tickets needing minimal
processing, it helps speed up display.

One interesting quirk of this code is that MakeClicky is run on Whois
lookups and traceroute.  Traceroute provides no useful details (not even
the ticket) so it's kind of a lost cause.  Whois on the other hand
provides lookup_params, and when we get that we should punt on the
caching so we don't provide links to the wrong ticket object.
Merge branch '3.0/incident-reply-warnings' into 3.0-trunk
Merge branch '3.0/makeclicky-cache' into 3.0-trunk
Add even more mentions of incompatibility with RT 4.2