5

I am currently converting my API from JS to TS. However, I'm having some difficulties with mongoose & typescript. Specifically, this is not available inside my instance methods.

My code:

AccountSchema.methods.comparePassword = async function (candidatePassword: string) {
  const isMatch = await bcrypt.compare(candidatePassword, this.password);
  return isMatch;
};

Which generates the following error as this refers to the function rather than the actual document:

Argument of type 'Function' is not assignable to parameter of type 'string'.

However, according to Mongoose's documentation, there shouldn't be an error as this should refer to the actual document.

What this actually refers to:

this: {
    [name: string]: Function;
}

How I defined my schema:

const AccountSchema = new mongoose.Schema<IAccount>({...})

My approach worked just fine with JS, so I know it has something to do with TypeScript. Mongoose documentation confirms that my implementation is the right way to define instance methods, so I'm quite confused why it doesn't work.

Is there a specific TS way of declaring & using instance methods in mongoose that I don't know of?

Any help would be greatly appreciated!

11
  • "Argument of type 'Function' is not assignable to parameter of type 'string'." is pretty clearly telling you you're passing a function where a string is expected. What argument is the error on? What is the definition of the function/method you're calling? What is the definition of the argument? Jan 13, 2021 at 15:19
  • May be try the shorthand.AccountSchema.methods.comparePassword = async (candidatePassword: string): boolean => { const isMatch = await bcrypt.compare(candidatePassword, this.password); return isMatch; };
    – user906573
    Jan 13, 2021 at 15:20
  • @T.J.Crowder there shouldn't even be such an error in the first place, as "this", according to Mongoose's documentation, should refer to the actual document rather than the function Jan 13, 2021 at 15:21
  • @user906573 that doesn't work as it prevents this from being bound (according to Mongoose's docs. I receive an error message that "this" is possibly undefined. Jan 13, 2021 at 15:23
  • @T.J.Crowder updated my question, should be more clear now Jan 13, 2021 at 15:28

1 Answer 1

6

All I needed was a this parameter on that function. That's special in TS, and specifies the type of this within the function.

Like so:

async function (this: IAccount, candidatePassword: string) { ... }

this parameters are not passed explicitly, so the new parameter doesn't change how the function is called.

1

Your Answer

Reminder: Answers generated by Artificial Intelligence tools are not allowed on Stack Overflow. Learn more

By clicking “Post Your Answer”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Not the answer you're looking for? Browse other questions tagged or ask your own question.