Project:
View Issue Details[ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||
ID | ||||||||
0023385 | ||||||||
Type | Category | Severity | Reproducibility | Date Submitted | Last Update | |||
defect | [Openbravo ERP] 01. General setup | minor | always | 2013-03-25 12:08 | 2013-06-20 02:08 | |||
Reporter | caristu | View Status | public | |||||
Assigned To | jecharri | |||||||
Priority | high | Resolution | fixed | Fixed in Version | ||||
Status | closed | Fix in branch | pi | Fixed in SCM revision | bde534f03bd4 | |||
Projection | none | ETA | none | Target Version | 3.0MP25 | |||
OS | Any | Database | Any | Java version | ||||
OS Version | Database version | Ant version | ||||||
Product Version | pi | SCM revision | ||||||
Merge Request Status | ||||||||
Review Assigned To | caristu | |||||||
OBNetwork customer | OBPS | |||||||
Web browser | ||||||||
Modules | Core | |||||||
Support ticket | 20136 | |||||||
Regression level | ||||||||
Regression date | ||||||||
Regression introduced in release | ||||||||
Regression introduced by commit | ||||||||
Triggers an Emergency Pack | No | |||||||
Summary | 0023385: Organization is generated even if the Initial Org Setup ends with errors | |||||||
Description | In the Initial Organization Setup process, the organization is generated even if not ends successfully. For example, if we Include the accounting file in the process and use an invalid COA, the create organization process ends successfully but the create accounting process fail. The organization should not be created if the whole Initial Organization Setup process ends with errors. | |||||||
Steps To Reproduce | 1) Go to Inital Organization Setup window 2) Fill the required fields 3) Check off the "Include Accounting File or Accounting Reference Data" checkbox 4) Include an invalid accounting file 5) Click OK. The process shows the error message in the accounting process and the organization is generated. | |||||||
Tags | No tags attached. | |||||||
Attached Files | ![]() | |||||||
![]() |
|
![]() |
|
(0059083) hgbot (developer) 2013-05-31 12:26 |
Repository: erp/devel/pi Changeset: 6792989a9124996136f6f7ad00061ece81fda599 Author: Javier Etxarri <javier.echarri <at> openbravo.com> Date: Fri May 31 12:25:02 2013 +0200 URL: http://code.openbravo.com/erp/devel/pi/rev/6792989a9124996136f6f7ad00061ece81fda599 [^] Fixes issue 23385: Organization is generated even if the Initial Org Setup ends with errors --- M src/org/openbravo/erpCommon/ad_forms/InitialOrgSetup.java M src/org/openbravo/erpCommon/businessUtility/InitialOrgSetup.java --- |
(0059087) caristu (viewer) 2013-05-31 15:53 |
Verified |
(0059088) hgbot (developer) 2013-05-31 18:23 |
Repository: erp/devel/pi Changeset: e84ef22ffd85a762ef5b29aae938d26ff556fb24 Author: Stefan Hühner <stefan.huehner <at> openbravo.com> Date: Fri May 31 18:19:45 2013 +0200 URL: http://code.openbravo.com/erp/devel/pi/rev/e84ef22ffd85a762ef5b29aae938d26ff556fb24 [^] Issue 23385: , Backed out: 6792989a9124 As is breaks OrgSetup when using the optional 'Location' field. --- M src/org/openbravo/erpCommon/ad_forms/InitialOrgSetup.java M src/org/openbravo/erpCommon/businessUtility/InitialOrgSetup.java --- |
(0059089) shuehner (administrator) 2013-05-31 18:24 |
Backed out. This break creations orgs when using the optional 'Location' field in the OrgSetup. Detected by smoketest: http://builds.openbravo.com/job/int-smoke-pgsql-test/980/ [^] |
(0059090) shuehner (administrator) 2013-05-31 18:27 |
The attached patch on top of the original commit seems to fix that specific bug for me. But needs further testing to ensure no more problems are introduced by the patch. Probable cause: insert of ad_org does run insert trigger to run ad_orginfo insert. The original commitAndClose did indirectly force a reload of the dal object. After it not being done in the middle of the process an explicit refresh of all previously loaded dal-object (like ad-org) must be done to refresh their state. |
(0059113) hudsonbot (viewer) 2013-06-01 02:15 |
A changeset related to this issue has been promoted main and to the Central Repository, after passing a series of tests. Promotion changeset: https://code.openbravo.com/erp/devel/main/rev/31f9b82dde77 [^] Maturity status: Test |
(0059115) hudsonbot (viewer) 2013-06-01 02:15 |
A changeset related to this issue has been promoted main and to the Central Repository, after passing a series of tests. Promotion changeset: https://code.openbravo.com/erp/devel/main/rev/31f9b82dde77 [^] Maturity status: Test |
(0059321) jecharri (viewer) 2013-06-11 11:00 |
This changeset solve the issue https://code.openbravo.com/erp/devel/pi/rev/3f47396281c68c8fd8adb5b800f552b95bd225dd [^] |
(0059372) hgbot (developer) 2013-06-11 18:20 |
Repository: erp/devel/pi Changeset: da0c04eed6ebf4c4494d04464bf709a308260d81 Author: Javier Etxarri <javier.echarri <at> openbravo.com> Date: Tue Jun 11 18:20:35 2013 +0200 URL: http://code.openbravo.com/erp/devel/pi/rev/da0c04eed6ebf4c4494d04464bf709a308260d81 [^] backout issue 23385: Organization is generated even if the Initial Org Setup ends with errors --- M src/org/openbravo/erpCommon/ad_forms/InitialOrgSetup.java M src/org/openbravo/erpCommon/businessUtility/InitialOrgSetup.java --- |
(0059422) hgbot (developer) 2013-06-13 15:39 |
Repository: erp/devel/pi Changeset: bde534f03bd48726603235e604e241af47bf0e3e Author: Javier Etxarri <javier.echarri <at> openbravo.com> Date: Thu Jun 13 15:39:26 2013 +0200 URL: http://code.openbravo.com/erp/devel/pi/rev/bde534f03bd48726603235e604e241af47bf0e3e [^] Fixes issue 23385: Organization is generated even if the Initial Org Setup ends with errors --- M src/org/openbravo/dal/core/OBContext.java M src/org/openbravo/erpCommon/ad_forms/InitialOrgSetup.java M src/org/openbravo/erpCommon/businessUtility/InitialOrgSetup.java --- |
(0059476) caristu (viewer) 2013-06-18 10:11 |
Verified with steps to reproduce |
(0059569) hudsonbot (viewer) 2013-06-20 02:08 |
A changeset related to this issue has been promoted main and to the Central Repository, after passing a series of tests. Promotion changeset: https://code.openbravo.com/erp/devel/main/rev/5caf871044ad [^] Maturity status: Test |
(0059575) hudsonbot (viewer) 2013-06-20 02:08 |
A changeset related to this issue has been promoted main and to the Central Repository, after passing a series of tests. Promotion changeset: https://code.openbravo.com/erp/devel/main/rev/5caf871044ad [^] Maturity status: Test |
![]() |
|||
Date Modified | Username | Field | Change |
2013-03-25 12:08 | caristu | New Issue | |
2013-03-25 12:08 | caristu | Assigned To | => dmiguelez |
2013-03-25 12:08 | caristu | Modules | => Core |
2013-03-25 12:08 | caristu | OBNetwork customer | => Yes |
2013-03-25 12:08 | caristu | Support ticket | => 20136 |
2013-03-25 12:08 | caristu | Resolution time | => 1370210400 |
2013-03-25 12:08 | caristu | Triggers an Emergency Pack | => No |
2013-03-25 12:08 | caristu | Issue Monitored: networkb | |
2013-03-25 12:09 | caristu | Target Version | => 3.0MP23 |
2013-03-27 17:38 | dmiguelez | Assigned To | dmiguelez => jonalegriaesarte |
2013-04-17 17:20 | jonalegriaesarte | Target Version | 3.0MP23 => 3.0MP24 |
2013-05-20 12:02 | jonalegriaesarte | Assigned To | jonalegriaesarte => ioritzCia |
2013-05-30 10:45 | jonalegriaesarte | Assigned To | ioritzCia => jecharri |
2013-05-31 12:26 | hgbot | Checkin | |
2013-05-31 12:26 | hgbot | Note Added: 0059083 | |
2013-05-31 12:26 | hgbot | Status | new => resolved |
2013-05-31 12:26 | hgbot | Resolution | open => fixed |
2013-05-31 12:26 | hgbot | Fixed in SCM revision | => http://code.openbravo.com/erp/devel/pi/rev/6792989a9124996136f6f7ad00061ece81fda599 [^] |
2013-05-31 15:53 | caristu | Review Assigned To | => caristu |
2013-05-31 15:53 | caristu | Note Added: 0059087 | |
2013-05-31 15:53 | caristu | Status | resolved => closed |
2013-05-31 18:23 | hgbot | Checkin | |
2013-05-31 18:23 | hgbot | Note Added: 0059088 | |
2013-05-31 18:24 | shuehner | Note Added: 0059089 | |
2013-05-31 18:24 | shuehner | Status | closed => new |
2013-05-31 18:24 | shuehner | Resolution | fixed => open |
2013-05-31 18:25 | shuehner | File Added: org_setup_fix_23385.diff | |
2013-05-31 18:27 | shuehner | Note Added: 0059090 | |
2013-06-01 02:15 | hudsonbot | Checkin | |
2013-06-01 02:15 | hudsonbot | Note Added: 0059113 | |
2013-06-01 02:15 | hudsonbot | Checkin | |
2013-06-01 02:15 | hudsonbot | Note Added: 0059115 | |
2013-06-03 16:50 | jonalegriaesarte | Target Version | 3.0MP24 => 3.0MP25 |
2013-06-11 10:59 | jecharri | Status | new => scheduled |
2013-06-11 10:59 | jecharri | fix_in_branch | => pi |
2013-06-11 11:00 | jecharri | Status | scheduled => resolved |
2013-06-11 11:00 | jecharri | Fixed in Version | => 3.0MP25 |
2013-06-11 11:00 | jecharri | Fixed in SCM revision | http://code.openbravo.com/erp/devel/pi/rev/6792989a9124996136f6f7ad00061ece81fda599 [^] => https://code.openbravo.com/erp/devel/pi/rev/3f47396281c68c8fd8adb5b800f552b95bd225dd [^] |
2013-06-11 11:00 | jecharri | Resolution | open => fixed |
2013-06-11 11:00 | jecharri | Note Added: 0059321 | |
2013-06-11 18:20 | hgbot | Checkin | |
2013-06-11 18:20 | hgbot | Note Added: 0059372 | |
2013-06-11 18:21 | jecharri | Status | resolved => new |
2013-06-11 18:21 | jecharri | Resolution | fixed => open |
2013-06-11 18:21 | jecharri | Fixed in Version | 3.0MP25 => |
2013-06-13 15:39 | hgbot | Checkin | |
2013-06-13 15:39 | hgbot | Note Added: 0059422 | |
2013-06-13 15:39 | hgbot | Status | new => resolved |
2013-06-13 15:39 | hgbot | Resolution | open => fixed |
2013-06-13 15:39 | hgbot | Fixed in SCM revision | https://code.openbravo.com/erp/devel/pi/rev/3f47396281c68c8fd8adb5b800f552b95bd225dd [^] => http://code.openbravo.com/erp/devel/pi/rev/bde534f03bd48726603235e604e241af47bf0e3e [^] |
2013-06-18 10:11 | caristu | Note Added: 0059476 | |
2013-06-18 10:11 | caristu | Status | resolved => closed |
2013-06-20 02:08 | hudsonbot | Checkin | |
2013-06-20 02:08 | hudsonbot | Note Added: 0059569 | |
2013-06-20 02:08 | hudsonbot | Checkin | |
2013-06-20 02:08 | hudsonbot | Note Added: 0059575 |
Copyright © 2000 - 2009 MantisBT Group |