Project:
| View Issue Details[ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||
| ID | ||||||||
| 0009027 | ||||||||
| Type | Category | Severity | Reproducibility | Date Submitted | Last Update | |||
| defect | [Openbravo ERP] 09. Financial management | major | always | 2009-05-13 18:54 | 2009-07-27 19:38 | |||
| Reporter | networkb | View Status | public | |||||
| Assigned To | rafaroda | |||||||
| Priority | immediate | Resolution | fixed | Fixed in Version | 2.50MP3 | |||
| Status | closed | Fix in branch | Fixed in SCM revision | 672dad0c98b0 | ||||
| Projection | none | ETA | none | Target Version | 2.40MP3 | |||
| OS | Any | Database | PostgreSQL | Java version | ||||
| OS Version | Database version | Ant version | ||||||
| Product Version | 2.40MP3 | SCM revision | ||||||
| Review Assigned To | ||||||||
| Web browser | ||||||||
| Modules | Core | |||||||
| Regression level | ||||||||
| Regression date | ||||||||
| Regression introduced in release | ||||||||
| Regression introduced by commit | ||||||||
| Triggers an Emergency Pack | No | |||||||
| Summary | 0009027: General Ledger Report executed with "show all" doesn't respect column's width | |||||||
| Description | When executing General Ledger Report with option "show all" selected, minimum width of columns is not respected. See image attached (For example: columns with no data in description field are not 75% width) | |||||||
| Steps To Reproduce | 1. Financial Management || Accounting || Analysis Tools || General Ledger Report 2. Check "show all" and execute the report clicking "HTML format" button | |||||||
| Tags | 250MP3releasecandidate | |||||||
| Attached Files |  showAllGL.jpg [^] (128,876 bytes) 2009-05-13 18:54
 GLShowAll.png [^] (226,615 bytes) 2009-05-28 13:04
 9027.diff [^] (643 bytes) 2009-07-03 17:17 [Show Content]
WithoutFix.png [^] (169,624 bytes) 2009-07-08 10:55
WithFix.png [^] (171,669 bytes) 2009-07-08 10:55
| |||||||
 Relationships [ Relation Graph ]
[ Dependency Graph ]
|
|||||||||||||||
|
|||||||||||||||
 Relationships |
|
Notes |
|
|
(0016610) dbaz (developer) 2009-05-25 10:32 |
The only possible solution is set a fixed width of each column (the maximun column with should be asked to Localization team, because once the maximun is reached it won't be able to see any more digit of a column). |
|
(0016614) rafaroda (developer) 2009-05-25 11:21 |
David, Could you please see if the solution proposed by dbaz is valid? Thanks. |
|
(0016618) dalsasua (reporter) 2009-05-25 11:47 |
Hi, In my opinion, it's not necessary to change the standard code, because it would imply a lot of testing to check that continues to work trough several browsers and screen resolutions. I would apply the proposed solution from David to the customer that reports the bug, and just test it in their environment. It is true that a financial report is not well built if columns are not aligned from one block to another, but the official report is the one printed through the pdf file, and this is correctly built. The html report is just a draft view of the report, to easy check and look up numbers. Once this is clear, I would say that the width of each column would be: date column: necessary to show the text '99-99-9999' dates debit, credit and total columns: necessary with to show the text '-999.999.999,99' description column: the rest of available width Regards. |
|
(0016703) rafaroda (developer) 2009-05-28 10:27 |
This issue is waiting for feedback regarding David Alsasua suggestion. |
|
(0016708) networkb (developer) 2009-05-28 10:53 |
David, Could you please explain why % width does not work? Thanks |
|
(0016712) dbaz (developer) 2009-05-28 10:56 |
Because the text contained in the cells exceeds the % |
|
(0016727) networkb (developer) 2009-05-28 13:04 |
We have been doing some tests and we have realized that in GeneralLedgerReport.html, there is a table which first row has width=100% so it is doing that the rest of width columns are not being respected. Modifying this and adjusting % width related to DAL indications above, we will make % work and appearance will be aligned and adapted to text. (screenshot attached GLShowAll.png) |
|
(0016728) rafaroda (developer) 2009-05-28 13:09 |
David Baz, Do you think that the fix proposed by networkb should be included in Openbravo ERP? If so please schedule the issue. Thanks. |
|
(0016729) dbaz (developer) 2009-05-28 13:12 |
Since it is a particular report and not a standard behaviour for all the application, localization team should take the decision... I reasign it to David Alsasua |
|
(0016730) dalsasua (reporter) 2009-05-28 13:41 |
In my opinion, this should be applied only to the reporter's environment. This change of the report, makes it expand horizontally as much as needed, no matter if description is so large that the width of the report is much bigger than the width of the filters section. This is not comfortable for lower resolution monitors, and breaks the standard behavior of the ERP. Regards. |
|
(0016731) networkb (developer) 2009-05-28 16:59 |
Just one final doubt: the only thing that we have done is to change the following line "<TD width="100%" class="DataGrid_Header_Cell">Cliente<SPAN id="fieldPartner2"></SPAN></TD>" by this other one "<TD colspan="5" class="DataGrid_Header_Cell">Cliente<SPAN id="fieldPartner2"></SPAN></TD></TR>" in ReportGeneralLedger.html file. The behavior which follows the report when description is too large expanding horizontally as much as needed (being bigger than the width of the filters section, if necessary) is due to clause "style=table-layout: auto;" which is set in the standard pattern of table definition... so it is following the standard behavior, isn't it? In case we want that reports data don't exceed filters section, we will need to set "style=table-layout: fixed;" and revise other table definitions... |
|
(0016734) rafaroda (developer) 2009-05-28 18:24 |
David Baz, Any comment on this? Thanks. |
|
(0016791) rafaroda (developer) 2009-06-01 09:22 |
Reminder sent to: dbaz David Baz, could you please determine whether there is an issue in the HTML coding of this report? Thanks. |
|
(0016801) rafaroda (developer) 2009-06-01 11:43 |
According to David Alsasua, this issue will not be fixed in standard Openbravo ERP but change can be applied to the customer. |
|
(0017430) networkb (developer) 2009-06-18 17:51 |
Hi again, We would like to understand why this bug was closed once we had located the problem and without known David Baz's opinion. Because of that, we continue having the same doubt: "the only thing that we have done is to change the following line "<TD width="100%" class="DataGrid_Header_Cell">Cliente<SPAN id="fieldPartner2"></SPAN></TD>" by this other one "<TD colspan="5" class="DataGrid_Header_Cell">Cliente<SPAN id="fieldPartner2"></SPAN></TD></TR>" in ReportGeneralLedger.html file. The behavior which follows the report when description is too large expanding horizontally as much as needed (being bigger than the width of the filters section, if necessary) is due to clause "style=table-layout: auto;" which is set in the standard pattern of table definition... so it is following the standard behavior, isn't it? In case we want that reports data don't exceed filters section, we will need to set "style=table-layout: fixed;" and revise other table definitions..." Thanks |
|
(0017534) networkb (developer) 2009-06-22 19:10 |
Hi again, We would like to understand why this bug was closed once we had located the problem and without known David Baz's opinion. Because of that, we continue having the same doubt: "the only thing that we have done is to change the following line "<TD width="100%" class="DataGrid_Header_Cell">Cliente<SPAN id="fieldPartner2"></SPAN></TD>" by this other one "<TD colspan="5" class="DataGrid_Header_Cell">Cliente<SPAN id="fieldPartner2"></SPAN></TD></TR>" in ReportGeneralLedger.html file. The behavior which follows the report when description is too large expanding horizontally as much as needed (being bigger than the width of the filters section, if necessary) is due to clause "style=table-layout: auto;" which is set in the standard pattern of table definition... so it is following the standard behavior, isn't it? In case we want that reports data don't exceed filters section, we will need to set "style=table-layout: fixed;" and revise other table definitions..." Thanks |
|
(0017536) dbaz (developer) 2009-06-22 19:29 |
Hi networkb At the end after reading your question I don't understand if with "style=table-layout: fixed;" you have finally succeed to fix the bug or not. If the answer is "yes" it is ok for me. Even more, since this "bug" is only going to be fixed in your client, any solution that you find which has your client's agree is valid. |
|
(0017898) networkb (developer) 2009-07-03 17:17 |
Hi, I attach the file we have applied to our customer. |
|
(0017961) vmromanos (manager) 2009-07-07 12:06 |
It seems that the bug is already fixed in the customer's environment and, from Engineering, we have decided not to change core. Shall we close the bug or do you need more clarifications? |
|
(0018009) rafaroda (developer) 2009-07-08 10:54 |
I do agree that ReportGeneralLedger.html has an issue in lines: <div id="sectionPartner"> <table border="1" cellspacing="0" cellpadding="0" width="90%" class="DataGrid_Header_Table DataGrid_Body_Table" style="table-layout: auto;"> <tr class="DataGrid_Body_Row"> <td width="100%" class="DataGrid_Header_Cell">Customer <span id="fieldPartner2"></span></td> </tr> which should be replaced by: <div id="sectionPartner"> <table border="1" cellspacing="0" cellpadding="0" width="90%" class="DataGrid_Header_Table DataGrid_Body_Table" style="table-layout: auto;"> <tr class="DataGrid_Body_Row"> <td colspan="5" class="DataGrid_Header_Cell">Customer <span id="fieldPartner2"></span></td> </tr> The current behavior looks like WithoutFix.png where the Date column takes too much width. Applying the fix, WithFix.png, Date column takes a reasonable width but Description cell keeps being the one on which the other cells' width are based. |
|
(0018112) hgbot (developer) 2009-07-10 16:06 |
Repository: erp/devel/pi Changeset: 672dad0c98b0e1d7e4adcbf7068cbaab5323f281 Author: Rafa Roda Palacios <rafael.roda <at> openbravo.com> Date: Fri Jul 10 16:05:48 2009 +0200 URL: http://code.openbravo.com/erp/devel/pi/rev/672dad0c98b0e1d7e4adcbf7068cbaab5323f281 [^] Fixes issue 0009027 Replaced width=\\"100%\\" by colspan=\\"5\\" to construct ReportGeneralLedger.html properly. --- M src/org/openbravo/erpCommon/ad_reports/ReportGeneralLedger.html --- |
|
(0018635) psarobe (manager) 2009-07-27 19:38 |
Following Withfix.png image, the bug seems to be fixed |
|
Notes |
|
Issue History |
|||
| Date Modified | Username | Field | Change |
| 2009-05-13 18:54 | networkb | New Issue | |
| 2009-05-13 18:54 | networkb | Assigned To | => rafaroda |
| 2009-05-13 18:54 | networkb | File Added: showAllGL.jpg | |
| 2009-05-13 18:54 | networkb | Regression testing | => No |
| 2009-05-15 08:21 | rafaroda | Assigned To | rafaroda => dalsasua |
| 2009-05-15 10:39 | vmromanos | Assigned To | dalsasua => vmromanos |
| 2009-05-15 10:40 | vmromanos | Status | new => scheduled |
| 2009-05-15 10:40 | vmromanos | fix_in_branch | => pi |
| 2009-05-15 18:09 | vmromanos | Status | scheduled => feedback |
| 2009-05-20 12:07 | networkb | Priority | immediate => high |
| 2009-05-20 12:07 | networkb | Status | feedback => new |
| 2009-05-20 12:07 | networkb | fix_in_branch | pi => |
| 2009-05-20 12:07 | networkb | Summary | General Ledger Report's appearance is aligned when executed with "show all" selected => General Ledger Report executed with "show all" doesn't respect column's width |
| 2009-05-20 12:07 | networkb | Description Updated | |
| 2009-05-20 12:22 | networkb | Assigned To | vmromanos => dbaz |
| 2009-05-20 12:49 | psarobe | Status | new => scheduled |
| 2009-05-20 12:49 | psarobe | fix_in_branch | => pi |
| 2009-05-20 12:50 | psarobe | Priority | high => immediate |
| 2009-05-20 12:50 | psarobe | fix_in_branch | pi => |
| 2009-05-25 10:32 | dbaz | Note Added: 0016610 | |
| 2009-05-25 10:33 | dbaz | Assigned To | dbaz => rafaroda |
| 2009-05-25 11:21 | rafaroda | Note Added: 0016614 | |
| 2009-05-25 11:21 | rafaroda | Assigned To | rafaroda => dalsasua |
| 2009-05-25 11:47 | dalsasua | Note Added: 0016618 | |
| 2009-05-25 11:47 | dalsasua | Assigned To | dalsasua => rafaroda |
| 2009-05-28 10:27 | rafaroda | Note Added: 0016703 | |
| 2009-05-28 10:27 | rafaroda | Status | scheduled => feedback |
| 2009-05-28 10:53 | networkb | Note Added: 0016708 | |
| 2009-05-28 10:53 | networkb | Assigned To | rafaroda => dbaz |
| 2009-05-28 10:53 | networkb | Status | feedback => new |
| 2009-05-28 10:56 | dbaz | Note Added: 0016712 | |
| 2009-05-28 10:56 | dbaz | Assigned To | dbaz => rafaroda |
| 2009-05-28 10:57 | dalsasua | Note Added: 0016713 | |
| 2009-05-28 13:04 | networkb | File Added: GLShowAll.png | |
| 2009-05-28 13:04 | networkb | Note Added: 0016727 | |
| 2009-05-28 13:09 | rafaroda | Note Added: 0016728 | |
| 2009-05-28 13:09 | rafaroda | Assigned To | rafaroda => dbaz |
| 2009-05-28 13:09 | rafaroda | Note Deleted: 0016713 | |
| 2009-05-28 13:12 | dbaz | Note Added: 0016729 | |
| 2009-05-28 13:12 | dbaz | Assigned To | dbaz => dalsasua |
| 2009-05-28 13:41 | dalsasua | Note Added: 0016730 | |
| 2009-05-28 13:42 | dalsasua | Assigned To | dalsasua => rafaroda |
| 2009-05-28 16:59 | networkb | Note Added: 0016731 | |
| 2009-05-28 18:24 | rafaroda | Note Added: 0016734 | |
| 2009-05-28 18:24 | rafaroda | Assigned To | rafaroda => dbaz |
| 2009-06-01 09:22 | rafaroda | Note Added: 0016791 | |
| 2009-06-01 11:42 | rafaroda | Assigned To | dbaz => dalsasua |
| 2009-06-01 11:43 | rafaroda | Status | new => closed |
| 2009-06-01 11:43 | rafaroda | Note Added: 0016801 | |
| 2009-06-01 11:43 | rafaroda | Resolution | open => no change required |
| 2009-06-02 00:00 | anonymous | sf_bug_id | 0 => 2799683 |
| 2009-06-18 17:51 | networkb | Note Added: 0017430 | |
| 2009-06-22 19:10 | networkb | Assigned To | dalsasua => rafaroda |
| 2009-06-22 19:10 | networkb | Status | closed => new |
| 2009-06-22 19:10 | networkb | Resolution | no change required => open |
| 2009-06-22 19:10 | networkb | Note Added: 0017534 | |
| 2009-06-22 19:29 | dbaz | Note Added: 0017536 | |
| 2009-07-01 09:23 | psarobe | Status | new => feedback |
| 2009-07-03 17:17 | networkb | File Added: 9027.diff | |
| 2009-07-03 17:17 | networkb | Note Added: 0017898 | |
| 2009-07-06 08:53 | rafaroda | Assigned To | rafaroda => dalsasua |
| 2009-07-06 08:53 | rafaroda | Status | feedback => new |
| 2009-07-06 08:54 | rafaroda | Status | new => scheduled |
| 2009-07-06 08:54 | rafaroda | fix_in_branch | => pi |
| 2009-07-07 12:06 | vmromanos | Note Added: 0017961 | |
| 2009-07-07 12:06 | vmromanos | Assigned To | dalsasua => networkb |
| 2009-07-07 12:06 | vmromanos | Status | scheduled => feedback |
| 2009-07-08 10:54 | rafaroda | Note Added: 0018009 | |
| 2009-07-08 10:54 | rafaroda | Assigned To | networkb => rafaroda |
| 2009-07-08 10:54 | rafaroda | Status | feedback => new |
| 2009-07-08 10:54 | rafaroda | fix_in_branch | pi => |
| 2009-07-08 10:55 | rafaroda | File Added: WithoutFix.png | |
| 2009-07-08 10:55 | rafaroda | File Added: WithFix.png | |
| 2009-07-08 10:56 | rafaroda | Status | new => acknowledged |
| 2009-07-10 16:06 | hgbot | Checkin | |
| 2009-07-10 16:06 | hgbot | Note Added: 0018112 | |
| 2009-07-10 16:06 | hgbot | Status | acknowledged => resolved |
| 2009-07-10 16:06 | hgbot | Resolution | open => fixed |
| 2009-07-10 16:06 | hgbot | Fixed in SCM revision | => http://code.openbravo.com/erp/devel/pi/rev/672dad0c98b0e1d7e4adcbf7068cbaab5323f281 [^] |
| 2009-07-21 16:24 | psarobe | Fixed in Version | => 2.50MP3 |
| 2009-07-21 16:26 | psarobe | Tag Attached: 250MP3releasecandidate | |
| 2009-07-27 19:38 | psarobe | Status | resolved => closed |
| 2009-07-27 19:38 | psarobe | Note Added: 0018635 | |
|
Issue History |
| Copyright © 2000 - 2009 MantisBT Group |
 |