Commit message (Collapse) | Author | Age | Files | Lines | ||
---|---|---|---|---|---|---|
... | ||||||
| * | | | | | Allow access to current object_counter variable from layout when rendering ↵ | Carlos Antonio da Silva | 2012-04-29 | 3 | -5/+14 | |
| | | | | | | | | | | | | | | | | | | | | | | | | with partial + collection | |||||
| * | | | | | Allow layout to access current object being rendered when using render ↵ | Carlos Antonio da Silva | 2012-04-29 | 2 | -1/+5 | |
| | | | | | | | | | | | | | | | | | | | | | | | | partial + object | |||||
| * | | | | | Allow layout rendering to access current object being rendered when using ↵ | Carlos Antonio da Silva | 2012-04-29 | 3 | -8/+18 | |
| |/ / / / | | | | | | | | | | | | | | | | partial + collection | |||||
* | | | | | Merge pull request #6017 from larzconwell/remove_sanitize_dom_id | Jeremy Kemper | 2012-04-29 | 1 | -6/+1 | |
|\ \ \ \ \ | |/ / / / |/| | | | | Removed the sanitize_dom_id method | |||||
| * | | | | Removed the sanitize_dom_id method because HTML5 doctype let's us use ↵ | Larz Conwell | 2012-04-27 | 1 | -6/+1 | |
| | | | | | | | | | | | | | | | | | | | | anything except nothing and whitespace for id's | |||||
* | | | | | Add changelog entry for jsonp mimetype change, fix failing test | Carlos Antonio da Silva | 2012-04-29 | 2 | -1/+3 | |
| |_|_|/ |/| | | | | | | | | | | | Fix failing test: Mime::JS generates "text/javascript" | |||||
* | | | | Don't convert params if the request isn't HTML - fixes #5341 | Andrew White | 2012-04-29 | 2 | -8/+58 | |
| | | | | ||||||
* | | | | Avoid calling content type multiple times | José Valim | 2012-04-29 | 1 | -4/+6 | |
| | | | | ||||||
* | | | | Merge pull request #2321 from omjokine/master | José Valim | 2012-04-29 | 2 | -2/+5 | |
|\ \ \ \ | | | | | | | | | | | JSONP should use mimetype application/javascript | |||||
| * \ \ \ | Merge remote-tracking branch 'upstream/master' | Olli Jokinen | 2011-12-01 | 180 | -6991/+9152 | |
| |\ \ \ \ | ||||||
| * | | | | | fixed test case test_render_json_with_callback to use content_type ↵ | Olli Jokinen | 2011-07-28 | 1 | -1/+1 | |
| | | | | | | | | | | | | | | | | | | | | | | | | application/javascript | |||||
| * | | | | | Change mimetype to Mime::JS if JSONP is used | Olli Jokinen | 2011-07-28 | 1 | -1/+4 | |
| | | | | | | ||||||
* | | | | | | Lazy load `default_form_builder` if it's passed as a string | Piotr Sarnacki | 2012-04-28 | 3 | -1/+26 | |
| | | | | | | | | | | | | | | | | | | | | | | | | closes #3341 | |||||
* | | | | | | Merge session arg with existing session instead of overwriting | Andrew White | 2012-04-28 | 3 | -1/+28 | |
| |_|_|/ / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This may break existing tests that are asserting the whole session contents but should not break existing tests that are asserting individual keys - e.g: class SomeControllerTest < ActionController::TestCase setup do session['user_id'] = 1 end test "some test" do get :some_action, nil, { 'another_var' => 2 } # This assertion will now fail assert_equal({ 'another_var' => 2 }, session) # This assertion will still pass assert_equal 2, session['another_var] end end Fixes #1529. | |||||
* | | | | | Merge branch 'master' of github.com:lifo/docrails | Vijay Dev | 2012-04-28 | 2 | -4/+4 | |
|\ \ \ \ \ | |_|_|/ / |/| | | | | ||||||
| * | | | | Use <tt>Foo::Bar</tt> instead of +Foo::Bar+ | Mark Rushakof | 2012-04-27 | 2 | -4/+4 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The latter doesn't render as code in HTML output. Regex used in Rubymine to locate the latter form: (\+)(:*\w+:(?::|\w)+)(\+) | |||||
* | | | | | Merge pull request #5989 from arunagw/warning_removed_master_ap | Jon Leighton | 2012-04-27 | 1 | -3/+3 | |
|\ \ \ \ \ | | | | | | | | | | | | | Shadowing variable warning removed | |||||
| * | | | | | Shadowing variable warning removed | Arun Agrawal | 2012-04-26 | 1 | -3/+3 | |
| | | | | | | ||||||
* | | | | | | remove calls to deprecated find(:first) in actionpack test suite | Sergey Nartimov | 2012-04-26 | 1 | -2/+2 | |
| | | | | | | ||||||
* | | | | | | refactor content_tag_for helper | Sergey Nartimov | 2012-04-26 | 1 | -3/+3 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - do not use merge! just for 2 values - there is no need to check block arity | |||||
* | | | | | | use safe_join in number helper | Sergey Nartimov | 2012-04-26 | 1 | -1/+1 | |
| | | | | | | ||||||
* | | | | | | remove unnecessary else statement | Vasiliy Ermolovich | 2012-04-26 | 1 | -3/+3 | |
| | | | | | | ||||||
* | | | | | | fix number_to_human docs [ci skip] | Vijay Dev | 2012-04-26 | 1 | -2/+2 | |
|/ / / / / | ||||||
* | | | | | Fix the build | Yehuda Katz | 2012-04-25 | 1 | -1/+2 | |
| | | | | | ||||||
* | | | | | Allow loading external route files from the router | Jose and Yehuda | 2012-04-25 | 2 | -0/+18 | |
|/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This feature enables the ability to load an external routes file from the router via: draw :filename External routes files go in +config/routes+. This feature works in both engines and applications. | |||||
* | | | | Merge branch 'master' of github.com:lifo/docrails | Vijay Dev | 2012-04-25 | 2 | -1/+4 | |
|\ \ \ \ | ||||||
| * | | | | Missed checked in Form Helper example | Alexey Vakhov | 2012-04-24 | 1 | -1/+1 | |
| | | | | | ||||||
| * | | | | Add distance_of_time_in_words example for seconds | Alexey Vakhov | 2012-04-22 | 1 | -0/+3 | |
| | | | | | ||||||
* | | | | | Merge pull request #5980 from gazay/valid_ips | Jeremy Kemper | 2012-04-25 | 2 | -42/+182 | |
|\ \ \ \ \ | | | | | | | | | | | | | Remote ip logic and validation. IPv6 support. | |||||
| * | | | | | Valid ips v4 and v6. Right logic for working with X-FORWARDED-FOR header and ↵ | Alexey Gaziev | 2012-04-25 | 2 | -42/+182 | |
| |/ / / / | | | | | | | | | | | | | | | | tests. | |||||
* | | | | | Remove unnecessary empty line | José Valim | 2012-04-25 | 1 | -1/+0 | |
| | | | | | ||||||
* | | | | | Remove default match without specified method | Jose and Yehuda | 2012-04-24 | 40 | -343/+353 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In the current router DSL, using the +match+ DSL method will match all verbs for the path to the specified endpoint. In the vast majority of cases, people are currently using +match+ when they actually mean +get+. This introduces security implications. This commit disallows calling +match+ without an HTTP verb constraint by default. To explicitly match all verbs, this commit also adds a :via => :all option to +match+. Closes #5964 | |||||
* | | | | | Merge pull request #5961 from bogdan/routes | Jeremy Kemper | 2012-04-24 | 1 | -20/+26 | |
|\ \ \ \ \ | | | | | | | | | | | | | Refactor ActionDispatch::HTTP::Url#url_for | |||||
| * | | | | | ActionDispatch::HTTP::Url#url_for refactor method | Bogdan Gusiev | 2012-04-24 | 1 | -20/+26 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Separated right side url generation(before query string) from left side url generation(after query string) | |||||
* | | | | | | Merge pull request #5927 from avakhov/distance_of_time_in_words-swap | Jeremy Kemper | 2012-04-24 | 2 | -16/+41 | |
|\ \ \ \ \ \ | | | | | | | | | | | | | | | Distance of time in words should work correctly if from_time > to_time | |||||
| * | | | | | | Use leap years trick in distance_of_time_in_words only for distances between ↵ | Alexey Vakhov | 2012-04-24 | 2 | -12/+18 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | real date points | |||||
| * | | | | | | fixed non matching documentation behaviour with method semantics on ↵ | Marcelo Casiraghi | 2012-04-24 | 2 | -3/+20 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | distance_of_time_in_words | |||||
| * | | | | | | Distance of time in words should work correct if from time > to_time | Alexey Vakhov | 2012-04-24 | 2 | -2/+4 | |
| | | | | | | | ||||||
* | | | | | | | Refactor hash creation in routeset | Carlos Antonio da Silva | 2012-04-24 | 1 | -3/+4 | |
| |/ / / / / |/| | | | | | ||||||
* | | | | | | RouteSet: decomplecting a way to handle positional args | Bogdan Gusiev | 2012-04-24 | 1 | -26/+20 | |
|/ / / / / | ||||||
* | | | | | RouteSet: simplify routes helpers generation code | Bogdan Gusiev | 2012-04-24 | 1 | -13/+21 | |
| | | | | | ||||||
* | | | | | Add missing requires in routes | Bogdan Gusiev | 2012-04-23 | 2 | -0/+3 | |
|/ / / / | ||||||
* | | | | Merge branch 'master' of github.com:lifo/docrails | Vijay Dev | 2012-04-21 | 3 | -4/+4 | |
|\ \ \ \ | ||||||
| * | | | | Comma missed | Alexey Vakhov | 2012-04-21 | 1 | -1/+1 | |
| | | | | | ||||||
| * | | | | Fix example in ActionView::Helpers::FormHelper | Thibaut Courouble | 2012-04-20 | 1 | -1/+1 | |
| | | | | | ||||||
| * | | | | Fix formatting | Oscar Del Ben | 2012-04-18 | 1 | -2/+2 | |
| | | | | | ||||||
* | | | | | Merge pull request #5865 from tiegz/minor_fixes | Aaron Patterson | 2012-04-20 | 2 | -35/+22 | |
|\ \ \ \ \ | | | | | | | | | | | | | Catch nil.to_sym errors in partial_renderer, and raise ArgumentError instead | |||||
| * | | | | | converting some tests to assert_raises, and DRY'ing retrieve_variable changes | Tieg Zaharia | 2012-04-16 | 2 | -40/+18 | |
| | | | | | | ||||||
| * | | | | | catch nil.to_sym errors in partial_renderer, and raise ArgumentError instead | Tieg Zaharia | 2012-04-16 | 2 | -1/+10 | |
| | | | | | | ||||||
* | | | | | | Freeze the middleware stack after it's built | Jeremy Kemper | 2012-04-20 | 1 | -1/+1 | |
| |/ / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | So apps that accidentally add middlewares later aren't unwittingly dumping them in a black hole. Closes #5911 |