From 418c174d56a5e53f174cd531719b291522470b23 Mon Sep 17 00:00:00 2001
From: Baris Usakli <barisusakli@gmail.com>
Date: Thu, 23 Jan 2020 12:48:21 -0500
Subject: [PATCH] fix: dont return flag data to client

---
 src/socket.io/flags.js    | 2 +-
 test/controllers-admin.js | 4 ++--
 test/flags.js             | 6 +-----
 3 files changed, 4 insertions(+), 8 deletions(-)

diff --git a/src/socket.io/flags.js b/src/socket.io/flags.js
index 90115e470e..e2c9a56215 100644
--- a/src/socket.io/flags.js
+++ b/src/socket.io/flags.js
@@ -21,7 +21,7 @@ SocketFlags.create = async function (socket, data) {
 
 	const flagObj = await flags.create(data.type, data.id, socket.uid, data.reason);
 	await flags.notify(flagObj, socket.uid);
-	return flagObj;
+	return flagObj.flagId;
 };
 
 SocketFlags.update = async function (socket, data) {
diff --git a/test/controllers-admin.js b/test/controllers-admin.js
index 476bd65ad4..371685866f 100644
--- a/test/controllers-admin.js
+++ b/test/controllers-admin.js
@@ -684,10 +684,10 @@ describe('Admin Controllers', function () {
 			var socketFlags = require('../src/socket.io/flags');
 			var oldValue = meta.config['min:rep:flag'];
 			meta.config['min:rep:flag'] = 0;
-			socketFlags.create({ uid: regularUid }, { id: pid, type: 'post', reason: 'spam' }, function (err, data) {
+			socketFlags.create({ uid: regularUid }, { id: pid, type: 'post', reason: 'spam' }, function (err, flagId) {
 				meta.config['min:rep:flag'] = oldValue;
 				assert.ifError(err);
-				request(nconf.get('url') + '/api/flags/' + data.flagId, { jar: moderatorJar, json: true }, function (err, res, body) {
+				request(nconf.get('url') + '/api/flags/' + flagId, { jar: moderatorJar, json: true }, function (err, res, body) {
 					assert.ifError(err);
 					assert(body);
 					assert.equal(body.reporter.username, 'regular');
diff --git a/test/flags.js b/test/flags.js
index 4e7af09905..95ac38a3fa 100644
--- a/test/flags.js
+++ b/test/flags.js
@@ -596,9 +596,7 @@ describe('Flags', function () {
 
 	describe('(websockets)', function () {
 		var SocketFlags = require('../src/socket.io/flags.js');
-		var tid;
 		var pid;
-		var flag;
 
 		before(function (done) {
 			Topics.post({
@@ -607,7 +605,6 @@ describe('Flags', function () {
 				title: 'Another topic',
 				content: 'This is flaggable content',
 			}, function (err, topic) {
-				tid = topic.postData.tid;
 				pid = topic.postData.pid;
 
 				done(err);
@@ -620,8 +617,7 @@ describe('Flags', function () {
 					type: 'post',
 					id: pid,
 					reason: 'foobar',
-				}, function (err, flagObj) {
-					flag = flagObj;
+				}, function (err) {
 					assert.ifError(err);
 
 					Flags.exists('post', pid, 1, function (err, exists) {