[
https://issues.apache.org/jira/browse/ISIS-1414?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15324642#comment-15324642
]
ASF subversion and git services commented on ISIS-1414:
-------------------------------------------------------
Commit 5d9185285b6f5268fad7807f25fa624a4bfb0c27 in isis's branch
refs/heads/ISIS-1427 from [~danhaywood]
[ https://git-wip-us.apache.org/repos/asf?p=isis.git;h=5d91852 ]
ISIS-1414: ensures that getChangedObjectProperties() is called
for the MetricsService, even if not called for auditing (ie were no audited
objects changed in the xactn)
> nextTransaction can cause null pointers with the new MetricsService.
> ---------------------------------------------------------------------
>
> Key: ISIS-1414
> URL: https://issues.apache.org/jira/browse/ISIS-1414
> Project: Isis
> Issue Type: Bug
> Components: Core
> Affects Versions: 1.12.1
> Reporter: Dan Haywood
> Assignee: Dan Haywood
> Priority: Minor
> Fix For: 1.13.0
>
>
> The reason is that request-scoped services are implemented as transaction
> services. Most of the time the scope of a request and the scope of a
> transacion are one and the same: there is just one transaction per scope.
> However, TransactionService#nextTransaction admits for there being multiple
> transactions per request.
> This bug is that the current implementation for RequestScoped does the
> init/close as part of the Transaction management. Instead, this should be
> part of the PersistenceSession#open/close (a PersistenceSession wraps a JDO
> session, so basically corresponds to the request).
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
