On Thu, Jul 24, 2008 at 11:13:24PM -0400, Charles Fry wrote:
> In that case do you mind just giving me an easy way to reproduce the
> bug,

Well, besides installing RT 3.8.0 or a Freeside CVS snapshot?  :)  

Subclass HTML::Mason::Request as described in the documentation 
(http://www.masonhq.com/docs/manual/Subclassing.html#request), i.e.:

package MyMasonRequest;

use strict;
use warnings;
use base 'HTML::Mason::Request';

sub new {
    my $class = shift;

    my $superclass = $HTML::Mason::ApacheHandler::VERSION ?
                     'HTML::Mason::Request::ApacheHandler' :
                     $HTML::Mason::CGIHandler::VERSION ?
                     'HTML::Mason::Request::CGI' :
                     'HTML::Mason::Request';

    $class->alter_superclass( $superclass );

    #huh... shouldn't alter_superclass take care of this for us?
    __PACKAGE__->valid_params( %{ $superclass->valid_params() } );

    $class->SUPER::new(@_);

}


and then use the subclass as described in the documentation
(http://www.masonhq.com/docs/manual/Subclassing.html#using_subclasses):

my $interp = HTML::Mason::Interp->new
          ( request_class  => 'MyMasonRequest' );

my $ah = HTML::Mason::ApacheHandler->new( interp => $interp );

> and thus test the patch, so that I can be somewhat confident that
> it is correct?

Yes, I'd be a bit wary about that too.  Wish the mason folks would at 
least respond.

-- 
_ivan


> On Thu, Jul 24, 2008 at 11:11 PM, Ivan Kohler <[EMAIL PROTECTED]> wrote:
> > On Thu, Jul 24, 2008 at 10:56:17PM -0400, Charles Fry wrote:
> >> Can someone from the Mason community please comment on this bug?
> >
> > No one commented on the original message I got the patch from (also sent
> > to mason-devel), and it was the only list traffic since March, so it may
> > be too much to hope someone responds in a week's time to make a decision
> > for lenny.
> >
> > It is worth pointing out that since the bug is 5.10-specific, this would
> > be regression from etch if left unfixed.
> >
> > I packaged a patched libhtml-mason-perl if that's useful.
> >
> > deb http://420.am/~ivan/freeside-sid/ ./
> >
> > --
> > _ivan
> >




-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]

Reply via email to