Skip to content

fix(list-view-android): app crashes on ListView item template change - #6634

Merged
ADjenkov merged 6 commits into
masterfrom
djenkov/list-view-templates
Jan 8, 2019
Merged

ADjenkov merged 6 commits into
masterfrom
djenkov/list-view-templates

Conversation

@ADjenkov

Copy link
Copy Markdown
Contributor

_itemTemplateSelector is called multiple times for Android because of the getItemViewType method which sets the bindingContext of itemTemplateSelector to the tapped dataItem on itemTap. After that some dataItem property is changed in itemTap user method and the bindingContext will point to that same dataItem reference without reevaluating the expression.

Added tests for dynamically changing ListView item template with itemTemplateSelector expression.

Fix #6284

@ADjenkov ADjenkov added the bug label Nov 28, 2018
@ADjenkov ADjenkov self-assigned this Nov 28, 2018
@ghost ghost added the in progress label Nov 28, 2018
@ADjenkov
ADjenkov force-pushed the djenkov/list-view-templates branch from 535d277 to 86629dc Compare November 28, 2018 13:58
@ghost ghost assigned dtopuzov Nov 29, 2018
@ADjenkov
ADjenkov force-pushed the djenkov/list-view-templates branch from d122373 to ee46d3d Compare November 29, 2018 14:18
@ghost ghost assigned vakrilov Jan 7, 2019
@ADjenkov
ADjenkov merged commit 2085d1e into master Jan 8, 2019
@ghost ghost removed bug in progress labels Jan 8, 2019
@ADjenkov
ADjenkov deleted the djenkov/list-view-templates branch January 8, 2019 11:48
SvetoslavTsenov pushed a commit that referenced this pull request Jan 13, 2019
…6634)

* fix(list-view): app crashes on first ListView item template change

* tests: add tests for changing ListView item template with expression
@lock

lock Bot commented Jan 8, 2020

Copy link
Copy Markdown

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock Bot locked and limited conversation to collaborators Jan 8, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

ListView causing app to crash

4 participants