aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* to_sql is passingAaron Patterson2014-04-082-120/+160
|
* fixing some tests in to_sqlAaron Patterson2014-04-083-81/+133
|
* bind visitor working with collector objectAaron Patterson2014-04-087-19/+49
|
* initial commit for collector implementationAaron Patterson2014-03-251-0/+26
|
* Merge branch 'rm_a' into ww2lin-rm_aAaron Patterson2014-03-252-2/+2
|\ | | | | | | | | * rm_a: take in to account quoted and translated values
| * take in to account quoted and translated valuesAaron Patterson2014-03-252-2/+2
| |
* | remove the default parameter and updated commentAlex Lin2014-03-252-3/+3
| |
* | removed the attribute that was left in for testingAlex Lin2014-03-251-1/+1
| |
* | Removed all the fiels in lib/arel/visitors/ which needs dependency on 'a' ↵Alex Lin2014-03-2416-381/+368
|/ | | | also fixed the test case for : test/visitors/test_to_sql.rb:22 which pass in the parameter attribute e.g the parameter a.
* build quoted nodes in factory methodsAaron Patterson2014-03-243-15/+24
|
* build quoted stringsAaron Patterson2014-03-243-10/+10
|
* add the casting node to the AST at build timeAaron Patterson2014-03-2420-63/+166
| | | | | If we add the casting node to the ast at build time, then we can avoid doing the lookup at visit time.
* Merge pull request #246 from cmohanprasath/assign_value_from_attribute_testAaron Patterson2014-02-242-2/+18
|\ | | | | Assign value from attribute test
| * Test case for Issue No 242.cmohanprasath2014-02-181-0/+11
| | | | | | | | The evaluates the assignment of two unqualified columns.
| * Allow assignment right member to reference columnsLoic Nageleisen2014-02-111-2/+7
| | | | | | | | | | | | | | | | | | | | | | | | Such queries become possible for mortals: ```ruby table.where(table[:qux].eq 'zomg').compile_update(table[:foo] => table[:bar]) ``` Arguably much more sane than using `engine.connection.quote_column_name` or `engine.connection.visitor.accept` on the right hand (bar), which is totally leaking the abstraction.
* | Merge pull request #247 from zenspider/fix-readmeAaron Patterson2014-02-181-2/+4
|\ \ | | | | | | fixed the readme per discussion w/ tenderlove
| * | fixed the readme per discussion w/ tenderloveRyan Davis2014-02-181-2/+4
|/ /
* | Merge branch 'remove-deprecated'Rafael Mendonça França2014-02-1020-104/+39
|\ \ | |/ |/|
| * Update historyRafael Mendonça França2014-02-103-16/+24
| |
| * Remove deprecated Arel::ExpressionRafael Mendonça França2014-02-107-31/+0
| |
| * Remove deprecated Arel::SqlLiteralRafael Mendonça França2014-02-109-27/+15
| |
| * Remove deprecated constant aliasesRafael Mendonça França2014-02-102-5/+0
| |
| * Remove deprecated `SelectManager#joins`Vipul A M2014-02-101-8/+0
| |
| * Remove deprecated `SelectManager#to_a`Vipul A M2014-02-101-6/+0
| |
| * Removed deprecated `Arel::Sql::Engine` and make use of `FakeRecord::Base` ↵Vipul A M2014-02-103-12/+1
|/ | | | inplace directly for test helper.
* Merge pull request #244 from vanderhoorn/masterRafael Mendonça França2014-02-081-10/+25
|\ | | | | Add tests for Nodes::Equality and Nodes::NotEqual to the ToSql class.
| * Add tests for Nodes::Equality and Nodes::NotEqual to the ToSql class.Roel van der Hoorn2014-02-071-10/+25
| |
* | Merge pull request #237 from vipulnsward/extract-arrayRafael Mendonça França2014-02-081-2/+4
|\ \ | |/ |/| Extract comparison array to a constant for the time being until removing the checks completely.
| * Extract comparison array to a constant for the time being until removing the ↵Vipul A M2014-01-071-2/+4
| | | | | | | | check completely.
* | Merge pull request #232 from vipulnsward/bump-minitestRafael Mendonça França2014-02-0512-12/+12
|\ \ | | | | | | | | | Bump minitest to 5.1.
| * | Bump minitest to 5.1.Vipul A M2014-01-0412-12/+12
| | |
* | | Allow failures on rubiniusRafael Mendonça França2014-02-051-0/+3
| | |
* | | Use rbx on travisRafael Mendonça França2014-02-051-1/+1
| | |
* | | Sync CHANGELOGRafael Mendonça França2014-02-051-0/+7
| | |
* | | Merge pull request #243 from vanderhoorn/patch-1Rafael Mendonça França2014-02-053-0/+72
|\ \ \ | | | | | | | | PostgreSQL bugfix for invalid SQL in subqueries
| * | | Add tests for PostgreSLQ bugfix regarding invalid SQL in subqueries when ↵Roel van der Hoorn2014-02-052-0/+70
| | | | | | | | | | | | | | | | using matches() or does_not_match().
| * | | PostgreSQL bugfix for invalid SQL in subqueriesvanderhoorn2014-02-051-0/+2
|/ / / | | | | | | | | | | | | | | | | | | In commit 68a95542e1a7a79d9777223fbffd2b982fed0268 the last_column feature of ToSql was removed. The visit_Arel_Nodes_Matches and visit_Arel_Nodes_DoesNotMatch methods are overwritten in the PostgreSQL class, but were not updated appropriately. This commit fixes the issue accordingly. This bug affects at least all update_all statements in Rails 4.0.2 that have subqueries with ILIKE statements on PostgreSQL. The bug is present in Arel 4.0.1 and later, so it probably affects most Rails 4.0.2 projects. It would be highly appreciated if Arel 4 could get a point release as well. Thanks for your continued work.
* | / add bind values to the manager classAaron Patterson2014-01-142-0/+10
| |/ |/|
* | Merge pull request #234 from aackerman/remove-test-engine-proxyRafael Mendonça França2014-01-041-68/+8
|\ \ | | | | | | Removed EngineProxy in SelectManager tests
| * | Removed EngineProxy in SelectManager testsAaron Ackerman2014-01-041-68/+8
| |/
* | Merge pull request #231 from vipulnsward/travis-2-1Rafael Mendonça França2014-01-041-1/+1
|\ \ | | | | | | Test against ruby 2.1 on travis.
| * | Test against ruby 2.1 on travis.Vipul A M2014-01-041-1/+1
| |/
* | Merge pull request #233 from aackerman/fix-select-manager-project-testRafael Mendonça França2014-01-041-4/+2
|\ \ | |/ |/| Assert that SelectManager#project accepts a raw String
| * Assert that SelectManager#project accepts a raw StringAaron Ackerman2014-01-041-4/+2
|/
* Merge pull request #229 from aackerman/readme-inaccuracyRafael Mendonça França2014-01-021-6/+0
|\ | | | | Remove README inaccuracy, where does not allow multiple arguments
| * Remove README inaccuracy, where does not allow multiple argumentsAaron Ackerman2014-01-021-6/+0
|/
* Merge pull request #226 from dogweather/masterRafael Mendonça França2013-12-061-0/+4
|\ | | | | Documented the case-insensitive nature of the match feature.
| * Documented the case-insensitive nature of the match feature.Robb Shecter2013-12-051-0/+4
|/ | | Documenting the library's behavior. People and other libraries (cf. Squeel) have come to depend on this behavior, and so am submitting this pull request to specify it authoritatively.
* Test on 2.1.0 previewRafael Mendonça França2013-12-051-0/+1
|
* Remove joins methodRafael Mendonça França2013-12-051-8/+0
|