Metadata was extracted unconditionally before.
Only do so if it is actually requested.
Details
Details
- Reviewers
bruns ngraham - Group Reviewers
Baloo - Commits
- R286:538c34b58057: honor the extractMetaData flag
Diff Detail
Diff Detail
- Repository
- R286 KFileMetaData
- Branch
- honorFlag
- Lint
No Linters Available - Unit
No Unit Test Coverage - Build Status
Buildable 25937 Build 25955: arc lint + arc unit
| src/extractors/office2007extractor.cpp | ||
|---|---|---|
| 79 | Long line bool extractMetaData = ...
if (extractMetaData && docPropsEntries.contains(QStringLiteral("core.xml")) { | |
| 144 | Long line if (extractMetaData && docPropsEntries.contains(QStringLiteral("app.xml")) { | |
| 185 | ... like done here | |
| src/extractors/taglibextractor.cpp | ||
| 99 | when you add the flag check here, you can avoid the extra indentation below. Likewise in some other places. | |
| src/extractors/xmlextractor.cpp | ||
| 103–107 | correct place for the flags check | |
| src/extractors/exiv2extractor.cpp | ||
|---|---|---|
| 178 | if (!....) return | |
| src/extractors/taglibextractor.cpp | ||
| 99 | Still trivially possible: if (savedProperties.isEmpty()) {
return;
}
if (extractFulltext) {
// extract lyrics
}
if (!extractMetadata) {
return
} | |
| src/extractors/xmlextractor.cpp | ||
| 103–107 | skipped, if (!....) { continue; } | |
| src/extractors/xmlextractor.cpp | ||
|---|---|---|
| 103–107 | sorry, that went lost somehow | |