Skip Menu |

This queue is for tickets about the HTML-WikiConverter-MoinMoin CPAN distribution.

Report information
The Basics
Id: 18083
Status: resolved
Worked: 15 min
Priority: 0/
Queue: HTML-WikiConverter-MoinMoin

People
Owner: diberri [...] cpan.org
Requestors: matthew.brett [...] gmail.com
Cc:
AdminCc:

Bug Information
Severity: Important
Broken in: (no value)
Fixed in: (no value)



Subject: Tests fail with complaint about strip_comments
On installation test rig, I get: ... t/moinmoin........The following parameter was passed in the call to HTML::WikiConverter::__validate_attributes but was not listed in the validation options: strip_comments at t/runtests.pl line 74 # Looks like your test died before it could output anything.
On Thu Mar 09 17:44:44 2006, guest wrote: Show quoted text
> On installation test rig, I get: > > ... > t/moinmoin........The following parameter was passed in the call to > HTML::WikiConverter::__validate_attributes but was not listed in the > validation options: strip_comments > at t/runtests.pl line 74 > # Looks like your test died before it could output anything.
I haven't tried to reproduce this myself, but I know what's going on. In H::WC 0.52 the strip_* attributes have been coalesced into a single strip_tags attribute which takes an arrayref. The test failure can safely be ignored; force install from cpan and H::WC should work fine with the MoinMoin dialect (so long as you don't attempt to use the old strip_* attributes). I'm working on a compatibility patch for H::WC that I'm planning to release with the 0.53 update. This will allow you to use the strip_* tags for the sake of backwards compatibility. (Though you could probably convince me quite easily just to patch MoinMoin and the other dialects so they don't use the old strip_* attributes. I'm open to suggestions.) Thanks for the report. -- David Iberri
Subject: Re: [rt.cpan.org #18083] Tests fail with complaint about strip_comments
Date: Sat, 11 Mar 2006 21:27:46 +0000
To: bug-HTML-WikiConverter-MoinMoin [...] rt.cpan.org
From: "Matthew Brett" <matthew.brett [...] gmail.com>
Hi, Show quoted text
> The test failure can safely be ignored; force install from cpan and > H::WC should work fine with the MoinMoin dialect (so long as you don't > attempt to use the old strip_* attributes).
Thanks a lot for the reply; this is what I did - and it has been working well. Show quoted text
> I'm working on a compatibility patch for H::WC that I'm planning to > release with the 0.53 update. This will allow you to use the strip_* > tags for the sake of backwards compatibility. (Though you could probably > convince me quite easily just to patch MoinMoin and the other dialects > so they don't use the old strip_* attributes. I'm open to suggestions.)
Ah - I'm afraid I have no view on that - but thanks for this nice utility and the support, Best, Matthew
On Sat Mar 11 14:04:45 2006, DIBERRI wrote: Show quoted text
> On Thu Mar 09 17:44:44 2006, guest wrote:
> > On installation test rig, I get: > > > > ... > > t/moinmoin........The following parameter was passed in the call to > > HTML::WikiConverter::__validate_attributes but was not listed in the > > validation options: strip_comments > > at t/runtests.pl line 74 > > # Looks like your test died before it could output anything.
> > I haven't tried to reproduce this myself, but I know what's going on.
This has been fixed in H::WC::MoinMoin 0.51. -- David Iberri