Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0024954Openbravo ERPA. Platformpublic2013-10-16 12:512013-10-18 10:19
Reporteregoitz 
Assigned ToAugustoMauch 
PriorityimmediateSeveritycriticalReproducibilityrandom
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target Version3.0MP29Fixed in Version3.0MP29 
Merge Request Status
Review Assigned Toshankarb
OBNetwork customer
Web browser
ModulesCore
Support ticket
Regression levelProduction - QA Approved
Regression date2013-07-22
Regression introduced in release
Regression introduced by commithttps://code.openbravo.com/erp/devel/pi/rev/f796eda7fdee3d278f41e15e16c6a6d42bd94ecb [^]
Triggers an Emergency PackNo
Summary0024954: The insert on grid view does not work after get an error on a previous insert
DescriptionThe insert on grid view does not work after get an error on a previous insert.
That problem does not happen always.

Seems to be related with grid or window personalization.
Steps To Reproduce-Go to the Tables and Columns window
-Add a new line on grid view with this data:
   * Data package: org.openbravo.model.ad.alert
   * Name: ADAlert1
   * DB Table Name: AD_Alert1
   * Leave Java Class Name empty
- Try to save, you will get an error
- Enter ADAlert1 in the Java Class Name, and save. That will work
- Create a new record in the grid view:
   * Data package: org.openbravo.model.ad.alert
   * Name: ADAlert2
   * DB Table Name: AD_Alert2
   * Java Class Name: ADAlert2
-You will get an error
Proposed Solution
Additional Information
TagsNo tags attached.
Relationships
depends on defect 0024863 closed AugustoMauch Not possible to save a row on grid having a window personalization 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2013-10-16 12:51egoitzNew Issue
2013-10-16 12:51egoitzAssigned To => AugustoMauch
2013-10-16 12:51egoitzModules => Core
2013-10-16 12:51egoitzTriggers an Emergency Pack => No
2013-10-16 12:52egoitzRegression level => Production - QA Approved
2013-10-16 12:52egoitzRegression date => 2013-07-22
2013-10-16 12:52egoitzRegression introduced by commit => https://code.openbravo.com/erp/devel/pi/rev/f796eda7fdee3d278f41e15e16c6a6d42bd94ecb [^]
2013-10-16 12:53egoitzRelationship addeddepends on 0024863
2013-10-16 15:55AugustoMauchIssue Monitored: shankarb
2013-10-16 15:55AugustoMauchReview Assigned To => shankarb
2013-10-16 16:03AugustoMauchSteps to Reproduce Updatedbug_revision_view_page.php?rev_id=5114#r5114
2013-10-16 16:04hgbotCheckin
2013-10-16 16:04hgbotNote Added: 0061717
2013-10-16 16:04hgbotStatusnew => resolved
2013-10-16 16:04hgbotResolutionopen => fixed
2013-10-16 16:04hgbotFixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/dd9eab1adea06e284d6a77e9ebacf3f45a2d6d92 [^]
2013-10-17 10:01shankarbNote Added: 0061764
2013-10-17 10:01shankarbStatusresolved => closed
2013-10-17 10:01shankarbFixed in Version => 3.0MP29
2013-10-17 18:21hudsonbotCheckin
2013-10-17 18:21hudsonbotNote Added: 0061774
2013-10-18 10:19AugustoMauchNote Added: 0061797

Notes
(0061717)
hgbot   
2013-10-16 16:04   
Repository: erp/devel/pi
Changeset: dd9eab1adea06e284d6a77e9ebacf3f45a2d6d92
Author: Augusto Mauch <augusto.mauch <at> openbravo.com>
Date: Wed Oct 16 15:54:59 2013 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/dd9eab1adea06e284d6a77e9ebacf3f45a2d6d92 [^]

Fixes bug 24954: Inserting a new recort not working a previous insert error

The problem was that the view._savingNewRecord was not being reset after the a new record request to the datasource coming from the grid resulted in an error. Now this case in handled in the editFailed function of ob-view-grid.

---
M modules/org.openbravo.client.application/web/org.openbravo.client.application/js/grid/ob-view-grid.js
---
(0061764)
shankarb   
2013-10-17 10:01   
Code reviewed and tested in pi changeset 89268df786de.
(0061774)
hudsonbot   
2013-10-17 18:21   
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/121b38a3673a [^]

Maturity status: Test
(0061797)
AugustoMauch   
2013-10-18 10:19   
Note:

In order to reproduce it, you have to first set the Core module in development