Code Monkey home page Code Monkey logo

Comments (10)

rip747 avatar rip747 commented on May 20, 2024

was this working before or is it something that i messed up when trying to get the oracle support working?

would you be able to write a test for this? I can then do a bisect and see which commit is the culprit that is causing the regression.

from cfwheels.

rip747 avatar rip747 commented on May 20, 2024

what database was this on?

from cfwheels.

andybellenie avatar andybellenie commented on May 20, 2024

SQL Server.

I've added a test for it here: andybellenie@76710fb

from cfwheels.

rip747 avatar rip747 commented on May 20, 2024

You are the man!
On Dec 13, 2012 8:47 AM, "Andy Bellenie" [email protected] wrote:

SQL Server.

I've added a test for it here: andybellenie@76710fbhttps://github.com/andybellenie/cfwheels/commit/76710fba179d2508c0150ac94b66b2cdd78db89c


Reply to this email directly or view it on GitHubhttps://github.com//issues/100#issuecomment-11334824.

from cfwheels.

rip747 avatar rip747 commented on May 20, 2024

looking into this, I don't see this as a bug. When performing a paginated search, the first query passed to the database is just a count seeing if there are any records. If there are no records, then there is no reason to perform the extra work to perform the second query which actually queries the database for the records.

from cfwheels.

andybellenie avatar andybellenie commented on May 20, 2024

The second query contains the column list, which is often used to drive functionality. It also means that the result of the query depends on whether or not you enable pagination.

from cfwheels.

rip747 avatar rip747 commented on May 20, 2024

to me this seems like a non-issue and logic in the findall method is a complete mess. But if you're willing to give it a shot, you have my blessing :)

from cfwheels.

andybellenie avatar andybellenie commented on May 20, 2024

I thought you'd say that :)

from cfwheels.

rip747 avatar rip747 commented on May 20, 2024

hehehe... yes I am evil 😈, but you know I'll be doing some research on this as well. I'll update the ticket with my findings.

from cfwheels.

perdjurner avatar perdjurner commented on May 20, 2024

Correct me if I am wrong but this was not an issue on 1.1.8, right?

Closing.

from cfwheels.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.