Wrapper function for functions with similar functionality

[5saviahv]

Wrapper function "_matcher" for functions with similar functionality.

• It changes .parents function sorting (test shows strange sorting order)
• it changes .siblings function matched elements (it matches same elements what jQuery [first example on their website] )

other functions work like before, simply repeating parts of the functions are kept in "wrapper" function

[fb55]

Hi @5saviahv, this is an awesome idea, thanks for opening this PR! I just started a new job, so might need some time to review this properly – sorry about that. Also no need to rebase the PR (that is great, but GitHub can deal with it).

[5saviahv]

Thanks for telling, take your time - in jQuery there are many such functions, it reduces repeating same constructs over and over.

[5saviahv]

• added fixture eleven so order of elements will be visible also it is separated by LF so tests have to deal with text nodes as well.
• added some comments to function

[fb55]

This is a great idea! Right now, it will lead to a slowdown due to uniqueSort being used quite a bit, when some of the results were taken without a change before. I've left some comments, let me know if they are useful.

[5saviahv]

Sry, while loop in siblings function kind of broke thing (first element was discarded) and I wasnt able revert it so I overwrite it.

[5saviahv]

I added Cheerio-Select #7 and it is related to this. We can later come back and change how uniqueSort works.

[fb55]

I added cheeriojs/cheerio-select#7 and it is related to this. We can later come back and change how uniqueSort works.

I have updated cheerio-select's filter method to no longer change the order of the input. That hopefully (?) removes the need for sorting in here entirely.