[opencms-dev] bug in opencmslucene module, cmsfile extensions are case sensitive.

[M Butcher]

Ernesto,

Sorry for the delayed response. I've been on holiday.

I'll make the changes and add them into CVS.

Matt

Ernesto De Santis wrote:
> Hi Matt
> 
> I found a bug in the module, the extensions are case sensitive. .pdf != .PDF
> :(
> 
> My client have pdf´s with .pdf and .PDF extensions and the .PDF files are
> not indexed.
> 
> My suggestion is pass to lower case the extension.
> 
> as following:
> 
> in
> SearchParameters class:
> method: private ArrayList getExtensionMappings(Element ele_plain)
> line 545:
>  extMap.addExtension(extText);
>  ->
>  extMap.addExtension(extText.toLowerCase());
> 
> and
> 
> SearchParameters class:
> method: public I_FileDocumentFactory getFileDocumentFactory(CmsObject cmso,
> CmsFile file) throws CmsException
> line 235:
>  return getFileDocumentFactory(file.getExtension(), (ArrayList)temp);
>  ->
>  return getFileDocumentFactory(file.getExtension().toLowerCase(),
> (ArrayList)temp);
> 
> Bye
> Ernesto.
> 
> 
> 
> ---
> Outgoing mail is certified Virus Free.
> Checked by AVG anti-virus system (http://www.grisoft.com).
> Version: 6.0.703 / Virus Database: 459 - Release Date: 10/06/2004
> 
> _______________________________________________
> This mail is send to you from the opencms-dev mailing list
> To change your list options, or to unsubscribe from the list, please visit
> http://mail.opencms.org/mailman/listinfo/opencms-dev