xdev-software / vaadin-grid-exporter

Makes it possible to export Vaadin Grids to different formats
Apache License 2.0
23 stars 2 forks source link

Grid columns truncate data. #229

Closed mcfar3199 closed 2 months ago

mcfar3199 commented 2 months ago

Checklist

Affected version

3.1.0

Steps to reproduce the bug

  1. Create a grid with 8-10 columns and text that is long or multiple words.
  2. Try to export the grid in either pdf or .xlsx

Expected behavior

I would expect the text to wrap in the cells the same way it does in the 3.0.3 release.

Actual behavior

When the preview is shown the text is truncated in both the pdf and .xlsx formats. If you download the grid, the data is still missing.

Additional information

No response

AB-xdev commented 2 months ago

Can confirm the problem, looks like something was broken with Jasper 7

AB-xdev commented 2 months ago

So there is an error in the upstream dynamicreports where getTextFieldTextAdjust ALWAYS returns null: https://github.com/dynamicreports/dynamicreports/blob/3f80c6f7555a52761e751767783117fd5892a826/dynamicreports-core/src/main/java/net/sf/dynamicreports/report/defaults/Default.java#L1065

Before Jasper 7 this had no consequences as isTextFieldStretchWithOverflow hid this mistake, however this was removed in Jasper 7 and now the error occurs.

AB-xdev commented 2 months ago

Reported in original project: https://github.com/dynamicreports/dynamicreports/issues/180

AB-xdev commented 2 months ago

Should be fixed in v3.2.1 :)

mcfar3199 commented 2 months ago

Thank you for your prompt response. You have built a much needed and awesome plugin!MitchSent from my iPhoneOn Jul 17, 2024, at 4:58 AM, Alex B @.***> wrote: Should be fixed in v3.2.1 :)

—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>