> On Sept. 17, 2015, 9:28 nachm., Thomas Lübking wrote: > > a) fix the test? > > b) the patch first limits to a Url subset and then guesses what it was ... > > if that is the "fix" it smell like okular or the test scenario cannot deal > > with remote files? > > Jeremy Whiting wrote: > As I said in the description this isn't meant to fix the test, but to > remove another warning. The warning states that openDocument is ignored > because QUrl isn't a type registered with QtDBus. I agree and am not sure why > this function was changed from taking a QString url over DBus to trying (and > failing) to take a QUrl instead since QDBus doesn't know what QUrl even is.
No idea about why this is processed via dbus, but "qDBusRegisterMetaType<QUrl>();" - Thomas ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125297/#review85590 ----------------------------------------------------------- On Sept. 17, 2015, 9:46 nachm., Jeremy Whiting wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/125297/ > ----------------------------------------------------------- > > (Updated Sept. 17, 2015, 9:46 nachm.) > > > Review request for kdelibs and Albert Astals Cid. > > > Repository: okular > > > Description > ------- > > When running the mainshelltest the tests that fail report being unable to > call openDocument because of it's QUrl parameter. With this it no longer > complains about that (but the tests still fail). > > > Diffs > ----- > > shell/okular_main.cpp 1c988d9 > shell/shell.h c16a0b2 > shell/shell.cpp d0204f9 > > Diff: https://git.reviewboard.kde.org/r/125297/diff/ > > > Testing > ------- > > Test no longer complains about being unable to call openDocument. > > > Thanks, > > Jeremy Whiting > >