Openbravo Issue Tracking System - Openbravo ERP |
| View Issue Details |
|
| ID | Project | Category | View Status | Date Submitted | Last Update |
| 0019779 | Openbravo ERP | B. User interface | public | 2012-02-15 16:55 | 2012-03-02 15:38 |
|
| Reporter | dbaz | |
| Assigned To | dbaz | |
| Priority | high | Severity | major | Reproducibility | sometimes |
| Status | closed | Resolution | fixed | |
| Platform | | OS | 5 | OS Version | |
| Product Version | | |
| Target Version | | Fixed in Version | | |
| Merge Request Status | |
| Review Assigned To | |
| OBNetwork customer | No |
| Web browser | |
| Modules | Core |
| Support ticket | |
| Regression level | |
| Regression date | |
| Regression introduced in release | |
| Regression introduced by commit | |
| Triggers an Emergency Pack | No |
|
| Summary | 0019779: [PNE] Clicking twice fastly the "Pick and Execute" button process can open the window with two grids |
| Description | Clicking twice fastly the "Pick and Execute" button process can open the window with two grids |
| Steps To Reproduce | As "Openbravo Admin":
* Go to "Return from Customer"
* Create new (and save)
* Double-Click (or triple-click) "Pick/Edit Lines".
Sometimes the opened view has two "popup" inside it instead one. In this case a vertical screen is shown and you can scroll down to see it. |
| Proposed Solution | |
| Additional Information | |
| Tags | No tags attached. |
| Relationships | | related to | defect | 0027621 | | closed | AugustoMauch | If you get a network interruption when clicking on the edit item button , then the button does not work anymore |
|
| Attached Files | 19779.diff (1,249) 2012-02-21 12:37 https://issues.openbravo.com/file_download.php?file_id=5014&type=bug |
|
| Issue History |
| Date Modified | Username | Field | Change |
| 2012-02-15 16:55 | dbaz | New Issue | |
| 2012-02-15 16:55 | dbaz | Assigned To | => dbaz |
| 2012-02-15 16:55 | dbaz | Modules | => Core |
| 2012-02-15 16:55 | dbaz | OBNetwork customer | => No |
| 2012-02-15 16:59 | hgbot | Checkin | |
| 2012-02-15 16:59 | hgbot | Note Added: 0045155 | |
| 2012-02-15 16:59 | hgbot | Status | new => resolved |
| 2012-02-15 16:59 | hgbot | Resolution | open => fixed |
| 2012-02-15 16:59 | hgbot | Fixed in SCM revision | => http://code.openbravo.com/erp/devel/pi/rev/f2585e9deaf598718190a8b3ad40b23df10d655c [^] |
| 2012-02-21 12:37 | iperdomo | Note Added: 0045302 | |
| 2012-02-21 12:37 | iperdomo | Status | resolved => new |
| 2012-02-21 12:37 | iperdomo | Resolution | fixed => open |
| 2012-02-21 12:37 | iperdomo | File Added: 19779.diff | |
| 2012-02-21 13:14 | hgbot | Checkin | |
| 2012-02-21 13:14 | hgbot | Note Added: 0045307 | |
| 2012-02-21 13:14 | hgbot | Status | new => resolved |
| 2012-02-21 13:14 | hgbot | Resolution | open => fixed |
| 2012-02-21 13:14 | hgbot | Fixed in SCM revision | http://code.openbravo.com/erp/devel/pi/rev/f2585e9deaf598718190a8b3ad40b23df10d655c [^] => http://code.openbravo.com/erp/devel/pi/rev/74e03242ccdc41c3eb242fb05d7f63969f433e8d [^] |
| 2012-02-21 13:53 | iperdomo | Note Added: 0045308 | |
| 2012-02-21 13:53 | iperdomo | Status | resolved => closed |
| 2012-03-02 15:37 | hudsonbot | Checkin | |
| 2012-03-02 15:37 | hudsonbot | Note Added: 0045858 | |
| 2012-03-02 15:38 | hudsonbot | Checkin | |
| 2012-03-02 15:38 | hudsonbot | Note Added: 0045910 | |
| 2014-09-19 12:57 | AugustoMauch | Relationship added | related to 0027621 |
|
Notes |
|
|
(0045155)
|
|
hgbot
|
|
2012-02-15 16:59
|
|
Repository: erp/devel/pi
Changeset: f2585e9deaf598718190a8b3ad40b23df10d655c
Author: David Baz Fayos <david.baz <at> openbravo.com>
Date: Wed Feb 15 16:58:28 2012 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/f2585e9deaf598718190a8b3ad40b23df10d655c [^]
Fixed issue 19779: 'Double-click' is not possible anymore in new process buttons
---
M modules/org.openbravo.client.application/web/org.openbravo.client.application/js/toolbar/ob-action-button.js
---
|
|
|
|
|
Testing on pi @ rev 8e3bca6e8153
Disabling the button gives a strange feeling since none of the other buttons do that. You can click as many times you want, but you only have 1 modal dialog.
I propose that instead of disabling, just append a flag to the button, opening = true after the first call and reset the flag on the callback function. This produces the same behavior than the other buttons.
Attached a sample diff |
|
|
|
(0045307)
|
|
hgbot
|
|
2012-02-21 13:14
|
|
Repository: erp/devel/pi
Changeset: 74e03242ccdc41c3eb242fb05d7f63969f433e8d
Author: David Baz Fayos <david.baz <at> openbravo.com>
Date: Tue Feb 21 13:14:31 2012 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/74e03242ccdc41c3eb242fb05d7f63969f433e8d [^]
Fixed issue 19779: 'Double-click' is not possible anymore in new process buttons
---
M modules/org.openbravo.client.application/web/org.openbravo.client.application/js/toolbar/ob-action-button.js
---
|
|
|
|
|
|
Code review, tested on pi @ rev 74e03242ccdc |
|
|
|
|
|
|
|
|
|