Googlish/Googleish confusion in Googleish.pm

RT Developers
1

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

2

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: The rt-devel Archives

Best Practical is hiring! Come hack Perl for us: Careers — Best Practical Solutions

Best regards, Ruslan.