aboutsummaryrefslogtreecommitdiffstats
path: root/actionpack/lib
Commit message (Collapse)AuthorAgeFilesLines
* Merge branch 'master' of git://github.com/lifo/docrailsXavier Noria2010-10-062-6/+3
|\
| * `render :text => proc { ... }` is no longer supported.John Firebaugh2010-10-041-4/+0
| |
| * Template::Error is also used if rendering fails.John Firebaugh2010-10-041-2/+3
| |
* | Provide a cleaner syntax for paths configuration that does not rely on ↵José Valim2010-10-062-6/+8
| | | | | | | | method_missing.
* | refactoring to use fewer intermediate variablesAaron Patterson2010-10-041-11/+12
| |
* | cleaning up more crazy!Aaron Patterson2010-10-041-3/+3
| |
* | deleting more crazyAaron Patterson2010-10-041-6/+4
| |
* | dry up some crazy codesAaron Patterson2010-10-041-8/+6
| |
* | only forwarding enough methods to work. People should grab the delegate ↵Aaron Patterson2010-10-041-5/+8
| | | | | | | | tempfile if they really need to do hard work
* | making sure respond_to? works properlyAaron Patterson2010-10-041-0/+5
| |
* | raising an argument error if tempfile is not providedAaron Patterson2010-10-041-0/+1
| |
* | delegate to the @tempfile instance variableAaron Patterson2010-10-041-13/+5
| |
* | avoid method call to compactAaron Patterson2010-10-041-4/+4
| |
* | Ensure the proper content type is returned for static files.José Valim2010-10-041-3/+3
| |
* | Initialize sid should just skip instance variables.José Valim2010-10-041-0/+7
| |
* | two argument String#slice is faster than single argument, also avoid ↵Aaron Patterson2010-10-031-1/+1
| | | | | | | | creating a Range object
* | Solve some warnings and a failing test.José Valim2010-10-032-1/+2
| |
* | Move ETag and ConditionalGet logic from AD::Response to the middleware stack.José Valim2010-10-032-22/+2
| |
* | Rely on Rack::Session stores API for more compatibility across the Ruby world.José Valim2010-10-035-329/+81
| |
* | PERF: Hash[] + map is faster than this silly inject, and var[1..-1] is ↵Santiago Pastorino2010-10-031-3/+3
| | | | | | | | faster than var.sub('@', '')
* | Rename _assigns to view_assigns in AV::TCDavid Chelimsky2010-10-031-9/+21
| | | | | | | | | | | | | | | | | | - also add tests - also deprecate _assigns [#5751 state:resolved] Signed-off-by: Santiago Pastorino <santiago@wyeworks.com>
* | Fixing search_field to remove object attribute from options hash [#5730 ↵Aditya Sanghi2010-10-021-1/+1
| | | | | | | | | | | | state:resolved] Signed-off-by: José Valim <jose.valim@gmail.com>
* | Revert "Perf: refactor _assign method to avoid inject and defining unneeded ↵José Valim2010-10-021-1/+5
| | | | | | | | | | | | | | | | local var." _assigns must return a hash. This reverts commit e66c1cee86aba1c81152f3d0872313e65cec85a9.
* | removing AS::Testing::Default in favor of just undefing default_testAaron Patterson2010-10-011-2/+1
| |
* | Revert "removing crazy finalizer code until there is proof that we need it"José Valim2010-09-301-0/+11
| | | | | | | | This reverts commit a40e3c1a9604ab3737ad2465c8f6a6db0fe0cc78.
* | Properly reload routes defined in class definitionPiotr Sarnacki2010-09-301-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Sometimes it's easier to define routes inside Engine or Application class definition (e.g. one file applications). The problem with such case is that if there is a plugin that has config/routes.rb file, it will trigger routes reload on application. Since routes definition for application is not in config/routes.rb file routes_reloader will fail to reload application's routes properly. With this commit you can pass routes definition as a block to routes method, which will allow to properly reload it: class MyApp::Application < Rails::Application routes do resources :users end end
* | Allow mounting engines at '/'Piotr Sarnacki2010-09-301-1/+1
| | | | | | | | | | Without that commit script_name always become '/', which results in paths like //posts/1 instead of /posts/1
* | Use .find here as it is simpler and faster.José Valim2010-09-301-1/+1
| |
* | avoid creating a block if possibleAaron Patterson2010-09-291-2/+2
| |
* | fixing regexp warningsAaron Patterson2010-09-291-1/+1
| |
* | tag value can be false, so nil? check is necessaryAaron Patterson2010-09-291-1/+1
| |
* | another case of extra nil? checkNeeraj Singh2010-09-301-1/+1
| |
* | no need to check for nil?Neeraj Singh2010-09-301-2/+2
| |
* | no need to check for nil?Neeraj Singh2010-09-301-1/+1
| |
* | no need of nil checkNeeraj Singh2010-09-301-1/+1
| |
* | removing crazy finalizer code until there is proof that we need itAaron Patterson2010-09-291-9/+0
| |
* | removing useless codeAaron Patterson2010-09-291-1/+0
| |
* | dry up method checking in the request objectAaron Patterson2010-09-291-10/+9
| |
* | reduce function calls on ArrayAaron Patterson2010-09-291-3/+2
| |
* | removing more lolinjectAaron Patterson2010-09-291-8/+7
| |
* | removing lollerjectAaron Patterson2010-09-291-1/+1
| |
* | oops, missed a uniqAaron Patterson2010-09-291-1/+1
| |
* | dry up action_methodsAaron Patterson2010-09-291-4/+4
| |
* | Merge remote branch 'miloops/warnings'José Valim2010-09-292-7/+3
|\ \ | |/ |/|
| * We can't assign @view_context_class here, define super() in test instead if ↵Emilio Tagua2010-09-291-1/+1
| | | | | | | | we want to avoid warnings.
| * _ is not a valid scheme name character, \w includes it and also is redundant ↵Emilio Tagua2010-09-291-1/+1
| | | | | | | | | | | | with \d. 'The scheme name consists of a letter followed by any combination of letters, digits, and the plus ("+"), period ("."), or hyphen ("-") characters; and is terminated by a colon (":").'
| * Remove redundant conditional.Emilio Tagua2010-09-291-5/+1
| |
| * Don't try to interpolate string if there's no interpolation point at all.Emilio Tagua2010-09-291-1/+1
| |
| * Initialize @view_context_class and cache view_context_class value.Emilio Tagua2010-09-291-1/+1
| |
* | no need to call e.classAaron Patterson2010-09-291-1/+1
| |