Issue

Versions

• sequelize: ^6.17.0
• sequelize-typescript: ^2.1.3
• typescript: ^4.5.4
• fastify: ^3.0.0

Issue type

• bug report

Actual behavior

[App] (sequelize) Warning: Model "Users" is declaring public class fields for attribute(s): "firstName", "lastName", "email", "userName", "phone", "dob", "password", "role_id", "role_title".
[App] These class fields are shadowing Sequelize's attribute getters & setters.
[App] See https://sequelize.org/main/manual/model-basics.html#caveat-with-public-class-field

Expected behavior

a clean CLI log without this warning, what am i doing wrong.

Steps to reproduce

I created simple models with annotations and use their objects to save or retrieve data from the database

Related code

model

import {Table, Column, Model, DataType, Unique, BelongsToMany} from 'sequelize-typescript';
//import bcrypt from 'fastify-bcrypt'

import Roles from './roles';
import userRoles from './userRoles';
@Table({
  timestamps: true,
  paranoid: true  
})

export default class Users extends Model {
  @Column(DataType.VIRTUAL)
  get object() {
    return this.constructor.name.toLowerCase();
  }    


  @Column({field: 'firstName',
    type: DataType.STRING, 
    allowNull: false , 
    validate: {
      notNull: {msg: 'firstName is required'}
    }
  })
  public firstName!: string;


  @Column({field: 'lastName',
    type: DataType.STRING, 
    allowNull: false , 
    validate: {
      notNull: {msg: 'lastName is required'}
    }
  })
  public lastName!: string;

  @Unique({
    name: 'email',
    msg: 'email already taken, Please try a different email'
  })
  @Column({field: 'email',
    type: DataType.STRING, 
    allowNull: false , 
    validate: {
      notNull: {msg: 'email is required'}
    }
  })
  public email!: string;

  @Unique({
    name: 'userName',
    msg: 'username already taken, Please try a different username'
  })
  @Column({field: 'userName',
    type: DataType.STRING, 
    allowNull: false, 
    validate: {
      notNull: {msg: 'userName is required'},      
    }
  })
  public userName!: string;

  @Unique({
    name: 'phone',
    msg: 'phone already taken, Please try a different phone'
  })
  @Column({field: 'phone',
    type: DataType.STRING, 
    allowNull: false , 
    validate: {
      notNull: {msg: 'phone is required'}
    }
  })
  public phone!: string;

  @Column({field: 'dob',
    type: DataType.STRING, 
    allowNull: false , 
    validate: {
      notNull: {msg: 'dob is required'}
    }
  })
  public dob!: string;


  @Column({field: 'password',
    type: DataType.STRING, 
    allowNull: false , 
    
    validate: {
      notNull: {msg: 'password is required'}
    }
  })  
  public password?: string

  @Column({field: 'role_id',
    type: DataType.NUMBER, 
    allowNull: false , 
    
    validate: {
      notNull: {msg: 'role_id is required'}
    }
  })  
  public role_id?: number

  @Column({field: 'role_title',
    type: DataType.STRING, 
    allowNull: false , 
    
    validate: {
      notNull: {msg: 'role_title is required'}
    }
  })  
  public role_title?: string
  
  


  @BelongsToMany(() => Roles, {
    through: {
      model: () => userRoles,
      unique: false,
    },
    foreignKey: 'user_id',
    constraints: false,
  })
  roles: Roles[] | undefined;


  

  static encrypt(password, bcrypt) {  
      
    
    //password = bcrypt.hash(password, 10);
    return bcrypt.hash(password)
  .then(hash => {return hash})
  .catch(err => err.message)
  }

 

  toJSON() {
    return {...super.toJSON(), password: null};
  }
  
}

handler

const role_inserted = await Users.findByPk(id,{include: ["roles"]})

Although I am not facing any blocking errors right now, but this warning keep throwing and I am concerned if this will be a problem as my code base increases to accomodate more complex scenarios.

[App] (sequelize) Warning: Model "Users" is declaring public class fields for attribute(s): "firstName", "lastName", "email", "userName", "phone", "dob", "password", "role_id", "role_title".
[App] These class fields are shadowing Sequelize's attribute getters & setters.
[App] See https://sequelize.org/main/manual/model-basics.html#caveat-with-public-class-fields