topic Re: How to get the proper exception message to the CMIS client in Alfresco Forum

How to get the proper exception message to the CMIS client

signjoy — Thu, 28 Jun 2018 14:30:36 GMT

Hi,We have a policy that moves the document according to the metadata called state id. So if the state id is 123456. It will move the document to 123 -> 456 -> 123456 -> c.pdfAs and when document comes into Alfresco through JAVA CMIS call, we check using NodeService whether the document is

Re: How to get the proper exception message to the CMIS client

afaust — Fri, 29 Jun 2018 08:13:01 GMT

You will never get the proper exception message from a transaction-level policy (bound using NotificationFrequency.TRANSACTION_COMMIT) to any client - CMIS or otherwise. All exceptions from such policies are wrapped and only via a stacktrace (not available to clients) could you get at the real cause.

This is one of the many reasons I typically recommend to NOT use transaction-level policies at all, and none of the policies I write are ever transaction-level. I do use custom transaction listeners combined with regular EVERY_EVENT policies on occasion to aggregate multiple events into a single processing step at the end of a transaction, but at least if a listener throws an exception, it is not wrapped excessively on the way up the call-chain.

Re: How to get the proper exception message to the CMIS client

signjoy — Fri, 29 Jun 2018 17:46:16 GMT

Axel Faust‌

If I keep policy for EVERY_EVENT then it will be called multiple times for a single file. I need to keep checking the metadata whether they are updated or not that I want to manipulate. So if I have 20 metadata then the code will run 20 times.. is that a good practice ?

But yes I have tested with EVERY_EVENT and it shows the actual exception thrown by the policy...

Thank you

Re: How to get the proper exception message to the CMIS client

afaust — Fri, 29 Jun 2018 19:12:06 GMT

You will only get called 20 times for 20 properties if some code in Alfresco is extremely inefficient and calling NodeService.setProperty for each property instead of calling NodeService.addProperties once for all properties. Unfortunately, it looks like the Alfresco CMIS implementation is that inefficient (I just checked the source).

But still, you should check in the code of your policy if the change is actually interesting for you, and such a check can be implement quite efficiently with two gets on the before/after maps and a single call to EqualsHelper.nullSafeEquals to check for a change/no-change.

Alfresco will always recommend using TRANSACTION_COMMIT because they have to deal with a lot of unqualified 3rd-party / customer developers that do not know how to properly / efficiently implement an EVERY_EVENT policy. I just see too many problems with TRANSACTION_COMMIT in almost every customer project where a non-professional (i.e. someone with little Alfresco development experience) is working on behaviour code, because the people know even less about properly using that frequency than they regarding to EVERY_EVENT.

Re: How to get the proper exception message to the CMIS client

signjoy — Fri, 29 Jun 2018 20:19:03 GMT

Thanks Axel Faust‌ for your time and information...

As we have lot of properties and CMIS calls I would rather stick to TRANSACTION_COMMIT option. But I understand why I was not able to catch the actual error message from the policy.

Re: How to get the proper exception message to the CMIS client

douglascrp — Thu, 12 Jul 2018 00:20:40 GMT

Interesting approach, and I didn't know that class.

I do use custom transaction listeners combined with regular EVERY_EVENT policies on occasion to aggregate multiple events into a single processing step at the end of a transaction

Do you mean something like this? Alfresco – Implementing delete behavior | Programming and So

Do you have any sample code you can share using the approach you described?

Thank you.

Re: How to get the proper exception message to the CMIS client

signjoy — Thu, 12 Jul 2018 13:32:02 GMT

Hi,

All you need to do is set the NotificationFrequency to EVERY_EVENT in the initialization method of your policy.

public class CustomDocPolicy implements NodeServicePolicies.OnCreateNodePolicy,
        NodeServicePolicies.OnUpdateNodePolicy,
        NodeServicePolicies.BeforeDeleteNodePolicy {


public void initialise() {
 this.policyComponent.bindClassBehaviour(QName.createQName(NamespaceService.ALFRESCO_URI, "onCreateNode"),
 CustomConstants.TYPE_BASE_DOC, new JavaBehaviour(this, "onCreateNode", Behaviour.NotificationFrequency.EVERY_EVENT));
 }



}