Moving KFIleMetadata into KDE SC (documentation and ExtractionResult)

Vishesh Handa me at vhanda.in
Wed Jan 22 17:47:37 GMT 2014


On Wednesday 22 January 2014 13:09:58 David Edmundson wrote:
> Add COPYING file

Fixed

> ----
> ExtractorPluginManager::fetchExtractors seems odd to me.
> If it can't find any plugins it searches for all plugins that start
> the same prefix.
> 
> I assume it's designed so I can have a plugin with the mimetype audio/
> that will still match the file mimetype audio/mp3
> But this means that if I add a special new plugin with the mimetype
> audio/mp3 the original plugin will stop running? Given you're trying
> to build a list of all valid plugins, is that meant to happen?
> 

Yes.

I've added the relevant documentation.

> --
> 
> ExtractionResult:
> 
> Having setInputUrl / setInputMimetype public in a way which can be set
> by the plugins seems wrong; personally I'd put the two in the
> constructor.
> 

Fixed

> 
> --
> 
> We should document the list of names of properties that can be extracted.
> Some things are in lowerCamelCase i.e "wordCount" "author" except for
> the eviv plugin which is in the form:
> Exif.Image.Make
> 
> and the odf extractor which is in the form
> 
> "dc:title"
> 
> to me it seems fairly random.

Fixed. Added a proper property and type system.

-- 
Vishesh Handa




More information about the kde-core-devel mailing list