[
http://jira.magnolia-cms.com/browse/MAGNOLIA-3018?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Philipp Bärfuss updated MAGNOLIA-3018:
--------------------------------------
Summary: content API cleanup: Content.getNodeData() should handle
exceptions properly (was: content API cleanup: Content.getNodeData() should
throw exceptions)
Description:
Content.getNodeData(name) doesn't define any throws declaration. To be
compliant to the former implementation (4.2) we will do the following:
- PathNotFound: return EmptyNodeData instance
- AccessDenied: return EmptyNodeData instance
setValue will fail always, but getString(), .. methods will return default
values.
Because of the amount of usages we decided against adding a throws declaration.
We might rethink that in the future but for now it is far the safest/cleanest
solution.
was:At least it should throw AccessDeniedExceptions
> content API cleanup: Content.getNodeData() should handle exceptions properly
> ----------------------------------------------------------------------------
>
> Key: MAGNOLIA-3018
> URL: http://jira.magnolia-cms.com/browse/MAGNOLIA-3018
> Project: Magnolia
> Issue Type: Sub-task
> Components: core
> Reporter: Philipp Bärfuss
> Assignee: Philipp Bärfuss
> Fix For: 4.3
>
>
> Content.getNodeData(name) doesn't define any throws declaration. To be
> compliant to the former implementation (4.2) we will do the following:
> - PathNotFound: return EmptyNodeData instance
> - AccessDenied: return EmptyNodeData instance
> setValue will fail always, but getString(), .. methods will return default
> values.
> Because of the amount of usages we decided against adding a throws
> declaration. We might rethink that in the future but for now it is far the
> safest/cleanest solution.
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
http://jira.magnolia-cms.com/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira
----------------------------------------------------------------
For list details see
http://www.magnolia-cms.com/home/community/mailing-lists.html
To unsubscribe, E-mail to: <[email protected]>
----------------------------------------------------------------
