fixing bug where new installs exposed the administrators group.

11 years ago · 252af4e8cc
parent feaea6dcea
commit 252af4e8cc
1 changed files with 1 additions and 9 deletions
--- a/src/install.js
+++ b/src/install.js
@ -410,15 +410,7 @@ var async = require('async'),
 							return callback(new Error('invalid-values'));
 						}

-						Groups.getGidFromName('administrators', function (err, gid) {
-							if (gid) {
-								Groups.join(gid, uid, callback);
-							} else {
-								Groups.create('administrators', 'Forum Administrators', function (err, groupObj) {
-									Groups.join(groupObj.gid, uid, callback);
-								});
-							}
-						});
+						Groups.joinByGroupName('administrators', uid, callback);
 					});
 				},
 				retryPassword = function (originalResults) {