logical operators templates.parse support

[akhoury]

if would be nice if you can just evaluate the expression in JS if truthy or not
<!-- IF (foo && bar) || (you && me) || okletmein -->

[psychobunny]

This isn't something that is supported in handlebars but I've waiting for
someone to request this so I can build it in, I already figured out how to
accomplish it ;)
On 2014-02-19 11:50 PM, "Aziz Khoury" notifications@github.com wrote:

if would be nice if you can just evaluate the expression in JS it truthy
or not

Reply to this email directly or view it on GitHubhttps://github.com//issues/1065
.

[akhoury]

Awesome thanks!
you're right, it's not supported in handlebars, but if can be done using a helper.

[psychobunny]

Yeah lets see :) it can be done with some other engines (ironically
supported in Smarty). More complex conditionals like array.length > x is
something I want to try too. I refuse to eval() if I can help it but I have
an idea. Have just been waiting until it's actually needed so since you
requested it I can give it a shot
On 2014-02-20 12:59 AM, "Aziz Khoury" notifications@github.com wrote:

Awesome thanks!
you're right, it's not supported in handlebars, but if can be done using a
helper.

Reply to this email directly or view it on GitHubhttps://github.com//issues/1065#issuecomment-35591702
.

[psychobunny]

Although maybe integrating a helper system might be easier overall... hmm...

(Or switch to handlebars /trollshield activate)
On 2014-02-20 1:11 AM, "Andrew Rodrigues" rodrigues.andrew@gmail.com
wrote:

Yeah lets see :) it can be done with some other engines (ironically
supported in Smarty). More complex conditionals like array.length > x is
something I want to try too. I refuse to eval() if I can help it but I have
an idea. Have just been waiting until it's actually needed so since you
requested it I can give it a shot
On 2014-02-20 12:59 AM, "Aziz Khoury" notifications@github.com wrote:

Awesome thanks!
you're right, it's not supported in handlebars, but if can be done using
a helper.

Reply to this email directly or view it on GitHubhttps://github.com//issues/1065#issuecomment-35591702
.

[psychobunny]

For now its okay to nest your IF statements, I'm assigning as low priority because we haven't seen a real dire need for this in core just yet. ETA post 0.4x

[akhoury]

haha, why do you refuse to use eval?
ok how about something without literally using eval. I did something similar here

var IFEVAL = function (expression, parseData) {
   var fn;
   try {
     // create a new function from code
     // pass the parseData so that could be the context of the function
     // app, meta, foo, bar these are just example arguments 
     // that you can pass to every functions and use inside expressions if needed, so your expression could be something like:
    //         topic.title && typeof meta.config.something == \'number\' && app.yo
   // yes you would need to escape quotes, well, it depends.. if single or double, but you get the gist
     fn = Function.apply(parseData, ['app, meta, foo, bar', 'return ' + expression + ' ; ' ] );
   } catch (e) {
     fn = function(){return false;}; 
   }
   if ( fn(app, meta, foo, bar) ) {
      /// .... print IF block, or revaluate
   } else {
      // .... print ELSE block if there is one
   }
}

i dont know something like that, you could also just have an EVAL helper that will just output the result of the expression, no?

[psychobunny]

Interesting 👻 I'll have a think about that, good idea.

haha, why do you refuse to use eval?

trying to reduce the amount of tpd* I get

*trolls per day

[akhoury]

here's a Handlebars gist, if you ever end up switching to it :P

https://gist.github.com/akhoury/9118682

[julianlam]

@psychobunny Did you still want this in for 0.4.0?

[psychobunny]

Yup. Helpers are in
On 2014-03-19 10:31 AM, "Julian Lam" notifications@github.com wrote:

@psychobunny https://github.com/psychobunny Did you still want this in
for 0.4.0?

Reply to this email directly or view it on GitHubhttps://github.com//issues/1065#issuecomment-38056722
.

[julianlam]

Oh, we're going to use helpers instead of logical operators? I think that's
better anyway 👍

[psychobunny]

k just merged it in, ex:

templates.registerHelper('somefn', function(data) {
    return (data.propOne && data.propTwo) ? "Something" : "another";
});

or

templates.registerHelper('somefn', function(array, iterator, total) {
    return array[iterator].someProperty || array[iterator].another ? "something" : "something else";
});

[cnvo]

Nice work @psychobunny 👍

On Fri, Mar 28, 2014 at 4:23 PM, psychobunny notifications@github.comwrote:

Closed #1065 #1065.

—
Reply to this email directly or view it on GitHubhttps://github.com//issues/1065
.