- 26 6月, 2014 15 次提交
-
-
由 Sean Griffin 提交于
Making this change revealed several subtle bugs related to models with no primary key, and anonymous classes. These have been fixed as well, with regression tests added.
-
由 Rafael Mendonça França 提交于
Deprecate automatic counter caches on has_many :through
-
由 Sean Griffin 提交于
Reliant on https://github.com/rails/rails/pull/15747 but pulled to a separate PR to reduce noise. `has_many :through` associations have the undocumented behavior of automatically detecting counter caches. However, the way in which it does so is inconsistent with counter caches everywhere else, and doesn't actually work consistently. As with normal `has_many` associations, the user should specify the counter cache on the `belongs_to`, if they'd like it updated.
-
由 Rafael Mendonça França 提交于
`Attribute` should know about its name
-
由 Sean Griffin 提交于
This allows using polymorphism for the uninitialized attributes raising an exception behavior.
-
由 Rafael Mendonça França 提交于
Make the isolated tests run on JRuby
-
由 Rafael Mendonça França 提交于
Allow proc and symbol as values for `only_integer` of `NumericalityValidator`
-
由 Rafael Mendonça França 提交于
Closes #15919
-
由 Sean Griffin 提交于
This will make it less painful to add additional properties, which should persist across writes, such as `name`. Conflicts: activerecord/lib/active_record/attribute_set.rb
-
由 Rafael Mendonça França 提交于
Move behavior of `read_attribute` to `AttributeSet` Conflicts: activerecord/lib/active_record/attribute_set.rb activerecord/test/cases/attribute_set_test.rb
-
由 Rafael Mendonça França 提交于
Encapsulate knowledge of type objects on `ActiveRecord::Result`
-
由 Rafael Mendonça França 提交于
Move `attributes_before_type_cast` to `AttributeSet` Conflicts: activerecord/lib/active_record/attribute_set.rb activerecord/test/cases/attribute_set_test.rb
-
由 Yves Senn 提交于
-
由 Yves Senn 提交于
Noticed that there was no section about Action View and added the heading. Notable changes need to be compiled. /cc @chancancode @zzak
-
由 Yves Senn 提交于
[ci skip] /cc @chancancode @zzak
-
- 25 6月, 2014 9 次提交
-
-
由 Sean Griffin 提交于
Moved `Builder` to its own file, as it started looking very weird once I added private methods to the `AttributeSet` class and the `Builder` class started to grow. Would like to refactor `fetch_value` to change to ```ruby self[name].value(&block) ``` But that requires the attributes to know about their name, which they currently do not.
-
由 Yves Senn 提交于
This is a partial fix for #15853. It only works when a `preload` is issued and not an `eager_load`. I've added a skipped failing test-case to keep in mind that we need to deal with `eager_load`.
-
由 Rafael Mendonça França 提交于
Fix a bug where NameError#name returns a qualified name in string
-
由 Rafael Mendonça França 提交于
Replace x.sort_by!.select! with x.select!.sort_by!
-
由 Yuki Nishijima 提交于
Ruby's original behaviour is that : * It only returns a const name, not a qualified aname * It returns a symbol, not a string
-
由 Rafael Mendonça França 提交于
Add a '--skip-routes' flag for the Controller generator.
-
由 Lucas Mazza 提交于
This way it is possible to skip the addition of the 'get "foo/bar"' routes when generating a controller.
-
由 Rafael Mendonça França 提交于
Deal with regex match groups in excerpt
-
由 Gareth Rees 提交于
Original implementation has bugs if the regex contains a match group. Example: excerpt('This is a beautiful? morning', /\b(beau\w*)\b/i, :radius => 5) Expected: "...is a beautiful? mor..." Actual: "...is a beautifulbeaut..." The original phrase was being converted to a regex and returning the text either side of the phrase as expected: 'This is a beautiful? morning'.split(/beautiful/i, 2) # => ["This is a ", "? morning"] When we have a match with groups the match is returned in the array. Quoting the ruby docs: "If pattern is a Regexp, str is divided where the pattern matches. [...] If pattern contains groups, the respective matches will be returned in the array as well." 'This is a beautiful? morning'.split(/\b(beau\w*)\b/iu, 2) # => ["This is a ", "beautiful", "? morning"] If we assume we want to split on the first match – this fix makes that assumption – we can pass the already assigned `phrase` variable as the place to split (because we already know that a match exists from line 168). Originally spotted by Louise Crow (@crowbot) at https://github.com/mysociety/alaveteli/pull/1557
-
- 24 6月, 2014 16 次提交
-
-
由 Arthur Nogueira Neves 提交于
Improves readability of this line...
-
由 Kevin Musiorski 提交于
[ci skip]
-
由 Sean Griffin 提交于
Add warning section for `rescue_from Exception` in Guides
-
由 Sean Griffin 提交于
-
由 Yves Senn 提交于
Always assume strings with non-numeric characters change numeric types
-
由 Sean Griffin 提交于
We previously only did this if the old value was zero, to make sure numericality validations run and failed if the user gave 'wibble' as the value, which would be type cast to 0. However, numericality validations will fail if there are any non-numeric characters in the string, so 5 -> '5wibble' should also be marked as changed.
-
由 Yves Senn 提交于
Silence warning in test
-
由 Sean Griffin 提交于
We still had one file using `column_for_attribute` when it could return nil, causing deprecation warnings in the tests.
-
由 Yves Senn 提交于
Need to add individual `:nodoc:` for nested classes / modules to completely remove the constants from the API.
-
由 Yves Senn 提交于
-
由 David Heinemeier Hansson 提交于
Add #model_name instance method to ActiveModel::Naming
-
由 Yves Senn 提交于
Adding `# :nodoc:` to the parent `class` / `module` is not going to ignore nested classes or modules. There is a modifier `# :nodoc: all` but sadly the containing class or module will continue to be in the docs. /cc @sgrif
-
由 Yves Senn 提交于
Add rails:update to the upgrading guides
-
由 Rafael Mendonça França 提交于
Remove unused params
-
由 Zachary Scott 提交于
Disable autolink in example code blocks in guides. [ci skip]
-
由 Juanito Fatas 提交于
/cc @chancancode @robin850 configuration reference: http://alexgorbatchev.com/SyntaxHighlighter/manual/configuration/ Related issues: * https://github.com/vmg/redcarpet/issues/334 * https://github.com/docrails-tw/guides/issues/62
-