input pattern requires double-escaping

[shanewholloway]

Trying to figure out why my input pattern was not working, I asked html2jade to translate the following input pattern:

> html2jade
<input pattern='\S+(\.\S+)+'>

html
body
    input(pattern='\S+(\.\S+)+')

But it does not work as advertised:

> jade
input(pattern='\S+(\.\S+)+')

<input pattern="S+(.S+)+"/>

Single escaping does not work:

> jade
input(pattern='\\S+(\\.\\S+)+')

<input pattern="S+(.S+)+"/>

Only after double-escaping the slashes do I get the desired escaping for my input pattern:

> jade
input(pattern='\\\\S+(\\\\.\\\\S+)+')

<input pattern="\S+(\.\S+)+"/>

[tj]

attribute strings are regular js strings, so single escapes should work, something funky going on otherwise

[shanewholloway]

My sentiments exactly — I was very surprised that single escaping didn't work. It probably means that the attribute string is being parsed twice on it's way to the output.

On Dec 7, 2012, at 7:09 AM, TJ Holowaychuk notifications@github.com wrote:

attribute strings are regular js strings, so single escapes should work, something funky going on otherwise

—
Reply to this email directly or view it on GitHub.

[mashihua]

HI, visionmedia. After I read the source https://github.com/visionmedia/jade/blob/master/lib/compiler.js#L571 .
Found give val.buf value to 'pattern':('\S+(.\S+)+'), the buf become { pattern: 'S+(.S+)+' } after eval invoked。

[ForbesLindesay]

The problem is that we're 'undoing' the escaping when we eval. For example we convert '\n' to an actual new line I assume.

[whaaaley]

I'm still experiencing this bug.

[ForbesLindesay]

You should have to do a single layer of escaping (because the value is a JavaScript expression):

input(pattern='\\S+(\\.\\S+)+')

should produce:

<input pattern="\S+(\.\S+)+"/>

Which is working for me. If that doesn't work for you, please open a new issue. I'm locking this issue as this bug was definitely fixed, any new bugs should be opened as a fresh issue.