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.

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

IDEA(dbal): A set of interfaces for constructing queries #7562

Closed
wants to merge 3 commits into from

Commits on Nov 29, 2014

  1. IDEA(dbal): A set of interfaces for constructing queries

    These interfaces allow us to construct most basic queries within Elgg
    in a programmatic and safe way. Have not attempted to find out where
    the more complicated entities queries could be converted to this,
    but it does seem pretty generic and able to handle quite a range, so
    I would be remiss not to try.
    
    The major downside of all this of course is that it would be mostly custom
    implementation. The main attempt at "innovation" is using pass-by-reference
    to alias tables instead of strings. This guarantees that all aliases are unique
    and allows you to refer to columns/tables in the query by references rather
    than strings.
    ewinslow committed Nov 29, 2014
    Configuration menu
    Copy the full SHA
    5f6bf79 View commit details
    Browse the repository at this point in the history

Commits on Dec 1, 2014

  1. Renamed Storage namespace to Sql. Fleshed out some classes.

    Added some more comments to Elgg\Database\* classes to demonstrate intended conversion
    ewinslow committed Dec 1, 2014
    Configuration menu
    Copy the full SHA
    2037276 View commit details
    Browse the repository at this point in the history
  2. Several more explorations of converting plain SQL. Progress on Insert…

    …Statement (which feels like the easiest one to start with).
    ewinslow committed Dec 1, 2014
    Configuration menu
    Copy the full SHA
    257e4ef View commit details
    Browse the repository at this point in the history