jmix-framework / jmix-ui-samples

Jmix v.1 Classic UI Samples. Online at https://demo.jmix.io/sampler
https://www.jmix.io
Apache License 2.0
5 stars 5 forks source link

PivotTable samples enhancement #47

Open konyashkina opened 3 years ago

konyashkina commented 3 years ago

I. There is a mistape in a description:

Open "PivotTable > PivotTable with explicit renderer and aggregation" http://cuba-kub-01.haulmont.com/sampler/#main/sample?id=pivottable-explicit-renderer-aggregation Open the link and change Locale to Russian. Expeted result: "представляет собой" Actual result:"представляет собор"

II. PivotTable data export fails with NullPointerExceptions

Please check some cases below.

Case 1. PivotTable > PivotTable Data Export http://cuba-kub-01.haulmont.com/sampler/#main/sample?id=pivottable-data-export Open the link and press export button on the top. Actual result: NullPointerException

Case 2. PivotTable > Export data to PivotTable using ShowPivotAction http://cuba-kub-01.haulmont.com/sampler/#main/sample?id=pivottable-show-pivot-action 1, Open the link

  1. Click "ShowPivotAction with custom settings" on the top. (Pivot table appears).
  2. Click ' Excel' button on the top of the page. Actual result: NullPointerException

Case 3. PivotTable > Export data to PivotTable using ShowPivotAction http://cuba-kub-01.haulmont.com/sampler/#main/sample?id=pivottable-show-pivot-action 1, Open the link

  1. Click "Show pivot table" (Pivot table appears).
  2. Select "TSV export" in the table.
  3. Click ' Excel' button on the top of the page. Actual result: IllegalStateException: 'TSV_EXPORT' renderer is not supported for data export

Case 4 Steps 1-2 from the previouse case. Select "Line Chart" in the table. Click ' Excel' button on the top of the page. Actual result: IllegalStateException: 'LINE_CHART' renderer is not supported for data export

Flaurite commented 3 years ago

Case 1 - fixed in dev Case 2 - fixed in dev Case 3-4 - will be fixed after updating Jmix version: https://github.com/Haulmont/jmix-sampler/issues/57