Calling user.destroy() doesn't actually delete database record

Question

We started the app with Parse and have been hitting problems since it shut down.

Destroying a user is not working in this deleteUser function

Parse.Cloud.define('deleteUser', function(request, response) {
var objectId = request.params.objectId;

if(!objectId || objectId.length === 0 || !objectId.trim()) {
    response.error("The user object id is needed to perform this action.");
}

Parse.Cloud.useMasterKey();
var query = new Parse.Query(Parse.User);
query.get(objectId, {
  success: function(user) {
    user.destroy(null, {
      success: response.success(user),
      error: function(user, error) {
        response.error(error);
      }
    });
  },
  error: function(error) {
    response.error("Could not find the user.");
  }
});
});

deleteEvent works just fine and I've also tried copying this exact syntax for deleteUser with no luck.

Parse.Cloud.define('deleteEvent', function(request, response) {

var objectId = request.params.objectId;

if(!objectId || objectId.length === 0 || !objectId.trim()) {
    response.error('The event object id is needed to perform this action.');
    return;
}

Parse.Cloud.useMasterKey();
var query = new Parse.Query(Parse.Object.extend('Events'));
query.get(objectId, {
    success: function(event) {
        event.destroy();
        response.success('Successfully deleted event.');
    },
    error: function(error) {
        response.error('Could not find the event.');
    }
});
});

The function finishes with the success call in the logs, but user.destroy() never actually deletes the user.

I console.log all of the params, IDs, and arguments to make sure I'm getting the expected. Is there a special way to delete Parse users?

This suddenly stopped working last week.

I've tried on Parse v1.6.14 and v1.9.2

Thanks in advance for the help!

Cipi · Accepted Answer · 2017-02-03 13:44:50Z

1

As someone said, Parse.Cloud.useMasterKey() is deprecated in parse-server. You now need to specify useMasterKey flag, or pass a sessionToken (to impersonate User) to methods accessing DB (get, save, first, count,...). So, change your code accordingly:

Parse.Cloud.define('deleteUser', function(request, response) {
var objectId = request.params.objectId;

if(!objectId || objectId.length === 0 || !objectId.trim()) {
    response.error("The user object id is needed to perform this action.");
}

var query = new Parse.Query(Parse.User);
query.get(objectId, {
  useMasterKey: true, //HERE (1/2)
  success: function(user) {
    user.destroy({
      useMasterKey: true, //HERE (2/2)
      success: response.success(user),
      error: function(user, error) {
        response.error(error);
      }
    });
  },
  error: function(error) {
    response.error("Could not find the user.");
  }
});
});

answered Feb 3, 2017 at 13:44

Cipi

11.4k9 gold badges50 silver badges60 bronze badges

Sign up to request clarification or add additional context in comments.

1 Comment

Andrew Pham Over a year ago

I see, sweet and simple reason. Thank you Cipi and Cliffordwh for your help!

alvaro · Accepted Answer · 2017-06-01 02:06:54Z

1

This is because Parse.Cloud.useMasterKey() is deprecated since Parse-server version 2.3.0. You now need to make use of useMasterKey: true when using the users table.

   var query = new Parse.Query(Parse.User);
       query.get({ useMasterKey: true }).then(.......)

edited Jun 1, 2017 at 2:06

alvaro

2,7362 gold badges21 silver badges19 bronze badges

answered Feb 3, 2017 at 5:42

Cliffordwh

1,4301 gold badge10 silver badges18 bronze badges

Collectives™ on Stack Overflow

Calling user.destroy() doesn't actually delete database record

2 Answers 2

1 Comment

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

1 Comment

Comments

Your Answer

Sign up or log in

Post as a guest

Related