Commit message (Collapse) | Author | Age | Files | Lines | ||
---|---|---|---|---|---|---|
... | ||||||
* | | Merge pull request #444 from alassek/dot-visitor-bind-param | Aaron Patterson | 2016-08-26 | 2 | -1/+8 | |
|\ \ | |/ |/| | Fix invalid BindParam output in Dot visitor | |||||
| * | Change BindParam visit method for Dot to a noop | Adam Lassek | 2016-08-26 | 1 | -1/+2 | |
| | | | | | | | | | | | | Since BindParam contains no information, treating it like a string adds no useful information to the graph, and results in an invalid label format. | |||||
| * | Add failing test for Dot visitor and BindParam | Adam Lassek | 2016-08-26 | 1 | -0/+6 | |
|/ | | | | | | | | | Since BindParam has no value, treating it like a string causes it to fallback to Object#to_s, leading to output like `#<Arel::Nodes::BindParam:0x007fa43c866108>`. Since angle brackets are significant in Dot labels, this causes `Error: bad label format` when passing the graph into dot. | |||||
* | Release 7.1.1 | Rafael Mendonça França | 2016-07-27 | 2 | -1/+7 | |
| | ||||||
* | Fix Casted#hash | Matthew Draper | 2016-07-28 | 1 | -1/+1 | |
| | | | | There is no @class variable. | |||||
* | Release 7.1.0 | Rafael Mendonça França | 2016-07-19 | 2 | -2/+4 | |
| | ||||||
* | Merge pull request #417 from tlkahn/master | Rafael França | 2016-07-19 | 2 | -0/+16 | |
|\ | | | | | Fix issue #415 - Should Arel::Nodes::True.new() be 1 in sqlite3? | |||||
| * | Fix issue #415 - Should Arel::Nodes::True.new() be 1 in sqlite3? | Yong Guo | 2016-03-01 | 2 | -0/+16 | |
| | | ||||||
* | | Merge pull request #328 from vipulnsward/dryup-collectors | Rafael França | 2016-07-19 | 1 | -33/+13 | |
|\ \ | | | | | | | DRY up visit_Arel_Nodes_SelectCore | |||||
| * | | DRY up visit_Arel_Nodes_SelectCore and extract nodes collection to ↵ | Vipul A M | 2016-04-11 | 1 | -33/+13 | |
| | | | | | | | | | | | | collect_nodes_for, for collecting wheres, projections, groups, windows | |||||
* | | | Merge pull request #427 from vipulnsward/disable-jruby-travis | Rafael França | 2016-07-19 | 1 | -0/+1 | |
|\ \ \ | | | | | | | | | - Disable jruby-head since bundle fetching is failing on travis | |||||
| * | | | - Disable jruby-head since bundle fetching is failing on travis. | Vipul A M | 2016-04-11 | 1 | -0/+1 | |
| |/ / | ||||||
* | | | Bump to 7.0.1.pre in anticipation of next release | Jeremy Daer | 2016-05-29 | 2 | -1/+7 | |
| | | | | | | | | | | | | [ci skip] | |||||
* | | | Merge pull request #437 from yui-knk/fix_head | Jeremy Daer | 2016-05-29 | 3 | -0/+3 | |
|\ \ \ | | | | | | | | | | | | | Support for unified Integer class in Ruby 2.4+ | |||||
| * | | | Support for unified Integer class in Ruby 2.4+ | yui-knk | 2016-05-30 | 3 | -0/+3 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Ruby 2.4 unifies Fixnum and Bignum into Integer: https://bugs.ruby-lang.org/issues/12005 Ruby ~2.3 `1234.class` is `Fixnum` and `123456789012345678901234567890.class` is `Bignum`. Ruby 2.4+ `1234.class` is `Integer` and `123456789012345678901234567890.class` is `Integer`. So what we should do is defining `visit_Integer` method to visitors. | |||||
* | | | | CI: bump Ruby 2.2.4->2.2.5, 2.3.0->2.3.1 | Jeremy Daer | 2016-05-29 | 1 | -2/+2 | |
| | | | | ||||||
* | | | | CI: test JRuby 9.0.5.0. Allow failures due to flaky Travis bundler situation. | Jeremy Daer | 2016-05-29 | 1 | -0/+3 | |
|/ / / | ||||||
* | | | Merge pull request #422 from yahonda/oracle12_support_bind_value | Sean Griffin | 2016-05-06 | 2 | -2/+17 | |
|\ \ \ | | | | | | | | | Support Oracle bind parameter value for Oracle12 visitor | |||||
| * | | | Support Oracle bind parameter value for Oracle12 visitor | Yasuo Honda | 2016-04-11 | 2 | -2/+17 | |
| |/ / | ||||||
* | | | Merge pull request #432 from tradegecko/pg-grouping-elements-support | Rafael França | 2016-05-05 | 5 | -2/+128 | |
|\ \ \ | | | | | | | | | PG: Add Support for GroupBy's Cube, Rollup and Grouping Set | |||||
| * | | | Add Support for GroupBy Cube, Rollup and Grouping Set Syntax for PostgreSQL ↵ | Mohammad Habbab | 2016-05-04 | 5 | -2/+128 | |
| | | | | | | | | | | | | | | | | Visitor | |||||
* | | | | Merge pull request #430 from yahonda/oracle12_limit_and_lock_2 | Sean Griffin | 2016-05-04 | 2 | -6/+9 | |
|\ \ \ \ | |/ / / |/| | | | Raise ArgumentError if limit and lock are used for Oracle12 visitor | |||||
| * | | | Raise ArgumentError if limit and lock are used for Oracle12 visitor | Yasuo Honda | 2016-05-02 | 2 | -6/+9 | |
|/ / / | | | | | | | | | | | | | | | | | | | it would generates `SELECT ... FETCH FIRST n ROWS ONLY FOR UPDATE` which causes Oracle 12c database returns this error : ORA-02014: cannot select FOR UPDATE from view with DISTINCT, GROUP BY, etc. | |||||
* | | | Merge pull request #429 from zhufenggood/master | Aaron Patterson | 2016-04-14 | 1 | -3/+3 | |
|\ \ \ | | | | | | | | | Update to_sql.rb. More slightly performance improvement. | |||||
| * | | | Update to_sql.rb | zhufenggood | 2016-04-15 | 1 | -3/+3 | |
|/ / / | | | | | | | Update to_sql.rb. Slightly performance improvement. | |||||
* | | | Merge pull request #425 from vipulnsward/fix-warnings | Rafael França | 2016-04-11 | 1 | -1/+1 | |
|\ \ \ | |/ / |/| | | Fix warnings from test_to_sql test | |||||
| * | | Fix warnings from test_to_sql test | Vipul A M | 2016-04-11 | 1 | -1/+1 | |
|/ / | ||||||
* | | Merge pull request #420 from zhufenggood/master | Rafael França | 2016-03-30 | 1 | -1/+1 | |
|\ \ | |/ |/| | Update to_sql.rb. Slightly performance improment. | |||||
| * | Update to_sql.rb. Slightly performance improment. | zhufenggood | 2016-03-31 | 1 | -1/+1 | |
|/ | | | Update to_sql.rb. Slightly performance improment. | |||||
* | Merge pull request #416 from jeffcole/clean-up-readme-code | Rafael França | 2016-03-01 | 1 | -35/+75 | |
|\ | | | | | Clean up README code formatting and grammar | |||||
| * | Fix grammar in README | Jeff Cole | 2016-02-28 | 1 | -9/+9 | |
| | | | | | | | | | | | | * Fixes grammatical errors * Fixes capitalization * Fixes punctuation | |||||
| * | Clean up README code | Jeff Cole | 2016-02-28 | 1 | -26/+66 | |
|/ | | | | | * Fixes syntax highlighting of the output from `SqlLiteral` * Adds whitespace to improve readability | |||||
* | Merge pull request #414 from bertBruynooghe/master | Ernie Miller | 2016-02-17 | 2 | -0/+20 | |
|\ | | | | | added Casted#hash | |||||
| * | added Casted#hash | Bert Bruynooghe | 2016-02-17 | 2 | -0/+20 | |
|/ | ||||||
* | Merge pull request #412 from yui-knk/remove_not_used_codes | Sean Griffin | 2016-02-06 | 1 | -23/+0 | |
|\ | | | | | Delete not used codes | |||||
| * | Delete not used codes | yui-knk | 2016-02-06 | 1 | -23/+0 | |
|/ | | | | | These codes were introduced by 03724fb1789198cc394f6e8b69cf9404e03eddd7, and to be not used by 79411322ae225289e1c051f4f68ed84c6349e4a0. | |||||
* | Test with Ruby 2.3 and allow failures with ruby-head | Rafael Mendonça França | 2016-01-06 | 1 | -1/+4 | |
| | ||||||
* | Merge pull request #408 from sjaveed/bitwise_operations | Rafael França | 2016-01-06 | 9 | -1/+173 | |
|\ | | | | | Support for Bitwise Operations as InfixOperations | |||||
| * | * Support for bitwise operations as infix operators. Tests included. | Shahbaz Javeed | 2016-01-06 | 9 | -1/+173 | |
|/ | | | | | | | | | | | | | | | | | | | | | | *** Individual commit messages included below *** * Preliminary support for bitwise operations as infix operators. Tests to follow. * Added bitwise xor, shift left and shift right operators * Fixed the BitwiseOr class so it uses the :| operator instead of :& * All the methods for the bitwise operators in the Arel::Math module now wrap them up in Arel::Nodes::Grouping so the operation becomes isolated like addition and subtraction * Preliminary set of tests for the new operators * Updated README with examples of bitwise operations * Added a new UnaryOperation class which is a riff on the InfixOperation class * Added tests for UnaryOperation (inspired by InfixOperation tests) * Added the bitwise not (~) operator as a UnaryOperation * Added tests for the bitwise not operator * Added documentation for the bitwise not operator * Updated gemspec using `rake arel.gemspec` | |||||
* | Merge pull request #410 from kbrock/concat | Rafael França | 2016-01-06 | 8 | -3/+47 | |
|\ | | | | | Add database specific string concatenation | |||||
| * | Add database specific string concatenation | Keenan Brock | 2016-01-05 | 8 | -3/+47 | |
|/ | ||||||
* | Merge pull request #400 from felixbuenemann/case-node | Rafael França | 2016-01-05 | 8 | -1/+256 | |
|\ | | | | | Implement CASE Conditional Expression | |||||
| * | Implement CASE Conditional Expression | Felix Bünemann | 2016-01-05 | 8 | -1/+256 | |
|/ | ||||||
* | Merge pull request #409 from jeroenvisser101/patch-1 | Guillermo Iguaran | 2015-12-31 | 1 | -1/+1 | |
|\ | | | | | Update copyright year | |||||
| * | Update copyright year | Jeroen Visser | 2016-01-01 | 1 | -1/+1 | |
|/ | | | New year, new license! | |||||
* | Merge pull request #407 from tadeusrox/rename_markdown | Rafael França | 2015-12-29 | 4 | -6/+6 | |
|\ | | | | | Rename README.markdown to README.md | |||||
| * | Rename README.markdown to README.md | Vinicius Tadeu | 2015-12-27 | 4 | -6/+6 | |
|/ | ||||||
* | Release 7.0.0 | Rafael Mendonça França | 2015-12-17 | 3 | -2/+10 | |
| | ||||||
* | Merge pull request #349 from edpaget/delegate-where-sql-visitor | Sean Griffin | 2015-12-17 | 3 | -2/+32 | |
|\ | | | | | | | Delegate to Connection Visitor in WhereSQL Visitor | |||||
| * | Inject the visitor rather than relying on other objects internals | Sean Griffin | 2015-12-17 | 2 | -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. |