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

Removed the notifyDataSetChanged from setItems #62

Merged
merged 2 commits into from
Jan 20, 2017

Conversation

drakeet
Copy link
Owner

@drakeet drakeet commented Jan 20, 2017

NOTE: If you are updating from v2.4.0 to v2.4.1,
note that the setItems(items) method has removed the notifyDataSetChanged()
and you need to call it by yourself.

**NOTE**: If you are updating from v2.4.0 to v2.4.1,
note that the `setItems(items)` method removes the
`notifyDataSetChanged()`
and you need to call it yourself.
@drakeet
Copy link
Owner Author

drakeet commented Jan 20, 2017

/**
 * Update the items atomically and safely.
 * It is recommended to use this method to update the data.
 * <p>e.g. {@code adapter.setItems(new Items(changedItems));}</p>
 *
 * <p>Note: If you want to refresh the list views, you should
 * call {@link RecyclerView.Adapter#notifyDataSetChanged()} by yourself.</p>
 *
 * @param items The <b>new</b> items list.
 * @since v2.4.1
 */
public void setItems(@Nullable List<?> items) {
    this.items = items;
}

@drakeet drakeet merged commit a1a6132 into master Jan 20, 2017
@drakeet drakeet deleted the feature/remove-notify branch January 20, 2017 13:17
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.

1 participant