Deep cloning the findOptions in findAndCountAll

Since the ID was added automatically to the attributes, it caused duplicate fields in the generated queries on the findAll

Deep cloning the findOptions in findAndCountAll
Since the ID was added automatically to the attributes, it caused duplicate fields in the generated queries on the findAll
jValdron
Commit eaa76e68 authored Feb 12, 2014 by jValdron
Showing with 1 additions and 1 deletions
lib/dao-factory.js
--- a/lib/dao-factory.js
+++ b/lib/dao-factory.js
@@ -614,7 +614,7 @@ module.exports = (function() {
  DAOFactory.prototype.findAndCountAll = function(findOptions, queryOptions) {
    var self  = this
      // no limit, offset, order, attributes for the options given to count()
-      , countOptions = Utils._.omit(findOptions || {}, ['offset', 'limit', 'order', 'attributes'])
+      , countOptions = Utils._.omit(findOptions ? Utils._.merge({}, findOptions) : {}, ['offset', 'limit', 'order', 'attributes'])

    return new Utils.CustomEventEmitter(function (emitter) {
      var emit = {