Model.restore is broken when using use$neOperator = false

[supersime]

Prerequisites

• I have written a descriptive issue title
• I have searched existing issues to ensure the bug has not already been reported

Mongoose version

7.4.2

Node.js version

20.11

MongoDB server version

5.0.24

Typescript version (if applicable)

No response

Description

We have noticed that the Model.restore is removing the field deleted from the document.
This previously used to work.
We are using the use$neOperator with a value of false - in other words, all of our queries look for data where deleted = false as a result of dsanel/mongoose-delete#50.

The following change, made in version 1.0.1 has broken this functionality:

• dsanel/mongoose-delete@b8c3cea

Before this code change, the schema.statics.restore was:

    schema.statics.restore =  function (conditions, callback) {
        if (typeof conditions === 'function') {
            callback = conditions;
            conditions = {};
        }

        var doc = {
            deleted: false,
            deletedAt: undefined,
            deletedBy: undefined
        };

        return updateDocumentsByQuery(this, conditions, doc, callback);
    };

After this code change, the function is:

    schema.statics.restore =  function (conditions, callback) {
        if (typeof conditions === 'function') {
            callback = conditions;
            conditions = {};
        }

        var doc = {
            $unset:{
                deleted: true,
                deletedAt: true,
                deletedBy: true
            }
        };

        return updateDocumentsByQuery(this, conditions, doc, callback);
    };

The issue here, is that the deleted key is being removed from the model, which doesn't support how the use$neOperator works!

I have tested the following which appears to work - could this be considered for the 7.6.x release please?

    schema.statics.restore =  function (conditions, callback) {
        if (typeof conditions === 'function') {
            callback = conditions;
            conditions = {};
        }

        var doc = use$neOperator ? {
            $unset: {
                deleted: true,
                deletedAt: true,
                deletedBy: true
            }
        } : {
            deleted: false,
            $unset: {
                deletedAt: true,
                deletedBy: true
            }
        };

        return updateDocumentsByQuery(this, conditions, doc, callback);
    };

Steps to Reproduce

Configure the mongoose-delete plugin as follows:

import softDelete from 'mongoose-delete';

export const softDeletePlugin = [
    softDelete,
    {
        deletedAt: true,
        deletedBy: true,
        overrideMethods: true,
        use$neOperator: false,
    },
];

When you add this plugin to your schema, delete a document, and then try to restore it:

await Model.restore({ _id})

The deleted key has been removed from the underlying document.

Expected Behavior

The deleted key should remain in the document, and have a value of false

[vkarpov15]

This GitHub repo is for mongoose, not mongoose-delete, so it would likely be better to report this as a bug on the mongoose-delete GitHub repo.

That being said, this does look like an unintentional bug introduced in mongoose-delete. We will see if we can open a PR to fix

[supersime]

This GitHub repo is for mongoose, not mongoose-delete, so it would likely be better to report this as a bug on the mongoose-delete GitHub repo.

That being said, this does look like an unintentional bug introduced in mongoose-delete. We will see if we can open a PR to fix

My apologies @vkarpov15 ... silly mistake! Thank you so much!