Cannot share folders with the MDM plugin

bhuisman

@mcostan / @tosty I added some background to https://jira.kopano.io/projects/KMP/issues/KMP-65 and we’ll check it out soon

tosty

@bhuisman

Seems, this little change fixes the problem:

--- class.pluginmdmmodule.php.orig      2018-08-05 20:13:22.903000354 +0200
+++ class.pluginmdmmodule.php   2018-08-06 14:18:36.403006245 +0200
@@ -795,7 +795,7 @@
        if ($subfolder[PR_SUBFOLDERS]) {
                if ($specialFolder) {
                        // Special folders can be redirected again to getSubFolders(),
-                       $this->getSubFolders($specialFolder, $store, $properties, $storeData, $subfolder[PR_ENTRYID], $storeUserName);
+                       $this->getSubFolders($specialFolder, $store, $properties, $storeData, $storeUserName);
                } else {
                        $folderObject = mapi_msgstore_openentry($store, $subfolder[PR_ENTRYID]);
                        array_push($expand, array('folder' => $folderObject, 'props' => $subfolder));```

mcostan

@tosty

That’s great!

@bhuisman

Let us know when this is in the MDM build and I can test and report back if this fixes the issue on my side.

Thanks a lot!

bhuisman

@tosty Cool! If you could create your patch following this guide: https://stash.kopano.io/projects/KW/repos/kopano-webapp/browse/CONTRIBUTING.md we can put it in the code as a contribution.

tosty

Well, I tried to get this done yesterday. Sent the patch via email.

https://git.masilux.de/tosty/KWA-MDM/commit/7a5722e940bec0e61dca89957c55e81343ca5258

bhuisman

@tosty thanks! I found your email and exported a patch - it is attached to the ticket.

mcostan

@bhuisman

Good afternoon, I was wondering when the fix will be available in the overnight community builds.

marty

@mcostan
Better to track https://jira.kopano.io/browse/KMP-65

mcostan

@marty

Good afternoon, I noticed the JIRA is now set to completed. Can I assume that the community overnight builds will include the fix?

marty

@mcostan
Yes and I’ve put them online already

mcostan

Great thank you for your help. I’ll pick up the changes and test.

mcostan

@marty

Installed and tested and I can confirm that this fixes the bug. Much appreciated in particular to @tosty for finding the solution!

tosty

@mcostan
You’re welcome!

PRO123

… and unfortunately this tiny but very important fix (thx to @tosty) is still not released with a supported MDM plugin version … and that 8 months later?

So I really wonder what we pay subscription for?

Since yesterday we wanted to make shared calendars and address books from the public folders available for mobile devices (can’t imagine anyone else doing that?) and have wasted some time because of this (long solved) bug because we as paying customer use the supported repo :/

Just not satisfied
Robert

marty

@PRO123 I’m sorry you feel this way, but you could have also send an email to support and ask about a status update, since you have a paid subscription.

In the mean time I will check if an update version can be released.

PRO123

@marty said in Cannot share folders with the MDM plugin:

but you could have also send an email to support and ask about a status update, since you have a paid subscription.

Yes we could contact the support every time, but you could save effort for everyone (also for Kopano support), if you release (important) fixes also in the supported repo, at least not sometime after (8) months.

Btw. thanks for your quick reaction here.

marty

@PRO123 You are correct.
The new version can be found in the repositories. Short announcement will be posted later.

PRO123

@marty said in Cannot share folders with the MDM plugin:

The new version can be found in the repositories.

THX a lot for your quick reaction & have a nice weekend :)