Redefine permission logic and Add tests for anonymous field and permissions

src/api/posts.js

@@ -40,6 +40,10 @@ postsAPI.get = async function (caller, data) {
 		post.content = '[[topic:post-is-deleted]]';
 	}
 
+	// Reference to ChatGPT
+	// Add the anonymous field to the response
+	post.anonymous = post.isAnon || false;
+
 	return post;
 };
 
@@ -62,6 +66,11 @@ postsAPI.getSummary = async (caller, { pid }) => {
 
 	const postsData = await posts.getPostSummaryByPids([pid], caller.uid, { stripTags: false });
 	posts.modifyPostByPrivilege(postsData[0], topicPrivileges);
+
+	// Reference to ChatGPT
+	// Add the anonymous field to the post summary
+	postsData[0].anonymous = postsData[0].isAnon || false;
+
 	return postsData[0];
 };
 

src/controllers/admin/privileges.js

@@ -36,6 +36,7 @@ privilegesController.get = async function (req, res) {
 			}
 		}
 	});
+
 	if (!selectedCategory) {
 		selectedCategory = await categories.getCategoryFields(cid, ['cid', 'name', 'icon', 'bgColor', 'color']);
 	}

src/controllers/search.js

@@ -78,6 +78,14 @@ searchController.search = async function (req, res, next) {
 		recordSearch(data),
 	]);
 
+	// Reference to ChatGPT
+	// Add anonymous field to each post
+	searchData.posts = searchData.posts.map((post) => {
+		post.anonymous = post.isAnon ? 'Anonymous User' : '';
+		delete post.isAnon;
+		return post;
+	});
+
 	searchData.pagination = pagination.create(page, searchData.pageCount, req.query);
 	searchData.multiplePages = searchData.pageCount > 1;
 	searchData.search_query = validator.escape(String(req.query.term || ''));

src/middleware/user.js

@@ -36,6 +36,16 @@ const passportAuthenticateAsync = function (req, res) {
 	});
 };
 
+// permisssions for admin to see username on anonymous posts
+// const showUsername = async (userId, currentUser) => {
+// const isAdmin = await user.isAdministrator(currentUser.uid);
+
+// if (isAnonymous && !isAdmin) {
+// return 'Anonymous User';
+// }
+
+// return user.getDisplayName(userId);
+// };
 
 module.exports = function (middleware) {
 	async function authenticate(req, res) {

src/posts/create.js

@@ -19,7 +19,12 @@ module.exports = function (Posts) {
 		const content = data.content.toString();
 		const timestamp = data.timestamp || Date.now();
 		const isMain = data.isMain || false;
-		const isAnon = true;
+		// const anonymous = false; // hard code anonymous to become false
+		const anonymous = true; // hard code anonymous to become true
+		// attempted to get id from the tpl but we don't know how to do it
+		// const anonymous = data.getElementById('anonymousInput').value === 'true';
+		// log anonymous field to see which variable it is
+		// console.log('get anon value:', anonymous);
 
 		if (!uid && parseInt(uid, 10) !== 0) {
 			throw new Error('[[error:invalid-uid]]');
@@ -36,7 +41,7 @@ module.exports = function (Posts) {
 			tid: tid,
 			content: content,
 			timestamp: timestamp,
-			anonymous: isAnon,
+			anonymous: anonymous, // set anonymous datafield to be anonymous value
 		};
 
 		if (data.toPid) {

src/posts/summary.js

@@ -20,7 +20,7 @@ module.exports = function (Posts) {
 		options.parse = options.hasOwnProperty('parse') ? options.parse : true;
 		options.extraFields = options.hasOwnProperty('extraFields') ? options.extraFields : [];
 
-		const fields = ['pid', 'tid', 'content', 'uid', 'timestamp', 'deleted', 'upvotes', 'downvotes', 'replies', 'handle'].concat(options.extraFields);
+		const fields = ['pid', 'tid', 'content', 'uid', 'timestamp', 'anonymous', 'deleted', 'upvotes', 'downvotes', 'replies', 'handle'].concat(options.extraFields);
 
 		let posts = await Posts.getPostsFields(pids, fields);
 		posts = posts.filter(Boolean);
@@ -45,14 +45,21 @@ module.exports = function (Posts) {
 				post.uid = 0;
 			}
 			post.user = uidToUser[post.uid];
+			// Check if the post is anonymous
+			// Reference to ChatGPT
+			if (post.anonymous && !user.isAdministrator(uid)) {
+				post.user.username = 'Anonymous User';
+			} else {
+				post.user.username = users.find(u => u.uid === post.uid).username;
+			}
 			Posts.overrideGuestHandle(post, post.handle);
 			post.handle = undefined;
 			post.topic = tidToTopic[post.tid];
 			post.category = post.topic && cidToCategory[post.topic.cid];
 			post.isMainPost = post.topic && post.pid === post.topic.mainPid;
 			post.deleted = post.deleted === 1;
 			post.timestampISO = utils.toISOString(post.timestamp);
-			post.anonymous = post.anonymous ? post.anonymous : 'false'; // checks if anonymous is true if not then false
+			post.anonymous = post.anonymous ? post.anonymous : 'false'; // makes sure anonymous is a required field for every post, false if anonymous field is undefined
 		});
 
 		posts = posts.filter(post => tidToTopic[post.tid]);

src/upgrades/1.4.4/sound_settings.js

@@ -11,7 +11,7 @@ const map = {
 	'waterdrop-low.mp3': 'Default | Water drop (low)',
 };
 
-console.log('Sophie Feng');
+// console.log('Sophie Feng');
 function soundSettings(cb) {
 	const keys = ['chat-incoming', 'chat-outgoing', 'notification'];
 

test/posts.js

@@ -701,6 +701,49 @@ describe('Post\'s', () => {
 		});
 	});
 
+	// tests for admin permission to see anonymous poster's user name
+	// Reference to ChatGPT
+	describe('Post Anonymity', () => {
+		let uid;
+		let postData;
+		let adminUid;
+
+		before(async () => {
+			// Create a regular user
+			uid = await user.create({ username: 'regularuser' });
+
+			// Create an admin user
+			adminUid = await user.create({ username: 'adminuser' });
+			await groups.join('administrators', adminUid); // Make the user an admin
+
+			// Create an anonymous post
+			({ postData } = await topics.post({
+				uid: uid,
+				cid: cid,
+				title: 'Anonymous Post Test',
+				content: 'This is an anonymous post',
+				anonymous: true,
+			}));
+		});
+
+		it('should display "Anonymous User" if the post is anonymous and the user is not an admin', async () => {
+			// Simulate a request by a non-admin user
+			const postSummary = await posts.getPostSummaryByPids([postData.pid], uid, {});
+			// console.log(postSummary[0]);
+
+			// Check that the username is set to "Anonymous User"
+			assert.strictEqual(postSummary[0].user.username, 'Anonymous User');
+		});
+
+		it('should display the real username if the post is anonymous but the user is an admin', async () => {
+			// Simulate a request by an admin user
+			const postSummary = await posts.getPostSummaryByPids([postData.pid], adminUid, {});
+			// console.log(postSummary[0]);
+			// Check that the username is the actual user's username
+			assert.strictEqual(postSummary[0].user.username, 'regularuser');
+		});
+	});
+
 	it('should get recent poster uids', (done) => {
 		topics.reply({
 			uid: voterUid,