fix #187 #70 options.filter bug

[eos3tion]

#187 #70

Files only filter each file while ever returns true, if the first file on the folder or the folder name dont pass the filter, the copy stops.

[jprichardson]

#216 (comment)

Would you be able to add a test? Thank you :)

[eos3tion]

var filter=/.*[.]json/ig;
var samples=["c:/11/22/33/aaa.json","c:/11/22/33/b.json","c:/11/22/33/abcdefg.json"];
samples.forEach( (item, index, array)=>console.log(filter.test(item)));

true
false
true

set filter's lastIndex to 0:

var filter=/.*[.]json/ig;
var samples=["c:/11/22/33/aaa.json","c:/11/22/33/b.json","c:/11/22/33/abcdefg.json"];
samples.forEach( (item, index, array)=>{
     filter.lastIndex=0;
     console.log(filter.test(item))
});

true
true
true

[RyanZim]

@jprichardson regex filters are deprecated and aren't even tested anymore. Should we merge this without tests or just close it?

[jprichardson]

@jprichardson regex filters are deprecated and aren't even tested anymore. Should we merge this without tests or just close it?

Indeed, thanks for checking.