Dynamic Helper about meteor-jade HOT 3 CLOSED

@dgreensp This is a bug with the $specials field. The compiler seems to expect this field to be an array (in the AST) and it doesn't work with a non-array value. This behavior is inconsistent with the other constructors.

// First case: $specials value wrapped in an array
Spacebars.codeGen(HTML.P({
  "$specials": [
    HTMLTools.Special(Spacebars.TemplateTag.parse('{{xxx}}'))
  ]
}));

// This works as expected and returns
'(function() {
  var self = this;
  return HTML.P({
    $dynamic: [ function() {
      return Spacebars.mustache(self.lookup("xxx"));
    } ]
  });
})'


// Second case: $specials value NOT wrapped in an array
Spacebars.codeGen(HTML.P({
  "$specials": HTMLTools.Special(Spacebars.TemplateTag.parse('{{xxx}}'))
}));

// returns
'(function() {
  var self = this;
  return HTML.P();
})'

// instead of
'(function() {
  var self = this;
  return HTML.P({
    $dynamic: function() {
      return Spacebars.mustache(self.lookup("xxx"));
    }
  });
})'

BTW I think that the array optimization (ie don't use an array if there is only one value and merge consecutive strings) should be done by the compiler.

from meteor-jade.

Hi! I still have an issue. I try to pass attributes to div: .post($dyn=attributes), where attributes are {class: "post animate", style: "top: 0px"}. But I get an error:

Error: Illegal HTML attribute name: 0

The error is thrown by extendAttrs function, which gets string "[object Object]", as src argument instead of an actual object. This seems to be done by Spacebars.mustache function.

from meteor-jade.

This is be fixed in the blaze-refactor branch.

from meteor-jade.