Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Hiding not-last column in the Grid causes IndexOutOfBoundsException on the client #6784

Closed
vaadin-bot opened this issue May 27, 2015 · 2 comments
Closed

Hiding not-last column in the Grid causes IndexOutOfBoundsException on the client #6784

vaadin-bot opened this issue May 27, 2015 · 2 comments
Labels
bug
Milestone
Vaadin 7.5.0.rc1

Comments

@vaadin-bot
Copy link
Collaborator

Originally by vilo

Try to run following code:

Grid grid = new Grid();
Column col = grid.addColumn("test1");
col.setHidable(true);
col.setHidden(true);
col = grid.addColumn("test2");
col.setHidable(true);
col.setHidden(false);

The first column is hidden and the second is not. It works, if only last columns are hidden.

On the client, it will fail with IndexOutOfBoundsException (stack trace from Firefox console):
Caused by: java.lang.IndexOutOfBoundsException: The given index(1) was outside of the current number of columns (0..0)
at Unknown.fillInStackTrace(sk.kios.maxp.widgetset.MpWidgetset-0.js)
at Unknown.Throwable_0(sk.kios.maxp.widgetset.MpWidgetset-0.js)
at Unknown.Exception_0(sk.kios.maxp.widgetset.MpWidgetset-0.js)
at Unknown.RuntimeException_0(sk.kios.maxp.widgetset.MpWidgetset-0.js)
at Unknown.IndexOutOfBoundsException_0(sk.kios.maxp.widgetset.MpWidgetset-0.js)
at Unknown.insertColumns(sk.kios.maxp.widgetset.MpWidgetset-0.js)
at Unknown.addColumnSkipSelectionColumnCheck(sk.kios.maxp.widgetset.MpWidgetset-0.js)
at Unknown.addColumn_0(sk.kios.maxp.widgetset.MpWidgetset-0.js)
at Unknown.addColumn(sk.kios.maxp.widgetset.MpWidgetset-0.js)
at Unknown.addColumnFromStateChangeEvent(sk.kios.maxp.widgetset.MpWidgetset-0.js)
at Unknown.onStateChanged_2(sk.kios.maxp.widgetset.MpWidgetset-0.js)
at Unknown.dispatch_78(sk.kios.maxp.widgetset.MpWidgetset-0.js)
at Unknown.dispatch_77(sk.kios.maxp.widgetset.MpWidgetset-0.js)
at Unknown.dispatch(sk.kios.maxp.widgetset.MpWidgetset-0.js)
at Unknown.dispatchEvent_3(sk.kios.maxp.widgetset.MpWidgetset-0.js)
at Unknown.doFire(sk.kios.maxp.widgetset.MpWidgetset-0.js)
at Unknown.fireEvent_1(sk.kios.maxp.widgetset.MpWidgetset-0.js)

Imported from https://dev.vaadin.com/ issue #18030
@vaadin-bot
Copy link
Collaborator Author

Originally by @Artur-

https://dev.vaadin.com/review/#/c/11126/

@vaadin-bot
Copy link
Collaborator Author

Originally by @samie

Scheduling fixes for 7.5.0

@vaadin-bot vaadin-bot added the bug label Dec 10, 2016
@vaadin-bot vaadin-bot added this to the Vaadin 7.5.0.rc1 milestone Dec 10, 2016
TatuLund added a commit that referenced this issue Sep 4, 2020
@TatuLund
Fix IndexOutOfBoundsException when adding columns in Grid
82682ae 
There is regression in Vaadin 7.7.16 and later, which is due patch  eafd446 that can lead to IndexOutOfBoundsException when there is hidden columns while adding new columns, which is similiar issue than earlier reported in #6784. Essentially the performance improvement patch overwrite some of the fix logic of 8453305 This fix bring overwriten escalator index compensation back.
TatuLund added a commit that referenced this issue Sep 4, 2020
@TatuLund
Fix IndexOutOfBoundsException when adding columns in Grid
ba0f8be 
There is regression in Vaadin 7.7.16 and later, which is due patch  eafd446 that can lead to IndexOutOfBoundsException when there is hidden columns while adding new columns, which is similiar issue than earlier reported in #6784. Essentially the performance improvement patch overwrite some of the fix logic of 8453305 This fix bring overwriten escalator index compensation back.
@TatuLund TatuLund mentioned this issue Sep 4, 2020
Merged
Ansku pushed a commit that referenced this issue Sep 4, 2020
@TatuLund
Fix IndexOutOfBoundsException when adding columns in Grid (#12095)
58bcc95 
There is regression in Vaadin 7.7.16 and later, which is due patch  eafd446 that can lead to IndexOutOfBoundsException when there is hidden columns while adding new columns, which is similiar issue than earlier reported in #6784. Essentially the performance improvement patch overwrite some of the fix logic of 8453305 This fix bring overwriten escalator index compensation back.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
Vaadin 7.5.0.rc1
Development

No branches or pull requests
1 participant
@vaadin-bot