Project:
| View Issue Details[ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||
| ID | ||||||||
| 0023651 | ||||||||
| Type | Category | Severity | Reproducibility | Date Submitted | Last Update | |||
| defect | [Openbravo ERP] A. Platform | trivial | always | 2013-04-26 12:41 | 2013-07-28 11:36 | |||
| Reporter | marvintm | View Status | public | |||||
| Assigned To | AugustoMauch | |||||||
| Priority | normal | Resolution | fixed | Fixed in Version | 3.0MP26 | |||
| Status | closed | Fix in branch | Fixed in SCM revision | 0b7bf7e1522c | ||||
| Projection | none | ETA | none | Target Version | ||||
| OS | Any | Database | Any | Java version | ||||
| OS Version | Database version | Ant version | ||||||
| Product Version | SCM revision | |||||||
| Review Assigned To | shankarb | |||||||
| Web browser | ||||||||
| Modules | Core | |||||||
| Regression level | ||||||||
| Regression date | ||||||||
| Regression introduced in release | ||||||||
| Regression introduced by commit | ||||||||
| Triggers an Emergency Pack | No | |||||||
| Summary | 0023651: Message in StandardWindowComponent is not always correct | |||||||
| Description | There is a check in StandardWindowComponent which checks whether the structure of the window definition is correct. This check is done at the end of the getRootTabComponent, and a message is shown in line 181 ("Window has more than on tab on level 0, choosing an arbitrary root tab") The check itself is correct, but the message is not always right. The check will also fail if two tabs have the same sequence number, and different tab levels. It is fine that the check fails in this case, but the message shown there should not be the one shown (because this can also happen in a level different from 0, and the problem then is not that the tabs have the same level on level 0, but that they have the same sequence number). | |||||||
| Steps To Reproduce | - Install retail pack version RMP21 or earlier. - Verify that the message is shown when going to the organization window. It is true that the definition of the window is incorrect, but the message itself is also not correct either. | |||||||
| Tags | No tags attached. | |||||||
| Attached Files | ||||||||
 Relationships [ Relation Graph ]
[ Dependency Graph ]
|
|
 Relationships |
|
Notes |
|
|
(0060308) hgbot (developer) 2013-07-26 13:48 |
Repository: erp/devel/pi Changeset: 0b7bf7e1522c81b8ca68677b58e5df2c23815a04 Author: Augusto Mauch <augusto.mauch <at> openbravo.com> Date: Fri Jul 26 13:46:55 2013 +0200 URL: http://code.openbravo.com/erp/devel/pi/rev/0b7bf7e1522c81b8ca68677b58e5df2c23815a04 [^] Fixes issue 23651: A proper warning message is shown in StandardWindowComponent The same warning message was being shown in these cases: - 2+ tabs with the same sequence number but different tab levels - 2+ tabs with tab level 0 A new check has been added to tell these two cases apart and to show the proper warning message in each case. --- M modules/org.openbravo.client.application/src/org/openbravo/client/application/window/StandardWindowComponent.java --- |
|
(0060310) AugustoMauch (manager) 2013-07-26 13:54 |
Test plan: Test that in these cases the proper warning is shown in the log - Two tabs with tabLevel 0 - Two tabs with same sequence number but different tab level, the tab level of one of them is 0 - Two tabs with same sequence number but different tab level, the tab level of both of them is not 0 |
|
(0060321) shankarb (reporter) 2013-07-28 03:09 |
Code reviewed and verified in pi changeset f49b9428b38d |
|
(0060325) hudsonbot (developer) 2013-07-28 11:36 |
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/9d1d514c85e8 [^] Maturity status: Test |
|
Notes |
|
Issue History |
|||
| Date Modified | Username | Field | Change |
| 2013-04-26 12:41 | marvintm | New Issue | |
| 2013-04-26 12:41 | marvintm | Assigned To | => AugustoMauch |
| 2013-04-26 12:41 | marvintm | Modules | => Core |
| 2013-04-26 12:41 | marvintm | Triggers an Emergency Pack | => No |
| 2013-07-26 08:21 | shankarb | Assigned To | AugustoMauch => shankarb |
| 2013-07-26 13:43 | AugustoMauch | Assigned To | shankarb => AugustoMauch |
| 2013-07-26 13:43 | AugustoMauch | Issue Monitored: shankarb | |
| 2013-07-26 13:43 | AugustoMauch | Review Assigned To | => shankarb |
| 2013-07-26 13:48 | hgbot | Checkin | |
| 2013-07-26 13:48 | hgbot | Note Added: 0060308 | |
| 2013-07-26 13:48 | hgbot | Status | new => resolved |
| 2013-07-26 13:48 | hgbot | Resolution | open => fixed |
| 2013-07-26 13:48 | hgbot | Fixed in SCM revision | => http://code.openbravo.com/erp/devel/pi/rev/0b7bf7e1522c81b8ca68677b58e5df2c23815a04 [^] |
| 2013-07-26 13:54 | AugustoMauch | Note Added: 0060310 | |
| 2013-07-28 03:09 | shankarb | Note Added: 0060321 | |
| 2013-07-28 03:09 | shankarb | Status | resolved => closed |
| 2013-07-28 03:09 | shankarb | Fixed in Version | => 3.0MP26 |
| 2013-07-28 11:36 | hudsonbot | Checkin | |
| 2013-07-28 11:36 | hudsonbot | Note Added: 0060325 | |
|
Issue History |
| Copyright © 2000 - 2009 MantisBT Group |
 |