Work update

[Jaroslaw Staniek]

On 21 June 2014 04:45, Harshi Mistry <harshimistry at gmail.com> wrote:
> Hi,
>
> I have been busy with exams in my college for past four days, so I have not
> been able to spend too much time.

Hello Harshita,
That's ok, good luck with exams!
Please refer to the shared calendar for particular deadlines.

> Though I figured out how to de-hardcode
> the path to access Java class in C++ code. I create jar file out of java
> file using cmake command add_jar() and install it in PLUGIN_INSTALL_DIR.
> Then I can access that jar file using KStandardDirs class, which provides
> locate() method to find absolute file paths of installed resources. I am
> still working on to make this work.

Good!

> There is one other problem which I am facing. I have installed .desktop file
> for OdbMigrate driver and defined its mime-type as
> 'application/vnd.oasis.opendocument.database'. So it clearly shows .odb
> option in dropdown menu of 'Choose file' dialogue box. But when I choose
> .odb file, kexi detects its mime-type as 'application/zip' and throws the
> error like 'no appropriate migration driver is available for this
> mime-type'. This is happening because odb files are actually zip files only,
> but kexi is somehow failing to detect correct mime-type of odb files. Please
> help me in solving this.

1. I am assuming you're working on addressing issues I raised at
https://git.reviewboard.kde.org/r/118736/.
Some are minor ones but in my opinion better to fix them (and get
habit to avoid them in the future) before too much is to fix.

2. I see the updated patch from you -
https://git.reviewboard.kde.org/r/118736/diff/2/#index_header  -
revision 2, but it if I click Download Diff the patch looks like an
incremental one. you can compare what appears when you click Jump to
revision: 1 then Download Diff, and what appears when you click Jump
to revision: 2 then Download Diff.

I recommend always uploading full updated patch to reviewboard.

3. In particular I mentioned that the plugin won't be found without
the K_EXPORT_KEXIMIGRATE_DRIVER(OdbMigrate, "odb"):

https://git.reviewboard.kde.org/r/118736/diff/1-2/?file=281101#file281101line15

You also do not seem to inherit KexiMigrate.
These are main reasons for not finding the plugin (the file exists but
it has no interface matching KexiMigrate).

Please note, I do not see relevant code at
https://github.com/harshitamistry/calligraRepo.
So maybe you have proper code but I just do not see it on reviewboard
on in the github.
I suppose your code is different since the patch I see has
odbmigrate.h that does not compile.
Could you push changes to that repo so I am sure I am looking at
actual source code?

4. Regarding the extenstion .odb, this is how Linux should found it:
I see the /usr/share/mime/application/vnd.oasis.opendocument.database.xml
file delivering the "application/vnd.oasis.opendocument.database" mime
type. Can you see this file on your system? You properly referenced
this type in keximigrate_odb.desktop.

Good (and bad) news: I found an issue in Kexi source code, so it does
not supports extra migration plugins (the MDB works because it's
hardcoded), I expect a proper fix in master branch by this Wednesday.

Thank you for your hard work.

-- 
regards / pozdrawiam, Jaroslaw Staniek
 Kexi & Calligra & KDE | http://calligra.org/kexi | http://kde.org
 Qt for Tizen | http://qt-project.org/wiki/Tizen
 Qt Certified Specialist | http://www.linkedin.com/in/jstaniek