On Fri, Jul 11, 2014 at 3:54 PM, Dan Haywood <[email protected]>
wrote:
> Um, yup. Looks like the IsisSessionTemplate [1] sets up a session, but no
> transaction.
>
> The background command service [2] actually subclasses from [1], in
> its BackgroundCommandExecution class [3]. If you look through the source
> code of that then you'll see how to set up a transaction; basically:
>
> transactionManager.executeWithinTransaction(new
> TransactionalClosureAbstract() {
> @Override
> public void execute() {
> ...
> }
> });
>
> Do you think that IsisSessionTemplate should set up a transaction
> automatically? I'm thinking it probably should...
>
+1. I came across a similar transaction issue with isis Session with Quartz
scheduler.
Would be nice to have IsisSessionTemplate setting up a transaction
automatically as I think it's an internal operation.
>
>
> HTH
> Dan
>
> [1] http://isis.apache.org/reference/non-ui/isis-session-template.html
> [2] http://isis.apache.org/reference/services/background-service.html
> [3]
> http://isis.apache.org/reference/non-ui/background-command-execution.html
>
>
>
>
>
> On 11 July 2014 11:13, Erik de Hair <[email protected]> wrote:
>
> > Hi,
> >
> > I have a service method that works correctly when called via the wicket
> > viewer. When calling the method by a Quartz scheduled job I get the
> > following exception:
> >
> > 12:09:42,129 [JobRunShell SchedulerQuartzConfigXml_Worker-1
> > ERROR] Job Isis.ProcessInitiatedMobileInportRequestsJob threw an
> unhandled
> > Exception:
> > java.lang.NullPointerException
> > at
> >
> org.apache.isis.core.runtime.system.transaction.IsisTransactionManager.flushTransaction(IsisTransactionManager.java:375)
> > at
> >
> org.apache.isis.core.runtime.persistence.internal.RuntimeContextFromSession$7.flush(RuntimeContextFromSession.java:234)
> > at
> >
> org.apache.isis.core.metamodel.services.container.DomainObjectContainerDefault.flush(DomainObjectContainerDefault.java:224)
> > at
> >
> org.apache.isis.core.metamodel.services.container.DomainObjectContainerDefault.allMatches(DomainObjectContainerDefault.java:397)
> > at
> >
> org.apache.isis.core.metamodel.services.container.DomainObjectContainerDefault.allInstances(DomainObjectContainerDefault.java:359)
> > at
> >
> org.apache.isis.core.metamodel.services.container.DomainObjectContainerDefault.allMatches(DomainObjectContainerDefault.java:368)
> > at
> >
> org.apache.isis.core.metamodel.services.container.DomainObjectContainerDefault.uniqueMatch(DomainObjectContainerDefault.java:449)
> > at
> >
> org.apache.isis.applib.AbstractContainedObject.uniqueMatch(AbstractContainedObject.java:238)
> > at
> >
> nl.pocos.dom.businessmobile.porting.MobileInportExportItem.viewModelInit(MobileInportExportItem.java:40)
> > at
> >
> org.apache.isis.core.progmodel.facets.object.viewmodel.annotation.ViewModelFacetForInterface.initialize(ViewModelFacetForInterface.java:35)
> > at
> >
> org.apache.isis.core.runtime.system.persistence.PersistenceSession.createViewModelInstance(PersistenceSession.java:366)
> > at
> >
> org.apache.isis.core.runtime.persistence.internal.RuntimeContextFromSession$7.createViewModelInstance(RuntimeContextFromSession.java:197)
> > at
> >
> org.apache.isis.core.metamodel.services.container.DomainObjectContainerDefault.doCreateViewModelInstance(DomainObjectContainerDefault.java:159)
> > at
> >
> org.apache.isis.core.metamodel.services.container.DomainObjectContainerDefault.newViewModelInstance(DomainObjectContainerDefault.java:104)
> > at
> >
> org.apache.isis.applib.AbstractContainedObject.newViewModelInstance(AbstractContainedObject.java:86)
> > at
> > nl.pocos.dom.businessmobile.porting.Portings.access$000(Portings.java:52)
> > at
> > nl.pocos.dom.businessmobile.porting.Portings$1.apply(Portings.java:329)
> > at
> > nl.pocos.dom.businessmobile.porting.Portings$1.apply(Portings.java:326)
> > at
> >
> com.google.common.collect.Lists$TransformingRandomAccessList$1.transform(Lists.java:582)
> > at
> >
> com.google.common.collect.TransformedIterator.next(TransformedIterator.java:48)
> > at
> >
> com.google.common.collect.TransformedIterator.next(TransformedIterator.java:48)
> > at
> >
> com.danhaywood.isis.domainservice.excel.impl.ExcelConverter.toFile(ExcelConverter.java:131)
> > at
> >
> com.danhaywood.isis.domainservice.excel.impl.ExcelServiceImpl.toExcel(ExcelServiceImpl.java:69)
> > at
> >
> nl.pocos.dom.businessmobile.porting.Portings.processInitiatedMobileInportRequests(Portings.java:336)
> > at
> >
> webapp.scheduler.InitiatedMobileInportRequestsProcessor.doExecute(InitiatedMobileInportRequestsProcessor.java:29)
> > at
> >
> org.apache.isis.core.runtime.sessiontemplate.AbstractIsisSessionTemplate.execute(AbstractIsisSessionTemplate.java:41)
> > at
> >
> webapp.scheduler.AbstractIsisQuartzJob.execute(AbstractIsisQuartzJob.java:53)
> > at org.quartz.core.JobRunShell.run(JobRunShell.java:202)
> > at
> >
> org.quartz.simpl.SimpleThreadPool$WorkerThread.run(SimpleThreadPool.java:573)
> >
> > Do I need to do additional configuration for calling methods via jobs?
> >
> > Thanks,
> > Erik
> >
>