adarshasnah / mongoose-rest-query

Restify your mongoose model

• 👋 Hi, I’m @adarshasnah
• 👀 I’m interested in ...
• 🌱 I’m currently learning ...
• 💞️ I’m looking to collaborate on ...
• 📫 How to reach me ...

mongoose-rest-query's Issues

BUG: controller > aggregate > parse(req.body)

BUG: number getting transformed into ObjectId.

What I did (from Angular App):

Error passed into aggregate's cb (and consoled in Chrome > Debugger):

Value before "parse":

Value after "parse":

Cause of bug:

ENHANCEMENT: controller > aggregate > parse(req.body) > ISOString

Description:

For equality comparisons to work when using aggregate with dates, Date instances must be used instead of "string" (even if it's in the ISOString form).

Proof:

Testing using "string"

Testing using Date instances

Tried and failed workaround:

Using Date instances from client app

Automatically gets converted to "string" (most probably by Angular's internal methods)

Inspected interceptor, but no conversion done here

Proposed solution:

Adding "ISOString to Date" conversion

Note: https://github.com/honeinc/is-iso-date used to check if a string is an ISOString,

Before conversion