- 14 8月, 2015 5 次提交
-
-
由 Aaron Patterson 提交于
the same value that is extracted from the options hash earlier is returned, so we don't need to pass it in in the first place. The caller already has the data, so stop passing it around.
-
由 Aaron Patterson 提交于
this way we don't have to mutate the options hash so far away from where the user passed it in
-
由 Rafael Mendonça França 提交于
Require explicit counter_cache option for has_many
-
由 Aaron Patterson 提交于
The `anchor` parameter [is overridden](https://github.com/rails/rails/blob/b4b4a611d0eb9aa1c640c5f521c6a43bf2a65bab/actionpack/lib/action_dispatch/routing/mapper.rb#L1528) unless it is directly passed to `match`, so setting it in a scope must be a mistake.
-
由 Rafael Mendonça França 提交于
Prevent duplicating `where` clauses
-
- 13 8月, 2015 35 次提交
-
-
由 Yves Senn 提交于
Closes #21201.
-
由 Robin Dupret 提交于
How to pass arguments to ActiveJob Jobs [ci skip]
-
由 Dhia Eddine Chouchane 提交于
A section explaining how to pass arguments to Jobs has been added. [ci skip] How to pass arguments to ActiveJob Jobs Removed the "how to pass arguments" from what you will know section [ci skip] improving Enqueue Job section Using GuestsCleanupJob rather than MyJob for coherence. [ci skip] Passing args section merged with enqueuing jobs Passing args is now explained through examples withing Enqueuing the Jobs section [ci skip] Unnecessary example removed [ci skip] Typo fixed (missing as)
-
由 Robin Dupret 提交于
[ci skip] Clarified asset pipeline guide
-
由 Andreas Lietz 提交于
-
由 Yves Senn 提交于
The latest, currently unreleased, version of queue_classic is required for this to work. See https://github.com/QueueClassic/queue_classic/pull/262 for more details.
-
由 Kasper Timm Hansen 提交于
Added helper methods to stub any instance
-
由 Yves Senn 提交于
As described in the "Follow Coding Conventions" section in our contribution guide (http://edgeguides.rubyonrails.org/contributing_to_ruby_on_rails.html#follow-the-coding-conventions) we favor `assert_not` over `refute`. While we don't usually make stylistic changes on it's own I opted to do it in this case. The reason being that test cases are usually copied as a starting point for new tests. This results in a spread of `refute` in files that have been using it already.
-
由 Yves Senn 提交于
Assert that the `:prefix` option of `number_to_human_size` is deprecated
-
由 Ronak Jangir 提交于
-
由 Sean Griffin 提交于
Replace the giant comment in routes.rb with a link to the guides
-
由 Roque Pinel 提交于
-
由 Zachary Scott 提交于
[ci skip] Documentation: Switch around a common phrase for readability
-
由 Aaron Patterson 提交于
I didn't like this method because it mutates the parameters. Now that the method is so small, just push it up to `initialize`
-
由 Rafael Mendonça França 提交于
If the through class has default scopes we should skip the statement cache. Closes #20745.
-
由 Aaron Patterson 提交于
now the `@defaults` variable doesn't need to be set before calling `normalize_defaults`
-
由 Aaron Patterson 提交于
These three options are stored in the `scope` chain outside of the options hash. If they are in the options hash, then someone passed them in to `match` and they don't really do anything. So lets remove the code.
-
由 Aaron Patterson 提交于
remove `format` from the options hash in the scope chain so that we don't need to remove it later
-
由 Aaron Patterson 提交于
this reduces the number of times we have to mutate the options hash.
-
由 Aaron Patterson 提交于
This just ensures that `format` is applied to things inside the scope
-
由 Rafael Mendonça França 提交于
Fewer objects and refactoring
-
由 Marcin Olichwirowicz 提交于
Since we are always responding with an array and using `any?`, we don't need to check if an array is empty
-
由 Marcin Olichwirowicz 提交于
-
由 Aaron Patterson 提交于
Now we don't have to manually remove this from the options hash since the scope stores it outside of "options"
-
由 Aaron Patterson 提交于
Remove the `options` reader from `Resource` because nobody needs to see that hash. Also remove mutations on the options hash in `apply_common_behavior_for` because leaving the side effects in that method makes it difficult to understand what is going on in the caller.
-
由 Aaron Patterson 提交于
these two keys have a different merge strategy, and they also just get removed from the options hash later in the code. If we store them in a separate place, then we don't need to remove them later
-
由 Aaron Patterson 提交于
-
由 Aaron Patterson 提交于
Eventually we don't want to expose the "options" hash from scope, only read values from it. Lets start by adding a reader method.
-
由 Aaron Patterson 提交于
now we don't need to construct a Mapping object just to get an ArgumentError if there is no `via` parameter provided.
-
由 Aaron Patterson 提交于
-
由 Aaron Patterson 提交于
-
由 Aaron Patterson 提交于
-
由 Aaron Patterson 提交于
We're going to try pulling this up further, and check `via` validity sooner. This way we don't have to do a bunch of processing on `options` hashes only to find out that the route is incorrect
-
由 Aaron Patterson 提交于
If we do the Regexp verification in a second method, then the `split_constraints` method gets much easier.
-
由 Aaron Patterson 提交于
-