Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

implement "sugar" wrapper method for findAll() followed by count() #533

Merged
merged 9 commits into from Jun 21, 2013
Merged

implement "sugar" wrapper method for findAll() followed by count() #533

merged 9 commits into from Jun 21, 2013

Commits on Apr 10, 2013

  1. implement "sugar" wrapper method for findAll() followed by count()

    @iamjochem
    iamjochem committed Apr 10, 2013
    Copy the full SHA
    596d78f View commit details
    Browse the repository at this point in the history

  2. also drop "order" property for options passed to count() when using f… 

    …indAndCount()
    @iamjochem
    iamjochem committed Apr 10, 2013
    Copy the full SHA
    e1dbc3a View commit details
    Browse the repository at this point in the history

  3. rename Model "findAndCount" method to "findAndCountAll"

    @iamjochem
    iamjochem committed Apr 10, 2013
    Copy the full SHA
    7a7f2d6 View commit details
    Browse the repository at this point in the history

Commits on Apr 11, 2013

  1. also strip the "include" option from options given to count()

    @iamjochem
    iamjochem committed Apr 11, 2013
    Copy the full SHA
    91b2be7 View commit details
    Browse the repository at this point in the history

Commits on May 7, 2013

  1. improve option handling ("deep" clone of the *where* property) in fin… 

    …dAndCountAll() so that generating the "count" query does not break the generated "select" query (due to the query generator overwriting the options *where* property, e.g. from `["name LIKE ?", "foo%"]` to `["foo%"]`)
    @iamjochem
    iamjochem committed May 7, 2013
    Copy the full SHA
    3ef5424 View commit details
    Browse the repository at this point in the history

Commits on Jun 6, 2013

  1. Merge remote-tracking branch 'upstream/master' into feature/findAndCo… 

    …untSugar
    @iamjochem
    iamjochem committed Jun 6, 2013
    Copy the full SHA
    70f3cde View commit details
    Browse the repository at this point in the history

  2. remove debug cruft

    @iamjochem
    iamjochem committed Jun 6, 2013
    Copy the full SHA
    99ed04c View commit details
    Browse the repository at this point in the history

  3. replace custom "deep" clone cruft with lo-dash deepClone() (seeing as… 

    … we now have lo-dash as a replacement for underscore)
    @iamjochem
    iamjochem committed Jun 6, 2013
    Copy the full SHA
    b8ba4a2 View commit details
    Browse the repository at this point in the history

  4. goodbye semi-colons, I will miss you ... but you are not welcome in s… 

    …ashas house
    @iamjochem
    iamjochem committed Jun 6, 2013
    Copy the full SHA
    6d49969 View commit details
    Browse the repository at this point in the history