Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Reuse `maybe_visit` method | claudiob | 2014-10-23 | 2 | -18/+4 |
| | | | | | This commit simply removes duplicated code by reusing the existing `maybe_visit` method. | ||||
* | {Matches,DoesNotMatch} support the ESCAPE clause | Tamir Duberstein | 2014-09-25 | 5 | -20/+46 |
| | |||||
* | move the dispatch table to be per-instance | Aaron Patterson | 2014-09-22 | 6 | -19/+37 |
| | | | | | | | visitors are not shared among threads, so any mutations to the cache should be OK. The cache is also pre-populated on construction, but we should pull that out so we can share the cache among visitors in the future. | ||||
* | connections should cache these values, so remove the cache in arel | Aaron Patterson | 2014-09-22 | 1 | -4/+3 |
| | |||||
* | Merge pull request #167 from pparidans/informix-v9-support | Rafael Mendonça França | 2014-09-17 | 1 | -1/+1 |
|\ | | | | | Informix versions < 10 use 'FIRST' keyword instead of 'LIMIT' | ||||
| * | Informix versions < 10 use 'FIRST' keyword instead of 'LIMIT' | Pierre Paridans | 2014-09-16 | 1 | -1/+1 |
| | | | | | | | | Still supported in versions 10+ | ||||
* | | Merge pull request #137 from dimko/visit_set | Rafael Mendonça França | 2014-09-17 | 3 | -0/+3 |
|\ \ | |/ |/| | Added a visitor for Set objects | ||||
| * | added a visitor for Set objects | Dimko | 2012-09-18 | 3 | -0/+3 |
| | | |||||
* | | Remove unused @alias, being referenced in hashing. | Vipul A M | 2014-09-16 | 1 | -1/+1 |
| | | |||||
* | | Wrap group_by sql with a subquery for limiting rows in Oracle. | Chris Dinger | 2014-09-16 | 1 | -1/+1 |
| | | | | | | | | Fixes #128 | ||||
* | | `Extract#as` should not mutate the receiver | Tamir Duberstein | 2014-09-13 | 2 | -18/+4 |
| | | | | | | | | Fixes https://github.com/rails/rails/issues/16913 | ||||
* | | Merge pull request #306 from mxhold/master | Rafael Mendonça França | 2014-09-12 | 1 | -0/+9 |
|\ \ | | | | | | | Add SelectManager#distinct_on to set/unset Arel::Nodes::DistinctOn quantifier | ||||
| * | | Add SelectManager#distinct_on to set/unset Arel::Nodes::DistinctOn quantifier | Max Holder | 2014-08-30 | 1 | -0/+9 |
| | | | |||||
* | | | Merge pull request #304 from bigxiang/fix_lt_and_lteq_error | Rafael Mendonça França | 2014-09-12 | 1 | -6/+6 |
|\ \ \ | | | | | | | | | Fix lt & lteq don't accept most of values. | ||||
| * | | | Fix lt & lteq don't accept most of values. | bigxiang | 2014-08-23 | 1 | -6/+6 |
| | | | | |||||
* | | | | Merge pull request #277 from sergey-alekseev/wrap-nested-gropings-only-once | Rafael Mendonça França | 2014-09-12 | 1 | -2/+6 |
|\ \ \ \ | |_|/ / |/| | | | Wrap nested Nodes::Grouping in brackets only once | ||||
| * | | | Wrap nested Nodes::Grouping in brackets only once | Sergey Alekseev | 2014-05-26 | 1 | -2/+6 |
| | | | | |||||
* | | | | Remove default aliases from aggregate functions | Joshua Cody | 2014-07-31 | 1 | -14/+8 |
| | | | | |||||
* | | | | Allow for alias omission in aggregate expressions | Joshua Cody | 2014-07-26 | 1 | -8/+15 |
| |/ / |/| | | |||||
* | | | Windowing: Calling #rows or #range should assign framing only once. | Alexander Staubo | 2014-06-22 | 2 | -2/+12 |
| | | | |||||
* | | | Windowing support for PARTITION BY clause. | Alexander Staubo | 2014-06-20 | 3 | -2/+21 |
| | | | |||||
* | | | Modify Visitors::Dot's API to use collectors | Jiri Pospisil | 2014-06-14 | 2 | -3/+5 |
| | | | |||||
* | | | Collectors: Introduce PlainString and make use of it in SQLString | Jiri Pospisil | 2014-06-14 | 2 | -14/+21 |
| | | | |||||
* | | | Merge pull request #290 from mekishizufu/fix_select_manager_as_subquery | Rafael Mendonça França | 2014-06-13 | 1 | -1/+1 |
|\ \ \ | | | | | | | | | Support passing of SelectManager to attribute methods | ||||
| * | | | No need to check for SelectStatement as it's a descendant of Node | Jiri Pospisil | 2014-06-12 | 1 | -1/+1 |
| | | | | |||||
| * | | | Make sure Arel::SelectManager is not mistaken for an attribute | Jiri Pospisil | 2014-06-12 | 1 | -1/+1 |
| | | | | |||||
* | | | | Make sure visit_Arel_Nodes_With works with collectors | Jiri Pospisil | 2014-06-12 | 1 | -2/+3 |
|/ / / | |||||
* | | | Merge pull request #284 from Octember/master | Rafael Mendonça França | 2014-06-06 | 1 | -18/+22 |
|\ \ \ | | | | | | | | | Cleaned up some edge cases with infinity in predications.rb | ||||
| * | | | Cleaned up some edge cases with infinity, the logic seems more intuitive now | Noah Lindner | 2014-06-01 | 1 | -18/+22 |
| | | | | |||||
* | | | | Merge pull request #281 from vipulnsward/remove-warning | Rafael Mendonça França | 2014-06-02 | 1 | -5/+1 |
|\ \ \ \ | | | | | | | | | | | - Remove four year old deprecation | ||||
| * | | | | - Remove four year old deprecation | Vipul A M | 2014-06-01 | 1 | -5/+1 |
| | | | | | |||||
* | | | | | Merge pull request #283 from vipulnsward/rm-values | Carlos Antonio da Silva | 2014-06-01 | 1 | -2/+0 |
|\ \ \ \ \ | |_|/ / / |/| | | | | Remove unused values variable | ||||
| * | | | | Remove unused values variable | Vipul A M | 2014-06-01 | 1 | -2/+0 |
| |/ / / | |||||
* / / / | Arel doesn't depend on activesupport and doesn't have "String#blank?" | Vipul A M | 2014-06-01 | 2 | -2/+2 |
|/ / / | | | | | | | | | | Remove usage of `.blank?` to match empty strings with a regular expression instead. | ||||
* | | | Merge pull request #276 from brocktimus/master | Rafael Mendonça França | 2014-05-28 | 1 | -1/+1 |
|\ \ \ | | | | | | | | | Fixing method signature for unsupported visits in the ToSQL Visitor | ||||
| * | | | Fixing method signature for unsupported visits in the ToSQL Visitor | Brock Trappitt | 2014-05-26 | 1 | -1/+1 |
| |/ / | |||||
* / / | Fix typo `delegte` => `delegate` | Vipul A M | 2014-05-25 | 1 | -1/+1 |
|/ / | |||||
* | | Merge pull request #271 from jordansexton/patch-2 | Rafael Mendonça França | 2014-05-19 | 1 | -0/+8 |
|\ \ | | | | | | | Added update and delete convenience methods | ||||
| * | | Added #update_manager and #delete_manager convenience methods for consistency | Jordan Sexton | 2014-05-17 | 1 | -0/+8 |
| | | | |||||
* | | | Merge pull request #272 from jduff/rebased_outer_join | Rafael Mendonça França | 2014-05-19 | 2 | -0/+8 |
|\ \ \ | | | | | | | | | Add convenience #outer_join | ||||
| * | | | convenience method for outer_join | Brian Hahn | 2014-05-18 | 2 | -0/+8 |
| |/ / | |||||
* | | | Merge pull request #267 from jordansexton/patch-1 | Rafael Mendonça França | 2014-05-19 | 1 | -0/+1 |
|\ \ \ | |/ / |/| | | Return self from distinct so it's chainable | ||||
| * | | Return self from #distinct so it's chainable | Jordan Sexton | 2014-05-17 | 1 | -0/+1 |
| | | | |||||
* | | | Remove old FIXME comment | Arthur Neves | 2014-05-07 | 1 | -3/+0 |
|/ / | | | | | | | The comment was introducted on https://github.com/rails/arel/commit/d8de55cee197d887b478b134ec692776613bf998 , and the code has changed | ||||
* | | Support to add the casting node to the AST at build time | Yasuo Honda | 2014-05-05 | 1 | -1/+1 |
| | | | | | | | | Ref https://github.com/rails/arel/commit/93d72131bcc24ccb5536bec672d2dac94f8de651 | ||||
* | | Add collector argument to address ArgumentError: wrong number of arguments ↵ | Yasuo Honda | 2014-05-02 | 1 | -1/+1 |
| | | | | | | | | (2 for 1) | ||||
* | | always add a space before keywords (it is easier) | Aaron Patterson | 2014-04-11 | 1 | -9/+3 |
| | | |||||
* | | Allow INSERT INTO <table> SELECT queries | Jean Boussier | 2014-04-10 | 3 | -3/+16 |
| | | |||||
* | | fix spacing so that Rails is happy | Aaron Patterson | 2014-04-09 | 1 | -3/+9 |
| | | |||||
* | | rhs bind parameters should be visited | Aaron Patterson | 2014-04-09 | 1 | -1/+1 |
| | |