Merge pull request #583 from topcoder-platform/feature/fix-crash

Gunasekar-K · web-flow · commit ce1a62260885 · 2023-03-26T20:47:33.000+05:30
fix: correctly unset direct project id
diff --git a/src/services/ChallengeService.js b/src/services/ChallengeService.js
@@ -1523,7 +1523,7 @@ async function updateChallenge(currentUser, challengeId, data) {
 
   // Make sure the user cannot change the direct project ID
   if (data.legacy && data.legacy.directProjectId) {
-    _.unset(data, "legacy.directProjectId", directProjectId);
+    _.unset(data, "legacy.directProjectId");
   }
 
   /* BEGIN self-service stuffs */

Original file line number	Diff line number	Diff line change
`@@ -1523,7 +1523,7 @@ async function updateChallenge(currentUser, challengeId, data) {`
`1523`	`1523`
`1524`	`1524`	`// Make sure the user cannot change the direct project ID`
`1525`	`1525`	`if (data.legacy && data.legacy.directProjectId) {`
`1526`		`- _.unset(data, "legacy.directProjectId", directProjectId);`
	`1526`	`+ _.unset(data, "legacy.directProjectId");`
`1527`	`1527`	`}`
`1528`	`1528`
`1529`	`1529`	`/* BEGIN self-service stuffs */`