Project:
View Issue Details[ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||
ID | ||||||||
0022625 | ||||||||
Type | Category | Severity | Reproducibility | Date Submitted | Last Update | |||
defect | [Openbravo ERP] 03. Procurement management | major | always | 2012-12-19 06:01 | 2013-02-20 04:49 | |||
Reporter | sureshbabu | View Status | public | |||||
Assigned To | AugustoMauch | |||||||
Priority | high | Resolution | fixed | Fixed in Version | 3.0MP21 | |||
Status | closed | Fix in branch | Fixed in SCM revision | 23f36f89bcf2 | ||||
Projection | none | ETA | none | Target Version | ||||
OS | Linux 64 bit | Database | PostgreSQL | Java version | 1.6.0_18 | |||
OS Version | Professional Appliance | Database version | 8.3.9 | Ant version | 1.7.1 | |||
Product Version | pi | SCM revision | ||||||
Merge Request Status | ||||||||
Review Assigned To | shankarb | |||||||
OBNetwork customer | No | |||||||
Web browser | Google Chrome | |||||||
Modules | Core | |||||||
Support ticket | ||||||||
Regression level | ||||||||
Regression date | ||||||||
Regression introduced in release | ||||||||
Regression introduced by commit | ||||||||
Triggers an Emergency Pack | No | |||||||
Summary | 0022625: In Purchase order, auto save functionality not working in grid view at header tab. | |||||||
Description | In purchase order, i insert a new row using Ctrl + I command, and then after i enter business partner details, when i click "Copy Lines" button i am getting the below error "java.lang.ArrayIndexOutOfBoundsException: 0" | |||||||
Steps To Reproduce | a) Login as Openbravo/openbravo Change the role to QA Test Admin role Then navigate to Purchase order window, then click Ctrl + I to insert new row Select Business Partner as: Vendor A And without saving the record, click "Copy Lines" button System throws below error, "java.lang.ArrayIndexOutOfBoundsException: 0" Note: When i click the Copy lines button once again, Copy Lines from order popup open with out any trouble, and i am not getting the above mentioned error. b) System throws error message as "Error:ProcessRunError" when the user selects any other details after clicking the "Copy From Order" button. c) In goods receipt window,when i create a new record in grid view and after entering the Business partner details without saving the record when i click "Create Lines from" option i am getting below error "Error: Error java.lang.NullPointerException" | |||||||
Tags | No tags attached. | |||||||
Attached Files | ![]() | |||||||
![]() |
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
![]() |
|
(0055185) hgbot (developer) 2012-12-26 06:24 |
Repository: erp/devel/pi Changeset: faf2790df3d255b0277e89718cc025aa96d68d12 Author: Shankar Balachandran <shankar.balachandran <at> openbravo.com> Date: Wed Dec 26 10:35:08 2012 +0530 URL: http://code.openbravo.com/erp/devel/pi/rev/faf2790df3d255b0277e89718cc025aa96d68d12 [^] Fixes Issue 0022625: Issue in auto save in grid view When a process that is executed for an unsaved record in grid view tries to access the record an error is generated because the process is executed before saving the record. Modified code to save and then execute the process. --- M modules/org.openbravo.client.application/web/org.openbravo.client.application/js/main/ob-standard-window.js --- |
(0055188) hgbot (developer) 2012-12-26 08:19 |
Repository: erp/devel/pi Changeset: a9605441d9c9ffcf76eaef82dfec97efeb20cbd5 Author: Shankar Balachandran <shankar.balachandran <at> openbravo.com> Date: Wed Dec 26 12:46:11 2012 +0530 URL: http://code.openbravo.com/erp/devel/pi/rev/a9605441d9c9ffcf76eaef82dfec97efeb20cbd5 [^] Related to Issue 22625 : Issue in auto save in grid view. The earlier fix does not work for the case where the record is the first record of the grid. This is because rowNum is compared, but it fails if the rowNum value is 0. Modified the check constraint to ignore null values alone. --- M modules/org.openbravo.client.application/web/org.openbravo.client.application/js/toolbar/ob-action-button.js --- |
(0055214) hudsonbot (viewer) 2012-12-26 15:43 |
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/631fc2e1c525 [^] Maturity status: Test |
(0055246) hgbot (developer) 2012-12-27 14:28 |
Repository: erp/devel/pi Changeset: 3a21b44068143119228e58ad6ad64fadc1a0fa89 Author: Shankar Balachandran <shankar.balachandran <at> openbravo.com> Date: Thu Dec 27 18:53:32 2012 +0530 URL: http://code.openbravo.com/erp/devel/pi/rev/3a21b44068143119228e58ad6ad64fadc1a0fa89 [^] Related to Issue 22625: Using smartclient method to check for number inputs --- M modules/org.openbravo.client.application/web/org.openbravo.client.application/js/toolbar/ob-action-button.js --- |
(0055302) AugustoMauch (administrator) 2013-01-02 11:03 |
The fix is not proper. The new code to save the row has been placed in a function that is called after the autosave has been done, instead of fixing the autosave itself. |
(0055320) hudsonbot (viewer) 2013-01-03 13:11 |
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/792728467ed0 [^] Maturity status: Test |
(0055329) hudsonbot (viewer) 2013-01-03 13:12 |
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/792728467ed0 [^] Maturity status: Test |
(0055406) hgbot (developer) 2013-01-09 10:23 |
Repository: erp/devel/pi Changeset: 07560f9523e0a6299fe838958c73840a895bc4c1 Author: Shankar Balachandran <shankar.balachandran <at> openbravo.com> Date: Wed Jan 09 14:49:50 2013 +0530 URL: http://code.openbravo.com/erp/devel/pi/rev/07560f9523e0a6299fe838958c73840a895bc4c1 [^] Related to issue 22625 : Backout Changeset faf2790df3d2 Backed out changeset as it the changeset gets called at various other instances like switching over tabs. --- M modules/org.openbravo.client.application/web/org.openbravo.client.application/js/main/ob-standard-window.js --- |
(0055407) hgbot (developer) 2013-01-09 10:23 |
Repository: erp/devel/pi Changeset: 005e47a80a3178c30c11ff0fc53fdb3e3f196d6f Author: Shankar Balachandran <shankar.balachandran <at> openbravo.com> Date: Wed Jan 09 14:50:48 2013 +0530 URL: http://code.openbravo.com/erp/devel/pi/rev/005e47a80a3178c30c11ff0fc53fdb3e3f196d6f [^] Related to issue 22625 : Backout Changeset 3a21b4406814 Backed out changeset as it the changeset gets called at various other instances like switching over tabs. --- M modules/org.openbravo.client.application/web/org.openbravo.client.application/js/toolbar/ob-action-button.js --- |
(0055432) hudsonbot (viewer) 2013-01-09 18:04 |
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/314b62ac40f7 [^] Maturity status: Test |
(0055433) hudsonbot (viewer) 2013-01-09 18:04 |
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/314b62ac40f7 [^] Maturity status: Test |
(0056306) AugustoMauch (administrator) 2013-02-08 16:58 |
A patch for this issue has been attached. It will be pushed to pi once it is unfrozen for MP21 issues |
(0056340) hgbot (developer) 2013-02-11 16:51 |
Repository: erp/devel/pi Changeset: 23f36f89bcf27986a40cc66c4f85b32fba526072 Author: Augusto Mauch <augusto.mauch <at> openbravo.com> Date: Mon Feb 11 16:49:32 2013 +0100 URL: http://code.openbravo.com/erp/devel/pi/rev/23f36f89bcf27986a40cc66c4f85b32fba526072 [^] Fixes issue 22625: Fixes autosave in grid for new records The problem was that the new and proper record id (which is not available until the row has been saved in the database) was already available when the context info was retrieved, but was being overwritten by the old dummy in this line [1]. This happened because the old id was not being cleared, due to the fact that smartclient does not clear the values of the primary keys [2]. This is fixed by replacing the dummy id with the proper one if: - The id returned by viewGrid.getEditValues (old one) begins with '_' (is dummy) - The id returned by viewGrid.getRecord (new one) does not begin with '_' (is not dummy) [1] https://code.openbravo.com/erp/devel/pi/file/fe46ee66e77a/modules/org.openbravo.client.application/web/org.openbravo.client.application/js/main/ob-standard-view.js#l1993 [^] [2] https://code.openbravo.com/erp/mods/org.openbravo.userinterface.smartclient.dev/file/8c2bc26694da/web/org.openbravo.userinterface.smartclient/isomorphic/client/widgets/ListGrid.js#l28859 [^] --- M modules/org.openbravo.client.application/web/org.openbravo.client.application/js/main/ob-standard-view.js --- |
(0056371) shankarb (viewer) 2013-02-12 12:21 |
Code reviwed and verified in pi changeset 5265a23b7dbe |
(0056564) hudsonbot (viewer) 2013-02-20 04:49 |
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/d828167a9e83 [^] Maturity status: Test |
![]() |
|||
Date Modified | Username | Field | Change |
2012-12-19 06:01 | sureshbabu | New Issue | |
2012-12-19 06:01 | sureshbabu | Assigned To | => dmiguelez |
2012-12-19 06:01 | sureshbabu | Web browser | => Google Chrome |
2012-12-19 06:01 | sureshbabu | Modules | => Core |
2012-12-19 06:01 | sureshbabu | OBNetwork customer | => No |
2012-12-19 06:01 | sureshbabu | Triggers an Emergency Pack | => No |
2012-12-19 06:13 | sureshbabu | Web browser | Google Chrome => Google Chrome |
2012-12-19 06:13 | sureshbabu | Steps to Reproduce Updated | View Revisions |
2012-12-19 07:01 | sureshbabu | Web browser | Google Chrome => Google Chrome |
2012-12-19 07:01 | sureshbabu | Steps to Reproduce Updated | View Revisions |
2012-12-19 18:49 | dmiguelez | Assigned To | dmiguelez => dbaz |
2012-12-20 17:35 | dbaz | Assigned To | dbaz => shankarb |
2012-12-26 06:23 | shankarb | Issue Monitored: AugustoMauch | |
2012-12-26 06:24 | shankarb | Review Assigned To | => AugustoMauch |
2012-12-26 06:24 | shankarb | Web browser | Google Chrome => Google Chrome |
2012-12-26 06:24 | shankarb | Status | new => scheduled |
2012-12-26 06:24 | shankarb | fix_in_branch | => pi |
2012-12-26 06:24 | hgbot | Checkin | |
2012-12-26 06:24 | hgbot | Note Added: 0055185 | |
2012-12-26 06:24 | hgbot | Status | scheduled => resolved |
2012-12-26 06:24 | hgbot | Resolution | open => fixed |
2012-12-26 06:24 | hgbot | Fixed in SCM revision | => http://code.openbravo.com/erp/devel/pi/rev/faf2790df3d255b0277e89718cc025aa96d68d12 [^] |
2012-12-26 08:19 | hgbot | Checkin | |
2012-12-26 08:19 | hgbot | Note Added: 0055188 | |
2012-12-26 15:43 | hudsonbot | Checkin | |
2012-12-26 15:43 | hudsonbot | Note Added: 0055214 | |
2012-12-27 14:28 | hgbot | Checkin | |
2012-12-27 14:28 | hgbot | Note Added: 0055246 | |
2013-01-02 11:03 | AugustoMauch | Note Added: 0055302 | |
2013-01-02 11:03 | AugustoMauch | Status | resolved => new |
2013-01-02 11:03 | AugustoMauch | Resolution | fixed => open |
2013-01-03 13:11 | hudsonbot | Checkin | |
2013-01-03 13:11 | hudsonbot | Note Added: 0055320 | |
2013-01-03 13:12 | hudsonbot | Checkin | |
2013-01-03 13:12 | hudsonbot | Note Added: 0055329 | |
2013-01-09 10:23 | hgbot | Checkin | |
2013-01-09 10:23 | hgbot | Note Added: 0055406 | |
2013-01-09 10:23 | hgbot | Checkin | |
2013-01-09 10:23 | hgbot | Note Added: 0055407 | |
2013-01-09 18:04 | hudsonbot | Checkin | |
2013-01-09 18:04 | hudsonbot | Note Added: 0055432 | |
2013-01-09 18:04 | hudsonbot | Checkin | |
2013-01-09 18:04 | hudsonbot | Note Added: 0055433 | |
2013-02-08 14:46 | AugustoMauch | File Added: issue22625.diff | |
2013-02-08 16:58 | AugustoMauch | Note Added: 0056306 | |
2013-02-08 16:58 | AugustoMauch | Status | new => scheduled |
2013-02-08 17:04 | AugustoMauch | Relationship added | has duplicate 0022240 |
2013-02-09 07:57 | shankarb | Assigned To | shankarb => AugustoMauch |
2013-02-11 13:10 | AugustoMauch | Relationship added | duplicate of 0022278 |
2013-02-11 13:36 | AugustoMauch | Relationship deleted | 0022278 |
2013-02-11 13:36 | AugustoMauch | Relationship added | has duplicate 0022278 |
2013-02-11 16:26 | AugustoMauch | Issue Monitored: shankarb | |
2013-02-11 16:26 | AugustoMauch | Review Assigned To | AugustoMauch => shankarb |
2013-02-11 16:26 | AugustoMauch | Web browser | Google Chrome => Google Chrome |
2013-02-11 16:26 | AugustoMauch | fix_in_branch | pi => |
2013-02-11 16:51 | hgbot | Checkin | |
2013-02-11 16:51 | hgbot | Note Added: 0056340 | |
2013-02-11 16:51 | hgbot | Status | scheduled => resolved |
2013-02-11 16:51 | hgbot | Resolution | open => fixed |
2013-02-11 16:51 | hgbot | Fixed in SCM revision | http://code.openbravo.com/erp/devel/pi/rev/faf2790df3d255b0277e89718cc025aa96d68d12 [^] => http://code.openbravo.com/erp/devel/pi/rev/23f36f89bcf27986a40cc66c4f85b32fba526072 [^] |
2013-02-12 12:21 | shankarb | Note Added: 0056371 | |
2013-02-12 12:21 | shankarb | Status | resolved => closed |
2013-02-12 12:21 | shankarb | Fixed in Version | => 3.0MP21 |
2013-02-20 04:49 | hudsonbot | Checkin | |
2013-02-20 04:49 | hudsonbot | Note Added: 0056564 | |
2013-03-05 18:19 | AugustoMauch | Relationship added | has duplicate 0022050 |
2013-03-05 18:22 | AugustoMauch | Relationship added | has duplicate 0022052 |
2013-03-05 18:23 | AugustoMauch | Relationship added | has duplicate 0022038 |
2013-03-05 18:26 | AugustoMauch | Relationship added | has duplicate 0022057 |
2013-03-06 11:05 | AugustoMauch | Relationship added | has duplicate 0022041 |
2013-03-06 11:05 | AugustoMauch | Relationship added | has duplicate 0022067 |
2013-03-06 11:10 | AugustoMauch | Relationship added | has duplicate 0022039 |
2013-11-04 17:50 | caristu | Relationship added | related to 0025072 |
2019-05-29 18:02 | caristu | Relationship added | related to 0040463 |
Copyright © 2000 - 2009 MantisBT Group |