joe/kami-parse-server
Public Access
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Remove unused parameter in Cloud Function (#6969)

Browse Source
This commit is contained in:
Diamond Lewis
2020-10-25 22:41:23 -05:00
committed by GitHub
parent e6ac3b6932
commit 74ba81104e
2 changed files with 7 additions and 10 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
spec/CloudCode.spec.js
View File

@@ -1313,7 +1313,7 @@ describe('Cloud Code', () => {
object.set('before', 'save'); object.set('before', 'save');
}); });
Parse.Cloud.define('removeme', (req, res) => { Parse.Cloud.define('removeme', () => {
const testObject = new TestObject(); const testObject = new TestObject();
return testObject return testObject
.save() .save()
@@ -1324,11 +1324,10 @@ describe('Cloud Code', () => {
.then(object => { .then(object => {
object.unset('remove'); object.unset('remove');
return object.save(); return object.save();
}) });
.catch(res.error);
}); });
Parse.Cloud.define('removeme2', (req, res) => { Parse.Cloud.define('removeme2', () => {
const testObject = new TestObject(); const testObject = new TestObject();
return testObject return testObject
.save() .save()
@@ -1339,8 +1338,7 @@ describe('Cloud Code', () => {
.then(object => { .then(object => {
object.unset('remove'); object.unset('remove');
return object.save(); return object.save();
}) });
.catch(res.error);
}); });
Parse.Cloud.run('removeme') Parse.Cloud.run('removeme')

7
src/Routers/FunctionsRouter.js
View File

@@ -96,7 +96,7 @@ export class FunctionsRouter extends PromiseRouter {
}); });
} }
static createResponseObject(resolve, reject, message) { static createResponseObject(resolve, reject) {
return { return {
success: function (result) { success: function (result) {
resolve({ resolve({
@@ -109,7 +109,6 @@ export class FunctionsRouter extends PromiseRouter {
const error = triggers.resolveError(message); const error = triggers.resolveError(message);
reject(error); reject(error);
}, },
message: message,
}; };
} }
static handleCloudFunction(req) { static handleCloudFunction(req) {
@@ -137,7 +136,7 @@ export class FunctionsRouter extends PromiseRouter {
return new Promise(function (resolve, reject) { return new Promise(function (resolve, reject) {
const userString = req.auth && req.auth.user ? req.auth.user.id : undefined; const userString = req.auth && req.auth.user ? req.auth.user.id : undefined;
const cleanInput = logger.truncateLogMessage(JSON.stringify(params)); const cleanInput = logger.truncateLogMessage(JSON.stringify(params));
const { success, error, message } = FunctionsRouter.createResponseObject( const { success, error } = FunctionsRouter.createResponseObject(
result => { result => {
try { try {
const cleanResult = logger.truncateLogMessage(JSON.stringify(result.response.result)); const cleanResult = logger.truncateLogMessage(JSON.stringify(result.response.result));
@@ -177,7 +176,7 @@ export class FunctionsRouter extends PromiseRouter {
return triggers.maybeRunValidator(request, functionName); return triggers.maybeRunValidator(request, functionName);
}) })
.then(() => { .then(() => {
return theFunction(request, { message }); return theFunction(request);
}) })
.then(success, error); .then(success, error);
}); });