Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | fix exception on polymorphic associations with predicates | Simon Woker | 2014-04-11 | 2 | -0/+16 |
| | |||||
* | Fix error for aggregate methods with select, see issue #13648 | Simon Woker | 2014-04-10 | 3 | -3/+19 |
| | |||||
* | Merge pull request #14679 from pbharadiya/master | Yves Senn | 2014-04-10 | 1 | -1/+1 |
|\ | | | | | Correct factual error for page not found [ci skip] | ||||
| * | Correct factual error for page not found | Parth Bharadiya | 2014-04-10 | 1 | -1/+1 |
|/ | |||||
* | +EOL | Akira Matsuda | 2014-04-10 | 1 | -1/+1 |
| | |||||
* | Merge pull request #11411 from dscataglini/master | Aaron Patterson | 2014-04-09 | 1 | -6/+12 |
|\ | | | | | Active support callback's before/after/around filters are not correctly making their singleton methods private | ||||
| * | private method doesn't work for singleton methods defined this way | Diego Scataglini | 2013-07-12 | 1 | -6/+12 |
| | | |||||
* | | Only call uniq on the conditional that actually needs it | Carlos Antonio da Silva | 2014-04-09 | 1 | -1/+1 |
| | | |||||
* | | Remove extra collect call | Carlos Antonio da Silva | 2014-04-09 | 1 | -1/+1 |
| | | |||||
* | | Merge pull request #14675 from ↵ | Aaron Patterson | 2014-04-09 | 2 | -7/+7 |
|\ \ | | | | | | | | | | | | | laurocaetano/make_reflection_caches_works_with_string_keys Make reflection and aggregate_reflection caches work with string keys. | ||||
| * | | Make sure the reflection test is passing a String to the reflection | Lauro Caetano | 2014-04-09 | 1 | -1/+1 |
| | | | | | | | | | | | | cache. | ||||
| * | | Make the aggregate_reflections cache work with strings as its keys. | Lauro Caetano | 2014-04-09 | 1 | -2/+2 |
| | | | |||||
| * | | No need to call `to_sym` on reflection name, since the cache now works | Lauro Caetano | 2014-04-09 | 1 | -2/+2 |
| | | | | | | | | | | | | | | | | | | with strings with string keys. Related #14668. | ||||
| * | | Make the reflections cache work with strings as its keys. | Lauro Caetano | 2014-04-09 | 1 | -2/+2 |
|/ / | |||||
* | | Merge pull request #14671 from arthurnn/travis_memcache | Rafael Mendonça França | 2014-04-09 | 1 | -0/+1 |
|\ \ | | | | | | | add services: memcache to travis | ||||
| * | | add services: memcache to travis | Arthur Neves | 2014-04-09 | 1 | -0/+1 |
|/ / | |||||
* | | Merge pull request #14668 from laurocaetano/source_option_with_string_value | Aaron Patterson | 2014-04-09 | 2 | -3/+3 |
|\ \ | | | | | | | The :source option for has_many => through should accept String values. | ||||
| * | | The `source` option for `has_many => through` should accept String | Lauro Caetano | 2014-04-09 | 2 | -3/+3 |
|/ / | | | | | | | | | | | | | | | values. With the changes introduced by 16b70fddd4dc7e7fb7be108add88bae6e3c2509b it was expecting the value to be a Symbol, while it could be also a String value. | ||||
* | | Merge pull request #14665 from andrielfn/avoid-uri-parsing | Rafael Mendonça França | 2014-04-09 | 2 | -5/+15 |
|\ \ | | | | | | | Avoid URI parsing | ||||
| * | | Avoid URI parsing | Andriel Nuernberg | 2014-04-09 | 2 | -5/+15 |
| | | | | | | | | | | | | | | | This parsing is unecessary once the Request object already has the needed information. | ||||
* | | | Merge pull request #14667 from lucasmazza/require-per-thread-registry | Rafael Mendonça França | 2014-04-09 | 1 | -0/+1 |
|\ \ \ | | | | | | | | | Add missing require so requiring `active_support/cache` works again. | ||||
| * | | | Add missing require so requiring `active_support/cache` works again. | Lucas Mazza | 2014-04-09 | 1 | -0/+1 |
| |/ / | |||||
* / / | depend_on_asset is not required anymore on sprockets-rails 2.1.2 | Rafael Mendonça França | 2014-04-09 | 3 | -25/+11 |
|/ / | |||||
* | | Merge pull request #14654 from Alamoz/secret_key | Rafael Mendonça França | 2014-04-09 | 1 | -4/+7 |
|\ \ | | | | | | | Expand explanation of how to set secrets.yml. | ||||
| * | | Expand explanation of how to set secrets.yml. [ci skip] | Adrien Lamothe | 2014-04-08 | 1 | -4/+7 |
| | | | |||||
* | | | upgrading section for 4.1 is no longer WIP. [ci skip] | Yves Senn | 2014-04-09 | 1 | -2/+0 |
| | | | |||||
* | | | Merge pull request #14653 from vdmgolub/patch-1 | Arun Agrawal | 2014-04-09 | 1 | -1/+1 |
|\ \ \ | | | | | | | | | Guides: minor typo fixed in Asset Pipeline guide | ||||
| * | | | Guides: minor typo fixed [ci skip] | Vadim Golub | 2014-04-09 | 1 | -1/+1 |
| |/ / | |||||
* | | | Merge pull request #14652 from Emily/duration_eqls | Guillermo Iguaran | 2014-04-08 | 3 | -0/+45 |
|\ \ \ | |/ / |/| | | Fixed problem where `1.day.eql?(1.day)` is false | ||||
| * | | Fixed problem where `1.day.eql?(1.day)` is false | Emily Dobervich | 2014-04-08 | 3 | -0/+45 |
|/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This fixes: 1.second.eql?(1.second) #=> false The new `eql?` requires that `other` is an `ActiveSupport::Duration`. This requirement makes `ActiveSupport::Duration`'s behavior consistent with other numeric types in Ruby. 1.eql?(1.0) #=> false 1.0.eql?(1) #=> false 1.second.eql?(1) #=> false (was true) 1.eql?(1.second) #=> false { 1 => "foo", 1.0 => "bar" } #=> { 1 => "foo", 1.0 => "bar" } { 1 => "foo", 1.second => "bar" } # now => { 1 => "foo", 1.second => "bar" } # was => { 1 => "bar" } And though the behavior here hasn't changed, for reference: 1 == 1.0 #=> true 1.0 == 1 #=> true 1 == 1.second #=> true 1.second == 1 #=> true | ||||
* | | new CHANGELOGs entries are in the top [ci skip] | Rafael Mendonça França | 2014-04-08 | 1 | -4/+4 |
| | | |||||
* | | Merge pull request #14647 from matiaskorhonen/updates-maintenance-policy | Rafael Mendonça França | 2014-04-08 | 2 | -3/+7 |
|\ \ | | | | | | | Updates the maintenance policy with new Rails versions | ||||
| * | | Updates the maintenance policy with new Rails versions | Matias Korhonen | 2014-04-08 | 2 | -3/+7 |
|/ / | |||||
* | | Dont abbreviate that which needs no abbreviation | David Heinemeier Hansson | 2014-04-08 | 1 | -2/+2 |
| | | |||||
* | | Dont encourage aliases now that we have variants | David Heinemeier Hansson | 2014-04-08 | 1 | -1/+0 |
| | | |||||
* | | Use short-form for the scaffold render calls and drop the needless test | David Heinemeier Hansson | 2014-04-08 | 2 | -9/+2 |
| | | |||||
* | | Merge pull request #14633 from matthewd/narrow_database_url | Rafael Mendonça França | 2014-04-08 | 14 | -101/+251 |
|\ \ | | | | | | | Only apply DATABASE_URL for Rails.env | ||||
| * | | Drop in @jeremy's new database.yml template text | Matthew Draper | 2014-04-09 | 13 | -44/+206 |
| | | | | | | | | | | | | | | | In passing, allow multi-word adapters to be referenced in a URL: underscored_name must become hyphened-name. | ||||
| * | | Don't deprecate after all | Matthew Draper | 2014-04-09 | 2 | -27/+11 |
| | | | |||||
| * | | Less ambition, more deprecation | Matthew Draper | 2014-04-09 | 2 | -75/+27 |
| | | | | | | | | | | | | | | | | | | | | | | | | The "DATABASE_URL_*" idea was moving in the wrong direction. Instead, let's deprecate the situation where we end up using ENV['DATABASE_URL'] at all: the Right Way is to explicitly include it in database.yml with ERB. | ||||
| * | | Ensure we correctly and immediately load all ENV entries | Matthew Draper | 2014-04-08 | 1 | -0/+21 |
| | | | | | | | | | | | | .. even when the supplied config made no hint that name was relevant. | ||||
| * | | Give a deprecation message even when the lookup fails | Matthew Draper | 2014-04-08 | 2 | -4/+6 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If the supplied string doesn't contain a colon, it clearly cannot be a database URL. They must have intended to do a key lookup, so even though it failed, give the explanatory deprecation warning, and raise the exception that lists the known configs. Conveniently, this also simplifies our logical behaviour: if the string matches a known configuration, or doesn't contain a colon (and is therefore clearly not a URL), then we output a deprecation warning, and behave exactly as we would if it were a symbol. | ||||
| * | | Rearrange the config merger some more | Matthew Draper | 2014-04-08 | 1 | -19/+16 |
| | | | | | | | | | | | | | | | | | | | | | This seems to simplify the operative part. Most importantly, by pre-loading all the configs supplied in ENV, we ensure the list is complete: if the developer specifies an unknown config, the exception includes a list of valid ones. | ||||
| * | | entry is always a Hash | Rafael Mendonça França | 2014-04-08 | 1 | -1/+1 |
| | | | |||||
| * | | Check env_url only once | Rafael Mendonça França | 2014-04-08 | 1 | -2/+7 |
| | | | |||||
| * | | Only call DEFAULT_ENV proc one time | Rafael Mendonça França | 2014-04-08 | 1 | -2/+5 |
| | | | |||||
| * | | Test DATABASE_URL has precendance over DATABASE_URL_#{environment} | Rafael Mendonça França | 2014-04-08 | 1 | -0/+9 |
| | | | |||||
| * | | Make sure DEFAULT_URL only override current environment database | Rafael Mendonça França | 2014-04-08 | 2 | -10/+30 |
| | | | | | | | | | | | | configuration | ||||
| * | | Only apply DATABASE_URL for Rails.env | Matthew Draper | 2014-04-08 | 2 | -45/+40 |
|/ / | | | | | | | | | As we like ENV vars, also support DATABASE_URL_#{env}, for more obscure use cases. | ||||
* | | Ensure we are looking with string keys | Godfrey Chan | 2014-04-07 | 1 | -1/+1 |
| | |