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

BUG FIX: Allow usage of Parcelable for IHeaders and IFlexible. #357

Merged
merged 1 commit into from
May 7, 2017
Merged

BUG FIX: Allow usage of Parcelable for IHeaders and IFlexible. #357

merged 1 commit into from
May 7, 2017

Conversation

long1eu
Copy link
Contributor

@long1eu long1eu commented May 7, 2017

If you pass the Items or Headers to an Intent as a Parcelable the headers
are recreated as different instances and are shown above each item and not
the whole section. The reason for this of the usage of reference equality
check. We need to use the equal() to fix this issue.

If you pass the Items or Headers to an Intent as a Parcelable the headers
are recreated as different instances and are shown above each item and not
the whole group. The reason for this of the usage of reference equality
check. We need to use the equal() to fix this issue.
@davideas davideas merged commit 61a58e0 into davideas:extensions May 7, 2017
davideas added a commit that referenced this pull request May 7, 2017
@long1eu
Copy link
Contributor Author

long1eu commented May 7, 2017

I forgot about that :))

@long1eu long1eu deleted the extensions branch May 7, 2017 17:02
davideas pushed a commit that referenced this pull request May 9, 2017
If you pass the Items or Headers to an Intent as a Parcelable the headers
are recreated as different instances and are shown above each item and not
the whole group. The reason for this of the usage of reference equality
check. We need to use the equal() to fix this issue.

(cherry picked from commit 61a58e0)
davideas added a commit that referenced this pull request May 9, 2017
(cherry picked from commit a5e949d)
davideas pushed a commit that referenced this pull request May 10, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants