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
> >
>

Reply via email to