aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
| * | | | | | | Correct Documentation for asset_tag_helpersAditya Kapoor2014-05-261-15/+16
| | | | | | | |
* | | | | | | | Merge pull request #15339 from sgrif/sg-mysql-booleansRafael Mendonça França2014-05-262-13/+18
|\ \ \ \ \ \ \ \ | |_|_|/ / / / / |/| | | | | | | Refactor the type casting of booleans in MySQL
| * | | | | | | Refactor the type casting of booleans in MySQLSean Griffin2014-05-262-13/+18
|/ / / / / / /
* | | | | | | Merge pull request #15336 from sgrif/sg-quotingMatthew Draper2014-05-275-54/+33
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Remove checks against `column.type` in abstract adapter quoting
| * | | | | | | Remove checks against `column.type` in abstract adapter quotingSean Griffin2014-05-265-54/+33
| |/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The intention is to eventually remove `column` from the arguments list both for `quote` and for `type_cast` entirely. This is the first step to that end.
* | | | | | | Merge pull request #15335 from sgrif/sg-missing-test-caseRafael Mendonça França2014-05-262-0/+11
|\ \ \ \ \ \ \ | |/ / / / / / |/| | | | | | Add missing test case for quoting behavior
| * | | | | | Add missing test case for quoting behaviorSean Griffin2014-05-262-0/+11
|/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It appears that the only time that `quote` is called with a column, but without first calling `type_cast` is when where is called with an array. My previous pull request broke this behavior, without failing tests. This adds a test for the only case I can think of that exercises the `if column.type == :integer` branch of `quote` effectively.
* | | | | | Merge pull request #15307 from sgrif/sg-type-cast-for-writeYves Senn2014-05-266-10/+19
|\ \ \ \ \ \ | | | | | | | | | | | | | | Add an interface for type objects to control Ruby => SQL
| * | | | | | Add an interface for type objects to control Ruby => SQLSean Griffin2014-05-266-10/+19
| |/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | Adds the ability to save custom types, which type cast to non-primitive ruby objects.
* | | | | | pg, add missing `:nodoc:` to adapter.Yves Senn2014-05-261-8/+8
| | | | | |
* | | | | | pg, remove unused code. Use `extract_schema_and_table` instead.Yves Senn2014-05-261-10/+0
| |_|_|_|/ |/| | | |
* | | | | Merge pull request #15325 from tgxworld/no_need_to_call_to_sYves Senn2014-05-262-2/+2
|\ \ \ \ \ | | | | | | | | | | | | Remove duplicated to_s method call.
| * | | | | Remove duplicated to_s method call.Guo Xiang Tan2014-05-252-2/+2
| | |/ / / | |/| | |
* | | | | Merge pull request #15316 from akshay-vishnoi/test-casesYves Senn2014-05-261-7/+8
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | Fix test cases for inflector.rb
| * | | | | Fix test cases for inflector.rbAkshay Vishnoi2014-05-251-7/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | `yield` was missing in #with_dup, due to which 57 assertions were not running in real, and all tests were passing by default. Error was introduced in commit - 31ceb5e.
* | | | | | Merge pull request #15324 from andriytyurnikov/simple_rake_stats_for_everythingYves Senn2014-05-261-18/+22
|\ \ \ \ \ \ | | | | | | | | | | | | | | simple solution that brings rake stats task to all rails project
| * | | | | | simple solution that brings rake stats task to every kind of rails projectAndriy Tyurnikov2014-05-261-18/+22
| | |/ / / / | |/| | | |
* | | | | | Merge pull request #15328 from kuldeepaggarwal/remove-warningsYves Senn2014-05-261-1/+1
|\ \ \ \ \ \ | | | | | | | | | | | | | | remove unused variable warning.
| * | | | | | remove unused variable warningKuldeep Aggarwal2014-05-261-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | `warning: assigned but unused variable - album`
* | | | | | | Fix docs for ActionController::Renderers.addGuillermo Iguaran2014-05-261-1/+1
|/ / / / / /
* | | | | | Merge pull request #15310 from tgxworld/small_refactor_get_routes_as_headCarlos Antonio da Silva2014-05-251-3/+1
|\ \ \ \ \ \ | |/ / / / / |/| | | | | Remove unnecessary flatten! method call.
| * | | | | Remove unnecessary flatten! method call.Guo Xiang Tan2014-05-251-3/+1
| | | | | |
* | | | | | Merge pull request #15315 from JuanitoFatas/guides-guidelinesSantiago Pastorino2014-05-251-1/+1
|\ \ \ \ \ \ | |_|/ / / / |/| | | | | [ci skip] Update GFM link in guides guidelines.
| * | | | | [ci skip] Update GFM link in guides guidelines.Juanito Fatas2014-05-251-1/+1
| |/ / / / | | | | | | | | | | | | | | | Also remove a duplicated sentence.
* | | | | Merge pull request #15312 from JuanitoFatas/action_view/asset_pathYves Senn2014-05-251-1/+1
|\ \ \ \ \ | | | | | | | | | | | | Convert source to string if it is present.
| * | | | | Convert source to string if it is present.Juanito Fatas2014-05-251-1/+1
| |/ / / /
* | | | | Merge pull request #15311 from zuhao/refactor_actionpack_bare_metal_testYves Senn2014-05-251-2/+2
|\ \ \ \ \ | |/ / / / |/| | | | The correct status to test should be :switching_protocols.
| * | | | The correct status to test should be :switching_protocols.Zuhao Wan2014-05-251-2/+2
|/ / / /
* | | | Merge pull request #15305 from tgxworld/remove_unnecessary_requireGuillermo Iguaran2014-05-244-5/+2
|\ \ \ \ | | | | | | | | | | Remove unnecessary require of Minitest.
| * | | | Remove unnecessary require of Minitest.Guo Xiang Tan2014-05-244-5/+2
| | | | | | | | | | | | | | | | | | | | Minitest has already been required when calling Minitest.autorun.
* | | | | unwrap the constraints object on initialization, eliminate loopsAaron Patterson2014-05-244-3/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Unwrap Constraints objects. I don't actually think it's possible to pass a Constraints object to this constructor, but there were multiple places that kept testing children of this object. I *think* they were just being defensive, but I have no idea.
* | | | | Merge pull request #15304 from JohnKellyFerguson/guides-4.1.1Xavier Noria2014-05-256-58/+61
|\ \ \ \ \ | | | | | | | | | | | | Update all Rails 4.1.0 references in guides to 4.1.1 [ci skip]
| * | | | | Update all Rails 4.1.0 references in guides to 4.1.1 [ci skip]John Kelly Ferguson2014-05-246-58/+61
| | | | | |
* | | | | | integration tests do not need to load fixtures by handXavier Noria2014-05-253-11/+0
|/ / / / / | | | | | | | | | | | | | | | | | | | | I don't know when this changed, checked back up to 3.2 and integration tests instantiate fixtures just fine.
* | | | | Merge pull request #15303 from JohnKellyFerguson/guides-ar-queryingArthur Nogueira Neves2014-05-242-16/+41
|\ \ \ \ \ | | | | | | | | | | | | Improve readability of Explain Queries table in guides
| * | | | | Improve readability of Explain Queries table in guides [ci skip]John Kelly Ferguson2014-05-242-16/+41
|/ / / / /
* | | | | Merge pull request #15299 from andriytyurnikov/engine_friendly_rake_statsRafael Mendonça França2014-05-242-19/+24
|\ \ \ \ \ | | | | | | | | | | | | rake stats for engines
| * | | | | rake stats for enginesAndriy Tyurnikov2014-05-242-19/+24
| | | | | |
* | | | | | Merge pull request #15264 from Gaurav2728/gaurav-documentation_fixRafael Mendonça França2014-05-241-2/+2
|\ \ \ \ \ \ | |_|_|_|/ / |/| | | | | documentation link fix and removing unused reader
| * | | | | documentation fixGaurav Sharma2014-05-241-2/+2
|/ / / / /
* | | | | Merge pull request #15298 from ShunsukeAida/fix_capitalizationRafael Mendonça França2014-05-241-1/+1
|\ \ \ \ \ | | | | | | | | | | | | fixed capitalization [ci skip]
| * | | | | fixed capitalizationShunsukeAida2014-05-251-1/+1
| | | | | |
* | | | | | Add FIXME note about the thor bugRafael Mendonça França2014-05-241-0/+2
| | | | | |
* | | | | | Merge pull request #9083 from bronson/patch-1Rafael Mendonça França2014-05-241-1/+4
|\ \ \ \ \ \ | |/ / / / / |/| | | | | | | | | | | avoid using alias in generators
| * | | | | avoid using alias in generatorsScott Bronson2013-01-251-1/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Ruby's alias produces public methods, causing a spurious Thor task to be created.  For example, this is the reason MigrationGenerator currently has two tasks:     > ActiveRecord::Generators::MigrationGenerator.all_tasks.keys     => ["singular_name", "create_migration_file"] singular_name was meant to be an attribute, not a task.  Because it's public, it gets called as a task every time the generator is invoked. The fix is to ensure all generator methods have the correct visibility.
* | | | | | Merge pull request #13988 from zzak/guides_configuring_reverse_proxyZachary Scott2014-05-241-4/+38
|\ \ \ \ \ \ | |_|_|_|_|/ |/| | | | | [Guides] Add documentation for using Rails with a reverse proxy [ci skip]
| * | | | | Add documentation for using Rails with a reverse proxyZachary Scott2014-05-241-4/+38
| | | | | |
* | | | | | Merge pull request #14986 from dlangevin/trailing-slash-url-generationRafael Mendonça França2014-05-243-7/+37
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes URL generation with trailing_slash: true Conflicts: actionpack/lib/action_dispatch/http/url.rb
| * | | | | | Fixes URL generation with trailing_slash: trueDan Langevin2014-05-223-9/+40
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | URL generation with trailing_slash: true was adding a trailing slash after .:format Routes.draw do resources :bars end bars_url(trailing_slash: true, format: 'json') # => /bars.json/ This commit removes that extra trailing slash
* | | | | | | pg, remove duplicated code.Yves Senn2014-05-241-4/+0
| | | | | | |
generated by cgit v1.2.3 (git 2.25.1) at 2025-03-24 13:48:40 +0000