Merge pull request #329 from gets0ul/connect_app_issue-3112

Fix for connect-app issue appirio-tech/connect-app#3112
topcoder-platform · Jul 6, 2019 · ef2331d · ef2331d
2 parents 5d31391 + 2f387af
commit ef2331d
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/src/routes/productCategories/create.js b/src/routes/productCategories/create.js
@@ -41,10 +41,10 @@ module.exports = [
     });
 
     // Check if duplicated key
-    return models.ProductCategory.findById(req.body.param.key)
+    return models.ProductCategory.findById(req.body.param.key, { paranoid: false })
       .then((existing) => {
         if (existing) {
-          const apiErr = new Error(`Product category already exists for key ${req.params.key}`);
+          const apiErr = new Error(`Product category already exists(may be deleted) for key "${req.body.param.key}"`);
           apiErr.status = 422;
           return Promise.reject(apiErr);
         }

diff --git a/src/routes/projectTypes/create.js b/src/routes/projectTypes/create.js
@@ -42,10 +42,10 @@ module.exports = [
     });
 
     // Check if duplicated key
-    return models.ProjectType.findById(req.body.param.key)
+    return models.ProjectType.findById(req.body.param.key, { paranoid: false })
       .then((existing) => {
         if (existing) {
-          const apiErr = new Error(`Project type already exists for key ${req.params.key}`);
+          const apiErr = new Error(`Project type already exists(may be deleted) for key "${req.body.param.key}"`);
           apiErr.status = 422;
           return Promise.reject(apiErr);
         }