Skip to content

Add selectItems and associated tests #437

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

Closed
wants to merge 6 commits into from

Conversation

Westbrook
Copy link
Contributor

Fixes #436

As mentioned in the issue above, this may actually be an array-selector issue more than an iron-list issue, but as this specific code seems better managed in iron-list than in user space code as initially noted with the potential solution in #124, I figured this was a good step in getting this conversation more definitively completed.

This code adds a selectItems method that selects the entirety of this.items when no arguments are found, or casts the content of arguments into an Array such that they can be selected in batch regardless of them being an array, a single item, or a list of items.

Tests for "select items: all", "select items: array" (sending the method an array), "select items: arguments" (sending the batch as an arguments list), and "select items: single" to maintain this functionality moving forward.

@Westbrook Westbrook mentioned this pull request Jun 23, 2017
@Westbrook Westbrook changed the base branch from master to 1.x June 23, 2017 00:59
@keanulee
Copy link
Contributor

Closing in favor of #457

@keanulee keanulee closed this Jun 23, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants