aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
| * Inject the visitor rather than relying on other objects internalsSean Griffin2015-12-172-2/+7
| | | | | | | | | | | | This is ultimately messy no matter what, and increases the coupling to the database backend, but we can at least contain it somewhat into an object that's already coupled.
| * Delegate to Connection Visitor in WhereSQL VisitorEdward Paget2015-12-172-1/+26
|/ | | | | | | | | | | | The WhereSQL visitor always uses the generic ToSQL visitor to create the where clause sql statement. This means that it'll miss database specific statements, such as 'ILIKE' in PostgreSQL. Since the `#where_sql` method is mainly used for ActiveRecord error reporting, this discrepancy could be confusing to users. This patch changes the WhereSQL visitor to use the its connection visitor to generate SQL for each statement in the SelectManager's wheres array. Then lets them be joined together with ' AND '.
* Merge pull request #329 from vipulnsward/add-attribute-aggregate-testsRafael França2015-12-171-1/+10
|\ | | | | Added test for verifying proper sql generated by minimum aggregate method
| * - Added test for verifying proper sql generated by minimum aggregate method.Vipul A M2014-10-241-1/+10
| |
* | Merge pull request #369 from codeodor/patch-1Rafael França2015-12-172-4/+10
|\ \ | | | | | | Improve error message when passed unsupported type
| * | Use a specific exception for unsupported visitsSammy Larbi2015-07-142-4/+10
| | |
| * | Improve error message when passed unsupported typeSammy Larbi2015-07-141-1/+1
| | |
* | | Merge pull request #355 from jemc/patch-1Rafael Mendonça França2015-12-171-4/+2
|\ \ \ | | | | | | | | | | | | Test with rbx-2 on Travis-CI
| * | | Use bundle exec on Travis-CIJoe Eli McIlvain2015-02-191-1/+1
| | | | | | | | | | | | This avoids problems with using the wrong version of gems that are also included in the Ruby standard library, like `minitest`
| * | | Test with rbx-2 on Travis-CIJoe Eli McIlvain2015-02-191-4/+1
| | | | | | | | | | | | Using `rbx-2` is the recommended way to test with the 2.0 branch of Rubinius.
* | | | Merge pull request #381 from carsonreinke/function_orderRafael França2015-12-172-0/+10
|\ \ \ \ | | | | | | | | | | Add OrderPredications back into Nodes::Function
| * | | | Add OrderPredications back into Nodes::Function (removed with deprecation of ↵Carson Reinke2015-07-232-0/+10
| | |/ / | |/| | | | | | | | | | Expression)
* | | | Merge pull request #390 from yahonda/expr_oracle12Rafael França2015-12-171-2/+2
|\ \ \ \ | | | | | | | | | | No need to quote limit for Oracle12 visitor
| * | | | No need to quote limit for Oracle12 visitorYasuo Honda2015-09-011-2/+2
| | | | |
* | | | | Merge pull request #403 from kbrock/regexpAaron Patterson2015-12-167-23/+91
|\ \ \ \ \ | | | | | | | | | | | | Case insensitivite match/regular expressions
| * | | | | test match predicate case sensitivity attributeKeenan Brock2015-12-051-0/+6
| | | | | |
| * | | | | introduce predicate {does_not_}matches_regexp Keenan Brock2015-12-052-6/+21
| | | | | |
| * | | | | use valid regular expression in regexp testKeenan Brock2015-12-051-12/+12
| | | | | |
| * | | | | Add case sensitive regexpKeenan Brock2015-12-055-4/+33
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Explicitly declare if this is case sensitive or not currently postgres assumes case insensitive regexp no other databases support regexps
| * | | | | Add case sensitive matchKeenan Brock2015-12-054-11/+29
|/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Explicitly declare if this is case sensitive or not most implementation assume case sensitive postgres assumes case insensitive
* | | | | Merge pull request #396 from ur0/masterSean Griffin2015-10-021-0/+0
|\ \ \ \ \ | | | | | | | | | | | | Rename MIT-LICENSE to MIT-LICENSE.txt
| * | | | | Rename MIT-LICENSE to MIT-LICENSE.txtUmang Raghuvanshi2015-10-021-0/+0
|/ / / / / | | | | | | | | | | | | | | | This renames MIT-LICENSE to MIT-LICENSE.txt. Closes #395, prevents builds from breaking.
* | | | | Merge pull request #392 from maclover7/contributingRafael Mendonça França2015-09-094-6/+113
|\ \ \ \ \ | | | | | | | | | | | | Add CONTRIBUTING.md
| * | | | | Add CONTRIBUTING.md [ci skip]maclover72015-09-014-6/+113
| |/ / / /
* | | | | Merge pull request #391 from yahonda/add_oracle12Rafael Mendonça França2015-09-011-1/+1
|\ \ \ \ \ | |/ / / / |/| | | | Add "lib/arel/visitors/oracle12.rb" to its gemspec
| * | | | Add "lib/arel/visitors/oracle12.rb" to its gemspecYasuo Honda2015-09-011-1/+1
|/ / / /
* | | | Use bundled gemsRafael Mendonça França2015-08-041-1/+1
| | | |
* | | | Run with Ruby 2.2Rafael Mendonça França2015-08-041-1/+2
| | | |
* | | | Improve travis configurationRafael Mendonça França2015-08-041-1/+10
|/ / / | | | | | | | | | | | | | | | | | | * Enable container builds * Notifications on Campfire * Bundle cache * Fast finish
* | | Merge pull request #337 from yahonda/support_oracle12_top_n_2Aaron Patterson2015-07-144-2/+107
|\ \ \ | | | | | | | | Support Oracle 12c top-N query
| * | | Create Arel::Visitors::Oracle12 to provide better top-N queryYasuo Honda2015-06-193-0/+101
| | | | | | | | | | | | | | | | to support `FETCH FIRST n ROWS` and `OFFSET` for Oracle 12c database
| * | | Extract visit_Arel_Nodes_SelectOptionsYasuo Honda2015-06-191-2/+6
| | | |
* | | | Merge pull request #361 from evgenim/masterAaron Patterson2015-07-142-4/+40
|\ \ \ \ | | | | | | | | | | Fix visit_Arel_Nodes_FullOuterJoin/RightOuterJoin
| * | | | Fix visit_Arel_Nodes_FullOuterJoin and visit_Arel_Nodes_RightOuterJoin to ↵evgenim2015-03-162-4/+40
| | | | | | | | | | | | | | | | | | | | make them work with collectors.
* | | | | Merge pull request #377 from Eric-Guo/masterRafael Mendonça França2015-07-132-3/+3
|\ \ \ \ \ | | | | | | | | | | | | Follow up #370 for Oracle, to fix undefined method `expr' for 10:Fixnum bug.
| * | | | | Follow up #370 for Oracle, to fix undefined method `expr' for 10:Fixnum bug.Eric Guo2015-07-142-3/+3
|/ / / / /
* | | | | Merge pull request #374 from Shopify/no-need-to-quote-mysql-limitRafael Mendonça França2015-07-101-1/+1
|\ \ \ \ \ | | | | | | | | | | | | No need to quote MySQL LIMIT
| * | | | | No need to quote MySQL LIMITJean Boussier2015-07-101-1/+1
|/ / / / /
* | | | | Merge pull request #372 from dzjuck/add_nodes_casted_to_gemspecRafael Mendonça França2015-06-241-1/+1
|\ \ \ \ \ | |_|/ / / |/| | | | Add nodes/casted.rb to gemspec file
| * | | | Add nodes/casted.rb to gemspec fileIlya Lavrov2015-06-241-1/+1
|/ / / /
* | | | Merge pull request #370 from arthurnn/dont_quote_limitAaron Patterson2015-06-181-3/+3
|\ \ \ \ | | | | | | | | | | Dont need to quote limit
| * | | | Dont need to quote limitArthur Neves2015-06-181-3/+3
|/ / / /
* | | | Merge pull request #367 from vipulnsward/move-casedRafael Mendonça França2015-06-012-40/+41
|\ \ \ \ | | | | | | | | | | Move casted to its own file
| * | | | Move casted to its own fileVipul A M2015-05-312-40/+41
|/ / / /
* | | | Merge pull request #362 from caldwecr/masterRafael Mendonça França2015-03-231-1/+1
|\ \ \ \ | |/ / / |/| | | Fix a typo in the sql literal spec for grouped "and" equality
| * | | Fix a typo in the sql literal spec for grouped "and" equalityCourtland Caldwell2015-03-231-1/+1
|/ / /
* | | Merge pull request #356 from tamird/limited-deleteRafael Mendonça França2015-02-237-5/+47
|\ \ \ | |/ / |/| | Limited delete
| * | Special limited delete handling in MSSQLTamir Duberstein2015-02-233-2/+28
| | | | | | | | | | | | | | | Refernce: https://technet.microsoft.com/en-us/library/ms175486%28v=sql.105%29.aspx
| * | Delete should respect 'limit'Bradford Folkens2015-02-235-3/+19
|/ / | | | | | | | | Conflicts: lib/arel/visitors/to_sql.rb
* | Change the interface of `having` to match that of `where`Sean Griffin2015-01-2710-29/+29
| | | | | | | | | | | | These two clauses have nearly identical semantics with regards to how they would be constructed as an AST. It doesn't make sense for their interfaces to be separate.