Dynamic Mongoose Schema based on property

Question

I have the following user schema (where all distinct properties of different user types are consolidated):

var UserSchema = new mongoose.Schema({
  status: String,
  firstName: String,
  lastName: String,
  address: Object,
  email: {type: String, lowercase: true, unique: true, required: [true, "can't be blank"], match: [/\S+@\S+\.\S+/, 'is invalid'], index: true},
  organization: [{ type: mongoose.Schema.Types.ObjectId, ref: 'Organization' }],
  phone: {type: Number, unique: true, required: true, required: [true, "can't be blank"]}, 
  role: String, 
  hash: String,
  salt: String,
  deliverySchedule:  [{type: String, required: true}]

}

"Common" Schema (what all user types share in common):

var UserSchema = new mongoose.Schema({
  status: String,
  firstName: String,
  lastName: String,
  email: {type: String, lowercase: true, unique: true, required: [true, "can't be blank"], match: [/\S+@\S+\.\S+/, 'is invalid'], index: true},
  phone: {type: Number, unique: true, required: true, required: [true, "can't be blank"]}, 
  role: String, 
  hash: String,
  salt: String
}

Role = "Customer":

  address: [{type: Object, required: true}]

Role = "DeliveryMan":

      deliverySchedule:  [{type: String, required: true}]
      organization: [{ type: mongoose.Schema.Types.ObjectId, required: true, 
ref: 'Organization' }],

Role = "Seller":

      organization: [{ type: mongoose.Schema.Types.ObjectId, required: true, 
ref: 'Organization' }],

I would like to add (and REQUIRE if possible) some fields to the "common" schema based on the user's role. However, I want to store them in the same collection.

How can I add a method to my models/Users.js to add properties to the schema based on "user.role"

Talha Awan · Accepted Answer · 2017-06-07 21:25:13Z

3

Make require validation optional for each role dependent fields.

var UserSchema = new mongoose.Schema({
  status: String,
  firstName: String,
  lastName: String,
  email: {type: String, lowercase: true, unique: true, required: [true, "can't be blank"], match: [/\S+@\S+\.\S+/, 'is invalid'], index: true},
  phone: {type: Number, unique: true, required: true, required: [true, "can't be blank"]}, 
  role: {type: String, enum: ['Customer', 'DeliveryMan', 'Seller'], required: true},
  address: {type: [Object],  required: isRequired("address")},
  deliverySchedule: {type: [String], required: isRequired("deliverySchedule")},
  organization: { type: [mongoose.Schema.Types.ObjectId], ref: 'Organization', required: isRequired("organization")},
  hash: String,
  salt: String
});

function isRequired(field){
    return function(){
        if(field == "address"){
            return this.role === "Customer"
        }
        else if(field == "deliverySchedule"){
            return this.role === "DeliveryMan"
        }
        else if(field == "organization"){
            return this.role === "Seller" || this.role === "DeliveryMan"
        }
    }
};

edited Jun 7, 2017 at 21:25

answered Jun 7, 2017 at 20:18

Talha Awan

4,6194 gold badges27 silver badges40 bronze badges

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

6 Comments

Moshe Over a year ago

This is more elegant than using discriminators. Thank you so much

Moshe Over a year ago

I want to refactor inline functions to one function. How do I do that? function authorize(roles) { roles.forEach(role =>if (role === this.role) return true else return false }

Moshe Over a year ago

for clarity, I meant to ask where can I place this function and how do I access it ?

Talha Awan Over a year ago

You can certainly take the functions out but you can't merge them into one, because that one function will have no context to tell which field is calling it and which required should be set to true.

Moshe Over a year ago

I see what you're saying, I keep getthing this.role as undefined. Can you show me the right practice of refactoring this? Thanks :)

|

Collectives™ on Stack Overflow

Dynamic Mongoose Schema based on property

1 Answer 1

6 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

6 Comments

Your Answer

Sign up or log in

Post as a guest

Related