Googlish/Googleish confusion in Googleish.pm

From the bottom of lib/RT/Search/Googleish.pm in the 3.6.0 release:

eval “require RT::Search::Googlish_Vendor”;
die $@ if ($@ && $@ !~ qr{^Can’t locate RT/Search/Googlish_Vendor.pm});
eval “require RT::Search::Googlish_Local”;
die $@ if ($@ && $@ !~ qr{^Can’t locate RT/Search/Googlish_Local.pm});

Googleish.pm has an E; Googlish_(Vendor)|(Local) does not. Confusion ensues!

I guess the Right Thing to do is to give it an E?

Thanks,
Tim Smith
Office of Compliance

Fix commited as revision 5418. Thanks.On 6/23/06, Timothy Smith tsmith@loc.gov wrote:

From the bottom of lib/RT/Search/Googleish.pm in the 3.6.0 release:

eval “require RT::Search::Googlish_Vendor”;
die $@ if ($@ && $@ !~ qr{^Can’t locate RT/Search/Googlish_Vendor.pm});
eval “require RT::Search::Googlish_Local”;
die $@ if ($@ && $@ !~ qr{^Can’t locate RT/Search/Googlish_Local.pm});

Googleish.pm has an E; Googlish_(Vendor)|(Local) does not. Confusion ensues!

I guess the Right Thing to do is to give it an E?

Thanks,
Tim Smith
Office of Compliance


List info: http://lists.bestpractical.com/cgi-bin/mailman/listinfo/rt-devel

Best Practical is hiring! Come hack Perl for us: http://bestpractical.com/about/jobs.html

Best regards, Ruslan.