| Commit message (Collapse) | Author | Age | Files | Lines |
... | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
* Loop rather than recurse in retrieve_connection_pool
* Key the hash by class rather than class name. This avoids creating
unnecessary strings.
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
This reverts commit 7256cb53e0c34e510a4d59a50d120c0358cf1d99, reversing
changes made to 6ebe22c3ae716d089af1e5090ddb0d12b31af8ac.
Reason: A test was failing.
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
This reverts commit e4b33b08d6d2b88b627b1e52c4f349e57c5b89fc.
https://github.com/rails/rails/pull/7452#issuecomment-8094302
|
|\ \ \ \ \
| |_|_|/ /
|/| | | | |
the example for contraint( :ip => XXXX) has an invalid regex
|
| | | | |
| | | | |
| | | | | |
escape the '.'s
|
| | | | |
| | | | |
| | | | | |
Since it uses request and response methods
|
|\ \ \ \ \
| | | | | |
| | | | | | |
Fix build about TemplateDigestorTest.
|
|/ / / / / |
|
|\ \ \ \ \
| | | | | |
| | | | | | |
Fixes warning: & interpreted as argument prefix
|
|/ / / / / |
|
|\ \ \ \ \
| | | | | |
| | | | | | |
Better ActionView::Digestor nested template inference
|
| | | | | | |
|
| | | | | | |
|
| | | | | | |
|
| | | | | | |
|
|\ \ \ \ \ \
| |/ / / / /
|/| | | | | |
A missing require is added to fix build. When performing rake test:isolated in actionmailer, test/base_test.rb was break.
|
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
was break.
|
|\ \ \ \ \ \
| | | | | | |
| | | | | | | |
Fix grammar
|
| | | | | | | |
|
|/ / / / / / |
|
|\ \ \ \ \ \
| | | | | | |
| | | | | | | |
Fix markdown rendering of the ActionPack changelog [ci skip]
|
| | |/ / / /
| |/| | | | |
|
|/ / / / / |
|
| | | | |
| | | | |
| | | | |
| | | | | |
computation *Jeremy Kemper/DHH*
|
| | | | |
| | | | |
| | | | |
| | | | | |
spiked in the cache_digests plugin) *DHH*
|
| | | | | |
|
| | | | | |
|
|\ \ \ \ \
| | | | | |
| | | | | | |
Add Missing Keys from Journey on Failed URL Format
|
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
The result of Generator with or without the @extras instance variable set contains the desired information. Rather than preserving state when initializing the original object, we can simply extract the keys from the resultant parameters.
ATP Actionpack, railties
|
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
Many named routes have keys that are required to successfully resolve. If a key is left off like this:
<%= link_to 'user', user_path %>
This will produce an error like this:
No route matches {:action=>"show", :controller=>"users"}
Since we know that the :id is missing, we can add extra debugging information to the error message.
No route matches {:action=>"show", :controller=>"users"} missing required keys: [:id]
This will help new and seasoned developers look closer at their parameters. I've also subclassed the routing error to be clear that this error is a result of attempting to generate a url and not because the user is trying to visit a bad url.
While this may sound trivial this error message is misleading and confuses most developers. The important part isn't what's in the options its's what's missing. Adding this information to the error message will make debugging much more obvious.
This is the sister pull request of https://github.com/rails/journey/pull/44 which will be required to get they missing keys into the correct error message.
Example Development Error in Rails: http://cl.ly/image/3S0T0n1T3421
|
| |/ / / /
|/| | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
We need to anchor to remove the extension. In addition to
be the correct way to do that, files in ~/.rbenv get that
.rb removed, so it is a real source of bugs, as reported in
https://github.com/rails/rails/commit/b33700f5580b4cd85379a1dc60fa341ac4d8deb2#commitcomment-1781840
|
|\ \ \ \ \
| | | | | |
| | | | | | |
Fixed indendation
|
|/ / / / / |
|
| | | | |
| | | | |
| | | | |
| | | | | |
scope conditions
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Changes in old branches needed to be manually synched in CHANGELOGs of newer ones.
This has proven to be brittle, sometimes one just forgets this manual step.
With this commit we switch to CHANGELOGs per branch. When a new major version is
cut from master, the CHANGELOGs in master start being blank.
A link to the CHANGELOG of the previous branch allows anyone interested to
follow the history.
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Nowadays circular autoloads do not work, but the user gets a NameError
that says some constant is undefined. That's puzzling, because he is
normally trying to autoload a constant he knows can be autoloaded.
With this check we can give a better error message.
|
|\ \ \ \ \
| | | | | |
| | | | | | |
Fix method redefined warnings.
|
|/ / / / / |
|
|\ \ \ \ \
| | | | | |
| | | | | | |
Fixes warnings when executing rake test in ActionMailer.
|
|/ / / / /
| | | | |
| | | | |
| | | | | |
582a7f459990487659886b90e54c22e055c65870
|
|\ \ \ \ \
| |/ / / /
|/| | | | |
Tiny fixes in railties/test/application.
|
| | | | | |
|
|/ / / / |
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
loaded stores file names without the .rb extension, but search_for_file
returns file names with the extension.
The solution is hackish, but this file needs a revamp.
|
|\ \ \ \
| | | | |
| | | | | |
removed : warning: `*' interpreted as argument prefix
|
|/ / / / |
|
| | | | |
|
|\ \ \ \
| | | | |
| | | | | |
Memcached to dalli for actionpack test.
|
| | | | | |
|
| | | | |
| | | | |
| | | | | |
There is no memcache gem left in repo.
|