aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* Merge pull request #14855 from laurocaetano/fix_polymorphic_with_string_keyYves Senn2014-05-215-4/+50
|\ | | | | | | Fix polymorphic eager load with foreign_key as String.
| * Fix polymorphic eager load with foreign_key as String.Lauro Caetano2014-05-205-2/+42
|/ | | | | | | | | The foreign_key could be `String` and just doing `owners_map[owner_key]` could return `nil`. To prevent this bug, we should `to_s` both keys if their types are different. Fixes #14734.
* Merge pull request #15213 from tgxworld/remove_redundant_codeRafael Mendonça França2014-05-201-3/+2
|\ | | | | Remove redundant code.
| * Remove redundant code.Guo Xiang Tan2014-05-201-3/+2
| |
* | Merge pull request #10887 from sakuro/deep_transform_keys_in_nested_arraysRafael Mendonça França2014-05-203-16/+67
|\ \ | | | | | | | | | | | | | | | | | | Hash#deep_*_keys(!) recurse into nested arrays. Conflicts: activesupport/CHANGELOG.md
| * | Hash#deep_*_keys(!) recurse into nested arrays.OZAWA Sakuro2013-06-083-16/+67
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Following methods now recursively transform nested arrays, too. * Hash#deep_transform_keys * Hash#deep_transform_keys! * Hash#deep_stringify_keys * Hash#deep_stringify_keys! * Hash#deep_symbolize_keys * Hash#deep_symbolize_keys!
* | | Merge pull request #14544 from jefflai2/named_scope_stiRafael Mendonça França2014-05-208-5/+42
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | Fixes Issue #13466. Conflicts: activerecord/CHANGELOG.md
| * | | Fixes Issue #13466.Jefferson Lai2014-04-238-5/+42
| | | | | | | | | | | | | | | | | | | | | | | | Changed the call to a scope block to be evaluated with instance_eval. The result is that ScopeRegistry can use the actual class instead of base_class when caching scopes so queries made by classes with a common ancestor won't leak scopes.
* | | | Merge pull request #15205 from sgrif/sg-delegate-klassRafael Mendonça França2014-05-209-14/+33
|\ \ \ \ | | | | | | | | | | Delegate `klass` to the injected type object
| * | | | Delegate `klass` to the injected type objectSean Griffin2014-05-209-14/+33
| | | | |
* | | | | Merge pull request #14979 from brocktimus/masterRafael Mendonça França2014-05-203-1/+16
|\ \ \ \ \ | | | | | | | | | | | | Making belongs_to: touch behaviour be consistent with save updating updated_at
| * | | | | Prevented belongs_to: touch propagating up if there are no changes being savedBrock Trappitt2014-05-213-1/+16
| | | | | |
* | | | | | Revert "Revert "Merge pull request #8313 from ↵Rafael Mendonça França2014-05-203-8/+22
|/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | alan/only_save_changed_has_one_objects"" This reverts commit e94e6c27af495a2460c811bb506459f1428dec6b. Conflicts: activerecord/CHANGELOG.md The original commit was reverted only to be safe since #14407 were reported. We don't have any proof we added a regression with the original commit so reverting it now will give us more problem. Closes #14407
* | | | | Merge pull request #15154 from msgehard/move_password_fieldRafael Mendonça França2014-05-201-2/+2
|\ \ \ \ \ | | | | | | | | | | | | Put attr_reader in with all of the other instance methods
| * | | | | Put attr_reader in with all of the other instance methodsMike Gehard2014-05-171-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | This makes the grouping make a little more sense
* | | | | | we don't use this parameter for anything, so rmAaron Patterson2014-05-203-22/+21
| | | | | |
* | | | | | fewer string allocations per url_forAaron Patterson2014-05-201-7/+5
| | | | | |
* | | | | | mutate the path string to avoid object allocationsAaron Patterson2014-05-201-4/+4
| | | | | |
* | | | | | push only_path conditional upAaron Patterson2014-05-201-18/+19
| | | | | |
* | | | | | push arg checking upAaron Patterson2014-05-201-4/+4
| | | | | |
* | | | | | fewer method calls and string garbage when there is no user/passwordAaron Patterson2014-05-201-9/+5
| | | | | |
* | | | | | fewer hash allocations when calling url_forAaron Patterson2014-05-201-9/+12
| | | | | |
* | | | | | fix variable names, only pass hashes to the positional args methodAaron Patterson2014-05-201-8/+10
| |/ / / / |/| | | |
* | | | | Merge pull request #15208 from sgrif/sg-delegate-type-cast-for-writeRafael Mendonça França2014-05-204-30/+14
|\ \ \ \ \ | | | | | | | | | | | | Delegate `type_cast_for_write` to injected type object
| * | | | | Delegate `type_cast_for_write` to injected type objectSean Griffin2014-05-204-30/+14
| | | | | |
* | | | | | Merge pull request #11346 from tomykaira/fix_10257Rafael Mendonça França2014-05-202-2/+21
|\ \ \ \ \ \ | |/ / / / / |/| | | | | Check authentication scheme in Basic auth
| * | | | | Run login_procedure only when the auth_scheme is validtomykaira2013-07-081-7/+14
| | | | | |
| * | | | | Check authentication scheme in Basic authtomykaira2013-07-072-1/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | `authenticate_with_http_basic` and its families should check the authentication schema is "Basic". Different schema, such as OAuth2 Bearer should be rejected by basic auth, but it was passing as the test shows. This fixes #10257.
* | | | | | Merge pull request #15207 from sgrif/sg-inline-column-helpersRafael Mendonça França2014-05-2010-120/+77
|\ \ \ \ \ \ | | | | | | | | | | | | | | Inline typecasting helpers from Column to the appropriate types
| * | | | | | Inline typecasting helpers from Column to the appropriate typesSean Griffin2014-05-2010-120/+77
| | | | | | |
* | | | | | | Merge pull request #15206 from sgrif/sg-type-map-postgresqlRafael Mendonça França2014-05-203-45/+31
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Use the generic type map for PostgreSQL OID registrations
| * | | | | | | Use the generic type map for PostgreSQL OID registrationsSean Griffin2014-05-203-45/+31
| |/ / / / / /
* | | | | | | Merge pull request #15204 from sgrif/sg-delegate-predicatesRafael Mendonça França2014-05-2012-24/+44
|\ \ \ \ \ \ \ | |/ / / / / / |/| | | | | | Delegate predicate methods to injected type object on Column
| * | | | | | Delegate predicate methods to injected type object on ColumnSean Griffin2014-05-2012-24/+44
|/ / / / / /
* | | | | | remove dead codeAaron Patterson2014-05-201-57/+0
| | | | | |
* | | | | | cache the formatter on the path objectAaron Patterson2014-05-202-2/+6
| | | | | |
* | | | | | translate AST to a formatter before url generationAaron Patterson2014-05-202-1/+72
| | | | | |
* | | | | | Merge pull request #15200 from sgrif/sg-type-map-mysqlRafael Mendonça França2014-05-204-23/+39
|\ \ \ \ \ \ | | | | | | | | | | | | | | Use the generic type map object for mysql field lookups
| * | | | | | Use the generic type map object for mysql field lookupsSean Griffin2014-05-204-23/+39
|/ / / / / /
* | | | | | Merge pull request #15191 from camsong/masterYves Senn2014-05-201-2/+8
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Distinguish rake assets:clobber from rake assets:clean [ci skip]
| * | | | | | Distinguish rake assets:clobber from rake assets:cleancam song2014-05-201-2/+5
| | | | | | |
* | | | | | | Merge pull request #15203 from sgrif/sg-delegate-type-castRafael Mendonça França2014-05-207-75/+7
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Replace `type_cast` case statement with delegation
| * | | | | | | Replace `type_cast` case statement with delegationSean Griffin2014-05-207-75/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | All subclasses of column were now delegating `type_cast` to their injected type object. We can remove the overriding methods, and generalize it on the `Column` class itself. This also enabled us to remove several column classes completely, as they no longer had any meaningful behavior of their own.
* | | | | | | | Merge pull request #15187 from v-yarotsky/fix_confusing_delegation_exceptionRafael Mendonça França2014-05-203-26/+20
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | Fix confusing exception in ActiveSupport delegation
| * | | | | | | | Fix confusing exception in ActiveSupport delegationVladimir Yarotsky2014-05-203-26/+20
| | | | | | | | |
* | | | | | | | | prepopulate the dispatch cache so we don't need the ThreadSafe cache.Aaron Patterson2014-05-201-7/+9
| | | | | | | | |
* | | | | | | | | make the each visitor top-down left-rightAaron Patterson2014-05-201-1/+1
| | | | | | | | |
* | | | | | | | | Merge pull request #15201 from sgrif/sg-types-postgresqlRafael Mendonça França2014-05-204-151/+85
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | Have Postgres OID types inherit from general types
| * | | | | | | | | Have Postgres OID types inherit from general typesSean Griffin2014-05-204-151/+85
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Using general types where possible. Several more can go away once infinity gets figured out.
* | | | | | | | | | fix multiple hash preloads. Fixes #14994Aaron Patterson2014-05-202-6/+14
| | | | | | | | | |