Commit 81b7c2aa authored by Volker Krause's avatar Volker Krause
Browse files

Handle multipart/related MIME sub-trees

This puts images referenced by HTML nodes below that node now, contrary
to the MIME structure, so HTML extractor scripts have access to images
(and their contained barcode content) that are referenced by cid: URL.
parent a936bbaa
Pipeline #83025 passed with stage
in 16 minutes and 22 seconds
......@@ -104,7 +104,7 @@ ExtractorDocumentNode MimeDocumentProcessor::createNodeFromContent(const QVarian
return node;
static void expandContentNode(ExtractorDocumentNode &node, KMime::Content *content, const ExtractorEngine *engine)
static ExtractorDocumentNode expandContentNode(ExtractorDocumentNode &node, KMime::Content *content, const ExtractorEngine *engine)
QString fileName;
const auto ct = content->contentType(false);
......@@ -125,17 +125,33 @@ static void expandContentNode(ExtractorDocumentNode &node, KMime::Content *conte
child = engine->documentNodeFactory()->createNode(content->decodedContent(), fileName);
return child;
static void expandContentNodeRecursive(ExtractorDocumentNode &node, KMime::Content *content, const ExtractorEngine *engine)
const auto ct = content->contentType(false);
const auto children = content->contents();
if (!content->contentType(false) || children.empty()) {
if (!ct || children.empty()) {
expandContentNode(node, content, engine);
// TODO special handling of multipart/related to add images to the corresponding HTML document
// special handling of multipart/related to add images to the corresponding HTML document
if (ct && ct->isMultipart() && ct->isSubtype("related") && ct->parameter(QLatin1String("type")) == QLatin1String("text/html") && children.size() >= 2) {
const auto child = children.front();
if (child->contentType(false) && child->contentType(false)->isHTMLText()) {
auto htmlNode = expandContentNode(node, child, engine);
for (auto it = std::next(children.begin()); it != children.end(); ++it) {
auto imgNode = expandContentNode(htmlNode, (*it), engine);
const auto cid = (*it)->contentID(false);
if (cid) {
for (const auto child : children) {
expandContentNodeRecursive(node, child, engine);
Supports Markdown
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment