remove extra check when delete idea (#463)

DiverVM · web-flow · commit 90b94bc223aa · 2025-04-25T14:24:23.000+03:00
diff --git a/routes/api/ideation.js b/routes/api/ideation.js
@@ -917,7 +917,6 @@ module.exports = function (app) {
             const ideaId = req.params.ideaId;
             const ideationId = req.params.ideationId;
             const topicId = req.params.topicId;
-            const sessToken = createHash('sha256').update(req.cookies[config.session.name]).digest('base64');
             const idea = await Idea.findOne({
                 where: {
                     id: ideaId
@@ -944,7 +943,7 @@ module.exports = function (app) {
                 res.forbidden();
             }
 
-            if ((!ideation.allowAnonymous && idea.authorId !== req.user.id) || (ideation.allowAnonymous && idea.sessionId !== sessToken)) {
+            if (idea.authorId !== req.user.id) {
                 return res.forbidden();
             }
 

Original file line number	Diff line number	Diff line change
`@@ -917,7 +917,6 @@ module.exports = function (app) {`
`917`	`917`	`const ideaId = req.params.ideaId;`
`918`	`918`	`const ideationId = req.params.ideationId;`
`919`	`919`	`const topicId = req.params.topicId;`
`920`		`- const sessToken = createHash('sha256').update(req.cookies[config.session.name]).digest('base64');`
`921`	`920`	`const idea = await Idea.findOne({`
`922`	`921`	`where: {`
`923`	`922`	`id: ideaId`
`@@ -944,7 +943,7 @@ module.exports = function (app) {`
`944`	`943`	`res.forbidden();`
`945`	`944`	`}`
`946`	`945`
`947`		`- if ((!ideation.allowAnonymous && idea.authorId !== req.user.id) \|\| (ideation.allowAnonymous && idea.sessionId !== sessToken)) {`
	`946`	`+ if (idea.authorId !== req.user.id) {`
`948`	`947`	`return res.forbidden();`
`949`	`948`	`}`
`950`	`949`