Added project dir to package read/save api

[rainman110]

Closes #617

Description

ABI CHANGE

• Added new package read/write functions to store miscelleneous data, like intelligraphs etc.
• The new functions are directly executed after calling the xml-based ones.
• Using the new APi, the currentProject()->path workaround can be avoided, which is error prone.
• Modules, that don't exploit this functionality require only a recompile
• Other modules (like intelligraph) should be adapted to use this new API

@mariusalexander I quickly adapted locally the intelligraph package to test this. Seems to be working. Please still have a look!

How Has This Been Tested?

Locally, with an adapted intelligraph.

Checklist:

• [ ] A test for the new functionality was added (if applicable).
• [ ] All tests run without failure.
• [ ] The changelog has been extended, if this MR contains important changes from the users's point of view.
• [ ] The new code complies with the GTlab's style guide.
• [ ] New interface methods / functions are exported via EXPORT. Non-interface functions are NOT exported.
• [ ] The number of code quality warnings is not increasing.

[rainman110]

@mariusalexander & @real-ct-ac Note, that we now must adapt the code of all modules using the package API.

This is a bit nasty, since we actually wanted to avoid this with the 2.1 release. The changes are minimal though but still required. Relying on the currentProject()->projectPath workaround is evil and should be erased in the long term.

Update: I think I have a better idea to overcome an API change. We add different function

• saveToProjectDir
• readFromProjectDir

These can be used, whenever non-xml files should be saved / read.

[mariusalexander]

Update: I think I have a better idea to overcome an API change. We add different function

• saveToProjectDir
  • readFromProjectDir

These can be used, whenever non-xml files should be saved / read.

So these functions ar now saveMiscData and readMiscData? I like the new API. It is a clever yet simple fix

[rainman110]

So these functions ar now saveMiscData and readMiscData? I like the new API. It is a clever yet simple fix

Correct. These functions should be used, when data should be read/saved that are not part of the xml document tree.

[mariusalexander]

Actually, I found a bug. The readMiscData and saveMiscData were not called if readData or saveData returned true. I had to change the operator && to ||.

[rainman110]

@mariusalexander This will be our first, true merged 2.1 MR :D

[mariusalexander]

@mariusalexander This will be our first, true merged 2.1 MR :D

Pioneers!