TWCable / grabbit

Grabbit - Fast Content Sync tool for AEM/CQ
Apache License 2.0
125 stars 64 forks source link

rep:cugPolicy Nodes Failing Sync #202

Open dscirto opened 6 years ago

dscirto commented 6 years ago

CUG policies on pages are causing syncs to fail (rep:cugPolicy nodes). It looks like it's related to this issue, since it complains about the mandatory rep:principalNames property missing.

I've been working on setting up scheduled syncs via the Sling Scheduler and a Granite UI tool for ad hoc syncing. The goal is to keep our environments and local environments reasonably synced to aid development and testing... So we can't unfortunately just exclude these nodes.

Am I right in assuming this is probably the expected behavior at this point and it wasn't addressed as a byproduct of something like the user/group support?

What version of AEM are you running? AEM 6.3 with SP1

How do you produce the issue? Add a CUG policy to a page (edit page properties, permissions tab, edit closed user group) and then try to sync the page. The rep:cugPolicy sub-node that gets created causes the error when trying to sync.

Does the issue occur consistently? Yes

jbornemann commented 6 years ago

CUG support was introduced in 6.3. Our work for other rep type nodes was done before 6.3.

It just simply hasn’t been implemented. Probably not too difficult.

dscirto commented 6 years ago

Ah, okay. That makes sense.

jbornemann commented 6 years ago

Want to open a PR :) ? Sagar and I will help with questions

dscirto commented 6 years ago

I don't have any time available to work on it, unfortunately. If I get a little break in the action, I'll definitely poke around