dfaure added a comment.
I'm OK with "only" a documentation fix at the KParts level. If you reimplement openUrl, you get to handle that URL, obviously. That includes possibly doing the async job that resolves it to a local file path. "making ReadOnlyPart::setUrl() set d->m_file if the url is a local file" would be fine if you mean QUrl::isLocalFile(), but NOT if you mean another KJob::exec(). So it wouldn't help. REPOSITORY R306 KParts REVISION DETAIL https://phabricator.kde.org/D27148 To: pdabrowski, elvisangelaccio, ngraham, #frameworks, dfaure Cc: ahmadsamir, marten, kde-frameworks-devel, LeGast00n, cblack, GB_2, michaelh, ngraham, bruns
