Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0024878Openbravo ERPA. Platformpublic2013-10-03 12:542013-10-18 12:13
Reportergorkaion 
Assigned Toshankarb 
PrioritynormalSeveritymajorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOS20OS VersionGentoo
Product Version 
Target Version3.0MP29Fixed in Version3.0MP29 
Merge Request Status
Review Assigned ToAugustoMauch
OBNetwork customer
Web browser
ModulesCore
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0024878: Pick and Execute grid does not truncate multple lines texts
DescriptionWhen you show in a Pick and Execute grid a column with Text reference.

If there is a record with a text with multiple lines the grid's row is expanded in height to show all the lines. This behaviour is different from regular grids where the text is truncated to show only the first line with trailing dots. The row shouldn't be expanded, the whole text is shown in the Tooltip of the field.
Steps To ReproduceOn a Pick and Execute add a column with reference text.
Add a record that contains multiple lines in that record.
Open the Pick and Execute and check that the row is expanded.

For example:
In Return from Customer pick and execute add the Description column of the Order.
Open the Pick and Execute to check that the column is shown.
In one of the Sales Orders that are shown in the Pick and Execute edit the Description field with a text that includes several lines.
Check that in the Sales ORder window if the Description is shown in the grid only the first line is shown.
Go back to the pick and execute and check that the row of the modified Sales Order is expanded to fit the whole description which is wrong.
Proposed Solution
Additional Information
TagsNo tags attached.
Relationships
related to defect 00175313.0MP4 closed mtaal Description field with 2 rows looks nasty in grid view 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2013-10-03 12:54gorkaionNew Issue
2013-10-03 12:54gorkaionAssigned To => AugustoMauch
2013-10-03 12:54gorkaionModules => Core
2013-10-03 12:54gorkaionTriggers an Emergency Pack => No
2013-10-07 07:42shankarbAssigned ToAugustoMauch => shankarb
2013-10-15 10:08shankarbRelationship addedrelated to 0017531
2013-10-15 10:12shankarbReview Assigned To => AugustoMauch
2013-10-15 10:13hgbotCheckin
2013-10-15 10:13hgbotNote Added: 0061673
2013-10-15 10:13hgbotStatusnew => resolved
2013-10-15 10:13hgbotResolutionopen => fixed
2013-10-15 10:13hgbotFixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/dc34fa9da48db4ecc9f8649fa3718a4b04ca7fec [^]
2013-10-16 19:47hudsonbotCheckin
2013-10-16 19:47hudsonbotNote Added: 0061752
2013-10-18 12:13AugustoMauchNote Added: 0061799
2013-10-18 12:13AugustoMauchStatusresolved => closed
2013-10-18 12:13AugustoMauchFixed in Version => 3.0MP29

Notes
(0061673)
hgbot   
2013-10-15 10:13   
Repository: erp/devel/pi
Changeset: dc34fa9da48db4ecc9f8649fa3718a4b04ca7fec
Author: Shankar Balachandran <shankar.balachandran <at> openbravo.com>
Date: Tue Oct 15 13:39:11 2013 +0530
URL: http://code.openbravo.com/erp/devel/pi/rev/dc34fa9da48db4ecc9f8649fa3718a4b04ca7fec [^]

Fixes Issue 0024878: Pick and Execute grid does not truncate multple lines texts

The fix for this issue is similar to the fix for issue 17531[1].
There the methods formatDisplayValue and cellHoverHTML were applied to ob-view-grid.js.
Moved it to ob-grid.js for access to view grid and p&e grids.


[1] https://issues.openbravo.com/view.php?id=17531 [^]

---
M modules/org.openbravo.client.application/web/org.openbravo.client.application/js/grid/ob-grid.js
M modules/org.openbravo.client.application/web/org.openbravo.client.application/js/grid/ob-view-grid.js
---
(0061752)
hudsonbot   
2013-10-16 19: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/51315988909a [^]

Maturity status: Test
(0061799)
AugustoMauch   
2013-10-18 12:13   
Code reviewed and verified in pi@eb0f3ecbdae4