Как исправить ошибку в TS, NestJS?

Question

[Артём Белых]

Здравствуйте, помогите исправить ошибку:

TS2339: Property 'mailerService' does not exist on type 'typeof UserMailService'

import { Injectable } from '@nestjs/common';
import {MailerService} from "@nestjs-modules/mailer";


@Injectable()
export class UserMailService {
    constructor(private readonly mailerService: MailerService) {}
    static async sendActivationMail(to: string, link: any): Promise<void>{


        await this.mailerService.sendMail({
            from: process.env.SMTP_USER,
            to,
            subject: 'Активация аккаунта на ' + process.env.API_URL,
            text: '',
            html:
                `
                    <div>
                        <h1>Для активации перейдите по ссылке</h1>
                        <a href="${link}">${link}</a>
                    </div>
                `
        }).then(() => {})
            .catch(() => {});

    }

}

users.module.ts

import {Module} from '@nestjs/common';
import {TypeOrmModule} from "@nestjs/typeorm";
import {UserEntity} from "./entities/user.entity";
import {UserController} from "./users.controller";
import {UserService} from "./users.service";
import {TokenEntity} from "./entities/token.entity";
import {JwtModule} from "@nestjs/jwt";
import {TokenGuard} from "./token.guard";
import {MailerModule} from "@nestjs-modules/mailer";
import {UserMailService} from "./userMail.service";

@Module({
    imports: [TypeOrmModule.forFeature([UserEntity, TokenEntity]), JwtModule.register({
        global: true,
        secret: "234",
        signOptions: { expiresIn: '60s' },
    }), MailerModule.forRoot({
            transport: {
                host: process.env.SMTP_HOST,
                port: process.env.SMTP_PORT,
                ignoreTLS: true,
                secure: false,
                auth: {
                    user: process.env.SMTP_USER,
                    pass: process.env.SMTP_PASSWORD,
                },
            },
        defaults: {
            from: '"nest-modules" <[e-mail]>',
        }
    })],
    controllers: [UserController],
    providers: [UserService, UserMailService, TokenGuard],

})
export class UsersModule {
}

При this.mailerService не видит функцию sendMail(), почему, помогите!

Answer 1

[Василий Банников]

У тебя метод sendActivationMail статичный. Он не имеет доступа к полям экземпляра.

Comments

[Артём Белых]

Как получить доступ к полям экземпляра?

[Василий Банников]

Артём, сделать метод sendActivationMail не статичным, либо передать mailerService как третий аргумент.

[Василий Банников]

Артём, кстати, почему у тебя link имеет тип any? Судя по коду, если это будет не строка - у тебя будет ошибка.

[Артём Белых]

Василий Банников, ну я сделал sendActivationMail не статичным, но тогда ошибка при вызове этого метода в users.service.ts:

TS2339: Property 'sendActivationMail' does not exist on type 'typeof UserMailService'.

[Василий Банников]

Артём, там где ты используешь sendActivationMail ты должен обращаться к экземпляру, а не к типу.

[Артём Белых]

Василий Банников, new UserMailService()

а потом к нему обращаться? Правильно понял?
если можете

вот контроллер где я вызываю функцию

import { Injectable } from '@nestjs/common';
import {InjectRepository} from "@nestjs/typeorm";
import {Repository} from "typeorm";
import {UserEntity} from "./entities/user.entity";
import {ApiError} from "./exceptions/api-error";
import {v4} from 'uuid';
import * as bcrypt from 'bcrypt';
import {CreateUserDto} from "./dto/create-user.dto";
import {TokenGuard} from "./token.guard";
import {UserMailService} from "./userMail.service";


@Injectable()
export class UserService {
  constructor(@InjectRepository(UserEntity) private readonly userRepository: Repository<UserEntity>) {}

  public async register(email: any, password: any) {
    const candidate = await this.userRepository.findOne({where: {email: email}})
    if (candidate) {
      throw ApiError.BadRequest(`Пользователь с почтовым адресом ${email} уже существует`)
    }
    const hashPassword = await bcrypt.hash(password, 3);
    const activationLink = v4(); // v34fa-asfasf-142saf-sa-asf

    const user = await this.userRepository.create({email, password: hashPassword, activationLink})
    
    await UserMailService.sendActivationMail(email, `${process.env.API_URL}/api/activate/${activationLink}`);

    const userDto = new CreateUserDto(user); // id, email, isActivated
    const tokens = TokenGuard.generateTokens({...userDto});
    await TokenGuard.saveToken(userDto.id, tokens.refreshToken);

    return {...tokens, user: userDto}
  }
}

[Алексей Ярков]

Артём, вот в контроллер свой UserMailService и прокинуть как зависимость. И обращаться к методу.

[Артём Белых]

Алексей Ярков, ой, код не контроллера а сервиса

[Алексей Ярков]

Артём, да без разницы ))

[Артём Белых]

Алексей Ярков, Не совсем понял "прокинуть как зависимость", я же его вызываю и обращаюсь к функции

[Алексей Ярков]

Артём,

@Injectable()
export class UserService {
  constructor(
    @InjectRepository(UserEntity) private readonly userRepository: Repository<UserEntity>,
    private readonly userMailService: UserMailService,
  ) {}

  public async register(email: any, password: any) {
    await this.userMailService.sendActivationMail(email, `${process.env.API_URL}/api/activate/${activationLink}`);
  }
}

[Артём Белых]

Алексей Ярков, Можете в моём коде исправить как правильно пожалуйста, а то я запутался куда что прокидывать в nestjs, только начал его учуть :))

[Василий Банников]

Артём, покажи место, где ты вызываешь sendActivationMail

[Василий Банников]

Артём, upd: Алексей Ярков написал как надо

[Артём Белых]

Алексей Ярков,

TS2339: Property 'UserMailService' does not exist on type 'UserService'.

Вот что пишет

[Алексей Ярков]

Артём, ну так надо добавить его в провайдеры модуля, к которому UserService относится

[Артём Белых]

Алексей Ярков, ААА, всё спасибо!

Answer 2

[Алексей Ярков]

@Inject()
private service

Comments

[Артём Белых]

@Inject() private mailerService: MailerService

Так? Но не работает.

[Алексей Ярков]

Артём, посмотрел доку, вроде всё так как надо. А вот static стоит убрать попробовать.

[Артём Белых]

Алексей Ярков, Тогда в users.service.ts при вызове функции

await UserMailService.sendActivationMail(email, `${process.env.API_URL}/api/activate/${activationLink}`);

Вот такая ошибка

TS2339: Property 'sendActivationMail' does not exist on type 'typeof UserMailService'