Skip to content

Conversation

@lalooola
Copy link

@lalooola lalooola commented Feb 1, 2016

Added the capability to be able to search a field where the value is an array. Matcher will run for each item in the array and return true with a partial match.

* Checks if pattern matches any of the strings in an array
*
* @param {string} str
* @param {array} strArray
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please declare explicitly that strArray has to be an array of strings: string[] or (preferably) Array.<string>.
Otherwise the function does not do what it claims to do

@jrencz
Copy link
Contributor

jrencz commented Feb 8, 2016

@lalooola thank you for contributing!

Please apply the changes I suggested in comments, squash them into the original commit and this one can be merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants