aboutsummaryrefslogtreecommitdiffstats
path: root/actionpack/lib
Commit message (Collapse)AuthorAgeFilesLines
...
* | render :template => 'foo/bar.json' now works as it should.José Valim2010-10-074-12/+28
| |
* | Make collection rendering faster.José Valim2010-10-072-3/+3
| |
* | Remove locals dependency from template.José Valim2010-10-078-85/+129
| | | | | | | | | | | | This means that templates does not need to store its source anymore, allowing us to reduce the ammount of memory taken by our Rails processes. Naively speaking, if your app/views contains 2MB of files, each of your processes (after being hit by a bunch of requests) will take 2MB less of memory after this commit. This is extremely important for the upcoming features. Since Rails will also render CSS and JS files, their source won't be stored as well allowing us to decrease the ammount of memory taken.
* | Get rid of ruby warnings in Resolvers. Move a few methods up to the abstract ↵José Valim2010-10-072-23/+26
|/ | | | class.
* 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 (":").'