[Development] QHelpEngineCore::documentsForIdentifier

[Martin Koller]

Hi,

I found myself getting an empty list when using QHelpEngineCore::documentsForIdentifier(id)
but getting 1 element when using the deprecated QHelpEngineCore::linksForIdentifier(id).

Checking the code I stumbled over something which I believe is a bug:

QHelpEngineCore::documentsForIdentifier(const QString &id, const QString &filterName)
does
if (!d->setup() || !d->usesFilterEngine)
        return QList<QHelpLink>();

so it checks if the new filter engine is active, but I think this is not needed here,
since in this code, the filter engine is not used at all, since the filterName
is already given as argument.

Since I ported older code to the new api but did not call setUsesFilterEngine(true)
(which I believe should not be needed when I don't work with filters),
this check was hit and I get no results.

Am I right ?
-- 
Best regards/Schöne Grüße

Martin
A: Because it breaks the logical sequence of discussion
Q: Why is top posting bad?

()  ascii ribbon campaign - against html e-mail 
/\                        - against proprietary attachments

Frühstück, Geschenkideen, Accessoires, Kulinarisches: www.lillehus.at