Tags with whitespace before the value are not allowed

[sethladd]

Not sure if this is by design.

This works:

{{ foo.bar }}

(notice the whitespace)

This does not work:

{{ #foo.bar }}

But this does work:

{{#foo.bar}}

[xxgreg]

I can't reproduce this. Could you send me some code. The tests below all pass.

test('Whitespace in section tags', () {
  expect(parse('{{#foo.bar}}oi{{/foo.bar}}').renderString({'foo': {'bar': true}}), equals('oi'));
  expect(parse('{{# foo.bar}}oi{{/foo.bar}}').renderString({'foo': {'bar': true}}), equals('oi'));
  expect(parse('{{#foo.bar }}oi{{/foo.bar}}').renderString({'foo': {'bar': true}}), equals('oi'));
  expect(parse('{{# foo.bar }}oi{{/foo.bar}}').renderString({'foo': {'bar': true}}), equals('oi'));
  expect(parse('{{#foo.bar}}oi{{/ foo.bar}}').renderString({'foo': {'bar': true}}), equals('oi'));
  expect(parse('{{#foo.bar}}oi{{/foo.bar }}').renderString({'foo': {'bar': true}}), equals('oi'));
  expect(parse('{{#foo.bar}}oi{{/ foo.bar }}').renderString({'foo': {'bar': true}}), equals('oi'));
  expect(parse('{{# foo.bar }}oi{{/ foo.bar }}').renderString({'foo': {'bar': true}}), equals('oi'));

});

test('Whitespace in variable tags', () {
  expect(parse('{{foo.bar}}').renderString({'foo': {'bar': true}}), equals('true'));
  expect(parse('{{ foo.bar}}').renderString({'foo': {'bar': true}}), equals('true'));
  expect(parse('{{foo.bar }}').renderString({'foo': {'bar': true}}), equals('true'));
  expect(parse('{{ foo.bar }}').renderString({'foo': {'bar': true}}), equals('true'));
});

[xxgreg]

I'm guessing you meant this:

{{ #libraries }} ... {{ /libraries }}

By my reading of the mustache spec these are two variable tags which looks up a variable named '#libraries' and '/libraries'.

For example:

 render('{{ #libraries }} + {{ /libraries }} = 3', {'#libraries': 1, '/libraries': 2});

Should result in "1 + 2 = 3".

But there are discussions about how to specify this:

mustache/spec#67

If you want to have portable template code it sounds like the best solution is to do it this way:

{{# libraries }} ... {{/ libraries }}

[xxgreg]

I checked pystache and node.js mustache they both treat the following as a section. So I guess instead of following the spec, I'll do as the other implementations do.

{{ #libraries }} ... {{ /libraries }}

I've implemented this, including adding a bunch of tests for strange corner cases.

[sethladd]

Ack, sorry, I mis-typed my original description. I've updated the description. You indeed did figure out what I was trying to say :)

Thanks for the fix!