https://bugs.kde.org/show_bug.cgi?id=396286
Bug ID: 396286
Summary: KF5 ofximporter "Map account" fails on MS Windows
Product: kmymoney
Version: git (master)
Platform: MS Windows
OS: MS Windows
Status: UNCONFIRMED
Severity: normal
Priority: NOR
Component: importer
Assignee: kmymoney-de...@kde.org
Reporter: jeffjl....@outlook.com
Target Milestone: ---
I have a couple of problems with ofxpartner.cpp on a craft-mingw-Qt5.11-windows
build.
First, the "get" function of the OfxHttpRequest constructor does not work.
Maybe because KIO::copy does not work on Windows. Maybe KIO::copy doesn't work
because of QUrl problems on Windows (see problem #2, below). Or maybe this has
been overlooked because if you have an old ofx-bank-index.xml file laying
around, KMM will use that file even if it fails to fetch a new one.
I hacked around that issue by just commenting out the "if(type.toLower() ==
QStringLiteral("get"))" section around ofxpartner.cpp line 297 and always using
the "post" code. A full fix would just remove the "type" parameter from that
function (it was unused in KMM4).
I have not been able to figure out a "clean" solution to the second problem. It
has to do with QUrl paths. I can not find a combination of QUrl, QFile, QDir,
QFileInfo calls that will construct a file name that succeeds with QFile::open
in the slotOfxConnected() method. Even though QUrl.isLocalFile() returns
"true", the file path either does not include the drive letter, e.g.
"/users/jeff/ofx-bank-index.xml", or includes the drive letter looking like a
network name, e.g. "//c/users/jeff/ofx-bank-index.xml". QFile fails to open
either of those. This may only be a problem because my KMM program is on D: and
my app folder is on C:. If they were on the same drive, maybe the "/users/jeff"
path would work.
If I just hack in a string replacement to change the "//c/" to "c:/" to pass to
QFile::open(), then KMM will fetch a new ofx-bank-index.xml and everything's
good from there (assuming the "get" fix is in).
The "static QString directory" member variable is correctly "C:/users/jeff" at
the troublesome spot. It just gets lost in the QUrl.
--
You are receiving this mail because:
You are watching all bug changes.
