Avoid number casting on number values that contain non-digit characters

[aberonni]

Fixes #1000 by adding an additional check: /^\d+$/.test(value)

I also took the liberty of pulling out the function and (hopefully) increasing readability

I haven't had a chance to test this with real-world data, but I'm hoping there might be integration / unit tests verifying that the updated code works. If not, happy to do some manual tested if needed.

[Helveg]

Thanks for the PR! Could you run npm run format so that the tests can run?

There aren't any tests specifically for handling different input values in the filter. If it's not too much to ask you could try adding some of them! You can add a few cases in paginate.spec.ts, here's what we should cover:

• Send a date input and check that results are correctly filtered.
• Send a number input and check that results are correctly filtered.
• Send a number-like input (which shouldn't be treated as a number)

It should be relatively simple to run the spec files with jest, but let me know if you need any help :)

[aberonni]

@Helveg I got as far as adding the date input test, but struggled a bit with how/where to add the other two, given that there isn't a field (or I couldn't see one) where I could easily add a dummy string for scenario 3.

[Helveg]

thanks! I'll take a look to complete the test cases when I have time 👍