AbstractTransform should not catch transformers exceptions
----------------------------------------------------------

                 Key: NXP-2171
                 URL: http://jira.nuxeo.org/browse/NXP-2171
             Project: Nuxeo Enterprise Platform 5
          Issue Type: Bug
    Affects Versions: 5.1.3.2
            Reporter: Olivier Grisel
            Assignee: Thierry Delprat
             Fix For: 5.2 M1


AbstractTransform should not catch transformer exception but throw them instead.

This would allow the caller to handle it correctly and help the developers find 
which file attachment name in which Nuxeo document has triggered the  problem.

The AbstractTransform is indeed two deep in the call stack and has no idea of 
the Nuxeo document being transform and therefore cannot log all necessary data  
to make debugging easy.

Fixing this problem requires an API change: add new throws declaration on the 
method:

    public List<TransformDocument> transform(
            Map<String, Map<String, Serializable>> options,
            TransformDocument... sources);

Therefore this change cannot be implemented in the 5.1 branch.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: 
http://jira.nuxeo.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        
_______________________________________________
ECM-tickets mailing list
[email protected]
http://lists.nuxeo.com/mailman/listinfo/ecm-tickets

Reply via email to