[jira] [Commented] (FINERACT-1091) PersistenceException: Field 'constitution_cv_id' doesn't have a default value

2020-07-16 Thread Yemdjih Kaze Nasser (Jira) 


[ 
https://issues.apache.org/jira/browse/FINERACT-1091?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17159522#comment-17159522
 ] 

Yemdjih Kaze Nasser commented on FINERACT-1091:
---

No this is not one of the issues. This is probably something else. I'd bet on 
what [~ptuomola] proposes.

 

> PersistenceException: Field 'constitution_cv_id' doesn't have a default value
> -
>
> Key: FINERACT-1091
> URL: https://issues.apache.org/jira/browse/FINERACT-1091
> Project: Apache Fineract
>  Issue Type: Bug
>Reporter: Michael Vorburger
>Priority: Blocker
>
> See FINERACT-932 for general background; I've found this in logs of 
> [https://www.fineract.dev|https://www.fineract.dev/]:
> {noformat}org.apache.openjpa.persistence.PersistenceException: The 
> transaction has been rolled back.  See the nested exceptions for details on 
> the errors that occurred.
>   at 
> org.apache.openjpa.kernel.BrokerImpl.newFlushException(BrokerImpl.java:2470) 
> ~[openjpa-3.1.1.jar:3.1.1]
>   at org.apache.openjpa.kernel.BrokerImpl.flush(BrokerImpl.java:2308) 
> ~[openjpa-3.1.1.jar:3.1.1]
>   at org.apache.openjpa.kernel.BrokerImpl.flushSafe(BrokerImpl.java:2199) 
> ~[openjpa-3.1.1.jar:3.1.1]
>   at org.apache.openjpa.kernel.BrokerImpl.flush(BrokerImpl.java:1962) 
> ~[openjpa-3.1.1.jar:3.1.1]
>   at 
> org.apache.openjpa.kernel.StateManagerImpl.assignObjectId(StateManagerImpl.java:618)
>  ~[openjpa-3.1.1.jar:3.1.1]
>   at 
> org.apache.openjpa.kernel.StateManagerImpl.assignField(StateManagerImpl.java:705)
>  ~[openjpa-3.1.1.jar:3.1.1]
>   at 
> org.apache.openjpa.kernel.StateManagerImpl.beforeAccessField(StateManagerImpl.java:1750)
>  ~[openjpa-3.1.1.jar:3.1.1]
>   at 
> org.apache.openjpa.kernel.StateManagerImpl.accessingField(StateManagerImpl.java:1679)
>  ~[openjpa-3.1.1.jar:3.1.1]
>   at 
> org.apache.fineract.infrastructure.core.domain.AbstractPersistableCustom.pcGetid(AbstractPersistableCustom.java)
>  ~[app/:na]
>   at 
> org.apache.fineract.infrastructure.core.domain.AbstractPersistableCustom.getId(AbstractPersistableCustom.java:50)
>  ~[app/:na]
>   at 
> org.apache.fineract.portfolio.address.service.AddressWritePlatformServiceImpl.addNewClientAddress(AddressWritePlatformServiceImpl.java:143)
>  ~[app/:na]
>   at 
> org.apache.fineract.portfolio.client.service.ClientWritePlatformServiceJpaRepositoryImpl.createClient(ClientWritePlatformServiceJpaRepositoryImpl.java:328)
>  ~[app/:na]
>   at 
> org.apache.fineract.portfolio.client.service.ClientWritePlatformServiceJpaRepositoryImpl$$FastClassBySpringCGLIB$$71ca1b7f.invoke()
>  ~[app/:na]
>   at 
> org.springframework.cglib.proxy.MethodProxy.invoke(MethodProxy.java:218) 
> ~[spring-core-5.2.6.RELEASE.jar:5.2.6.RELEASE]
>   at 
> org.springframework.aop.framework.CglibAopProxy$CglibMethodInvocation.invokeJoinpoint(CglibAopProxy.java:771)
>  ~[spring-aop-5.2.6.RELEASE.jar:5.2.6.RELEASE]
>   at 
> org.springframework.aop.framework.ReflectiveMethodInvocation.proceed(ReflectiveMethodInvocation.java:163)
>  ~[spring-aop-5.2.6.RELEASE.jar:5.2.6.RELEASE]
>   at 
> org.springframework.aop.framework.CglibAopProxy$CglibMethodInvocation.proceed(CglibAopProxy.java:749)
>  ~[spring-aop-5.2.6.RELEASE.jar:5.2.6.RELEASE]
>   at 
> org.springframework.transaction.interceptor.TransactionAspectSupport.invokeWithinTransaction(TransactionAspectSupport.java:366)
>  ~[spring-tx-5.2.6.RELEASE.jar:5.2.6.RELEASE]
>   at 
> org.springframework.transaction.interceptor.TransactionInterceptor.invoke(TransactionInterceptor.java:118)
>  ~[spring-tx-5.2.6.RELEASE.jar:5.2.6.RELEASE]
>   at 
> org.springframework.aop.framework.ReflectiveMethodInvocation.proceed(ReflectiveMethodInvocation.java:186)
>  ~[spring-aop-5.2.6.RELEASE.jar:5.2.6.RELEASE]
>   at 
> org.springframework.aop.framework.CglibAopProxy$CglibMethodInvocation.proceed(CglibAopProxy.java:749)
>  ~[spring-aop-5.2.6.RELEASE.jar:5.2.6.RELEASE]
>   at 
> org.springframework.aop.framework.CglibAopProxy$DynamicAdvisedInterceptor.intercept(CglibAopProxy.java:691)
>  ~[spring-aop-5.2.6.RELEASE.jar:5.2.6.RELEASE]
>   at 
> org.apache.fineract.portfolio.client.service.ClientWritePlatformServiceJpaRepositoryImpl$$EnhancerBySpringCGLIB$$3e889443.createClient()
>  ~[app/:na]
>   at 
> org.apache.fineract.portfolio.client.handler.CreateClientCommandHandler.processCommand(CreateClientCommandHandler.java:45)
>  ~[app/:na]
>   at 
> org.apache.fineract.portfolio.client.handler.CreateClientCommandHandler$$FastClassBySpringCGLIB$$6bce1ca9.invoke()
>  ~[app/:na]
>   at 
> org.springframework.cglib.proxy.MethodProxy.invoke(MethodProxy.java:218) 
> ~[spring-core-5.2.6.RELEASE.jar:5.2.6.RELEASE]
>   at 
>

[jira] [Commented] (FINERACT-1091) PersistenceException: Field 'constitution_cv_id' doesn't have a default value

2020-07-16 Thread Petri Tuomola (Jira) 


[ 
https://issues.apache.org/jira/browse/FINERACT-1091?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17159199#comment-17159199
 ] 

Petri Tuomola commented on FINERACT-1091:
-

At first glance it looks like the validator in 
ClientDataValidator.getDataValidationErrorsForCreateOnClientNonPerson() is 
broken... it only complains if the constitution code value is incorrect, not if 
it is missing altogether. It is mandatory at database level so a value should 
be passed in...

> PersistenceException: Field 'constitution_cv_id' doesn't have a default value
> -
>
> Key: FINERACT-1091
> URL: https://issues.apache.org/jira/browse/FINERACT-1091
> Project: Apache Fineract
>  Issue Type: Bug
>Reporter: Michael Vorburger
>Priority: Blocker
>
> See FINERACT-932 for general background; I've found this in logs of 
> [https://www.fineract.dev|https://www.fineract.dev/]:
> {noformat}org.apache.openjpa.persistence.PersistenceException: The 
> transaction has been rolled back.  See the nested exceptions for details on 
> the errors that occurred.
>   at 
> org.apache.openjpa.kernel.BrokerImpl.newFlushException(BrokerImpl.java:2470) 
> ~[openjpa-3.1.1.jar:3.1.1]
>   at org.apache.openjpa.kernel.BrokerImpl.flush(BrokerImpl.java:2308) 
> ~[openjpa-3.1.1.jar:3.1.1]
>   at org.apache.openjpa.kernel.BrokerImpl.flushSafe(BrokerImpl.java:2199) 
> ~[openjpa-3.1.1.jar:3.1.1]
>   at org.apache.openjpa.kernel.BrokerImpl.flush(BrokerImpl.java:1962) 
> ~[openjpa-3.1.1.jar:3.1.1]
>   at 
> org.apache.openjpa.kernel.StateManagerImpl.assignObjectId(StateManagerImpl.java:618)
>  ~[openjpa-3.1.1.jar:3.1.1]
>   at 
> org.apache.openjpa.kernel.StateManagerImpl.assignField(StateManagerImpl.java:705)
>  ~[openjpa-3.1.1.jar:3.1.1]
>   at 
> org.apache.openjpa.kernel.StateManagerImpl.beforeAccessField(StateManagerImpl.java:1750)
>  ~[openjpa-3.1.1.jar:3.1.1]
>   at 
> org.apache.openjpa.kernel.StateManagerImpl.accessingField(StateManagerImpl.java:1679)
>  ~[openjpa-3.1.1.jar:3.1.1]
>   at 
> org.apache.fineract.infrastructure.core.domain.AbstractPersistableCustom.pcGetid(AbstractPersistableCustom.java)
>  ~[app/:na]
>   at 
> org.apache.fineract.infrastructure.core.domain.AbstractPersistableCustom.getId(AbstractPersistableCustom.java:50)
>  ~[app/:na]
>   at 
> org.apache.fineract.portfolio.address.service.AddressWritePlatformServiceImpl.addNewClientAddress(AddressWritePlatformServiceImpl.java:143)
>  ~[app/:na]
>   at 
> org.apache.fineract.portfolio.client.service.ClientWritePlatformServiceJpaRepositoryImpl.createClient(ClientWritePlatformServiceJpaRepositoryImpl.java:328)
>  ~[app/:na]
>   at 
> org.apache.fineract.portfolio.client.service.ClientWritePlatformServiceJpaRepositoryImpl$$FastClassBySpringCGLIB$$71ca1b7f.invoke()
>  ~[app/:na]
>   at 
> org.springframework.cglib.proxy.MethodProxy.invoke(MethodProxy.java:218) 
> ~[spring-core-5.2.6.RELEASE.jar:5.2.6.RELEASE]
>   at 
> org.springframework.aop.framework.CglibAopProxy$CglibMethodInvocation.invokeJoinpoint(CglibAopProxy.java:771)
>  ~[spring-aop-5.2.6.RELEASE.jar:5.2.6.RELEASE]
>   at 
> org.springframework.aop.framework.ReflectiveMethodInvocation.proceed(ReflectiveMethodInvocation.java:163)
>  ~[spring-aop-5.2.6.RELEASE.jar:5.2.6.RELEASE]
>   at 
> org.springframework.aop.framework.CglibAopProxy$CglibMethodInvocation.proceed(CglibAopProxy.java:749)
>  ~[spring-aop-5.2.6.RELEASE.jar:5.2.6.RELEASE]
>   at 
> org.springframework.transaction.interceptor.TransactionAspectSupport.invokeWithinTransaction(TransactionAspectSupport.java:366)
>  ~[spring-tx-5.2.6.RELEASE.jar:5.2.6.RELEASE]
>   at 
> org.springframework.transaction.interceptor.TransactionInterceptor.invoke(TransactionInterceptor.java:118)
>  ~[spring-tx-5.2.6.RELEASE.jar:5.2.6.RELEASE]
>   at 
> org.springframework.aop.framework.ReflectiveMethodInvocation.proceed(ReflectiveMethodInvocation.java:186)
>  ~[spring-aop-5.2.6.RELEASE.jar:5.2.6.RELEASE]
>   at 
> org.springframework.aop.framework.CglibAopProxy$CglibMethodInvocation.proceed(CglibAopProxy.java:749)
>  ~[spring-aop-5.2.6.RELEASE.jar:5.2.6.RELEASE]
>   at 
> org.springframework.aop.framework.CglibAopProxy$DynamicAdvisedInterceptor.intercept(CglibAopProxy.java:691)
>  ~[spring-aop-5.2.6.RELEASE.jar:5.2.6.RELEASE]
>   at 
> org.apache.fineract.portfolio.client.service.ClientWritePlatformServiceJpaRepositoryImpl$$EnhancerBySpringCGLIB$$3e889443.createClient()
>  ~[app/:na]
>   at 
> org.apache.fineract.portfolio.client.handler.CreateClientCommandHandler.processCommand(CreateClientCommandHandler.java:45)
>  ~[app/:na]
>   at 
> org.apache.fineract.portfolio.client.handler.CreateClientCommandHandler$$FastClassBySpringCGLIB$$6bce1ca9.invoke()
>  ~[app/:na]
>

[jira] [Commented] (FINERACT-1091) PersistenceException: Field 'constitution_cv_id' doesn't have a default value

2020-07-16 Thread Michael Vorburger (Jira) 


[ 
https://issues.apache.org/jira/browse/FINERACT-1091?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17159120#comment-17159120
 ] 

Michael Vorburger commented on FINERACT-1091:
-

[~xurror] I remember you saying somewhere in FINERACT-849 or 
[https://github.com/apache/fineract/pull/928] (I don't remember / can't find 
anymore exactly where) that as part of your exploration of migrating from 
OpenJPA to EclipseLink, you have fixed a few things which seemed real bugs that 
EclipseLink pointed out because it was somehow more strict than OpenJPA? Was 
this perhaps one of them? If you do, I highly encourage you to break those 
fixes out of the overall (big) OpenJPA to EclipseLink migration, and submit 
them as smaller PR specifically fixing bugs such as this. (And if this 
"'constitution_cv_id" doesn't ring any bell, just ignore.)

> PersistenceException: Field 'constitution_cv_id' doesn't have a default value
> -
>
> Key: FINERACT-1091
> URL: https://issues.apache.org/jira/browse/FINERACT-1091
> Project: Apache Fineract
>  Issue Type: Bug
>Reporter: Michael Vorburger
>Priority: Blocker
>
> See FINERACT-932 for general background; I've found this in logs of 
> [https://www.fineract.dev|https://www.fineract.dev/]:
> {noformat}org.apache.openjpa.persistence.PersistenceException: The 
> transaction has been rolled back.  See the nested exceptions for details on 
> the errors that occurred.
>   at 
> org.apache.openjpa.kernel.BrokerImpl.newFlushException(BrokerImpl.java:2470) 
> ~[openjpa-3.1.1.jar:3.1.1]
>   at org.apache.openjpa.kernel.BrokerImpl.flush(BrokerImpl.java:2308) 
> ~[openjpa-3.1.1.jar:3.1.1]
>   at org.apache.openjpa.kernel.BrokerImpl.flushSafe(BrokerImpl.java:2199) 
> ~[openjpa-3.1.1.jar:3.1.1]
>   at org.apache.openjpa.kernel.BrokerImpl.flush(BrokerImpl.java:1962) 
> ~[openjpa-3.1.1.jar:3.1.1]
>   at 
> org.apache.openjpa.kernel.StateManagerImpl.assignObjectId(StateManagerImpl.java:618)
>  ~[openjpa-3.1.1.jar:3.1.1]
>   at 
> org.apache.openjpa.kernel.StateManagerImpl.assignField(StateManagerImpl.java:705)
>  ~[openjpa-3.1.1.jar:3.1.1]
>   at 
> org.apache.openjpa.kernel.StateManagerImpl.beforeAccessField(StateManagerImpl.java:1750)
>  ~[openjpa-3.1.1.jar:3.1.1]
>   at 
> org.apache.openjpa.kernel.StateManagerImpl.accessingField(StateManagerImpl.java:1679)
>  ~[openjpa-3.1.1.jar:3.1.1]
>   at 
> org.apache.fineract.infrastructure.core.domain.AbstractPersistableCustom.pcGetid(AbstractPersistableCustom.java)
>  ~[app/:na]
>   at 
> org.apache.fineract.infrastructure.core.domain.AbstractPersistableCustom.getId(AbstractPersistableCustom.java:50)
>  ~[app/:na]
>   at 
> org.apache.fineract.portfolio.address.service.AddressWritePlatformServiceImpl.addNewClientAddress(AddressWritePlatformServiceImpl.java:143)
>  ~[app/:na]
>   at 
> org.apache.fineract.portfolio.client.service.ClientWritePlatformServiceJpaRepositoryImpl.createClient(ClientWritePlatformServiceJpaRepositoryImpl.java:328)
>  ~[app/:na]
>   at 
> org.apache.fineract.portfolio.client.service.ClientWritePlatformServiceJpaRepositoryImpl$$FastClassBySpringCGLIB$$71ca1b7f.invoke()
>  ~[app/:na]
>   at 
> org.springframework.cglib.proxy.MethodProxy.invoke(MethodProxy.java:218) 
> ~[spring-core-5.2.6.RELEASE.jar:5.2.6.RELEASE]
>   at 
> org.springframework.aop.framework.CglibAopProxy$CglibMethodInvocation.invokeJoinpoint(CglibAopProxy.java:771)
>  ~[spring-aop-5.2.6.RELEASE.jar:5.2.6.RELEASE]
>   at 
> org.springframework.aop.framework.ReflectiveMethodInvocation.proceed(ReflectiveMethodInvocation.java:163)
>  ~[spring-aop-5.2.6.RELEASE.jar:5.2.6.RELEASE]
>   at 
> org.springframework.aop.framework.CglibAopProxy$CglibMethodInvocation.proceed(CglibAopProxy.java:749)
>  ~[spring-aop-5.2.6.RELEASE.jar:5.2.6.RELEASE]
>   at 
> org.springframework.transaction.interceptor.TransactionAspectSupport.invokeWithinTransaction(TransactionAspectSupport.java:366)
>  ~[spring-tx-5.2.6.RELEASE.jar:5.2.6.RELEASE]
>   at 
> org.springframework.transaction.interceptor.TransactionInterceptor.invoke(TransactionInterceptor.java:118)
>  ~[spring-tx-5.2.6.RELEASE.jar:5.2.6.RELEASE]
>   at 
> org.springframework.aop.framework.ReflectiveMethodInvocation.proceed(ReflectiveMethodInvocation.java:186)
>  ~[spring-aop-5.2.6.RELEASE.jar:5.2.6.RELEASE]
>   at 
> org.springframework.aop.framework.CglibAopProxy$CglibMethodInvocation.proceed(CglibAopProxy.java:749)
>  ~[spring-aop-5.2.6.RELEASE.jar:5.2.6.RELEASE]
>   at 
> org.springframework.aop.framework.CglibAopProxy$DynamicAdvisedInterceptor.intercept(CglibAopProxy.java:691)
>  ~[spring-aop-5.2.6.RELEASE.jar:5.2.6.RELEASE]
>   at 
>