(#276) Fixed dollar sign group replace in vue preprocessor

[adamDilger]

replace#specifying_a_string_as_the_replacement

When vue code contained groupings such as $' or $$, the call to "".replace() would treat these as special replacements producing invalid code.

Switching the replacement arg to a function returning the same string skips this logic.

The tests pass, and as this is only removing special functionality I would assume this is a safe change

EDIT: I've also added another commit to this PR which fixes having both a script and a script/setup tags in a SFC

[seanogdev]

I've applied this as a patch in our repo and this works perfectly, thanks @adamDilger 🙌

[seanogdev]

@ayusharma @byara Would we be able to get a release for this whenever you guys had a spare moment? 🙏

[adamDilger]

I've added another bugfix onto this branch/PR, for #218 , as there would have been conflicting changes if these fixes were separated out.

If this is not valid I'm happy to revert and re-PR once the first fix has been merged

[adamkdean]

I found this issue even when $ exists in a string

[byara]

Hey @adamDilger I changes the base branch to v5. I hope it is OK. Can you please check wether everything is working as expected? If so, I'll merge this into v5 branch so we can release it later this week.

[adamDilger]

@byara I've rebased on v5, re-ran tests, and tested on a dummy project, everything looks good 👍

Thanks!