'==' works locally but '=' required for comparison on server

Question

I couldnt figure out why my Meteor template helper wasnt working on my Ubuntu server, so I just hacked together a couple variations and this one ended up working...locally I use if(user[0].trusted == true) but for some reason that conditional wasnt getting triggered on the server.

Handlebars.registerHelper('isTrusted', function(user_id){
        var user = Meteor.users.find({_id: user_id}).fetch();

        console.log(user, 'user');
        console.log(user[0].trusted);

        if(user[0].trusted = true){
            console.log(user[0].trusted, 'user trusted field');
            return true;
        } else {
            false;
        }

});

Why?

and all you are doing is assigning true to the property which will evaluate to true. That if is not checking that trusted is true. — epascarello, May 20 '15 at 01:33
= will not work for comparisons either... == compares and type casts, === does straight comparison as if same type — Asheliahut, May 20 '15 at 01:33
`if(user[0].trusted = true){ }` is equivalent to `user[0].trusted = true; if (true) { }`. — Felix Kling, May 20 '15 at 01:36
@humble.rumble that's exactly what I said... In different words — Asheliahut, May 20 '15 at 02:34

score 3 · Accepted Answer · answered May 20 '15 at 01:40

Your hack is wrong. It will always enter in the if branch, mostly because you are not comparing but assigning:

if(user[0].trusted = true)

Here you are assigning to user[0].trusted the true value. Because it's inside an if, javascript is checking if the assignment is correct. Because it assigned correctly, then it enters in the branch (always).

Send to console the value of user[0].trusted. Maybe it's a number, or has another value. Anyway, your code has a bug, it's not a thing with the server.

in the server db I cast the value of `trusted` as a string as opposed to a boolean — redress, May 20 '15 at 01:46

score 1 · Answer 2 · answered May 20 '15 at 05:52

1

This may be a nice way to do it too:

Handlebars.registerHelper('isTrusted', function(user_id){
    return !!Meteor.users.findOne({_id: user_id, trusted: true});
});

Or if you cast it to a string you can use 'true' instead of true

answered May 20 '15 at 05:52

Tarang

72,789
35
206
268

using ! ! returns a boolean as opposed to an object ? – redress May 20 '15 at 05:56
answer found here http://stackoverflow.com/questions/784929/what-is-the-not-not-operator-in-javascript – redress May 20 '15 at 06:04

'==' works locally but '=' required for comparison on server

2 Answers2