From ee17367cc1738254ddb00ff39cd4c49e0afe3e3c Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Bar=C4=B1=C5=9F=20Soner=20U=C5=9Fakl=C4=B1?= Date: Thu, 28 May 2015 15:27:55 -0400 Subject: [PATCH] removed unused requires, firehook on sucess --- src/groups.js | 14 +++++++------- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/src/groups.js b/src/groups.js index f876591399..cdc99950bc 100644 --- a/src/groups.js +++ b/src/groups.js @@ -2,19 +2,15 @@ var async = require('async'), winston = require('winston'), - path = require('path'), nconf = require('nconf'), - fs = require('fs'), validator = require('validator'), user = require('./user'), - meta = require('./meta'), db = require('./database'), plugins = require('./plugins'), posts = require('./posts'), privileges = require('./privileges'), - utils = require('../public/src/utils'), - util = require('util'); + utils = require('../public/src/utils'); (function(Groups) { @@ -235,8 +231,12 @@ var async = require('async'), }; Groups.setGroupField = function(groupName, field, value, callback) { - plugins.fireHook('action:group.set', {field: field, value: value, type: 'set'}); - db.setObjectField('group:' + groupName, field, value, callback); + db.setObjectField('group:' + groupName, field, value, function(err) { + if (err) { + return callback(err); + } + plugins.fireHook('action:group.set', {field: field, value: value, type: 'set'}); + }); }; Groups.isPrivate = function(groupName, callback) {