aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* Partially revert deprecation of *_filterRafael Mendonça França2014-06-033-36/+9
| | | | | | | | | | | We are going to deprecate only on Rails 5 to make easier plugin maintainers support different Rails versions. Right now we are only discouraging their usage. This reverts commit 6c5f43bab8206747a8591435b2aa0ff7051ad3de. Conflicts: actionpack/CHANGELOG.md
* Merge pull request #15438 from sgrif/sg-null-columnRafael Mendonça França2014-06-035-14/+40
|\ | | | | | | | | | | | | Return a null column when no column exists for an attribute Conflicts: activerecord/CHANGELOG.md
| * Return a null column when no column exists for an attributeSean Griffin2014-06-035-14/+40
| |
* | Merge pull request #15349 from tgxworld/remove_duplicated_method_callRafael Mendonça França2014-06-033-7/+3
|\ \ | | | | | | Remove duplicated HashWithIndifferentAccess#with_indifferent_access.
| * | Remove duplicated HashWithIndifferentAccess#with_indifferent_access.Guo Xiang Tan2014-05-263-7/+3
| | |
* | | Merge pull request #15172 from notEthan/active_record_pretty_printRafael Mendonça França2014-06-033-0/+95
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | implement ActiveRecord::Base#pretty_print Conflicts: activerecord/CHANGELOG.md
| * | | implement ActiveRecord::Base#pretty_print + changelogEthan2014-05-293-0/+95
| | | |
* | | | Merge pull request #15492 from sgrif/sg-dirty-defaultsRafael Mendonça França2014-06-035-2/+40
|\ \ \ \ | | | | | | | | | | Keep column defaults in type cast form
| * | | | Keep column defaults in type cast formSean Griffin2014-06-035-2/+40
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The contract of `_field_changed?` assumes that the old value is always type cast. That is not the case for the value in `Column#default` as things are today. It appears there are other public methods that assume that `Column#default` is type cast, as well. The reason for this change originally was because the value gets put into `@raw_attributes` in initialize. This reverts to the old behavior on `Column`, and updates `initialize` to make sure that the values are in the right format.
* | | | | Merge pull request #15493 from bf4/relax_mail_depRafael Mendonça França2014-06-031-1/+1
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | Relax mail gem constraint from ~> 2.5.4 to ~> 2.5
| * | | | | Relax mail gem constraint from ~> 2.5.4 to ~> 2.5, >= 2.5.4Benjamin Fleischer2014-06-031-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This allows Rails users to install mail 2.6 which relaxes the mime-types dependency, which is a big win for a lot of people. Previously, the mail gem restricted mime-types to ~> 1.16 but now it has expanded to [">= 1.16", "< 3"] And the mime-types maintainer will also be checking that 2.x releases don't break mail. See https://github.com/mikel/mail/pull/713 https://rubygems.org/gems/mail/versions/2.6.0
* | | | | | force table creationAaron Patterson2014-06-031-2/+3
| | | | | |
* | | | | | Merge pull request #15447 from akshay-vishnoi/test-casesRafael Mendonça França2014-06-031-0/+8
|\ \ \ \ \ \ | | | | | | | | | | | | | | Test `except!` in other cases too
| * | | | | | Test `except!` in other cases tooAkshay Vishnoi2014-05-311-0/+8
| | | | | | |
* | | | | | | oops! :bomb:Aaron Patterson2014-06-032-92/+0
| | | | | | |
* | | | | | | Routes specifying 'to:' must be a string that contains a "#" or a rackAaron Patterson2014-06-036-27/+149
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | application. Use of a symbol should be replaced with `action: symbol`. Use of a string without a "#" should be replaced with `controller: string`.
* | | | | | | Revert "test pg, we don't care about the internal state of `column#default`."Yves Senn2014-06-037-0/+23
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Revert "test pg, remove unused column assignments. Follow up to 254cdf47" Related to #15492 This reverts commit 254cdf4728291277f3fbaa854f34495030e476b4. This reverts commit 4bcf9029452e0c760af04faab6b549710401e8cf. There are public methods that assume `Column#default` is type casted. The return value of `Column#default` is publicly relevant and should not change. /cc @sgrif
* | | | | | | use the factory method to construct the mappingAaron Patterson2014-06-031-1/+1
| | | | | | |
* | | | | | | add tests for mixing :to and controller / actionAaron Patterson2014-06-031-0/+48
| | | | | | |
* | | | | | | push some options munging to an alternate constructorAaron Patterson2014-06-031-13/+19
| | | | | | |
* | | | | | | add tests for nested lambda constraintsAaron Patterson2014-06-031-0/+27
| | | | | | |
* | | | | | | move options_constraints tests next to each otherAaron Patterson2014-06-031-2/+2
| | | | | | |
* | | | | | | only do is_a checks on `formatted` onceAaron Patterson2014-06-031-9/+5
| | | | | | |
* | | | | | | push options_constraints processing upAaron Patterson2014-06-031-22/+24
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | this lets us reduce is_a checks on the options_constraints and push "callable constraints" verification to the right place.
* | | | | | | remove options as an ivarAaron Patterson2014-06-031-6/+8
| | | | | | |
* | | | | | | only do Regexp === option onceAaron Patterson2014-06-031-3/+4
| | | | | | |
* | | | | | | return early from add_request_methodAaron Patterson2014-06-031-11/+11
| | | | | | |
* | | | | | | only loop over `options` once (hopefully)Aaron Patterson2014-06-031-13/+11
| | | | | | |
* | | | | | | only loop through constraints onceAaron Patterson2014-06-031-19/+15
| | | | | | |
* | | | | | | change defaults allocation to a one-linerAaron Patterson2014-06-031-3/+2
| | | | | | |
* | | | | | | no longer need the scope ivarAaron Patterson2014-06-031-2/+1
| | | | | | |
* | | | | | | only look up scope[:module] onceAaron Patterson2014-06-031-6/+6
| | | | | | |
* | | | | | | remove IGNORE_OPTIONSAaron Patterson2014-06-031-4/+8
| | | | | | |
* | | | | | | :only is never used in Mapping, so rm the keyAaron Patterson2014-06-031-1/+2
| | | | | | |
* | | | | | | remove :defaults from the IGNORE_OPTIONS listAaron Patterson2014-06-031-4/+4
| | | | | | |
* | | | | | | remove :constraints from IGNORE_OPTIONSAaron Patterson2014-06-031-4/+5
| | | | | | |
* | | | | | | disconnect normalize_defaults! from options[:constraints]Aaron Patterson2014-06-031-6/+6
| | | | | | |
* | | | | | | disconnect options and scope from the `blocks` methodAaron Patterson2014-06-031-7/+8
| | | | | | |
* | | | | | | disconnect the constraints method from the options and scope hashesAaron Patterson2014-06-031-12/+15
| | | | | | |
* | | | | | | :on is removed from options before the Mapping is instantiated, so remove it ↵Aaron Patterson2014-06-031-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | from IGNORE_OPTIONS
* | | | | | | pull up via extraction and remove it from options / IGNORE_OPTIONSAaron Patterson2014-06-031-8/+6
| | | | | | |
* | | | | | | always make :via a listAaron Patterson2014-06-031-3/+2
| | | | | | |
* | | | | | | always pull out a via variable and simplify logicAaron Patterson2014-06-031-12/+15
| | | | | | |
* | | | | | | test with an empty viaAaron Patterson2014-06-031-0/+8
| | | | | | |
* | | | | | | add a test for missing "via" parameterAaron Patterson2014-06-032-1/+9
| | | | | | |
* | | | | | | remove :as and :anchor from IGNORE_OPTIONSAaron Patterson2014-06-031-3/+5
| | | | | | |
* | | | | | | remove another value from IGNORE_OPTIONSAaron Patterson2014-06-031-2/+2
| | | | | | |
* | | | | | | shorten up IGNORE_OPTIONSAaron Patterson2014-06-031-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | since we are now passing the format value around, we can remove it from the options hash, which means we don't need to consult as many values from IGNORE_OPTIONS
* | | | | | | only look up the format option from the hash onceAaron Patterson2014-06-031-16/+18
| | | | | | |
* | | | | | | Merge pull request #15486 from sgrif/sg-binary-quotingMatthew Draper2014-06-045-47/+83
|\ \ \ \ \ \ \ | |_|/ / / / / |/| | | | | | Refactor quoting of binary data to not be based on the column type