| Subject: | Bugs in YUI::Loader and feature requests |
| Date: | Tue, 25 May 2010 11:52:43 -0500 |
| To: | bug-YUI-Loader [...] rt.cpan.org |
| From: | Martin Holste <mcholste [...] gmail.com> |
Hi,
I was creating a YUI::Loader module and found yours already complete
and full-featured. However, there are some bugs. The module doesn't
work at all in its current incarnation because of a type-oh on line 5
of YUI/Loader/List.pm which has "requires" instead of "required" which
throws a Moose error and prevents anything from working. Because this
is a run-time error, it didn't show up in the smoke tests.
When the debug filter is on, it adds '-debug' to the css files, which
don't have debug versions. This makes the debug filter useless.
There are a lot of .js files referenced that have '-beta' and
'-experimental' suffixes which are no longer valid. This breaks most
modules.
The uploader module requires three additional modules so it should be:
'requires': ['yahoo', 'dom', 'event', 'element']
The other issues aren't exactly bugs but more feature requests: Could
you please add a paginator category to YUI/Loader/Catalog.pm and
update the current YUI version in YUI/Loader.pm's LATEST_YUI_VERSION
to be 2.8.1?
I think this would work for Catalog.pm:
'paginator': {
'type': 'js',
'path': 'paginator/paginator-min.js',
'requires': ['event', 'element'],
'optional': ['selector'],
'skinnable': true
}
also
"paginator":{name:"Paginator Control", type: "widget"}
Thanks very much.
--Martin