I might be mistaken, but it looks like you are not actually inheriting from CGI?

It may be easier to subclass it. CGI has a function for mixing and matching the object oriented and procedural interface called CGI::self_or_default(@_);

Used like so in your overridden functions it will either return the current CGI (or CGI inherited object) or created the default $Q.

my ($self,@p) = CGI::self_or_default(@_);
[download]

One problem I have noticed with mixing and matching the OO and functional interface while inheriting is that functional calls always appear to use $CGI::Q whether you have created an object via new or not. This can be a problem if you are inheriting from CGI and want to override param to store additional information in the object(like debugging)as they will get stored in different instances depending on how they where called.

A quick fix for me was using new in the following manner.

$CGI::Q = new CGI::DebugModule();
# Mix and match away
[download]

You could always provide your own self_or_default to be smart enought to use an existing object if one has been created via new as well.

-Lee

"To be civilized is to deny one's nature."