search

LibrePlan / libreplan

LibrePlan - Open Web Planning
https://www.libreplan.dev
GNU Affero General Public License v3.0
289 stars 170 forks source link

Branch scenarios: Error creating new order version #611

Closed kwoot closed 6 years ago

kwoot commented 14 years ago

(Original Bugzilla Bug ID: 467)

Date: 2010-05-21 16:09:55 From: Javier Morán \<jmoran@igalia.com> To: Oscar González \<ogonzalez@igalia.com> Version: navalplan-1.0 (1.0.x) Last updated: 2011-01-18 17:53:40

(Note: this issue was migrated automatically with bugzilla2github.py tool )

kwoot commented 6 years ago

Bugzilla Comment ID: 1065 Date: 2010-05-21 16:09:55 From: Javier Morán \<jmoran@igalia.com>

If you connect in the attached dump to the scenario "Simulacion optimista" and you go to the order "Pedido 1" and you change the planning points to have a planning point in the order element "Bloque 1" when you press "Save" you get the following error:

/20100303 Ubuntu/8.04 (hardy) Namoroka/3.6.2pre" 21-may-2010 15:57:29 org.zkoss.zk.ui.impl.UiEngineImpl handleError:1131 GRAVE: >>java.lang.IllegalArgumentException: id to load is required for loading

at org.hibernate.event.LoadEvent.(LoadEvent.java:51) at org.hibernate.event.LoadEvent.(LoadEvent.java:33) at org.hibernate.impl.SessionImpl.get(SessionImpl.java:813) at org.hibernate.impl.SessionImpl.get(SessionImpl.java:809) at org.navalplanner.business.common.daos.GenericDAOHibernate.find(GenericDAOHibernate.java:168) at org.navalplanner.business.common.daos.GenericDAOHibernate.remove(GenericDAOHibernate.java:196) ... 8890043 [298860130@qtp-1132612908-29] ERROR org.navalplanner.web.error.PageForErrorOnEvent - id to load is required for loading java.lang.IllegalArgumentException: id to load is required for loading at org.hibernate.event.LoadEvent.(LoadEvent.java:51) at org.hibernate.event.LoadEvent.(LoadEvent.java:33) at org.hibernate.impl.SessionImpl.get(SessionImpl.java:813) at org.hibernate.impl.SessionImpl.get(SessionImpl.java:809) at org.navalplanner.business.common.daos.GenericDAOHibernate.find(GenericDAOHibernate.java:168) at org.navalplanner.business.common.daos.GenericDAOHibernate.remove(GenericDAOHibernate.java:196) at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method) at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:39) at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:25) at java.lang.reflect.Method.invoke(Method.java:597) at org.springframework.aop.support.AopUtils.invokeJoinpointUsingReflection(AopUtils.java:307) at org.springframework.aop.framework.ReflectiveMethodInvocation.invokeJoinpoint(ReflectiveMethodInvocation.java:182) at org.springframework.aop.framework.ReflectiveMethodInvocation.proceed(ReflectiveMethodInvocation.java:149) at org.springframework.dao.support.PersistenceExceptionTranslationInterceptor.invoke(PersistenceExceptionTranslationInterceptor.java:138) at org.springframework.aop.framework.ReflectiveMethodInvocation.proceed(ReflectiveMethodInvocation.java:171) at org.springframework.aop.framework.JdkDynamicAopProxy.invoke(JdkDynamicAopProxy.java:204) at $Proxy39.remove(Unknown Source) at org.navalplanner.business.orders.entities.TaskSource$TaskGroupMustBeReplacedByTask.remove(TaskSource.java:161) at org.navalplanner.business.orders.entities.TaskSource$TaskGroupMustBeReplacedByTask.apply(TaskSource.java:149) at org.navalplanner.business.orders.entities.TaskSource$TaskGroupSynchronization.getChildren(TaskSource.java:198) at org.navalplanner.business.orders.entities.TaskSource$TaskGroupSynchronization.apply(TaskSource.java:190) at org.navalplanner.web.orders.OrderModel.synchronizeWithSchedule(OrderModel.java:568) at org.navalplanner.web.orders.OrderModel.saveOnTransaction(OrderModel.java:495) at org.navalplanner.web.orders.OrderModel.access$000(OrderModel.java:103) at org.navalplanner.web.orders.OrderModel$1.execute(OrderModel.java:471) at org.navalplanner.web.orders.OrderModel$1.execute(OrderModel.java:468) at org.navalplanner.business.common.AdHocTransactionService.runOnTransaction(AdHocTransactionService.java:110) at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method) at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:39) at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:25) at java.lang.reflect.Method.invoke(Method.java:597) at org.springframework.aop.support.AopUtils.invokeJoinpointUsingReflection(AopUtils.java:307) at org.springframework.aop.framework.ReflectiveMethodInvocation.invokeJoinpoint(ReflectiveMethodInvocation.java:182) at org.springframework.aop.framework.ReflectiveMethodInvocation.proceed(ReflectiveMethodInvocation.java:149) at org.springframework.transaction.interceptor.TransactionInterceptor.invoke(TransactionInterceptor.java:106) at org.springframework.aop.framework.ReflectiveMethodInvocation.proceed(ReflectiveMethodInvocation.java:171) at org.springframework.aop.framework.JdkDynamicAopProxy.invoke(JdkDynamicAopProxy.java:204) at $Proxy33.runOnTransaction(Unknown Source) at org.navalplanner.web.orders.OrderModel.save(OrderModel.java:468) at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method) at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:39) at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:25) at java.lang.reflect.Method.invoke(Method.java:597) at org.springframework.aop.support.AopUtils.invokeJoinpointUsingReflection(AopUtils.java:307) at org.springframework.aop.framework.JdkDynamicAopProxy.invoke(JdkDynamicAopProxy.java:198) at $Proxy131.save(Unknown Source) at org.navalplanner.web.orders.OrderCRUDController.save(OrderCRUDController.java:515) at org.navalplanner.web.orders.OrderCRUDController.saveAndContinue(OrderCRUDController.java:448) at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method) at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:39) at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:25) at java.lang.reflect.Method.invoke(Method.java:597) at bsh.Reflect.invokeMethod(Unknown Source) at bsh.Reflect.invokeObjectMethod(Unknown Source) at bsh.Name.invokeMethod(Unknown Source) at bsh.BSHMethodInvocation.eval(Unknown Source) at bsh.BSHPrimaryExpression.eval(Unknown Source) at bsh.BSHPrimaryExpression.eval(Unknown Source) at bsh.Interpreter.eval(Unknown Source) at bsh.Interpreter.eval(Unknown Source) at org.zkoss.zk.scripting.bsh.BSHInterpreter.exec(BSHInterpreter.java:119) at org.zkoss.zk.scripting.util.GenericInterpreter.interpret(GenericInterpreter.java:310) at org.zkoss.zk.ui.impl.PageImpl.interpret(PageImpl.java:787) at org.zkoss.zk.ui.impl.EventProcessor.process0(EventProcessor.java:176) at org.zkoss.zk.ui.impl.EventProcessor.process(EventProcessor.java:141) at org.zkoss.zk.ui.impl.EventProcessingThreadImpl.process0(EventProcessingThreadImpl.java:493) at org.zkoss.zk.ui.impl.EventProcessingThreadImpl.run(EventProcessingThreadImpl.java:425)

kwoot commented 6 years ago

Bugzilla Comment ID: 1069 Date: 2010-05-23 13:56:18 From: Oscar González \<ogonzalez@igalia.com>

commit 2eb7cca7688fbbe22b5068163a0e06528785db75 Author: Óscar González Fernández ogonzalez@igalia.com Date: Sun May 23 13:44:18 2010 +0200

ItEr58S14RecalculosConexionEscenariosItEr57S15: Avoid the creation of spurious domain dependencies.

When creating a Dependency is automatically added to the tasks. This
was causing the invisible dependencies to be created.

commit 8357c60ca7eb6fd8646dd78ceeaf73a66cdc71bb Author: Óscar González Fernández ogonzalez@igalia.com Date: Sun May 23 13:34:28 2010 +0200

ItEr58S14RecalculosConexionEscenariosItEr57S15: Fix bug.

When a tasks is removed its dependencies must be removed too. Otherwise when the task is not preexistent a TransientObjectException happens.

commit 90c24974e1395fc4bae1226395aed99ddc86b331 Author: Óscar González Fernández ogonzalez@igalia.com Date: Sat May 22 20:32:42 2010 +0200

ItEr58S14RecalculosConexionEscenariosItEr57S15: Guarantee removed tasks sources are removed from order element.

Using taskSourceRemoval the order element is updated.

commit 8f375a789163eb309570bc1a51f270a3cccc4570 Author: Óscar González Fernández ogonzalez@igalia.com Date: Sat May 22 20:28:34 2010 +0200

ItEr58S14RecalculosConexionEscenariosItEr57S15: It's better to do it after the transaction

commit 2d1a5682f03843dcee69ed9935d29152bcd66fa7 Author: Óscar González Fernández ogonzalez@igalia.com Date: Sat May 22 20:19:46 2010 +0200

ItEr58S14RecalculosConexionEscenariosItEr57S15: [Bug #465] Fix bug.

The condition was wrong. When the tasks are not preexistent the tasks cannot be removed
kwoot commented 6 years ago

Bugzilla Comment ID: 1066 Date: 2010-05-21 16:10:32 From: Javier Morán \<jmoran@igalia.com>

Attached file: dump.sql File description: database dump