Hmm… I don't think so. Might be better to just put a 'db_error' there.


-- 
evan moore | MIT ’14 | Computer Science

On February 12, 2014 at 2:12:49 AM, Christopher Van (notifications@github.com) wrote:

In views/user/acl.js:

> +            user.getUserFromID(client, userID, function(err, resp) {
> +                if (err || !resp) {
> +                    res.json(500, {error: 'db_error'});
> +                    done();
> +                    return;
> +                }
> +
> +                user.updateUser(client, resp, {
> +                    permissions: {
> +                        developer: isDev,
> +                        reviewer: isRev,
> +                        admin: isAdmin
> +                    }
> +                }, function(err, newData) {
> +                    if (err) {
> +                        res.json(500, {error: err});
what would the err be? is it something we should expose?

—
Reply to this email directly or view it on GitHub.