parseInt option for Queries - parseFloat missing?

[mweibel]

Hi,

we're currently implementing the function sum() for sequelize and ran across this line, dao factory#271.
In my opinion, this should rather be parseFloat and the respective options-parser in query-interface#295 should also check this option and do rather a parseFloat.

Do you know why you didn't add support this or is this just a missing functionality?

• Michael

[janmeier]

Agree, no reason this shouldn't be parseFloat

[durango]

I'll take care of this writing tests now

[durango]

@mweibel #583

I just pushed the update to fix this, if you immediately need it grab it from the Repo :)

[mweibel]

Thanks!

Am 02.05.2013 um 20:59 schrieb durango notifications@github.com:

@mweibel #583

I just pushed the update to fix this, if you immediately need it grab it from the Repo :)

—
Reply to this email directly or view it on GitHub.