Project:
View Issue Details[ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||
ID | ||||||||
0024783 | ||||||||
Type | Category | Severity | Reproducibility | Date Submitted | Last Update | |||
defect | [Openbravo ERP] A. Platform | minor | always | 2013-09-16 17:44 | 2013-10-30 20:47 | |||
Reporter | caristu | View Status | public | |||||
Assigned To | shankarb | |||||||
Priority | high | Resolution | fixed | Fixed in Version | 3.0MP29 | |||
Status | closed | Fix in branch | Fixed in SCM revision | 74ad554804c2 | ||||
Projection | none | ETA | none | Target Version | 3.0MP29 | |||
OS | Any | Database | Any | Java version | ||||
OS Version | Database version | Ant version | ||||||
Product Version | SCM revision | |||||||
Merge Request Status | ||||||||
Review Assigned To | AugustoMauch | |||||||
OBNetwork customer | OBPS | |||||||
Web browser | ||||||||
Modules | Core | |||||||
Support ticket | 23403 | |||||||
Regression level | ||||||||
Regression date | ||||||||
Regression introduced in release | ||||||||
Regression introduced by commit | ||||||||
Triggers an Emergency Pack | No | |||||||
Summary | 0024783: Time fields are not refreshed when setting their values trough an event handler | |||||||
Description | The fields that uses a time reference are not refreshed after setting their value using a business event handler (http://wiki.openbravo.com/wiki/How_to_implement_a_business_event_handler [^]) | |||||||
Steps To Reproduce | 1) Create a new event handler for a window (entity) that has a field with a time reference, for example Work Effort. (Attached is an example for this window) 2) Implement the onSave method(), setting the value for the time field. 3) Notice that after executing the event handler, it is necessary to click on the refresh button to see the new value for the time field. | |||||||
Tags | No tags attached. | |||||||
Attached Files | ![]() | |||||||
![]() |
||||||||
|
![]() |
|
(0061956) hgbot (developer) 2013-10-30 08:23 |
Repository: erp/devel/pi Changeset: 942524b9dfc07c617594c5640fce54faed410367 Author: Shankar Balachandran <shankar.balachandran <at> openbravo.com> Date: Wed Oct 30 12:50:08 2013 +0530 URL: http://code.openbravo.com/erp/devel/pi/rev/942524b9dfc07c617594c5640fce54faed410367 [^] Fixes Issue 0024783: Time fields are not refreshed on value change through an event handler The time fields were not updated when updated through event handlers. Used the formSaved event to check the fields current value and the data returned value and setting the new field value from data. --- M modules/org.openbravo.client.application/web/org.openbravo.client.application/js/form/formitem/ob-formitem-time.js --- |
(0061958) hgbot (developer) 2013-10-30 09:25 |
Repository: erp/devel/pi Changeset: 2eb60e1f2b727a6e1c0a6aec00714e5c1a3ecdd9 Author: Pandeeswari Ramakrishnan <pandeeswari.ramakrishnan <at> openbravo.com> Date: Wed Oct 30 13:54:05 2013 +0530 URL: http://code.openbravo.com/erp/devel/pi/rev/2eb60e1f2b727a6e1c0a6aec00714e5c1a3ecdd9 [^] Related to Issue 0024783 : Code formatting --- M modules/org.openbravo.client.application/web/org.openbravo.client.application/js/form/formitem/ob-formitem-time.js --- |
(0061960) AugustoMauch (administrator) 2013-10-30 09:57 |
Reopened becaues it does not handle the following cases: - When the event handler changes the time value from not null to null - When the event handler changes the time value from null to not null |
(0061962) hgbot (developer) 2013-10-30 10:22 |
Repository: erp/devel/pi Changeset: 74ad554804c2aaccce13cd65b39a7d8c40aae5d9 Author: Shankar Balachandran <shankar.balachandran <at> openbravo.com> Date: Wed Oct 30 14:50:14 2013 +0530 URL: http://code.openbravo.com/erp/devel/pi/rev/74ad554804c2aaccce13cd65b39a7d8c40aae5d9 [^] Fixes Issue 24783 : Time fields are not refreshed when setting their values trough an event handler The response data value of the field is set to the field value if not equal. --- M modules/org.openbravo.client.application/web/org.openbravo.client.application/js/form/formitem/ob-formitem-time.js --- |
(0061965) AugustoMauch (administrator) 2013-10-30 10:26 |
Code reviewed and verified in pi@74ad554804c2 |
(0061984) hudsonbot (viewer) 2013-10-30 20:47 |
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/80f375c041fe [^] Maturity status: Test |
(0061985) hudsonbot (viewer) 2013-10-30 20:47 |
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/80f375c041fe [^] Maturity status: Test |
(0061987) hudsonbot (viewer) 2013-10-30 20:47 |
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/80f375c041fe [^] Maturity status: Test |
![]() |
|||
Date Modified | Username | Field | Change |
2013-09-16 17:44 | caristu | New Issue | |
2013-09-16 17:44 | caristu | Assigned To | => AugustoMauch |
2013-09-16 17:44 | caristu | Modules | => Core |
2013-09-16 17:44 | caristu | OBNetwork customer | => Yes |
2013-09-16 17:44 | caristu | Support ticket | => 23403 |
2013-09-16 17:44 | caristu | Resolution time | => 1385766000 |
2013-09-16 17:44 | caristu | Triggers an Emergency Pack | => No |
2013-09-16 17:44 | caristu | Issue Monitored: networkb | |
2013-09-16 17:45 | caristu | File Added: TimeEventHandler.java | |
2013-09-16 17:55 | caristu | Relationship added | related to 0024491 |
2013-10-30 08:15 | shankarb | Assigned To | AugustoMauch => shankarb |
2013-10-30 08:16 | shankarb | Review Assigned To | => AugustoMauch |
2013-10-30 08:17 | shankarb | Issue Monitored: AugustoMauch | |
2013-10-30 08:23 | hgbot | Checkin | |
2013-10-30 08:23 | hgbot | Note Added: 0061956 | |
2013-10-30 08:23 | hgbot | Status | new => resolved |
2013-10-30 08:23 | hgbot | Resolution | open => fixed |
2013-10-30 08:23 | hgbot | Fixed in SCM revision | => http://code.openbravo.com/erp/devel/pi/rev/942524b9dfc07c617594c5640fce54faed410367 [^] |
2013-10-30 09:25 | hgbot | Checkin | |
2013-10-30 09:25 | hgbot | Note Added: 0061958 | |
2013-10-30 09:57 | AugustoMauch | Note Added: 0061960 | |
2013-10-30 09:57 | AugustoMauch | Status | resolved => new |
2013-10-30 09:57 | AugustoMauch | Resolution | fixed => open |
2013-10-30 10:22 | hgbot | Checkin | |
2013-10-30 10:22 | hgbot | Note Added: 0061962 | |
2013-10-30 10:22 | hgbot | Status | new => resolved |
2013-10-30 10:22 | hgbot | Resolution | open => fixed |
2013-10-30 10:22 | hgbot | Fixed in SCM revision | http://code.openbravo.com/erp/devel/pi/rev/942524b9dfc07c617594c5640fce54faed410367 [^] => http://code.openbravo.com/erp/devel/pi/rev/74ad554804c2aaccce13cd65b39a7d8c40aae5d9 [^] |
2013-10-30 10:26 | AugustoMauch | Note Added: 0061965 | |
2013-10-30 10:26 | AugustoMauch | Status | resolved => closed |
2013-10-30 10:26 | AugustoMauch | Fixed in Version | => 3.0MP29 |
2013-10-30 20:47 | hudsonbot | Checkin | |
2013-10-30 20:47 | hudsonbot | Note Added: 0061984 | |
2013-10-30 20:47 | hudsonbot | Checkin | |
2013-10-30 20:47 | hudsonbot | Note Added: 0061985 | |
2013-10-30 20:47 | hudsonbot | Checkin | |
2013-10-30 20:47 | hudsonbot | Note Added: 0061987 |
Copyright © 2000 - 2009 MantisBT Group |