[PATCH] Do not handle form enctype="text/html" as enctype="text/plain"

David Faure faure at kde.org
Thu May 11 17:15:34 BST 2006


On Thursday 11 May 2006 15:16, Aurélien Gâteau wrote:
> -    } else if (type.string().find("text", 0, false) != -1 || type.string().find("plain", 0, false) != -1)
> +    } else if (type.string().find("text", 0, false) != -1 && type.string().find("plain", 0, false) != -1)

Shouldn't this become type.string() == "text/plain" then? It seems strange to
do two substring matches if the only possibility that makes sense is "text/plain" anyway...

-- 
David Faure, faure at kde.org, sponsored by Trolltech to work on KDE,
Konqueror (http://www.konqueror.org), and KOffice (http://www.koffice.org).





More information about the kfm-devel mailing list