refactor: remove /users/{uid}/settings/{setting} route

@baris Also, I am now allowing the following properties to be saved in User.saveSettings:
- categoryTopicSort
- topicPostSort
- setCategorySort
- setTopicSort

src/api/users.js

@@ -106,10 +106,6 @@ usersAPI.updateSettings = async function (caller, data) {
 	return await user.saveSettings(data.uid, payload);
 };
 
-usersAPI.updateSetting = async function (caller, data) {
-	await user.setSetting(data.uid, data.setting, data.value);
-};
-
 usersAPI.changePassword = async function (caller, data) {
 	await user.changePassword(caller.uid, Object.assign(data, { ip: caller.ip }));
 	await events.log({

src/controllers/write/users.js

@@ -33,11 +33,6 @@ Users.updateSettings = async (req, res) => {
 	helpers.formatApiResponse(200, res, settings);
 };
 
-Users.updateSetting = async (req, res) => {
-	await api.users.updateSetting(req, { ...req.params, value: req.body.value });
-	helpers.formatApiResponse(200, res);
-};
-
 Users.changePassword = async (req, res) => {
 	await api.users.changePassword(req, { ...req.body, uid: req.params.uid });
 	helpers.formatApiResponse(200, res);

src/routes/write/users.js

@@ -22,7 +22,6 @@ function authenticatedRoutes() {
 	setupApiRoute(router, 'delete', '/:uid', [...middlewares, middleware.assert.user, middleware.exposePrivileges], controllers.write.users.delete);
 
 	setupApiRoute(router, 'put', '/:uid/settings', [...middlewares, middleware.checkRequired.bind(null, ['settings'])], controllers.write.users.updateSettings);
-	setupApiRoute(router, 'put', '/:uid/settings/:setting', [...middlewares, middleware.checkRequired.bind(null, ['value'])], controllers.write.users.updateSetting);
 
 	setupApiRoute(router, 'put', '/:uid/password', [...middlewares, middleware.checkRequired.bind(null, ['newPassword']), middleware.assert.user], controllers.write.users.changePassword);
 

src/user/settings.js

@@ -121,6 +121,10 @@ module.exports = function (User) {
 			upvoteNotifFreq: data.upvoteNotifFreq,
 			bootswatchSkin: data.bootswatchSkin,
 			categoryWatchState: data.categoryWatchState,
+			categoryTopicSort: data.categoryTopicSort,
+			topicPostSort: data.topicPostSort,
+			setCategorySort: data.setCategorySort,
+			setTopicSort: data.setTopicSort,
 		};
 		const notificationTypes = await notifications.getAllNotificationTypes();
 		notificationTypes.forEach(function (notificationType) {