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.

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

7.7.16: Grid scrolling with open editor in unbuffered mode throws IllegalStateException #11463

Closed
anasmi opened this issue Feb 21, 2019 · 1 comment
Labels

Comments

@anasmi
Copy link
Contributor

anasmi commented Feb 21, 2019

Version: 7.7.16
Code to reproduce:

------
 Collection<Person> people = personList();
        BeanItemContainer<Person> container =
                new BeanItemContainer<Person>(Person.class, people);
        final Grid grid = new Grid(container)
        grid.setSizeFull();
        grid.setEditorEnabled(true);
        grid.setEditorBuffered(false);
        setContent(grid);
------
 private List<Person>  personList(){
        List<Person> people=new ArrayList<>();
        for(int i=0;i<100;i++){
          people.add(new Person("Name" + i, 24+i));
        }
        return people;
    }

Steps to reproduce:

  • click on any cell and scroll, exception is thrown

Error: Caused by: java.lang.IllegalStateException: Row with logical index 72 is currently not available in the DOM

Exception is thrown at this point (Also, this is most likely the pr, which introduced the regression): 062caa5#diff-6cd1edd3140e7674e92302b06bbc34d9R2145

@TatuLund TatuLund added the v7 label Feb 21, 2019
TatuLund added a commit that referenced this issue Feb 25, 2019

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
IllegalStateException may occur if user has scrolled Grid so that Escalator has updated, and row under Editor is no longer there
Fixes #11463
@anasmi anasmi added the BFP label Feb 27, 2019
OlliTietavainenVaadin pushed a commit that referenced this issue Mar 11, 2019
…#11467)

IllegalStateException may occur if user has scrolled Grid so that Escalator has updated, and row under Editor is no longer there
Fixes #11463
@TatuLund
Copy link
Contributor

Closed via #11467

Ansku added a commit that referenced this issue May 15, 2020

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
…#12002) (#12009)

IllegalStateException may occur if user has scrolled Grid (compatibility library version) so that Escalator has updated, and row under Editor is no longer there

Issue #11463
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants