topic Re: Create new version with Behaviours/Policies in Alfresco Archive

Create new version with Behaviours/Policies

zhihailiu — Fri, 16 Dec 2016 19:50:54 GMT

I have a rather simple behaviour/policy implementation. When an aspect "signedDateAspect" is added to a document, check and set the value for the property "signedDate". The code (relevant part) is below.public class SignedDateAspectPolicy implements NodeServicePolicies.OnAddAspectPolicy {public void

Re: Create new version with Behaviours/Policies

zhihailiu — Fri, 16 Dec 2016 19:57:09 GMT

Forget to specify that it is tested on Alfresco 5.0

Re: Create new version with Behaviours/Policies

afaust — Sun, 18 Dec 2016 15:03:30 GMT

Your issue likely is related to the fact that you are using TRANSACTION_COMMIT for binding the policy. Unfortunately I see that quite a lot in blog posts or other community examples without people understanding implications / side effects of it. During "Upload New Version" a temporary node will be created (a working copy since "Upload New Version" consists of a check-out/check-in cycle) which will get deleted before the operation is complete. This node will cause the behaviour to be triggered, but since you specified TRANSACTION_COMMIT it will only be called at the end of the transaction after the temporary node has already been deleted again.

This is not related to how versions are created in general - it is just a specific of the "Upload New Version" UI action, which actually simulates three user actions (check-out, edit the working copy, check-in) to ensure some behavioural consistency to different ways a user could interact with the system and trigger these actions invidiually.

Re: Create new version with Behaviours/Policies

zhihailiu — Sun, 18 Dec 2016 16:47:50 GMT

Axel,

Thank you for your response. That clearly explained check in/out in "Upload New Version" and resulted "node not found". Extremely helpful.

As for the fix, do you think the "return if not exists" approach is acceptable? Or it should switch to other binding such as EVERY_EVENT?

Thanks again.

Re: Create new version with Behaviours/Policies

afaust — Sun, 18 Dec 2016 20:18:18 GMT

I prefer to use the EVERY_EVENT notification frequency in almost all instances. But that does not absolve the programmer from doing the necessary pre-condition checks, and "return if not exists" is one of the proper checks to perform for TRANSACTION_COMMIT. Another would be "is the node part of workspace://SpacesStore" by comparing the StoreRef of the NodeRef against the constant in StoreRef-class.

Re: Create new version with Behaviours/Policies

zhihailiu — Mon, 19 Dec 2016 13:58:57 GMT

Thank you!