aboutsummaryrefslogtreecommitdiffstats
path: root/actionpack/lib/action_dispatch
Commit message (Collapse)AuthorAgeFilesLines
* Remove method if exists, avoid calling Array#first so many times.Emilio Tagua2010-09-281-3/+5
|
* Remove more warnings on variables.Emilio Tagua2010-09-281-0/+1
|
* Initialize @compiled_at if it is not.Emilio Tagua2010-09-281-0/+1
|
* Remove warings from rescues: don't define more than once debug_hash method ↵Emilio Tagua2010-09-281-2/+2
| | | | and ask if @response if defined.
* Remove remaining warnings on _trace by adding parenthesis to gsub arguments.Emilio Tagua2010-09-281-3/+3
|
* Use parenthesis to avoid ambiguous first argument warning.Emilio Tagua2010-09-281-1/+1
|
* Avoid more uninitialized variable warnings.Emilio Tagua2010-09-281-1/+2
|
* @selected may not be defined here, avoid warning.Emilio Tagua2010-09-281-1/+1
|
* No need to call super here. Use yield instead block.callEmilio Tagua2010-09-281-3/+2
|
* Initialize @app if it doesn't exists.Emilio Tagua2010-09-281-1/+1
|
* Move uri parser to AS as URI.parser method to reuse it in AP and ARes.Emilio Tagua2010-09-271-8/+2
|
* Define @_routes inside method, makes more sense and will be initialized when ↵Emilio Tagua2010-09-272-2/+1
| | | | | | called anywhere. Signed-off-by: Santiago Pastorino <santiago@wyeworks.com>
* Remove more warnings shadowing outer local variable.Emilio Tagua2010-09-271-2/+2
| | | | Signed-off-by: Santiago Pastorino <santiago@wyeworks.com>
* Remove warning "URI.unescape is obsolete" from actionpack.Emilio Tagua2010-09-271-4/+2
| | | | Signed-off-by: Santiago Pastorino <santiago@wyeworks.com>
* Merge remote branch 'miloops/warnings'José Valim2010-09-278-35/+55
|\ | | | | | | | | Conflicts: actionpack/lib/action_controller/metal/url_for.rb
| * _routes must be inside @controller conditional.Emilio Tagua2010-09-271-1/+1
| |
| * Refactor method to avoid warnings and not run unnecessary code.Emilio Tagua2010-09-271-11/+11
| |
| * Initialize @cookies.Emilio Tagua2010-09-271-0/+1
| |
| * Remove more warnings shadowing outer local variable.Emilio Tagua2010-09-271-2/+2
| |
| * Don't shadow outer local variables.Emilio Tagua2010-09-271-1/+1
| |
| * Avoid uninitialized variable warning, reuse @integration_session.Emilio Tagua2010-09-271-8/+14
| |
| * Ask is @controller is defined to avoid warning.Emilio Tagua2010-09-271-1/+1
| |
| * Initialize @_routes if it doesn't exists.Emilio Tagua2010-09-271-0/+1
| |
| * Remove warning "too many arguments for format string" when interpolating ↵Emilio Tagua2010-09-271-1/+1
| | | | | | | | with empty hash.
| * Initialize @_etag.Emilio Tagua2010-09-271-0/+1
| |
| * Initialize @as before plural method is called.Emilio Tagua2010-09-271-0/+1
| |
| * Remove old method before redefining it.Emilio Tagua2010-09-271-0/+2
| |
| * Remove warning "URI.unescape is obsolete" from actionpack.Emilio Tagua2010-09-272-11/+19
| |
* | Convert unless/else into if/else.thedarkone2010-09-271-4/+4
| |
* | Hash#empty? is faster than Enumerable#any? when used on a Hash.thedarkone2010-09-271-1/+1
| |
* | options[:action] is very likely to be nil.thedarkone2010-09-271-1/+1
| |
* | Cache 2 of Request's commonly called methods.thedarkone2010-09-271-2/+2
| |
* | No need to create a separate lambda for each call.thedarkone2010-09-271-15/+14
|/
* Fix the precedence issue hereSantiago Pastorino2010-09-261-1/+1
|
* Not need to do this double ternarySantiago Pastorino2010-09-261-1/+1
|
* port is appended twice to HTTP_HOST when host already has the portSantiago Pastorino2010-09-261-3/+3
|
* Remove deprecated stuff in ActionControllerCarlos Antonio da Silva2010-09-261-2/+2
| | | | | | This removes all deprecated classes in ActionController related to Routing, Abstract Request/Response and Integration/IntegrationTest. All tests and docs were changed to ActionDispatch instead of ActionController.
* Don't duplicate Rack::Response functionality. [#5320 state:resolved]John Firebaugh2010-09-251-18/+4
| | | | Signed-off-by: José Valim <jose.valim@gmail.com>
* Improve performance of applications using file uploads by not busting the ↵Carl Lerche2010-09-221-29/+20
| | | | method cache on every request containing a file upload.
* fixing bug with rails use of rack-testAaron Patterson2010-09-221-2/+4
|
* Refactor routing methods.Emilio Tagua2010-09-222-11/+2
| | | | Signed-off-by: Santiago Pastorino <santiago@wyeworks.com>
* Raise ArgumentError instead of normalizing controller name when there is a ↵Andrew White2010-09-181-1/+5
| | | | leading slash [#5651 state:resolved]
* Remove leading slash from controller [#5651 state:resolved]Andrew White2010-09-181-1/+1
|
* Add RouteSet#appendCarl Lerche2010-09-171-4/+12
| | | Allows specifying blocks to the routeset that will get appended after the RouteSet is drawn.
* First pass at Rack::Cachewycats2010-09-132-0/+68
|
* Only send secure cookies over SSL.W. Andrew Loe III2010-09-131-1/+4
|
* removes /i from the TRUSTED_PROXIES regexp, adds /x and comments for ↵Xavier Noria2010-09-121-2/+10
| | | | readability, adds a pointer to a Wikipedia section that documents the matched IPs
* gets rid of a double negation, no need to force exactly true/false in a ↵Xavier Noria2010-09-121-4/+4
| | | | predicate
* Partial revert of commit #7fae0aa4ac116 removing ↵Mikel Lindsaar2010-09-111-5/+0
| | | | | | | action_dispatch.prepare_dispatcher per commit #48bf667a8b1 See line note: http://github.com/rails/rails/commit/7fae0aa4ac116bcbf522a9a4f19972c653d366fa#commitcomment-145148
* Merge remote branch 'drogus/engines'wycats2010-09-111-1/+1
|\