Commit message (Collapse) | Author | Age | Files | Lines | ||
---|---|---|---|---|---|---|
... | ||||||
* | | | simplify conditionals by assuming hash values will never be `false` | Aaron Patterson | 2011-07-26 | 1 | -10/+2 | |
| | | | ||||||
* | | | use regular ruby rather than clever ruby | Aaron Patterson | 2011-07-26 | 1 | -3/+3 | |
| | | | ||||||
* | | | fixing wildcard path matching when wildcard is inside parenthesis | Aaron Patterson | 2011-07-26 | 2 | -1/+8 | |
| | | | ||||||
* | | | Remove blank line | Santiago Pastorino | 2011-07-26 | 1 | -1/+0 | |
| | | | ||||||
* | | | Merge pull request #2273 from thoefer/refactor | Santiago Pastorino | 2011-07-26 | 1 | -5/+4 | |
|\ \ \ | | | | | | | | | refactored 'assert_redirected_to' | |||||
| * | | | refactored 'assert_redirected_to': local call to validate_request! will be ↵ | thoefer | 2011-07-26 | 1 | -5/+4 | |
| | | | | | | | | | | | | | | | | called in assert_response already. changed names of local variables in order to recognize the semantics a bit easier. | |||||
* | | | | Merge pull request #2095 from dgeb/issue-2094 | Aaron Patterson | 2011-07-26 | 2 | -1/+18 | |
|\ \ \ \ | | | | | | | | | | | Fixes overriding of options[:html][:remote] in form_for() (Issue #2094) | |||||
| * | | | | added test case for fix to issue #2094 | Dan Gebhardt | 2011-07-16 | 1 | -0/+17 | |
| | | | | | ||||||
| * | | | | fixed problem in which options[:html][:remote] would be overridden in ↵ | Dan Gebhardt | 2011-07-15 | 1 | -1/+1 | |
| | | | | | | | | | | | | | | | | | | | | form_for() - fixes #2094 | |||||
* | | | | | use sprocket's append_path and assert_match | Damien Mathieu | 2011-07-26 | 1 | -26/+26 | |
| |/ / / |/| | | | ||||||
* | | | | use_sprockets is not used anymore | Santiago Pastorino | 2011-07-26 | 1 | -5/+1 | |
| | | | | ||||||
* | | | | Bump sprockets up | Santiago Pastorino | 2011-07-26 | 1 | -1/+1 | |
| | | | | ||||||
* | | | | Bump rack up. Closes #2107 | Santiago Pastorino | 2011-07-25 | 1 | -1/+1 | |
| | | | | ||||||
* | | | | Merge pull request #2262 from sferik/format_true | José Valim | 2011-07-25 | 2 | -0/+9 | |
|\ \ \ \ | | | | | | | | | | | Allow a route to have :format => true | |||||
| * | | | | Allow a route to have :format => true | Erik Michaels-Ober | 2011-07-25 | 2 | -0/+9 | |
| | |/ / | |/| | | | | | | | | | | | | | | | | | | When format is true, it is mandatory (as opposed to :format => false). This is currently not possible with resource routes, which automatically make format optional by default. | |||||
* / | | | Remove unused use_sprockets config | Santiago Pastorino | 2011-07-25 | 2 | -8/+1 | |
|/ / / | ||||||
* | | | Merge pull request #2245 from ↵ | Santiago Pastorino | 2011-07-25 | 2 | -6/+25 | |
|\ \ \ | | | | | | | | | | | | | | | | | davidtrogers/constraints_block_passed_to_following_routes_in_same_scope Memoizing @blocks & disposing of constraints options properly | |||||
| * | | | Ensure the constraints block is only applied to the correct route | Dave Rogers | 2011-07-25 | 2 | -6/+25 | |
| | |/ | |/| | | | | | | | | | | addresses issue #1907 - any routes that follow a route with a constraints block are inheriting the previous route's constraints. | |||||
* | | | no need for an anonymous class on every setup call | Aaron Patterson | 2011-07-25 | 1 | -10/+9 | |
| | | | ||||||
* | | | Merge pull request #2258 from thedarkone/no-file-stat | José Valim | 2011-07-25 | 2 | -3/+3 | |
|\ \ \ | | | | | | | | | Use shorter class-level File methods instead of going through File.stat | |||||
| * | | | Use shorter class-level File methods instead of going through File.stat. | thedarkone | 2011-07-25 | 2 | -3/+3 | |
| |/ / | ||||||
* / / | Make polymorphic_url calls go through application helpers again. | thedarkone | 2011-07-25 | 3 | -9/+14 | |
|/ / | | | | | | | This brings back the ability to overwrite/extend url generating methods in application heleprs. | |||||
* | | Merge pull request #2243 from dmathieu/comment | Xavier Noria | 2011-07-24 | 1 | -0/+1 | |
|\ \ | | | | | | | Help the next guy with explanations - See bb3b3e306 | |||||
| * | | help the next guy with explanations - See bb3b3e306 | Damien Mathieu | 2011-07-24 | 1 | -0/+1 | |
| | | | ||||||
* | | | Use top level Rails constant | Andrew White | 2011-07-24 | 1 | -2/+2 | |
| | | | ||||||
* | | | Merge branch 'master' of git://github.com/lifo/docrails | Xavier Noria | 2011-07-24 | 7 | -11/+11 | |
|\ \ \ | |/ / |/| | | ||||||
| * | | Missed a few instances of British English spelling in the last commit | Oemuer Oezkir | 2011-07-24 | 3 | -3/+3 | |
| | | | | | | | | | | | | oh and obviously I meant according to Webster, not Weber | |||||
| * | | Changed a few instances of of words in the API docs written in British ↵ | Oemuer Oezkir | 2011-07-24 | 6 | -10/+10 | |
| | | | | | | | | | | | | | | | | | | English to American English(according to Weber) | |||||
| * | | now the documentation reflects the actual returned value for a missing ↵ | gnagno | 2011-07-23 | 1 | -1/+1 | |
| | | | | | | | | | | | | translation | |||||
* | | | Merge pull request #2233 from gramos/fix-issue-1951-master | Aaron Patterson | 2011-07-23 | 2 | -2/+30 | |
|\ \ \ | | | | | | | | | Fix issue 1951 master Namespaced model partial_path is wrong in namespaced controllers | |||||
| * | | | - added ActionView::PartialRenderer#merge_path_into_partial(path, partial) | Gaston Ramos | 2011-07-24 | 2 | -3/+17 | |
| | | | | | | | | | | | | | | | | fix issues/1951 | |||||
| * | | | - added test case for issue: | Gaston Ramos | 2011-07-24 | 1 | -0/+14 | |
| | | | | | | | | | | | | | | | | | | | | https://github.com/rails/rails/issues/1951 Namespaced model partial_path is wrong in namespaced controllers | |||||
* | | | | just use normal ruby for stubbing | Aaron Patterson | 2011-07-23 | 1 | -5/+3 | |
|/ / / | ||||||
* | | | Merge pull request #2228 from Empact/helper_tags_in_mailers | Aaron Patterson | 2011-07-23 | 2 | -1/+2 | |
|\ \ \ | | | | | | | | | Simple fix to the NoMethodError noted in #2177. | |||||
| * | | | Simple fix to the NoMethodError noted in #2177. | Ben Woosley | 2011-07-23 | 2 | -1/+2 | |
| | | | | | | | | | | | | Unfortunately #respond_to?(:controller) won't work as suggested, nor will respond_to?(:params), as #controller is present and #params is delegated to #controller. #delegate makes respond_to? return true regardless whether the target responds to it. | |||||
* | | | | rake assets:precompile should always perform caching - Closes #2199 | Damien Mathieu | 2011-07-24 | 1 | -0/+1 | |
|/ / / | ||||||
* / / | checked all .rb files in the project tree for missing magic comments, one ↵ | Xavier Noria | 2011-07-23 | 1 | -0/+2 | |
|/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | was missing Came with this one-liner for this: find . -name '*.rb' | \ xargs chardet | \ grep -v ascii | \ cut -d: -f1 -s | \ xargs -n1 ruby -0777 -ne 'puts $FILENAME if $_ !~ /#.*coding:\s*utf/i' Welcome $_. | |||||
* | | Merge branch 'master' of git://github.com/lifo/docrails | Xavier Noria | 2011-07-23 | 3 | -71/+95 | |
|\ \ | ||||||
| * | | little details seen while doing a pass through what's new in docrails | Xavier Noria | 2011-07-23 | 2 | -7/+7 | |
| | | | ||||||
| * | | use the doc conventions for arguments in all cases | Ray Baxter | 2011-07-20 | 1 | -3/+3 | |
| | | | | | | | | | | | | | | | See https://github.com/lifo/docrails/commit/5c71a4e76897844ab6aeabb581549d981d9768c9#commitcomment-485675 | |||||
| * | | Update form helper documentation to match code. | Paul Battley | 2011-07-19 | 1 | -2/+2 | |
| | | | | | | | | | | | | Change d3cfee11 removed the automatically generated object_name_submit id attributes on form submit elements. This makes the documentation match. | |||||
| * | | reword select_date's behaviour clearly when an incomplete options array is ↵ | Vijay Dev | 2011-07-20 | 1 | -3/+3 | |
| | | | | | | | | | | | | passed | |||||
| * | | minor corrections in form helpers guide and api docs | Vijay Dev | 2011-07-20 | 1 | -2/+2 | |
| | | | ||||||
| * | | examples for telephone_field, url_field and email_field | Ray Baxter | 2011-07-17 | 1 | -0/+12 | |
| | | | ||||||
| * | | document options to search_field | Ray Baxter | 2011-07-17 | 1 | -1/+22 | |
| | | | ||||||
| * | | use correct in description | Ray Baxter | 2011-07-17 | 1 | -1/+1 | |
| | | | ||||||
| * | | we're talking about days here, not hours | Ray Baxter | 2011-07-17 | 1 | -1/+1 | |
| | | | ||||||
| * | | wording between select_second, select_minute and | Ray Baxter | 2011-07-17 | 1 | -2/+2 | |
| | | | | | | | | | select_hour should be consistent and correct | |||||
| * | | fix typo | Ray Baxter | 2011-07-17 | 1 | -1/+1 | |
| | | | ||||||
| * | | make documentation agree with test expectations and | Ray Baxter | 2011-07-17 | 1 | -2/+2 | |
| | | | | | | | | | | | | | | | | | | behaviour See: test_select_date_with_incomplete_order in actionpack/test/template/date_helper_test.rb |